xfs
[Top] [All Lists]

Re: [PATCH] xfs_db: check on-disk structure sizes

To: Eric Sandeen <sandeen@xxxxxxxxxxx>
Subject: Re: [PATCH] xfs_db: check on-disk structure sizes
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Tue, 12 Jan 2016 23:46:40 -0800
Cc: Dave Chinner <david@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <10021348-43A2-465E-BC00-E0A0882FFD22@xxxxxxxxxxx>
References: <20160111234644.GB7831@xxxxxxxxxxxxxxxx> <20160112140122.GC12156@xxxxxxxxxxxxxxx> <20160113012945.GC2455@xxxxxxxxxxxxxxxx> <5695C01F.7030807@xxxxxxxxxxx> <20160113054730.GQ10456@dastard> <10021348-43A2-465E-BC00-E0A0882FFD22@xxxxxxxxxxx>
User-agent: Mutt/1.5.24 (2015-08-30)
On Wed, Jan 13, 2016 at 12:02:36AM -0600, Eric Sandeen wrote:
> > As I've pointed out previously to Darrick: xfstests:/tests/xfs/122
> > 
> > Make that build again, update it.
> > 
> Oh, I went looking for that and missed it somehow, thought it had been 
> removed.   Ok then!

The real issue is that build environment change alignments.  32 vs 64
bit builds are obvious, but on some architectures different ABIs have
different alignments (we had some fun with ARM in that regard),
nevermind the equivalents to IRIX n32 popping up everywhere these days
that make things complicated.

I'd really love to have Darrick's check in xfs_format.h as an opt-in
if a build time assert is provided - that way every user can check it
doesn't screw up the structures.

<Prev in Thread] Current Thread [Next in Thread>