2006-03-20 21:24:06

by Greg KH

[permalink] [raw]
Subject: [GIT PATCH] Remove devfs from 2.6.16

They are the same "delete devfs" patches that I submitted for 2.6.12 and
2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
and ends up saving a lot of space. Since 2.6.13 came out, I have seen
no complaints about the fact that devfs was not able to be enabled
anymore, and in fact, a lot of different subsystems have already been
deleting devfs support for a while now, with apparently no complaints
(due to the lack of users.)

It's also been over 8 months past the date when we said we would delete
devfs from the kernel tree in the file,
Documentation/feature-removal-schedule.txt, and over one and one half
years since we publicly announced to the world that devfs would be
removed from the kernel tree. So, I think people have had plenty of
advance notice that this was going to happen by now :)

Please pull from:
rsync://rsync.kernel.org/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
or if master.kernel.org hasn't synced up yet:
master.kernel.org:/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/

I've posted all of these patches before, but if people really want to look at them, they can be found at:
http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-05-devfs/

Also, if people _really_ are in love with the idea of an in-kernel
devfs, I have posted a patch that does this in about 300 lines of code,
called ndevfs. It is available in the archives if anyone wants to use
that instead (it is quite easy to maintain that patch outside of the
kernel tree, due to it only needing 3 hooks into the main kernel tree.)

thanks,

greg k-h


