On Wed, Nov 22, 2000 at 04:27:51PM -0500, Brian Kress wrote:
> "Heinz J. Mauelshagen" wrote:
> >
> > On Wed, Nov 22, 2000 at 04:05:39PM -0500, Brian Kress wrote:
> > > Question about /proc/partitions and LVM. LVM devices in
> > > /proc/partitions currently show up as lvma, lvmb, etc, depending on
> > > their device number. This breaks things like mount by filesystem
> > > name. Back when LVM existed as a patch, I believe there was some
> > > support in fs/partitions/check.c for showing the proper name for
> > > the device (something like <vgname>/<lvname>).
> >
> > Yes.
> > It actually was <vgname>/<lvname>.
> >
> > Linus didn't accept it though.
> >
> > The code is still available in case he changes his mind.
>
> Yes, I can see that now looking through lvm.c. (LVM_HD_NAME)
> I'm guessing why he didn't take it is the minor hack of adding a
> function pointer to genhd.c as a special case just for LVM.
> A general solution to this problem would be nice. The
> big switch statement in disk_name in fs/paritions/check.c is kind
> of ugly as it has generic code have to know things about specific
> drivers.
Yep.
> How's this for an idea to generalize this. Put a function
> pointer in the gendisk structure for a specific function to call
> for disk_name for disks for that gendisk.
Sounds resonable to me.
> That way, LVM gets its
> <vgname>/<lvname>, IDE gets its two disks per major (special cased
> right now), all the other special cases get what they need and future
> drivers can call their devices whatever they like without touching
> this file. Makes the whole thing more modular. Make a NULL for
> this function default to the old <name>a, <name>b, etc.
> What to people think about this? If this sounds reasonable
> I can come up with a patch.
Very good. I'ld appreciate it.
>
>
> Brian Kress
> [EMAIL PROTECTED]
>
>
>
>
>
>
>
>
> >
> > > Are their any plans for something like this to be added
> > > or is their a reason it was taken out?
> > >
> > >
> > > BTW, 2.4.0-test11 is the first "perfect" kernel for me. It
> > > finally has everything I use working correctly. (well, with a
> > > small raid5 patch, anyway).
> > >
> > >
> > > Brian Kress
> > > [EMAIL PROTECTED]
> > > -
> > > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > > the body of a message to [EMAIL PROTECTED]
> > > Please read the FAQ at http://www.tux.org/lkml/
> >
> > --
> >
> > Regards,
> > Heinz -- The LVM guy --
> >
> > =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
> >
> > Heinz Mauelshagen Sistina Software Inc.
> > Senior Consultant/Developer Bartningstr. 12
> > 64289 Darmstadt
> > Germany
> > [EMAIL PROTECTED] +49 6151 7103 86
> > FAX 7103 96
> > =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
--
Regards,
Heinz -- The LVM guy --
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Heinz Mauelshagen Sistina Software Inc.
Senior Consultant/Developer Bartningstr. 12
64289 Darmstadt
Germany
[EMAIL PROTECTED] +49 6151 7103 86
FAX 7103 96
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
Please read the FAQ at http://www.tux.org/lkml/
