]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
service: add explanatory comments to control_pid_good() and cgroup_good()
authorLennart Poettering <lennart@poettering.net>
Mon, 2 Oct 2017 10:57:03 +0000 (12:57 +0200)
committerLennart Poettering <lennart@poettering.net>
Mon, 2 Oct 2017 10:58:42 +0000 (12:58 +0200)
Let's add a similar comment to each as we already have for
main_pid_good(), emphasizing that these functions are supposed to be
have very similar.

src/core/service.c

index b6f929051e211cf790dc3cbca7ad4352c45681c1..56e07de69e009d9f553ad5426b4f2b0c93ebf9dc 100644 (file)
@@ -1425,6 +1425,10 @@ static int main_pid_good(Service *s) {
 _pure_ static int control_pid_good(Service *s) {
         assert(s);
 
+        /* Returns 0 if the control PID is dead, > 0 if it is good. We never actually return < 0 here, but in order to
+         * make this function as similar as possible to main_pid_good() and cgroup_good(), we pretend that < 0 also
+         * means: we can't figure it out. */
+
         return s->control_pid > 0;
 }
 
@@ -1433,6 +1437,9 @@ static int cgroup_good(Service *s) {
 
         assert(s);
 
+        /* Returns 0 if the cgroup is empty or doesn't exist, > 0 if it is exists and is populated, < 0 if we can't
+         * figure it out */
+
         if (!UNIT(s)->cgroup_path)
                 return 0;