Documentation/DocBook/kernel-api.tmpl | 5
Documentation/filesystems/devfs/ChangeLog | 1977 ---------------
Documentation/filesystems/devfs/README | 1959 ---------------
Documentation/filesystems/devfs/ToDo | 40
Documentation/filesystems/devfs/boot-options | 65
arch/i386/kernel/microcode.c | 1
arch/ppc/4xx_io/serial_sicc.c | 2
arch/sh/kernel/cpu/sh4/sq.c | 1
arch/sparc64/solaris/socksys.c | 4
arch/um/drivers/line.c | 2
arch/um/drivers/ssl.c | 1
arch/um/drivers/stdio_console.c | 1
arch/um/drivers/ubd_kern.c | 18
arch/um/include/line.h | 1
drivers/block/DAC960.c | 1
drivers/block/acsi.c | 5
drivers/block/acsi_slm.c | 10
drivers/block/cciss.c | 1
drivers/block/cpqarray.c | 5
drivers/block/floppy.c | 55
drivers/block/loop.c | 6
drivers/block/nbd.c | 5
drivers/block/paride/pg.c | 18
drivers/block/paride/pt.c | 21
drivers/block/pktcdvd.c | 1
drivers/block/ps2esdi.c | 1
drivers/block/rd.c | 5
drivers/block/swim3.c | 4
drivers/block/sx8.c | 5
drivers/block/ub.c | 6
drivers/block/umem.c | 1
drivers/block/viodasd.c | 3
drivers/block/xd.c | 1
drivers/block/z2ram.c | 1
drivers/cdrom/aztcd.c | 1
drivers/cdrom/cdu31a.c | 1
drivers/cdrom/cm206.c | 1
drivers/cdrom/gscd.c | 1
drivers/cdrom/mcdx.c | 1
drivers/cdrom/optcd.c | 1
drivers/cdrom/sbpcd.c | 6
drivers/cdrom/sjcd.c | 1
drivers/cdrom/sonycd535.c | 1
drivers/cdrom/viocd.c | 3
drivers/char/cyclades.c | 1
drivers/char/dsp56k.c | 10
drivers/char/dtlk.c | 5
drivers/char/epca.c | 1
drivers/char/esp.c | 1
drivers/char/ftape/zftape/zftape-init.c | 25
drivers/char/hvc_console.c | 1
drivers/char/hvcs.c | 1
drivers/char/hvsi.c | 1
drivers/char/ip2main.c | 24
drivers/char/ipmi/ipmi_devintf.c | 13
drivers/char/isicom.c | 1
drivers/char/istallion.c | 13
drivers/char/lp.c | 7
drivers/char/mem.c | 6
drivers/char/misc.c | 15
drivers/char/mmtimer.c | 2
drivers/char/moxa.c | 1
drivers/char/ppdev.c | 15
drivers/char/pty.c | 8
drivers/char/raw.c | 15
drivers/char/riscom8.c | 1
drivers/char/rocket.c | 5
drivers/char/serial167.c | 1
drivers/char/stallion.c | 14
drivers/char/tipar.c | 17
drivers/char/tty_io.c | 17
drivers/char/vc_screen.c | 11
drivers/char/viocons.c | 1
drivers/char/viotape.c | 10
drivers/char/vme_scc.c | 1
drivers/char/vt.c | 2
drivers/ide/ide-cd.c | 2
drivers/ide/ide-disk.c | 2
drivers/ide/ide-floppy.c | 1
drivers/ide/ide-probe.c | 11
drivers/ide/ide-tape.c | 12
drivers/ide/ide.c | 10
drivers/ieee1394/dv1394.c | 22
drivers/ieee1394/ieee1394_core.c | 14
drivers/ieee1394/ieee1394_core.h | 1
drivers/ieee1394/raw1394.c | 6
drivers/ieee1394/video1394.c | 14
drivers/input/serio/serio_raw.c | 1
drivers/isdn/capi/capi.c | 5
drivers/isdn/hardware/eicon/divamnt.c | 3
drivers/isdn/hardware/eicon/divasi.c | 3
drivers/isdn/hardware/eicon/divasmain.c | 3
drivers/isdn/i4l/isdn_tty.c | 3
drivers/macintosh/adb.c | 3
drivers/md/dm-ioctl.c | 30
drivers/md/dm.c | 2
drivers/md/md.c | 30
drivers/media/dvb/dvb-core/dvbdev.c | 13
drivers/media/dvb/dvb-core/dvbdev.h | 1
drivers/media/dvb/ttpci/av7110.h | 4
drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c | 11
drivers/media/radio/miropcm20-rds.c | 1
drivers/media/video/arv.c | 1
drivers/media/video/videodev.c | 7
drivers/message/i2o/i2o_block.c | 1
drivers/mmc/mmc_block.c | 4
drivers/net/ppp_generic.c | 9
drivers/net/tun.c | 1
drivers/net/wan/cosa.c | 14
drivers/s390/block/dasd.c | 4
drivers/s390/block/dasd_genhd.c | 2
drivers/s390/block/dasd_int.h | 1
drivers/s390/block/xpram.c | 6
drivers/s390/char/monreader.c | 1
drivers/s390/char/tty3270.c | 3
drivers/s390/crypto/z90main.c | 1
drivers/s390/net/ctctty.c | 2
drivers/sbus/char/bpp.c | 9
drivers/sbus/char/vfc.h | 2
drivers/sbus/char/vfc_dev.c | 7
drivers/scsi/osst.c | 24
drivers/scsi/scsi.c | 3
drivers/scsi/scsi_scan.c | 6
drivers/scsi/sd.c | 2
drivers/scsi/sg.c | 10
drivers/scsi/sr.c | 2
drivers/scsi/st.c | 20
drivers/serial/21285.c | 1
drivers/serial/8250.c | 1
drivers/serial/at91_serial.c | 1
drivers/serial/au1x00_uart.c | 1
drivers/serial/crisv10.c | 2
drivers/serial/dz.c | 4
drivers/serial/imx.c | 1
drivers/serial/ip22zilog.c | 1
drivers/serial/m32r_sio.c | 1
drivers/serial/mcfserial.c | 1
drivers/serial/mpc52xx_uart.c | 1
drivers/serial/mpsc.c | 2
drivers/serial/pmac_zilog.c | 1
drivers/serial/pxa.c | 1
drivers/serial/s3c2410.c | 2
drivers/serial/sa1100.c | 1
drivers/serial/serial_core.c | 5
drivers/serial/serial_txx9.c | 3
drivers/serial/sh-sci.c | 3
drivers/serial/sunsab.c | 1
drivers/serial/sunsu.c | 1
drivers/serial/sunzilog.c | 1
drivers/serial/v850e_uart.c | 1
drivers/serial/vr41xx_siu.c | 1
drivers/tc/zs.c | 3
drivers/telephony/phonedev.c | 4
drivers/usb/class/cdc-acm.c | 3
drivers/usb/gadget/serial.c | 3
drivers/usb/serial/usb-serial.c | 3
drivers/video/fbmem.c | 5
fs/Makefile | 1
fs/block_dev.c | 1
fs/char_dev.c | 1
fs/coda/psdev.c | 23
fs/compat_ioctl.c | 1
fs/devfs/Makefile | 8
fs/devfs/base.c | 2836 ----------------------
fs/devfs/util.c | 97
fs/partitions/Makefile | 1
fs/partitions/check.c | 32
fs/partitions/devfs.c | 130 -
fs/partitions/devfs.h | 10
include/asm-ppc/ocp.h | 1
include/linux/compat_ioctl.h | 5
include/linux/devfs_fs.h | 41
include/linux/devfs_fs_kernel.h | 58
include/linux/fb.h | 1
include/linux/genhd.h | 2
include/linux/ide.h | 1
include/linux/miscdevice.h | 1
include/linux/serial_core.h | 1
include/linux/tty_driver.h | 14
include/linux/videodev2.h | 1
include/scsi/scsi_device.h | 1
init/Makefile | 1
init/do_mounts.c | 8
init/do_mounts.h | 16
init/do_mounts_devfs.c | 137 -
init/do_mounts_initrd.c | 6
init/do_mounts_md.c | 7
init/do_mounts_rd.c | 4
init/main.c | 1
mm/shmem.c | 5
mm/tiny-shmem.c | 4
net/bluetooth/rfcomm/tty.c | 3
net/irda/ircomm/ircomm_tty.c | 1
net/irda/irnet/irnet.h | 1
sound/core/info.c | 1
sound/core/sound.c | 24
sound/oss/soundcard.c | 16
sound/sound_core.c | 6
198 files changed, 82 insertions(+), 8331 deletions(-)

