Re: [PATCH 3/4] xfs_db/mdrestore/repair: don't use the incore struct xfs_sb for offsets into struct xfs_dsb

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 01, 2024 at 10:54:39PM -0700, Christoph Hellwig wrote:
> >  	if (xfs_sb_version_hasmetauuid(sb))
> > -		size = offsetof(xfs_sb_t, sb_meta_uuid)
> > +		size = offsetof(struct xfs_dsb, sb_meta_uuid)
> >  			+ sizeof(sb->sb_meta_uuid);
> 
> It would be nice to fix the formatting here, or maybe even add a a
> field_end() or simіlar helper here..

I think that's part of the metadir patchset, in "xfs_repair: refactor
offsetof+sizeof to offsetofend".

> Otherwise looks good:
> 
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>

Thank you!

--D




[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux