Re: [PATCH] Allow use of block devices for guest filesystem

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

 



On 08/04/2011 09:04 AM, Daniel P. Berrange wrote:
From: "Daniel P. Berrange"<berrange@xxxxxxxxxx>

Currently the LXC driver can only populate filesystems from
host filesystems, using bind mounts. This patch allows host
block devices to be mounted. It autodetects the filesystem
format at mount time, and adds the block device to the cgroups
ACL. Example usage is

     <filesystem type='block' accessmode='passthrough'>
       <source dev='/dev/sda1'/>
       <target dir='/home'/>
     </filesystem>

* src/lxc/lxc_container.c: Mount block device filesystems
* src/lxc/lxc_controller.c: Add block device filesystems
   to cgroups ACL
---
  src/lxc/lxc_container.c  |  179 +++++++++++++++++++++++++++++++++++++++++++++-
  src/lxc/lxc_controller.c |   17 +++++
  2 files changed, 195 insertions(+), 1 deletions(-)

+
+/*
+ * This functions attempts todo automatic detection of filesystem

s/todo/to do/

+ * type following the same rules as the util-linux 'mount' binary.
+ *
+ * The main difference is that we don't (currently) try to use
+ * libblkid to detect the format first. We go straight to using
+ * /etc/filesystems, and then /proc/filesystems
+ */
+static int lxcContainerMountFSBlockAuto(virDomainFSDefPtr fs,
+                                        int flags,
+                                        const char *src,
+                                        const char *srcprefix)
+{


-    ret = 0;
+    /* We've got to the end of /etc/filesystems and saw
+     * a '*', so we mhust try /proc/filesystems next

s/mhust/must/

ACK with those typo fixes.

--
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]