---------------

Greg Kroah-Hartman:
devfs: Remove devfs from the kernel tree
devfs: Remove devfs documentation from the kernel tree
devfs: Remove devfs from the partition code
devfs: Remove devfs from the init code
devfs: Remove devfs support from the serial subsystem
devfs: Remove devfs support from the SCSI subsystem
devfs: Remove devfs support from the ide subsystem.
devfs: Remove devfs support from the IEE1394 subsystem
devfs: Remove devfs_*_tape() functions from the kernel tree
devfs: Remove devfs_mk_dir() function from the kernel tree
devfs: Remove devfs_mk_symlink() function from the kernel tree
devfs: Remove devfs_mk_bdev() function from the kernel tree
devfs: Remove devfs_mk_cdev() function from the kernel tree
devfs: Remove devfs_remove() function from the kernel tree
devfs: Remove the devfs_fs_kernel.h file from the tree
devfs: Remove the miscdevice devfs_name field as it's no longer needed
devfs: Remove the gendisk devfs_name field as it's no longer needed
devfs: Remove the videodevice devfs_name field as it's no longer needed
devfs: Remove the line_driver devfs_name field as it's no longer needed
devfs: Remove the tty_driver devfs_name field as it's no longer needed
devfs: Rename TTY_DRIVER_NO_DEVFS to TTY_DRIVER_DYNAMIC_DEV
devfs: Last little devfs cleanups throughout the kernel tree.


2006-03-20 21:30:28

by Randy Dunlap

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Mon, 20 Mar 2006 13:23:38 -0800 Greg KH wrote:

> They are the same "delete devfs" patches that I submitted for 2.6.12 and
> 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
> and ends up saving a lot of space. Since 2.6.13 came out, I have seen
> no complaints about the fact that devfs was not able to be enabled
> anymore, and in fact, a lot of different subsystems have already been
> deleting devfs support for a while now, with apparently no complaints
> (due to the lack of users.)
>
> It's also been over 8 months past the date when we said we would delete
> devfs from the kernel tree in the file,
> Documentation/feature-removal-schedule.txt, and over one and one half
> years since we publicly announced to the world that devfs would be
> removed from the kernel tree. So, I think people have had plenty of
> advance notice that this was going to happen by now :)
>
> Please pull from:
> rsync://rsync.kernel.org/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
> or if master.kernel.org hasn't synced up yet:
> master.kernel.org:/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
>
> I've posted all of these patches before, but if people really want to look at them, they can be found at:
> http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-05-devfs/
>
> Also, if people _really_ are in love with the idea of an in-kernel
> devfs, I have posted a patch that does this in about 300 lines of code,
> called ndevfs. It is available in the archives if anyone wants to use
> that instead (it is quite easy to maintain that patch outside of the
> kernel tree, due to it only needing 3 hooks into the main kernel tree.)
>
> thanks,
>
> greg k-h
>
>
> Documentation/DocBook/kernel-api.tmpl | 5
> Documentation/filesystems/devfs/ChangeLog | 1977 ---------------
> Documentation/filesystems/devfs/README | 1959 ---------------
> Documentation/filesystems/devfs/ToDo | 40
> Documentation/filesystems/devfs/boot-options | 65

