[CRIU] [PATCH v2 21/36] ns: Write/read ns entries in new way

Kirill Tkhai ktkhai at virtuozzo.com
Fri Feb 3 08:14:43 PST 2017


The patch introduces generic way for dumping all the namespaces
in a generic way (currently, only user ns entries are dumped).

Handler for old user ns images is remained on its place.

Signed-off-by: Kirill Tkhai <ktkhai at virtuozzo.com>
---
 criu/namespaces.c |  182 ++++++++++++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 173 insertions(+), 9 deletions(-)

diff --git a/criu/namespaces.c b/criu/namespaces.c
index d22f36415..520ff2ca9 100644
--- a/criu/namespaces.c
+++ b/criu/namespaces.c
@@ -1127,7 +1127,6 @@ static int dump_user_ns(struct ns_id *ns)
 	int ret, exit_code = -1;
 	pid_t pid = ns->ns_pid;
 	UsernsEntry *e = ns->user.e;
-	struct cr_img *img;
 
 	ret = parse_id_map(pid, "uid_map", &e->uid_map);
 	if (ret < 0)
@@ -1142,14 +1141,6 @@ static int dump_user_ns(struct ns_id *ns)
 	if (check_user_ns(ns))
 		return -1;
 
-	img = open_image(CR_FD_USERNS, O_DUMP, ns->id);
-	if (!img)
-		goto err;
-	ret = pb_write_one(img, e, PB_USERNS);
-	close_image(img);
-	if (ret < 0)
-		goto err;
-
 	return 0;
 err:
 	if (e->uid_map) {
@@ -1733,6 +1724,172 @@ static int read_old_user_ns_img(void)
 	return 0;
 }
 
