public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Oliver Neukum <oliver@neukum.org>
Cc: Jesse Stockall <stockall@magma.ca>,
	linux-kernel@vger.kernel.org, rgooch@safe-mbox.com,
	akpm@osdl.org
Subject: Re: [PATCH] delete devfs
Date: Thu, 22 Jul 2004 02:49:53 -0400	[thread overview]
Message-ID: <20040722064952.GC20561@kroah.com> (raw)
In-Reply-To: <200407220047.53153.oliver@neukum.org>

On Thu, Jul 22, 2004 at 12:47:53AM +0200, Oliver Neukum wrote:
> Am Donnerstag, 22. Juli 2004 00:05 schrieb Greg KH:
> > > That's the point that Oliver and I raised, the "leave it till 2.7" (not
> > > breaking things for real world users) argument seems stronger than the
> > > "rip it now" (because it makes things cleaner, easier to code, etc)
> > > argument. 
> > 
> > The kernel development model (the whole stable/development tree thing)
> > has changed based on the discussions at the kernel summit yesterday.
> > See lwn.net for more details. That is why I sent this patch at this
> > point in time.
> 
> Interesting, but we are not talking about an _internal_ API here.
> It's about blocking the upgrade path.

There is no such block.  udev has a full devfs compatibility mode, I made
sure of that before every suggesting that a change like this happen.

> System using a stable kernel will needlessly stop working after an
> upgrade to another stable kernel.

Userspace tools need to be upgraded/added due to different kernel
changes all the time.  This is just another one of them.

Also, everyone please, consider these points about the current devfs
code:
	- it is unmaintained, and has been for years.
	- it contains known bugs (race conditions), that are pretty much
	  unsolvable with the current architecture of the code, that
	  have been pointed out many times, for years.
	- there is almost no functionality that devfs provides that is
	  not provided with udev[1]
	- no distro supports devfs
	- no active, respected, kernel developer wants to see devfs
	  remain in the kernel tree.

Yes, this has always seemed to be a hot topic.  And yes, I did really
push a lot of people's buttons by posting this patch[2], but I did it
for two reasons:
	- It was going to be my first 2.7.0 patch anyway.
	- Based on the discussions at the kernel summit, the development
	  model has changed.  This patch was a test to see if anything
	  has really changed or not :)

I'm sorry about the fact that this change in development model was not
really made public before I posted it.  That probably caused the most
confusion in this thread, and with hindsight, I should have waited a few
days for that information to have gotten out to the whole world before
submitting it[3].

Again, I apologize.

thanks,

greg k-h

[1] ok, yeah, the floppy driver will not get loaded automatically if you
try to open a /dev node that is not present at that time.  But if you
still rely on this model of behavior to properly load your modules, you
somehow missed the memo that stated that the kernel has changed from
automatically loading modules based on userspace needs (with kmod), to
one where they are loaded based on the presence of a device.  This has
all been discussed in detail many times in the past on lkml.  See the
udev FAQ for some other details about this.

[2] It was said that I rubbed the lamp with this patch, causing Richard
to suddenly appear in public at OLS within 8 hours.  I personally doubt
this, and publicly welcome Richard back to the kernel development
community.

[3] Note to self, sending patches at 2:30am after spending the evening
in deep technical discussions[4] with other kernel developers is not the
best thing to do.

[4] Ok, they weren't that deep...

  reply	other threads:[~2004-07-22  6:51 UTC|newest]

