Skip to content

Commit 5d103e5

Browse files
authored
Fixes #146: cgroups_mem_max unit in config.proto
This commit fixes the incorrect cgroups_mem_max unit described in a config.proto comment. We do not perform any calculations on this value and we don't specify the values unit (k/M/G) when writing to memory cgroup controller files, so the value is specified in bytes.
1 parent f755488 commit 5d103e5

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

config.proto

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,7 @@ message NsJailConfig {
196196
optional bool seccomp_log = 60 [default = false];
197197

198198
/* If > 0, maximum cumulative size of RAM used inside any jail */
199-
optional uint64 cgroup_mem_max = 61 [default = 0]; /* In MiB */
199+
optional uint64 cgroup_mem_max = 61 [default = 0]; /* In bytes */
200200
/* Mount point for cgroups-memory in your system */
201201
optional string cgroup_mem_mount = 62 [default = "/sys/fs/cgroup/memory"];
202202
/* Writeable directory (for the nsjail user) under cgroup_mem_mount */

0 commit comments

Comments
 (0)