Re: [PATCH v2 0/3] Enable cached zone report

["Darrick J. Wong" <djwong@kernel.org>]

On Fri, Dec 19, 2025 at 03:56:02PM -0800, Darrick J. Wong wrote:
> On Fri, Dec 19, 2025 at 06:38:07PM +0900, Damien Le Moal wrote:
> > Enable cached zone report to speed up mkfs and repair on a zoned block
> > device (e.g. an SMR disk). Cached zone report support was introduced in
> > the kernel with version 6.19-rc1.  This was co-developped with
> > Christoph.
> 
> Just out of curiosity, do you see any xfsprogs build problems with
> BLK_ZONE_COND_ACTIVE if the kernel headers are from 6.18?
> 
> > Darrick,
> > 
> > It may be cleaner to have a common report zones helper instead of
> > repating the same ioctl pattern in mkfs/xfs_mkfs.c and repair/zoned.c.
> > However, I am not sure where to place such helper. In libxfs/ or in
> > libfrog/ ? Please advise.
> 
> libfrog/, please.

and since I hit [send] without providing a reason, I'll request a
do-over:

libfrog/, please.  That's where we stuff all the support code, ioctl
wrappers, and data structures for xfsprogs nowadays.  It's better than
cluttering up libxfs/.

--D

> --D
> 
> > Thanks !
> > 
> > Changes from v1:
> >  - Fix erroneous handling of ioctl(BLKREPORTZONEV2) error to correctly
> >    fallback to the regular ioctl(BLKREPORTZONE) if the kernel does not
> >    support BLKREPORTZONEV2.
> > 
> > Damien Le Moal (3):
> >   libxfs: define BLKREPORTZONEV2 if the kernel does not provide it
> >   mkfs: use cached report zone
> >   repair: use cached report zone
> > 
> >  libxfs/topology.h | 8 ++++++++
> >  mkfs/xfs_mkfs.c   | 7 ++++++-
> >  repair/zoned.c    | 7 ++++++-
> >  3 files changed, 20 insertions(+), 2 deletions(-)
> > 
> > -- 
> > 2.52.0
> > 
> > 
>