[lxc-devel] [lxc/master] cgroups: initialize cgroup root directory

brauner on Github lxc-bot at linuxcontainers.org
Wed Aug 28 18:34:30 UTC 2019


A non-text attachment was scrubbed...
Name: not available
Type: text/x-mailbox
Size: 364 bytes
Desc: not available
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20190828/a252d4f0/attachment.bin>
-------------- next part --------------
From 6607d6e91d271ee493a72c1bc3b2589c77222909 Mon Sep 17 00:00:00 2001
From: Christian Brauner <christian.brauner at ubuntu.com>
Date: Wed, 28 Aug 2019 20:33:39 +0200
Subject: [PATCH] cgroups: initialize cgroup root directory

Signed-off-by: Christian Brauner <christian.brauner at ubuntu.com>
---
 src/lxc/cgroups/cgfsng.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/lxc/cgroups/cgfsng.c b/src/lxc/cgroups/cgfsng.c
index 44ec5a0bec..be1fd87733 100644
--- a/src/lxc/cgroups/cgfsng.c
+++ b/src/lxc/cgroups/cgfsng.c
@@ -1764,7 +1764,7 @@ __cgfsng_ops static bool cgfsng_mount(struct cgroup_ops *ops,
 				      struct lxc_handler *handler,
 				      const char *root, int type)
 {
-	__do_free char *tmpfspath = NULL;
+	__do_free char *cgroup_root = NULL;
 	int ret;
 	bool has_cgns = false, retval = false, wants_force_mount = false;
 
@@ -1796,23 +1796,21 @@ __cgfsng_ops static bool cgfsng_mount(struct cgroup_ops *ops,
 		type = LXC_AUTO_CGROUP_FULL_MIXED;
 
 	if (ops->cgroup_layout == CGROUP_LAYOUT_UNIFIED) {
-		__do_free char *unified_path = NULL;
-
-		unified_path = must_make_path(root, "/sys/fs/cgroup", NULL);
+		cgroup_root = must_make_path(root, "/sys/fs/cgroup", NULL);
 		if (has_cgns && wants_force_mount) {
 			/* If cgroup namespaces are supported but the container
 			 * will not have CAP_SYS_ADMIN after it has started we
 			 * need to mount the cgroups manually.
 			 */
 			return cg_mount_in_cgroup_namespace(type, ops->unified,
-							    unified_path) == 0;
+							    cgroup_root) == 0;
 		}
 
-		return cg_mount_cgroup_full(type, ops->unified, unified_path) == 0;
+		return cg_mount_cgroup_full(type, ops->unified, cgroup_root) == 0;
 	}
 
 	/* mount tmpfs */
-	ret = safe_mount(NULL, tmpfspath, "tmpfs",
+	ret = safe_mount(NULL, cgroup_root, "tmpfs",
 			 MS_NOSUID | MS_NODEV | MS_NOEXEC | MS_RELATIME,
 			 "size=10240k,mode=755", root);
 	if (ret < 0)
@@ -1827,7 +1825,7 @@ __cgfsng_ops static bool cgfsng_mount(struct cgroup_ops *ops,
 			continue;
 		controller++;
 
-		controllerpath = must_make_path(tmpfspath, controller, NULL);
+		controllerpath = must_make_path(cgroup_root, controller, NULL);
 		if (dir_exists(controllerpath))
 			continue;
 


More information about the lxc-devel mailing list