mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Zeuthen <david@fubar.dk>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Kay Sievers <kay.sievers@suse.de>,
	Pekka J Enberg <penberg@cs.Helsinki.FI>, Greg KH <gregkh@suse.de>,
	Adrian Bunk <bunk@stusta.de>, Robert Love <rml@novell.com>,
	Andrew Morton <akpm@osdl.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	John Stultz <johnstul@us.ibm.com>
Subject: Re: 2.6.16-rc4: known regressions
Date: Wed, 22 Feb 2006 11:18:22 -0500	[thread overview]
Message-ID: <1140625103.21517.18.camel@daxter.boston.redhat.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0602220737170.30245@g5.osdl.org>

On Wed, 2006-02-22 at 07:44 -0800, Linus Torvalds wrote:
> 
> On Wed, 22 Feb 2006, Kay Sievers wrote:
> > 
> > Well, that's part of the contract by using an experimental version of HAL,
> > it has nothing to do with the kernel
> 
> NO NO NO!
> 
> Dammit, if this is the logic and mode of operation of HAL people, then we 
> must stop accepting patches to the kernel from HAL people.
> 
> THIS IS NOT DEBATABLE.
> 
> If you cannot maintain a stable kernel interface, then you damn well 
> should not send your patches in for inclusion in the standard kernel. Keep 
> your own "HAL-unstable" kernel and ask people to test it there.

Oh, you know, I don't think that's exactly how it works; HAL is pretty
much at the mercy of what changes goes into the kernel. And, trust me,
the changes we need to cope with from your so-called stable API are not
so nice. 

But I realize these changes are important because it's progress and back
in 2.6.0 things were horribly broken for at least desktop workloads [1].
It also makes me release note that newer HAL releases require newer
kernel and udev releases and that's alright. In fact it's perfectly
fine. We get users to upgrade to the latest and greatest and we keep
making good progress. That's open source at it's finest I think.

For just one example of API breaking see

 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=175998

This breaks stuff for end users in a stable distribution. Not good.

> It really is that easy. Once a system call or other kernel interface goes 
> into the standard kernel, it stays that way. It doesn't get switched 
> around to break user space.
> 
> Bugs happen, and sometimes we break user space by mistake. Sometimes it 
> really really is inevitable. But we NEVER EVER say what you say: "it's 
> your own fault". It's _our_ fault, and it's _our_ problem to work out.
> 
> Guys: you now have two choices: fix it by sending me a patch and an 
> explanation of what went wrong, or see the patch that broke things be 
> reverted. And STOP THIS DAMN APOLOGIA. 
> 
> I'm fed up with hearing how "breaking user space is ok because it's HAL or 
> hotplug". IT IS NOT OK. Get your damn act together, and stop blaming other 
> people. 

I think maintaining a stable syscall interface makes sense. Didn't you
once say that only the syscall interface was supposed to be stable? Or
was that Greg KH? I can't remember...

And I also think that breaking things like sysfs can be alright as long
as you coordinate it with major users of it, e.g. udev and HAL. Please
realize how useful the changes sysfs changes from 2.6.0 to present were
and... and that there still is a lot of work left to make certain things
work for desktop workloads.

I even think changing things like in the RH bug I linked to above is
fine provided that you mentioned it in the release notes...

One day perhaps sysfs will be "just right" and you can mark it as being
stable. I just don't think we're there yet. And I see no reason
whatsoever to paint things as black and white as you do.

    David (Please keep me Cc'ed, I can't keep up with lkml)

[1] : plug in a USB hub with other hubs attached and 10-20 USB devices;
works a lot better with current kernels and udev than it ever did in
2.6.0 with /sbin/hotplug (fork bomb)



  parent reply	other threads:[~2006-02-22 16:19 UTC|newest]

