| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH v8 2/9] dax: fix conversion of holes to PMDs, Ross Zwisler |
|---|---|
| Next by Date: | Re: [PATCH 1/6] Move from __uint*_t types to uint*_t and likewise for __int*_t, Christoph Hellwig |
| Previous by Thread: | Re: [PATCH] xfs_db: check on-disk structure sizes, Eric Sandeen |
| Next by Thread: | Re: [PATCH] xfs_db: check on-disk structure sizes, Darrick J. Wong |
| Indexes: | [Date] [Thread] [Top] [All Lists] |