More Documentation/ references to /devfs/i :

Changes
computone.txt
feature-removal-schedule.txt
initrd.txt
ioctl-number.txt
kernel-docs.txt
kernel-parameters.txt
README.DAC960


---
~Randy

2006-03-20 21:34:49

by H. Peter Anvin

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

Followup to: <[email protected]>
By author: "Randy.Dunlap" <[email protected]>
In newsgroup: linux.dev.kernel
> ioctl-number.txt

Do not remove from ioctl-number.txt; those ioctl numbers should not be
reused.

-hpa

2006-03-20 21:38:00

by Randy Dunlap

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Mon, 20 Mar 2006 13:34:07 -0800 (PST) H. Peter Anvin wrote:

> Followup to: <[email protected]>
> By author: "Randy.Dunlap" <[email protected]>
> In newsgroup: linux.dev.kernel
> > ioctl-number.txt
>
> Do not remove from ioctl-number.txt; those ioctl numbers should not be
> reused.

Ack. I didn't mean that they should all be removed, but at
least reviewed/considered.

---
~Randy

2006-03-20 22:03:42

by Greg KH

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Mon, Mar 20, 2006 at 01:32:30PM -0800, Randy.Dunlap wrote:
> More Documentation/ references to /devfs/i :
>
> Changes
> computone.txt
> feature-removal-schedule.txt
> initrd.txt
> ioctl-number.txt
> kernel-docs.txt
> kernel-parameters.txt
> README.DAC960

Yes, some of the surrounding documentation might still need to be
cleaned up. I'll do that in a few days (after I push out my other
trees...)

thanks,

greg k-h

2006-03-20 23:58:56

by Olivier Galibert

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Mon, Mar 20, 2006 at 01:23:38PM -0800, Greg KH wrote:
> They are the same "delete devfs" patches that I submitted for 2.6.12 and
> 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
> and ends up saving a lot of space. Since 2.6.13 came out, I have seen
> no complaints about the fact that devfs was not able to be enabled
> anymore, and in fact, a lot of different subsystems have already been
> deleting devfs support for a while now, with apparently no complaints
> (due to the lack of users.)

I'm an occasional user. I'm just able to add a config entry by hand.

Devfs for block devices is required for the fedora core 3 install
kernel. I haven't checked whether fc4 needs it too (DaveJ?), but if
it is the case it would be a real bad idea to remove it before fc6 is
out.

I know, I know, compatibility is for the weak.

OG.

2006-03-21 00:05:36

by Dave Jones

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Tue, Mar 21, 2006 at 12:58:46AM +0100, Olivier Galibert wrote:
> On Mon, Mar 20, 2006 at 01:23:38PM -0800, Greg KH wrote:
> > They are the same "delete devfs" patches that I submitted for 2.6.12 and
> > 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
> > and ends up saving a lot of space. Since 2.6.13 came out, I have seen
> > no complaints about the fact that devfs was not able to be enabled
> > anymore, and in fact, a lot of different subsystems have already been
> > deleting devfs support for a while now, with apparently no complaints
> > (due to the lack of users.)
>
> Devfs for block devices is required for the fedora core 3 install
> kernel. I haven't checked whether fc4 needs it too (DaveJ?)

Hell no. It's never been included in any Fedora release.
Make it dead already.

Dave

--
http://www.codemonkey.org.uk

2006-03-21 07:15:35

by Arjan van de Ven

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Tue, 2006-03-21 at 00:58 +0100, Olivier Galibert wrote:
> On Mon, Mar 20, 2006 at 01:23:38PM -0800, Greg KH wrote:
> > They are the same "delete devfs" patches that I submitted for 2.6.12 and
> > 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
> > and ends up saving a lot of space. Since 2.6.13 came out, I have seen
> > no complaints about the fact that devfs was not able to be enabled
> > anymore, and in fact, a lot of different subsystems have already been
> > deleting devfs support for a while now, with apparently no complaints
> > (due to the lack of users.)
>
> I'm an occasional user. I'm just able to add a config entry by hand.
>
> Devfs for block devices is required for the fedora core 3 install
> kernel.

