]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
cgroup-util: fix the reversed return value of cg_is_unified_systemd_contoller_wanted
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 20 Feb 2017 17:26:53 +0000 (12:26 -0500)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Mon, 20 Feb 2017 17:26:53 +0000 (12:26 -0500)
1d84ad944520fc3e062ef518c4db4e1 reversed the meaning of the option.
The kernel command line option has the opposite meaning to the function,
i.e. specifying "legacy=yes" means "unifed systemd controller=no".

src/basic/cgroup-util.c

index 67c2d3a681e7499bfd0cc86c33b6d8c71f1259fe..b5ca10a2de7f9eefbf8e765aae9b9fdc83f36940 100644 (file)
@@ -2378,7 +2378,9 @@ bool cg_is_unified_systemd_controller_wanted(void) {
         if (r < 0)
                 return false;
 
-        return (wanted = r > 0 ? b : false);
+        /* The meaning of the kernel option is reversed wrt. to the return value
+         * of this function, hence the negation. */
+        return (wanted = r > 0 ? !b : false);
 }
 
 bool cg_is_legacy_systemd_controller_wanted(void) {