Re: Ext2 groups descriptor corruption in 2.2 - Phillips's patch seems to work

From: Ville Herva (
Date: Fri Jan 04 2002 - 02:36:26 EST

On Thu, Jan 03, 2002 at 05:00:32PM -0700, you [Andreas Dilger] said:
> On Jan 02, 2002 11:53 +0000, Alan Cox wrote:
> > > Since you have done such a thorough job of documenting the whole thing, why
> > > not drop the other shoe and submit the patches?
> >
> > He did 8)
> >
> > I've asked him to run them past Stephen and the other ext2 folks
> Well, nobody else has spoken up on this, so I might as well. I remember
> when Daniel originally posted this fix, and at the time it was the right
> thing to do. Al's patch fixed more than Daniel's did,

"More", in what way? Fixed more problem cases or just better in sense that
he refromatted and splitted the long ext2_new_inode() function?

> but I think it is too much change to be adding to 2.2, so we should use
> the minimal fix from Daniel, unless there are objections.

Ok. The minimal approach is what I was after, too :).

Do you have any opionion about 2.0? The problem has been spotted on 2.0.39,
too, and the 2.0.40pre3 ext2_new_inode() is almost identical to 2.2 one
barring few trivialish changes. We plan to test the patch some more on 2.0
(we already know it compiles and boots, btw ;), but it would be nice if some
one who has played with 2.0 ext2 would give a "go ahead".

-- v --
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

This archive was generated by hypermail 2b29 : Mon Jan 07 2002 - 21:00:23 EST