Re: [PATCH v3 2/3] xfs: zero directory data block padding on write verification

[Dave Chinner]

On Mon, Apr 13, 2026 at 10:59:21AM -0700, Darrick J. Wong wrote:
> On Sat, Apr 11, 2026 at 03:24:14PM +0100, Yuto Ohnuki wrote:
> > Old kernels did not zero the pad field in xfs_dir3_data_hdr when
> > initializing directory data blocks, so existing filesystems may have
> > non-zero padding on disk.
> > 
> > Zero the pad field in xfs_dir3_data_write_verify alongside the existing
> > LSN and checksum updates. The pad field is pure alignment padding with
> > no runtime meaning, so zeroing it during write verification is safe and
> > has no additional I/O cost. This lets filesystems gradually self-heal
> > stale non-zero padding as directories are modified, without requiring an
> > explicit repair pass.
> > 
> > Suggested-by: Dave Chinner <dgc@xxxxxxxxxx>
> > Signed-off-by: Yuto Ohnuki <ytohnuki@xxxxxxxxxx>
> > ---
> >  fs/xfs/libxfs/xfs_dir2_data.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/fs/xfs/libxfs/xfs_dir2_data.c b/fs/xfs/libxfs/xfs_dir2_data.c
> > index 35ff119aa84b..aecbab61014c 100644
> > --- a/fs/xfs/libxfs/xfs_dir2_data.c
> > +++ b/fs/xfs/libxfs/xfs_dir2_data.c
> > @@ -382,6 +382,7 @@ xfs_dir3_data_write_verify(
> >  	struct xfs_mount	*mp = bp->b_mount;
> >  	struct xfs_buf_log_item	*bip = bp->b_log_item;
> >  	struct xfs_dir3_blk_hdr	*hdr3 = bp->b_addr;
> > +	struct xfs_dir3_data_hdr *datahdr3 = bp->b_addr;
> >  	xfs_failaddr_t		fa;
> >  
> >  	fa = xfs_dir3_data_verify(bp);
> > @@ -396,6 +397,11 @@ xfs_dir3_data_write_verify(
> >  	if (bip)
> >  		hdr3->lsn = cpu_to_be64(bip->bli_item.li_lsn);
> >  
> > +	/*
> > +	 * Zero padding that may be stale from old kernels.
> > +	 */
> > +	datahdr3->pad = 0;
> 
> This needs to be gated on the directory block actually having a dir3
> data header (aka xfs_has_crc()).

It already is gated correctly - this check is done 5 lines above:

	if (!xfs_has_crc(mp))
		return;

-Dave.

-- 
Dave Chinner
dgc@xxxxxxxxxx