Thread overview: 89+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-17 22:45 Linux 2.6.16-rc4 Linus Torvalds
2006-02-17 23:14 ` 2.6.16-rc4: known regressions Adrian Bunk
2006-02-19 11:06   ` Pekka Enberg
2006-02-19 14:54     ` Adrian Bunk
2006-02-19 17:50       ` Pekka Enberg
2006-02-19 21:14       ` Pekka Enberg
2006-02-20  1:02         ` Greg KH
2006-02-20  7:08           ` Pekka J Enberg
2006-02-21 22:51           ` Pekka Enberg
2006-02-21 22:57             ` Kay Sievers
2006-02-21 23:33               ` Andrew Morton
2006-02-22  0:04                 ` Kay Sievers
2006-02-22  0:15                   ` Mark Lord
2006-02-22  0:21                   ` Andrew Morton
2006-02-22  0:34                     ` Linus Torvalds
2006-02-22  0:46                       ` Con Kolivas
2006-02-22  1:06                       ` Linus Torvalds
2006-02-22 11:21                         ` Theodore Ts'o
2006-02-22 14:25                           ` uswsusp & initrd -- was " Pavel Machek
2006-02-22 15:48                           ` Joel Becker
2006-02-22 16:25                             ` Theodore Ts'o
2006-02-22 17:33                               ` Gabor Gombas
2006-02-22 17:57                                 ` Linus Torvalds
2006-02-22 18:37                                   ` Christian Trefzer
2006-02-22 18:59                                 ` Joel Becker
2006-02-22 19:18                                   ` Greg KH
2006-02-22 19:29                                     ` Arjan van de Ven
2006-02-22 19:40                                       ` Greg KH
2006-02-22 20:45                                         ` Jens Axboe
2006-02-22 22:51                                           ` Greg KH
2006-02-23  6:39                                             ` Jens Axboe
2006-02-23 17:29                                         ` Martin Bligh
2006-02-23 17:52                                           ` Greg KH
2006-02-23 18:01                                             ` Martin Bligh
2006-02-23 18:04                                             ` Arjan van de Ven
2006-02-23 20:26                                         ` Benjamin LaHaise
2006-02-24 23:42                                         ` Eric W. Biederman
2006-02-22 19:39                                     ` Linus Torvalds
2006-02-22 19:54                                   ` Andrew Morton
2006-02-22 20:02                                     ` Arjan van de Ven
2006-02-22 20:12                                     ` Linus Torvalds
2006-02-22 20:44                                       ` Andrew Morton
2006-02-22 20:26                                     ` Greg KH
2006-02-23  5:28                                       ` Jody McIntyre
2006-02-22 20:57                                     ` Diego Calleja
2006-02-22 21:19                                     ` Russell King
2006-02-22 21:30                                       ` Greg KH
2006-02-22 20:47                             ` Bryan O'Sullivan
2006-02-22 19:07                           ` Greg KH
2006-02-22 17:06                         ` Matthias Andree
2006-02-23 12:36                         ` Paulo Marques
2006-02-22 10:49                   ` Diego Calleja
2006-02-22  7:06               ` Pekka J Enberg
2006-02-22 15:27                 ` Kay Sievers
2006-02-22 15:44                   ` Linus Torvalds
2006-02-22 16:03                     ` Arjan van de Ven
2006-02-22 16:11                       ` Christoph Hellwig
2006-02-22 17:17                         ` sysfs regressions (was: 2.6.16-rc4: known regressions) Matthias Andree
2006-02-22 17:47                           ` Greg KH
2006-02-22 16:18                     ` David Zeuthen [this message]
2006-02-22 16:35                       ` 2.6.16-rc4: known regressions Christoph Hellwig
2006-02-22 16:46                         ` David Zeuthen
2006-02-22 16:51                           ` Christoph Hellwig
2006-02-22 17:08                       ` Linus Torvalds
2006-02-22 17:31                         ` Linus Torvalds
2006-02-22 18:04                           ` Al Viro
2006-02-23  3:01                             ` John Stoffel
2006-02-22 17:51                         ` Al Viro
2006-02-22 17:55                           ` Christoph Hellwig
2006-02-22 18:10                             ` Al Viro
2006-02-22 19:25                         ` David Zeuthen
2006-02-22 17:10                       ` Al Viro
2006-02-22 17:10                       ` grundig
2006-02-22 17:14                       ` Martin Bligh
2006-02-23  4:17                         ` Theodore Ts'o
2006-02-22 18:10                   ` Pekka Enberg
2006-02-22  8:28               ` Arjan van de Ven
2006-02-17 23:27 ` Linux 2.6.16-rc4 Nigel Cunningham
2006-02-18  8:59 ` Edmondo Tommasina
2006-02-18  9:19   ` Gene Heskett
2006-02-18 10:20     ` Con Kolivas
2006-02-18 11:26       ` Gene Heskett
2006-02-18 16:04 ` Jean Delvare
2006-02-22 22:02 ` Linux 2.6.16-rc4 edac oops Mark Rustad
2006-02-24 11:09   ` Andrew Morton
2006-02-22  2:39 2.6.16-rc4: known regressions Yu, Luming
2006-02-22  3:16 ` Adrian Bunk
2006-02-22  6:55 Yu, Luming
2006-02-22 12:23 ` Adrian Bunk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1140625103.21517.18.camel@daxter.boston.redhat.com \
    --to=david@fubar.dk \
    --cc=akpm@osdl.org \
    --cc=bunk@stusta.de \
    --cc=gregkh@suse.de \
    --cc=johnstul@us.ibm.com \
    --cc=kay.sievers@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=penberg@cs.Helsinki.FI \
    --cc=rml@novell.com \
    --cc=torvalds@osdl.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®