xfs
[Top] [All Lists]

Re: [PATCH] (and bad attr2 bug) - pack xfs_sb_t for 64-bit arches

To: sandeen@xxxxxxxxxxx
Subject: Re: [PATCH] (and bad attr2 bug) - pack xfs_sb_t for 64-bit arches
From: Nathan Scott <nscott@xxxxxxxxxx>
Date: Mon, 20 Nov 2006 10:11:46 +1100
Cc: David Chinner <dgc@xxxxxxx>, Timothy Shimmin <tes@xxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to: <48064.10.0.0.2.1163776850.squirrel@sandeen.net>
Organization: Aconex
References: <455CB54F.8080901@sandeen.net> <BB70F203E29C2D37A2F727C8@timothy-shimmins-power-mac-g5.local> <20061117023946.GN11034@melbourne.sgi.com> <CEB981736A0E8C7DF9ABD7C8@timothy-shimmins-power-mac-g5.local> <20061117055521.GS11034@melbourne.sgi.com> <52841.10.0.0.2.1163745285.squirrel@sandeen.net> <1163746343.4695.152.camel@edge> <48064.10.0.0.2.1163776850.squirrel@sandeen.net>
Reply-to: nscott@xxxxxxxxxx
Sender: xfs-bounce@xxxxxxxxxxx
On Fri, 2006-11-17 at 09:20 -0600, sandeen@xxxxxxxxxxx wrote:
> > On Fri, 2006-11-17 at 00:34 -0600, sandeen@xxxxxxxxxxx wrote:
> >> and really, now that this is out in the wild, maybe sb_features3
> >> instead of padding is appropriate, and check both for the attr2
> >> bit...? :(
> >
> > Thats not going to work, theres three or four other feature2 bits
> > preceding attr2 as well.
> >
> > The "take a 32 bit systems fs to a 64 bit system" is relatively
> > uncommon, so I suppose its just something we live with (as we did
> > with the log recovery issues in that situation for several years).
> 
> So you think this should not be fixed, then?  Because if it -is- fixed

I didn't say that.  It should be fixed.  Noone will notice though,
as its not actually biting anyone... (the attr2 problem will not
be related to this, its gonna be something else).

> then it's not an fs transfer problem; suddenly 64-bit attr2 filesystems
> will think they have attr1 if proper padding is added.

Now to really fry your noodle, attr2 is actually ondisk compatible
with attr1.  :)  (the SB bit was taken to prevent a repair buglet
from accidentally trashing all inodes using a non-fixed forkoff).

cheers.

-- 
Nathan


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