From e8cd390d263cfd05b90db4f2c71b2f8e12fefe79 Mon Sep 17 00:00:00 2001 From: Balazs Lecz <leczb@google.com> Date: Mon, 5 Jul 2010 18:27:10 +0000 Subject: [PATCH] hv_chroot: use utils.GetMounts() Signed-off-by: Balazs Lecz <leczb@google.com> Reviewed-by: Iustin Pop <iustin@google.com> --- lib/hypervisor/hv_chroot.py | 24 ++++++++---------------- 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/lib/hypervisor/hv_chroot.py b/lib/hypervisor/hv_chroot.py index e9f2317f5..f33fed204 100644 --- a/lib/hypervisor/hv_chroot.py +++ b/lib/hypervisor/hv_chroot.py @@ -83,23 +83,15 @@ class ChrootManager(hv_base.BaseHypervisor): def _GetMountSubdirs(path): """Return the list of mountpoints under a given path. - This function is Linux-specific. - """ - #TODO(iustin): investigate and document non-linux options - #(e.g. via mount output) - data = [] - fh = open("/proc/mounts", "r") - try: - for line in fh: - _, mountpoint, _ = line.split(" ", 2) - if (mountpoint.startswith(path) and - mountpoint != path): - data.append(mountpoint) - finally: - fh.close() - data.sort(key=lambda x: x.count("/"), reverse=True) - return data + result = [] + for _, mountpoint, _, _ in utils.GetMounts(): + if (mountpoint.startswith(path) and + mountpoint != path): + result.append(mountpoint) + + result.sort(key=lambda x: x.count("/"), reverse=True) + return result @classmethod def _InstanceDir(cls, instance_name): -- GitLab