Thread overview: 93+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-21 14:15 [PATCH] delete devfs Greg KH
2004-07-21 14:26 ` Oliver Neukum
2004-07-21 14:35   ` Lars Marowsky-Bree
2004-07-21 14:52   ` Greg KH
2004-07-21 21:19     ` Jesse Stockall
2004-07-21 21:27       ` Greg KH
2004-07-21 21:53         ` Jesse Stockall
2004-07-21 22:05           ` Greg KH
2004-07-21 22:17             ` Jesse Stockall
2004-07-21 22:47             ` Oliver Neukum
2004-07-22  6:49               ` Greg KH [this message]
2004-07-22  9:55                 ` Oliver Neukum
2004-07-22 10:08                   ` Paolo Ciarrocchi
2004-07-22 16:13                 ` Matt Porter
2004-07-23 19:06                   ` [RFC]: CONFIG_UNSUPPORTED (was: Re: [PATCH] delete devfs) R. J. Wysocki
2004-07-23 20:04                     ` Adrian Bunk
2004-07-23 21:17                       ` Russell King
2004-07-23 21:22                       ` R. J. Wysocki
2004-07-23 23:35                       ` Sam Ravnborg
2004-07-23 22:01                         ` [RFC]: CONFIG_UNSUPPORTED Stephen Wille Padnos
2004-07-22  1:08           ` [PATCH] delete devfs Grzegorz Jaśkiewicz
2004-07-22  1:48             ` Mike Snitzer
2004-07-21 22:02         ` Adrian Bunk
2004-07-21 22:07           ` Greg KH
2004-07-21 22:14             ` David Weinehall
2004-07-21 22:31             ` Brian Gerst
2004-07-21 23:11               ` New dev model (was [PATCH] delete devfs) Jonathan Corbet
2004-07-21 23:52                 ` Adrian Bunk
2004-07-22  9:55                   ` Andrew Morton
2004-07-22  7:04                     ` Greg KH
2004-07-22 10:19                       ` Andrew Morton
2004-07-22 12:55                         ` Josh Boyer
2004-07-22 11:32                       ` Giacomo A. Catenazzi
2004-07-22 19:12                         ` Greg KH
2004-07-22 19:33                     ` Adrian Bunk
2004-07-22 22:28                       ` Paul Jackson
2004-07-22 23:25                         ` Adrian Bunk
2004-07-23  2:22                           ` Tim Wright
2004-07-23  6:31                             ` Ville Herva
2004-07-23 21:04                               ` Valdis.Kletnieks
2004-07-23 21:08                                 ` Ville Herva
2004-07-25 11:59                             ` Jan Knutar
2004-07-25 18:53                               ` Jesper Juhl
2004-07-23  8:16                         ` szonyi calin
2004-07-23 12:21                           ` Jonathan Corbet
2004-07-23 19:59                             ` Adrian Bunk
2004-07-24 14:24                             ` Marcelo Tosatti
2004-07-23 14:54                           ` Geert Uytterhoeven
2004-07-23 15:50                             ` szonyi calin
2004-07-27 22:18                               ` Bill Davidsen
2004-07-28 21:25                                 ` Krzysztof Halasa
2004-08-02 18:48                                   ` Bill Davidsen
2004-08-03 22:07                                     ` Krzysztof Halasa
2004-07-24 16:21                           ` Ragnar Hojland Espinosa
2004-07-27 22:12                         ` Bill Davidsen
2004-07-28  7:24                           ` Paul Jackson
2004-07-22 23:01                       ` Andrew Morton
2004-07-22 20:18                         ` Adrian Bunk
2004-07-22 20:28                         ` Kevin Fox
2004-07-23 20:09                           ` Adrian Bunk
2004-07-22 21:01                         ` Martin Schlemmer
2004-07-23  0:39                         ` Jason Cooper
2004-07-23 20:57                         ` Timothy Miller
2004-07-25 13:30                           ` Adrian Bunk
2004-07-26  1:38                         ` Ben Hoskings
2004-07-26  2:12                           ` Bernd Eckenfels
2004-07-28  6:25                             ` Ben Hoskings
2004-07-28 21:23                             ` Krzysztof Halasa
2004-08-04 21:53                               ` Bernd Eckenfels
2004-07-28 21:22                           ` Krzysztof Halasa
2004-07-29 12:25                     ` Adrian Bunk
2004-07-22  1:33               ` [PATCH] delete devfs Mike Snitzer
2004-07-21 23:26             ` R. J. Wysocki
2004-07-21 22:11           ` Francois Romieu
2004-07-21 22:40             ` Adrian Bunk
2004-07-21 23:15               ` Francois Romieu
2004-07-22  8:23               ` sam
2004-07-22 10:24                 ` Gene Heskett
2004-07-22 10:58                   ` Nick Piggin
2004-07-22 21:06                   ` sam
2004-07-23  0:21                     ` Gene Heskett
2004-07-22 22:19                   ` Paul Jakma
2004-07-22 19:22         ` Martin Schlemmer
2004-07-22 17:56       ` Deepak Saxena
2004-07-21 14:52   ` Geert Uytterhoeven
2004-07-21 14:41 ` Matthew Garrett
2004-07-21 18:25   ` Greg KH
2004-07-21 19:55     ` Matthew Garrett
2004-07-21 19:34   ` Chris Wedgwood
2004-07-21 21:13   ` Ben Collins
2004-07-21 22:20   ` Wichert Akkerman
2004-07-22 19:44     ` Martin Schlemmer
2004-07-21 15:49 ` Kasper Sandberg

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=20040722064952.GC20561@kroah.com \
    --to=greg@kroah.com \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=oliver@neukum.org \
    --cc=rgooch@safe-mbox.com \
    --cc=stockall@magma.ca \
    /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