Re: [PATCH v2 5/5] block: add trace messages to zone write plugging

[Bart Van Assche <bvanassche@xxxxxxx>]

On 7/14/25 7:38 AM, Johannes Thumshirn wrote:
> +	TP_printk("%d,%d zone %u, BIO %llu + %u",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev), __entry->zno,
> +		  (unsigned long long)__entry->sector,
> +		  __entry->nr_segs)

This output format is confusing. All other block layer tracing code
uses the notation <sector> + ... to indicate a starting sector and a
length in sectors. Is the segment information relevant enough to
include it in the tracing output? If so, please use another way to
report the number of segments, e.g. "BIO %llu, nr_segs %u".

Thanks,

Bart.