Re: [RFC PATCH 9/9] Add NBD support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2009-06-01 at 06:55 -0400, David Dillow wrote:
> On Mon, 2009-06-01 at 04:41 -0500, Victor Lowther wrote:
> > On Mon, 2009-06-01 at 00:59 -0400, David Dillow wrote:
> 
> > > +# XXX netroot expects to have the handler mount things, but we should
> > > +# XXX allow LVM, LUKS, etc over nbd
> > > +
> > > +nbd-client $preopts "$nbdserver" "$nbdport" /dev/nbd0 $opts || exit 1
> > 
> > Ah, but we are not really a netroot in the sense that NFS is -- all we
> > are doing is attaching a block device.  Once it is attached the usual
> > root-on-block-device rules should kick in as soon as udev sees the block
> > devices.
> 
> Exactly, hence the comment -- and one elsewhere about wanting to move
> the mount to the network block devices via udev. Unfortunately, NBD does
> all of its device adds at module load time, so we cannot trigger a mount
> on those as there's nothing behind the device.

ew, narsty.  Sounds just as annoying as the loopback device.

> I was thinking about 'echo add /path/to/nbd0/uevent' here so we could
> force another loop through udev.
> 
> > > +if ! mount -t $nbdfstype -o$fsopts /dev/nbd0 $NEWROOT; then
> > > +    # Mount failed, clean up after ourselves so if we try a different
> > > +    # interface it can succeed
> > > +    nbd-client -d /dev/nbd0
> > > +    exit 1
> > > +fi
> > 
> > why not just poke udev with udevadm trigger if there is any doubt?
> 
> Not sure what you mean by that in this context -- do you mean the root
> mount via udev as discussed above, or the cleanup right here? 

The root mount.  

> I'm not sure how to do the cleanup when mounting via udev; I'd need to
> think.

Perhaps a mount-failed hook?

> 
-- 
Victor Lowther
RHCE# 805008539634727
LPIC-2# LPI000140019

--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux