On 7/14/25 07:38, Johannes Thumshirn wrote: > blk_zone_update_request_bio() does two things. First it checks if the > request to be completed was written via ZONE APPEND and if yes it then > updates the sector to the one that the data was written to. > > This is small enough to be an inline function. But upcoming changes adding > a tracepoint don't work if the function is inlined. > > Split the function into two, the first is blk_req_bio_is_zone_append() > checking if the sector needs to be updated. This can still be an inline > function. The second is blk_zone_append_update_request_bio() doing the > sector update. > > Reviewed-by: Damien Le Moal<dlemoal@xxxxxxxxxx> > Signed-off-by: Johannes Thumshirn<johannes.thumshirn@xxxxxxx> Looks good. Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx> -ck