Re: [PATCH 4/5] block: add tracepoint for blkdev_zone_mgmt

[Bart Van Assche <bvanassche@xxxxxxx>]

On 7/9/25 4:47 AM, Johannes Thumshirn wrote:
> +	TP_printk("%d,%d %s %llu + %llu",
> +		  MAJOR(__entry->dev), MINOR(__entry->dev), __entry->rwbs,
> +		  (unsigned long long)__entry->sector,
> +		  __entry->nr_sectors)

sector_t is a synonym for u64. u64 is defined as unsigned long in
include/uapi/asm-generic/int-l64.h and is defined as unsigned long long
in include/uapi/asm-generic/int-ll64.h. Kernel code always includes
the int-ll64.h header file. In other words, I think the above cast is
superfluous for all CPU architectures supported by the Linux kernel.

Thanks,

Bart.