[CRIU] [PATCH 07/13] mounts: share one root for all namespaces
Andrey Vagin
avagin at openvz.org
Tue Apr 8 17:13:35 PDT 2014
Currently we support nested mount namespaces, only if all of them have
the same root.
crtools doesn't mount root, so it should be bind-mounted for all
namespaces.
Signed-off-by: Andrey Vagin <avagin at openvz.org>
---
mount.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/mount.c b/mount.c
index 53e4ef6..2fd0cbd 100644
--- a/mount.c
+++ b/mount.c
@@ -242,6 +242,12 @@ static struct mount_info *mnt_build_ids_tree(struct mount_info *list)
m->mnt_id, m->parent_mnt_id, m->mountpoint,
root ? "found" : "not found");
if (root && m->is_root) {
+ if (!mounts_equal(root, m, true) ||
+ (strcmp(root->root, m->root))) {
+ pr_err("Nested mount namespaces with different roots are not supported yet");
+ return NULL;
+ }
+
/*
* A root of a sub mount namespace is
* mounted in a temporary directory in the
@@ -1079,9 +1085,10 @@ static int propagate_mount(struct mount_info *mi)
skip_parent:
/*
* FIXME Currently non-root mounts can be restored
- * only if a proper root mount exists
+ * only if a proper root mount exists.
+ * Here is one exception for sub-roots. Roots of all mntns are the same.
*/
- if (fsroot_mounted(mi))
+ if (mi->parent == NULL || fsroot_mounted(mi))
list_for_each_entry(t, &mi->mnt_bind, mnt_bind) {
if (t->bind)
continue;
--
1.8.5.3
More information about the CRIU
mailing list