Re: cvs commit: src/sbin/mdconfig mdconfig.8

From: Ceri Davies <ceri_at_submonkey.net>
Date: Fri, 9 Nov 2007 22:27:27 +0000
On Sat, Nov 10, 2007 at 12:05:51AM +0200, Giorgos Keramidas wrote:
> On 2007-11-09 21:34, Ceri Davies <ceri_at_submonkey.net> wrote:
> > On Fri, Nov 09, 2007 at 06:02:52PM +0000, Giorgos Keramidas wrote:
> > > keramida    2007-11-09 18:02:51 UTC
> > >
> > >   FreeBSD src repository (doc committer)
> > >
> > >   Modified files:
> > >     sbin/mdconfig        mdconfig.8
> > >   Log:
> > >   Instead of hardcoding md10 as the device node in the cd9660 example,
> > >   use a trick submitted by Ruslan.
> >
> > > | _at__at_ -218,8 +218,7 _at__at_ mount /dev/md0c /mnt
> > > |  .Pp
> > > |  To mount an ISO 9660 CD image file:
> > > |  .Bd -literal -offset indent
> > > | -mdconfig -a -t vnode -u 10 -f cdimage.iso
> > > | -mount_cd9660 /dev/md10 /mnt
> > > | +mount -t cd9660 /dev/`mdconfig -a -t vnode -f cdimage.iso` /mnt
> > > |  .Ed
> > > |  .Sh SEE ALSO
> > > |  .Xr md 4 ,
> >
> > I'm concerned that this actually misses the point, which is to create
> > an md device backed by a .iso file.
> 
> Similar concerns have been raised by bde a while ago.  I have a slightly
> improved patch now, which describes *all* the steps of the example.
> 
> This one actually *notes* that the mdconfig command creates an md(4)
> device, and then mounts it, instead of saying that we 'mount an iso'.

Thanks!  I like this much better, but have one little nit to pick:

> -To mount an ISO 9660 CD image file:
> +To create an
> +.Xr md 4
> +disk out of an ISO 9660 CD image file

Can you use "device" rather than "disk" here please?

> +.Ns ( Fl a
> +and
> +.Fl t Ar vnode
> +are implied), using the first available
> +.Xr md 4
> +device node, and then mount the new memory disk:

Ditto.

Thanks!

Ceri
-- 
That must be wonderful!  I don't understand it at all.
                                                  -- Moliere

Received on Fri Nov 09 2007 - 22:27:35 UTC