that is not true; Fedora Core 3 does not even have devfs enabled, and
neither RHL nor FC has shipped devfsd like forever

FC3 uses udev


2006-03-21 12:48:01

by Olivier Galibert

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Tue, Mar 21, 2006 at 08:15:22AM +0100, Arjan van de Ven wrote:
> On Tue, 2006-03-21 at 00:58 +0100, Olivier Galibert wrote:
> > On Mon, Mar 20, 2006 at 01:23:38PM -0800, Greg KH wrote:
> > > They are the same "delete devfs" patches that I submitted for 2.6.12 and
> > > 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
> > > and ends up saving a lot of space. Since 2.6.13 came out, I have seen
> > > no complaints about the fact that devfs was not able to be enabled
> > > anymore, and in fact, a lot of different subsystems have already been
> > > deleting devfs support for a while now, with apparently no complaints
> > > (due to the lack of users.)
> >
> > I'm an occasional user. I'm just able to add a config entry by hand.
> >
> > Devfs for block devices is required for the fedora core 3 install
> > kernel.
>
> that is not true; Fedora Core 3 does not even have devfs enabled, and
> neither RHL nor FC has shipped devfsd like forever
>
> FC3 uses udev

The _install_ kernel does not use udev. It uses a mix of static /dev
and whatever it is which creates the block devices, but after checking
is not devfs after all, sorry about the mistake.

OG.

2006-03-23 03:32:32

by Kalin KOZHUHAROV

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

Greg KH wrote:
> They are the same "delete devfs" patches that I submitted for 2.6.12 and
> 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
> and ends up saving a lot of space. Since 2.6.13 came out, I have seen
> no complaints about the fact that devfs was not able to be enabled
> anymore, and in fact, a lot of different subsystems have already been
> deleting devfs support for a while now, with apparently no complaints
> (due to the lack of users.)
>
> It's also been over 8 months past the date when we said we would delete
> devfs from the kernel tree in the file,
> Documentation/feature-removal-schedule.txt, and over one and one half
> years since we publicly announced to the world that devfs would be
> removed from the kernel tree. So, I think people have had plenty of
> advance notice that this was going to happen by now :)
>
> Please pull from:
> rsync://rsync.kernel.org/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
> or if master.kernel.org hasn't synced up yet:
> master.kernel.org:/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
>
> I've posted all of these patches before, but if people really want to look at them, they can be found at:
> http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-05-devfs/
>
> Also, if people _really_ are in love with the idea of an in-kernel
> devfs, I have posted a patch that does this in about 300 lines of code,
> called ndevfs. It is available in the archives if anyone wants to use
> that instead (it is quite easy to maintain that patch outside of the
> kernel tree, due to it only needing 3 hooks into the main kernel tree.)

OK, I completely agree with that, but shouldn't there be left something in
the Documentation at least? A note like "devfs was superseded by udev since
2.6.?? and was completely removed since 2.6.??" or something along the same
line of thought? It will just make life easier for people finding lots of
old pages and HOWTOs on the Net mentioning devfs.

Unfortunately I couldn't find a proper place for that note...
Maybe a new file Documentation/filesystems/devfs.txt instead of the
Documentation/filesystems/devfs directory present now?


Kalin.

--
|[ ~~~~~~~~~~~~~~~~~~~~~~ ]|
+-> http://ThinRope.net/ <-+
|[ ______________________ ]|

2006-03-23 21:29:47

by Adrian Bunk

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

