]>
Commit | Line | Data |
---|---|---|
1 | From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 | |
2 | From: Wolfgang Bumiller <w.bumiller@proxmox.com> | |
3 | Date: Wed, 28 Mar 2018 13:41:46 +0200 | |
4 | Subject: [PATCH] PVE: [Up] start/initutils: make cgroupns separation level | |
5 | configurable | |
6 | ||
7 | Adds a new global config variable `lxc.cgroup.separate` | |
8 | which controls whether a separation directory for cgroup | |
9 | namespaces should be used. | |
10 | Can be empty, "privileged", "unprivileged" or "both". | |
11 | ||
12 | Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com> | |
13 | --- | |
14 | src/lxc/initutils.c | 17 +++++++++-------- | |
15 | src/lxc/initutils.h | 1 + | |
16 | src/lxc/start.c | 25 ++++++++++++++----------- | |
17 | 3 files changed, 24 insertions(+), 19 deletions(-) | |
18 | ||
19 | diff --git a/src/lxc/initutils.c b/src/lxc/initutils.c | |
20 | index 11c808662..8b5e2542a 100644 | |
21 | --- a/src/lxc/initutils.c | |
22 | +++ b/src/lxc/initutils.c | |
23 | @@ -63,14 +63,15 @@ static char *copy_global_config_value(char *p) | |
24 | const char *lxc_global_config_value(const char *option_name) | |
25 | { | |
26 | static const char * const options[][2] = { | |
27 | - { "lxc.bdev.lvm.vg", DEFAULT_VG }, | |
28 | - { "lxc.bdev.lvm.thin_pool", DEFAULT_THIN_POOL }, | |
29 | - { "lxc.bdev.zfs.root", DEFAULT_ZFSROOT }, | |
30 | - { "lxc.bdev.rbd.rbdpool", DEFAULT_RBDPOOL }, | |
31 | - { "lxc.lxcpath", NULL }, | |
32 | - { "lxc.default_config", NULL }, | |
33 | - { "lxc.cgroup.pattern", NULL }, | |
34 | - { "lxc.cgroup.use", NULL }, | |
35 | + { "lxc.bdev.lvm.vg", DEFAULT_VG }, | |
36 | + { "lxc.bdev.lvm.thin_pool", DEFAULT_THIN_POOL }, | |
37 | + { "lxc.bdev.zfs.root", DEFAULT_ZFSROOT }, | |
38 | + { "lxc.bdev.rbd.rbdpool", DEFAULT_RBDPOOL }, | |
39 | + { "lxc.lxcpath", NULL }, | |
40 | + { "lxc.default_config", NULL }, | |
41 | + { "lxc.cgroup.pattern", NULL }, | |
42 | + { "lxc.cgroup.use", NULL }, | |
43 | + { "lxc.cgroup.protect_limits", DEFAULT_CGPROTECT }, | |
44 | { NULL, NULL }, | |
45 | }; | |
46 | ||
47 | diff --git a/src/lxc/initutils.h b/src/lxc/initutils.h | |
48 | index 6bf23a706..b542e6015 100644 | |
49 | --- a/src/lxc/initutils.h | |
50 | +++ b/src/lxc/initutils.h | |
51 | @@ -42,6 +42,7 @@ | |
52 | #define DEFAULT_THIN_POOL "lxc" | |
53 | #define DEFAULT_ZFSROOT "lxc" | |
54 | #define DEFAULT_RBDPOOL "lxc" | |
55 | +#define DEFAULT_CGPROTECT "privileged" | |
56 | ||
57 | #ifndef PR_SET_MM | |
58 | #define PR_SET_MM 35 | |
59 | diff --git a/src/lxc/start.c b/src/lxc/start.c | |
60 | index f3b29d6cd..1cf792aa2 100644 | |
61 | --- a/src/lxc/start.c | |
62 | +++ b/src/lxc/start.c | |
63 | @@ -1820,17 +1820,20 @@ static int lxc_spawn(struct lxc_handler *handler) | |
64 | TRACE("Set up legacy device cgroup controller limits"); | |
65 | ||
66 | if (cgns_supported()) { | |
67 | - if (!cgroup_ops->payload_create(cgroup_ops, handler, true)) { | |
68 | - ERROR("failed to create inner cgroup separation layer"); | |
69 | - goto out_delete_net; | |
70 | - } | |
71 | - if (!cgroup_ops->payload_enter(cgroup_ops, handler->pid, true)) { | |
72 | - ERROR("failed to enter inner cgroup separation layer"); | |
73 | - goto out_delete_net; | |
74 | - } | |
75 | - if (!cgroup_ops->chown(cgroup_ops, handler->conf, true)) { | |
76 | - ERROR("failed chown inner cgroup separation layer"); | |
77 | - goto out_delete_net; | |
78 | + const char *tmp = lxc_global_config_value("lxc.cgroup.protect_limits"); | |
79 | + if (!strcmp(tmp, "both") || !strcmp(tmp, wants_to_map_ids ? "unprivileged" : "privileged")) { | |
80 | + if (!cgroup_ops->payload_create(cgroup_ops, handler, true)) { | |
81 | + ERROR("failed to create inner cgroup separation layer"); | |
82 | + goto out_delete_net; | |
83 | + } | |
84 | + if (!cgroup_ops->payload_enter(cgroup_ops, handler->pid, true)) { | |
85 | + ERROR("failed to enter inner cgroup separation layer"); | |
86 | + goto out_delete_net; | |
87 | + } | |
88 | + if (!cgroup_ops->chown(cgroup_ops, handler->conf, true)) { | |
89 | + ERROR("failed chown inner cgroup separation layer"); | |
90 | + goto out_delete_net; | |
91 | + } | |
92 | } | |
93 | } | |
94 | ||
95 | -- | |
96 | 2.20.1 | |
97 |