Re: [PATCH 1/2] block: only use the cached zone report if BLK_ZONE_REP_CACHED is set
From: Shashank Mohan Jain
Date: Mon Oct 12 2026 - 00:59:54 EST
On Sun, Oct 11, 2026, Damien Le Moal wrote:
> Can you reverse this if to be "if (rep.flags & BLK_ZONE_REP_CACHED) {" ...
> Also, the condition should probably be:
> if ((rep.flags & BLK_ZONE_REP_CACHED) &&
> blkdev_has_cached_report_zones(bdev))
Reworked exactly as suggested and reposted as v2:
https://lore.kernel.org/all/20261011154018.48913-1-jain.sm@xxxxxxxxx/
blkdev_has_cached_report_zones() is now used from the ioctl, so I moved
it (with its comment) above blkdev_report_zones_ioctl(). Patch 2 is
unchanged and carries your Reviewed-by. Retested on zoned null_blk.
Thanks,
Shashank