Re: Fw: Re: Fw: [DAC960] 2.6.0-test1: device files wrong in devfs

From: Andrey Borzenkov (arvidjaar_at_mail.ru)
Date: 08/30/03

  • Next message: Marcelo Tosatti: "Re: Freeze with HPT370 2.4.22-rc2 and dxr3" 
    To: Andrew Morton <akpm@osdl.org>
Date:	Sun, 31 Aug 2003 00:15:53 +0400

    On Saturday 30 August 2003 12:16, Andrew Morton wrote:
    > just going through my inbox...
    >
    >
    > Begin forwarded message:
    >
    > Date: Mon, 28 Jul 2003 22:14:45 -0700
    > From: Dave Olien <dmo@osdl.org>
    > To: Andrew Morton <akpm@osdl.org>
    > Cc: lkml@gert.robben.nu, Christoph Hellwig <hch@lst.de>
    > Subject: Re: Fw: [DAC960] 2.6.0-test1: device files wrong in devfs
    >
    >
    >
    > I just looked a little more closely at more of the drivers.
    > The following are OK. They call devfs_mk_dir() and devfs_mk_bdev()
    > directly to make entries in devfs.
    >
    > floppy98.c, floppy.c, umem.c, xd.c
    >
    > This leaves these that don't seem to make devfs entries:
    >
    > cciss.c and amiflop.c, ataflop.c, z2ram.c, acsi.c, ps2esdi.c
    >
    > A quick fix might be to just add initialization of devfs_name following
    > the init of disk_name, as I did in the DAC960.c
    >

    this is not quick fix, this is the only right fix. The only problem is what to
    set devfs_name to :)

    the attached patch suggests some possible names for non-floppy devices based
    on reading driver source. I have to ask if these make sense. At least for
    cciss Mandrake devfsd patch expects different names but it seems to be
    mistake (it assumes single controller always)

    For floppy it is not as simple. Floppy cannot use genhd and must create names
    manually; but I do not know what names are appropriate or expected.

    For acsi the target/lun name may have problem of creating compat names (if
    any) by devfsd.

    Please note that none of them created any devfs name under 2.4 as well. So it
    is not a regression ...

    -andrey

    > On Mon, Jul 28, 2003 at 06:35:28PM -0700, Dave Olien wrote:
    > > At quick glance, other drivers that look suspicious under drivers/block:
    > >
    > > acsi.c --- driver for Atari hard disks
    > > ataflop.c
    > > amiflop.c
    > > floppy98.c
    > > floppy.c
    > > ps2esdi.c
    > > z2ram.c
    > > umem.c
    > > xd.c
    > >
    > > A lot of these may be obsolete drivers, although they have been
    > > updated to use the new gendisk structures.
    > >
    > > On Mon, Jul 28, 2003 at 06:21:00PM -0700, Andrew Morton wrote:
    > > > I assume that IDE gets it right. And scsi appears to do it centrally.
    > > >
    > > > cciss.c appears to not be setting devfs_name. Christoph, could you
    > > > please suggest a fix?

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

  • Next message: Marcelo Tosatti: "Re: Freeze with HPT370 2.4.22-rc2 and dxr3"