+static int dump_ns_hookups(int for_dump)
+{
+	struct cr_img *img;
+	struct ns_id *ns;
+	NsHookupEntry e;
+	PidnsEntry p;
+	NetnsEntry2 n;
+	int ret = 0;
+
+	if (!for_dump)
+		return 0;
+
+	img = open_image(CR_FD_NS_HOOKUP, O_DUMP);
+	if (!img)
+		return -1;
+
+	for (ns = ns_ids; ns != NULL; ns = ns->next) {
+		if (ns->nd != &user_ns_desc &&
+		    ns->nd != &pid_ns_desc &&
+		    ns->nd != &net_ns_desc)
+			continue;
+		if (ns->type == NS_CRIU ||
+		    !(root_ns_mask & ns->nd->cflag))
+			continue;
+
+		ns_hookup_entry__init(&e);
+		e.id = ns->id;
+		e.ns_cflag = ns->nd->cflag;
+		if (ns->parent) {
+			e.has_parent_id = true;
+			e.parent_id = ns->parent->id;
+		}
+
+		if (ns->nd == &user_ns_desc) {
+			e.user_ext = ns->user.e;
+		} else if (ns->nd == &pid_ns_desc) {
+			pidns_entry__init(&p);
+			p.userns_id = ns->user_ns->id;
+			e.pid_ext = &p;
+		} else {
+			netns_entry2__init(&n);
+			n.userns_id = ns->user_ns->id;
+			e.net_ext = &n;
+		}
+		ret = pb_write_one(img, &e, PB_NS_HOOKUP);
+		if (ret < 0) {
+			pr_err("Can't write ns-hookup.img\n");
+			break;
+		}
+	}
+
+	close_image(img);
+	return ret;
+}
+
+struct delayed_ns {
+	struct ns_id *ns;
+	int userns_id;
+};
+
+static int read_ns_hookups(void)
+{
+	struct ns_id *ns, *p_ns, *u_ns;
+	struct delayed_ns *d_ns = NULL;
+	struct pstree_item fake;
+	NsHookupEntry *e = NULL;
+	int ret = 0, nr_d = 0;
+	struct ns_desc *desc;
+	struct cr_img *img;
+	struct pid pid;
+	u32 u_id;
+
+	pid.ns[0].virt = -1;
+	fake.pid = &pid;
+
+	img = open_image(CR_FD_NS_HOOKUP, O_RSTR);
+	if (!img)
+		return -1;
+	if (empty_image(img))
+		goto close;
+
+	while (1) {
+		ret = pb_read_one_eof(img, &e, PB_NS_HOOKUP);
+		if (ret <= 0)
+			break;
+		ret = -1;
+		desc = &pid_ns_desc;
+		if (e->ns_cflag == CLONE_NEWUSER) {
+			desc = &user_ns_desc;
+			/*
+			 * user_ns hierarhies may contain holes
+			 * (a ns, which is not connected to a task),
+			 * while pid_ns should already be populated.
+			 */
+			if (rst_add_ns_id(e->id, &fake, desc)) {
+				pr_err("Can't add user ns\n");
+				break;
+			}
+		} else if (e->ns_cflag == CLONE_NEWNET)
+			desc = &net_ns_desc;
+
+		ns = lookup_ns_by_id(e->id, desc);
+		if (!ns) {
+			pr_err("Can't find ns %d\n", e->id);
+			break;
+		}
+
+		if (e->user_ext && e->ns_cflag == CLONE_NEWUSER) {
+			ns->user.e = dup_userns_entry(e->user_ext);
+			if (!ns->user.e) {
+				pr_err("Can't dup map\n");
+				goto close;
+			}
+		} else if ((e->pid_ext && e->ns_cflag == CLONE_NEWPID) ||
+			   (e->net_ext && e->ns_cflag == CLONE_NEWNET)) {
+			u_id = e->pid_ext ? e->pid_ext->userns_id : e->net_ext->userns_id;
+			u_ns = lookup_ns_by_id(u_id, &user_ns_desc);
+			if (!u_ns) {
+				/* User_ns hasn't read yet; set aside this ns */
+				d_ns = xrealloc(d_ns, (nr_d + 1) * sizeof(*d_ns));
+				if (!d_ns)
+					goto close;
+				d_ns[nr_d].ns = ns;
+				d_ns[nr_d++].userns_id = e->pid_ext->userns_id;
+			} else
+				ns->user_ns = u_ns;
+		} else {
+			pr_err("No ns ext\n");
+			break;
+		}
+
+		if (e->has_parent_id) {
+			p_ns = lookup_ns_by_id(e->parent_id, desc);
+			if (!p_ns) {
+				pr_err("Can't find root ns %d\n", e->parent_id);
+				break;
+			}
+			ns->parent = p_ns;
+			list_add(&ns->siblings, &p_ns->children);
+		} else if (e->ns_cflag == CLONE_NEWUSER) {
+			ns->type = NS_ROOT;
+			root_user_ns = ns;
+			userns_entry = ns->user.e;
+		}
+
+		ns_hookup_entry__free_unpacked(e, NULL);
+	}
+close:
+	if (!ret) {
+		while (nr_d-- > 0) {
+			u_ns = lookup_ns_by_id(d_ns[nr_d].userns_id, &user_ns_desc);
+			if (!u_ns) {
+				pr_err("Can't find user_ns\n");
+				ret = -1;
+				break;
+			}
+			d_ns[nr_d].ns->user_ns = u_ns;
+		}
+	}
+	if (ret)
+		pr_err("Failed to read ns image\n");
+	xfree(d_ns);
+	close_image(img);
+	return ret;
+}
+
 int prepare_userns(struct pstree_item *item)
 {
 	UsernsEntry *e = userns_entry;
@@ -1766,6 +1923,10 @@ int collect_namespaces(bool for_dump)
 	if (ret < 0)
 		return ret;
 
+	ret = dump_ns_hookups(for_dump);
+	if (ret < 0)
+		return ret;
+
 	return 0;
 }
 
@@ -1962,6 +2123,9 @@ int prepare_namespace_before_tasks(void)
 	if (read_mnt_ns_img())
 		goto err_img;
 
+	if (read_ns_hookups())
+		goto err_img;
+
 	if (read_old_user_ns_img())
 		goto err_img;
 



More information about the CRIU mailing list