On Thu, Mar 23, 2006 at 12:33:13PM +0900, Kalin KOZHUHAROV wrote:
> Greg KH wrote:
> > They are the same "delete devfs" patches that I submitted for 2.6.12 and
> > 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
> > and ends up saving a lot of space. Since 2.6.13 came out, I have seen
> > no complaints about the fact that devfs was not able to be enabled
> > anymore, and in fact, a lot of different subsystems have already been
> > deleting devfs support for a while now, with apparently no complaints
> > (due to the lack of users.)
> >
> > It's also been over 8 months past the date when we said we would delete
> > devfs from the kernel tree in the file,
> > Documentation/feature-removal-schedule.txt, and over one and one half
> > years since we publicly announced to the world that devfs would be
> > removed from the kernel tree. So, I think people have had plenty of
> > advance notice that this was going to happen by now :)
> >
> > Please pull from:
> > rsync://rsync.kernel.org/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
> > or if master.kernel.org hasn't synced up yet:
> > master.kernel.org:/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
> >
> > I've posted all of these patches before, but if people really want to look at them, they can be found at:
> > http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-05-devfs/
> >
> > Also, if people _really_ are in love with the idea of an in-kernel
> > devfs, I have posted a patch that does this in about 300 lines of code,
> > called ndevfs. It is available in the archives if anyone wants to use
> > that instead (it is quite easy to maintain that patch outside of the
> > kernel tree, due to it only needing 3 hooks into the main kernel tree.)
>
> OK, I completely agree with that, but shouldn't there be left something in
> the Documentation at least? A note like "devfs was superseded by udev since
> 2.6.?? and was completely removed since 2.6.??" or something along the same
> line of thought? It will just make life easier for people finding lots of
> old pages and HOWTOs on the Net mentioning devfs.
>
> Unfortunately I couldn't find a proper place for that note...
> Maybe a new file Documentation/filesystems/devfs.txt instead of the
> Documentation/filesystems/devfs directory present now?

There's already a note in Documentation/Changes.

> Kalin.

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

2006-03-28 23:10:44

by Bill Davidsen

[permalink] [raw]
Subject: Re: [GIT PATCH] Remove devfs from 2.6.16

Adrian Bunk wrote:
> On Thu, Mar 23, 2006 at 12:33:13PM +0900, Kalin KOZHUHAROV wrote:
>> Greg KH wrote:
>>> They are the same "delete devfs" patches that I submitted for 2.6.12 and
>>> 2.6.13 and 2.6.14 and 2.6.15. It rips out all of devfs from the kernel
>>> and ends up saving a lot of space. Since 2.6.13 came out, I have seen
>>> no complaints about the fact that devfs was not able to be enabled
>>> anymore, and in fact, a lot of different subsystems have already been
>>> deleting devfs support for a while now, with apparently no complaints
>>> (due to the lack of users.)
>>>
>>> It's also been over 8 months past the date when we said we would delete
>>> devfs from the kernel tree in the file,
>>> Documentation/feature-removal-schedule.txt, and over one and one half
>>> years since we publicly announced to the world that devfs would be
>>> removed from the kernel tree. So, I think people have had plenty of
>>> advance notice that this was going to happen by now :)
>>>
>>> Please pull from:
>>> rsync://rsync.kernel.org/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
>>> or if master.kernel.org hasn't synced up yet:
>>> master.kernel.org:/pub/scm/linux/kernel/git/gregkh/devfs-2.6.git/
>>>
>>> I've posted all of these patches before, but if people really want to look at them, they can be found at:
>>> http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-05-devfs/
>>>
>>> Also, if people _really_ are in love with the idea of an in-kernel
>>> devfs, I have posted a patch that does this in about 300 lines of code,
>>> called ndevfs. It is available in the archives if anyone wants to use
>>> that instead (it is quite easy to maintain that patch outside of the
>>> kernel tree, due to it only needing 3 hooks into the main kernel tree.)
>> OK, I completely agree with that, but shouldn't there be left something in
>> the Documentation at least? A note like "devfs was superseded by udev since
>> 2.6.?? and was completely removed since 2.6.??" or something along the same
>> line of thought? It will just make life easier for people finding lots of
>> old pages and HOWTOs on the Net mentioning devfs.
>>
>> Unfortunately I couldn't find a proper place for that note...
>> Maybe a new file Documentation/filesystems/devfs.txt instead of the
>> Documentation/filesystems/devfs directory present now?
>
> There's already a note in Documentation/Changes.

I really don't think that's the most likely place people would look. The
suggested few lines in a file with a dead-obvious name would undoubtedly
save people time, as they would look there first.

No complaint about getting rid of the feature, but there are lots of
mentions of devfs, putting in a definitive explanation wouldn't be out
of place.

--
-bill davidsen ([email protected])
"The secret to procrastination is to put things off until the
last possible moment - but no longer" -me