Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 16 Feb 2008 20:26:03 +1300
From:      Andrew Thompson <thompsa@FreeBSD.org>
To:        Poul-Henning Kamp <phk@phk.freebsd.dk>
Cc:        Dag-Erling =?unknown-8bit?B?U23DuHJncmF2?= <des@des.no>, src-committers@FreeBSD.org, cvs-all@FreeBSD.org, obrien@FreeBSD.org, cvs-src@FreeBSD.org
Subject:   Re: cvs commit: src/sys/geom/lvm g_lvm.c g_lvm.h src/share/man/man4 geom_lvm.4
Message-ID:  <20080216072603.GD2144@heff.fud.org.nz>
In-Reply-To: <94261.1203122076@critter.freebsd.dk>
References:  <20080215235901.GA11610@dragon.NUXI.org> <94261.1203122076@critter.freebsd.dk>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Feb 16, 2008 at 12:34:36AM +0000, Poul-Henning Kamp wrote:
> In message <20080215235901.GA11610@dragon.NUXI.org>, "David O'Brien" writes:
> >On Sat, Feb 16, 2008 at 12:42:29AM +0100, Dag-Erling Smrgrav wrote:
> >> Andrew Thompson <thompsa@FreeBSD.org> writes:
> >> >   Added files:
> >> >     sys/geom/lvm         g_lvm.c g_lvm.h 
> >> >     share/man/man4       geom_lvm.4 
> >> >   Log:
> >> >   Add a geom class to map Linux LVM logical volumes.
> >> 
> >> I really wish you had chosen a less generic name...
> >
> >Its not too late to change the name.
> 
> I agree, it should be linux_lvm unless the Linux camp has a
> more precise name for it.

Fine by me. Just to be sure the correct module and manpage names should
be geom_linux_lvm?


Andrew



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20080216072603.GD2144>