Re: 2.6.16-rc4: known regressions

From: Kay Sievers
Date: Tue Feb 21 2006 - 19:03:06 EST


On Tue, Feb 21, 2006 at 03:33:05PM -0800, Andrew Morton wrote:
> Kay Sievers <kay.sievers@xxxxxxx> wrote:
> >
> > On Wed, Feb 22, 2006 at 12:51:01AM +0200, Pekka Enberg wrote:
> > > On Sun, 2006-02-19 at 17:02 -0800, Greg KH wrote:
> > > > If you revert this one patch, on top of a clean 2.6.16-rc4, do things
> > > > start working for you again?
> > >
> > > Okey dokey, bisecting with mrproper took little longer than expected but
> > > I found the bad changeset:
>
> Thanks - it helps heaps.
>
> > > 033b96fd30db52a710d97b06f87d16fc59fee0f1 is first bad commit
> > > diff-tree 033b96fd30db52a710d97b06f87d16fc59fee0f1 (from 0f76e5acf9dc788e664056dda1e461f0bec93948)
> > > Author: Kay Sievers <kay.sievers@xxxxxxx>
> > > Date: Fri Nov 11 06:09:55 2005 +0100
> > >
> > > [PATCH] remove mount/umount uevents from superblock handling
> >
> > Upgrade HAL, it's too old for that kernel.
> >
>
> We broke back-compatibility. The changelog _failed to tell us_ that we
> were breaking back-compatibility. The patch wouldn't have been applied if
> we'd been told that. At least, not without a lot of careful thought.
>
> The fact that the changelog failed to tell us this makes one suspect that
> the breakage was inadvertent.
>
>
> So no, upgrading HAL is not a good answer. Please fix the kernel.

No, these events I added for HAL and they were wrong as pointed out by
Al Viro and with his help we replaced them by a better solution, which
actually is the "fix".

HAL was prepared to make use of the new events and needs to be upgraded
when the kernel gets upgraded. This happens all the time as long as we
try to find the right way to interact with the kernel and need to
change the interfaces.

HAL is tightly bound to the kernel and this will countinuously happen in
the future too, cause we can't solve the problem that you don't know how
an interface works without a user and if you don't put it in the kernel you
certainly don't have a user.

Interfaces get stable by becoming good enough and not by someone that
declares them as stable. If you have a problem with that, please introduce
a list of stable interfaces where people can rely on and I will not put
any of the new interfaces that are under construction on that list. We
need to be able to use new interfaces and change them until we know that
they are good enough. It is very unlikely to get complex interfaces
right in the first place. In the mount event case they have been identified
as the wrong way to do it and got replaced.

Thanks,
Kay
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/