]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
crimson/osd: fix inaccurate comment about child early-exit in get_early_config 68093/head
authorKefu Chai <k.chai@proxmox.com>
Sun, 29 Mar 2026 11:41:24 +0000 (19:41 +0800)
committerKefu Chai <k.chai@proxmox.com>
Sun, 29 Mar 2026 11:55:02 +0000 (19:55 +0800)
The comment contained a typo ("taged") and vaguely referred to "one of
the parameters" without explaining what actually happens: the child
calls exit(0) for early-exit paths such as --help and --version, and
the parent detects this by checking for a clean exit with no pipe data.

Signed-off-by: Kefu Chai <k.chai@proxmox.com>
src/crimson/osd/main_config_bootstrap_helpers.cc

index 3454b5dbc3942d9a1e9953399fb8cf0e0e322dee..aafd52964a11365fc0b0b1f94ce7244d9bc963c1 100644 (file)
@@ -337,8 +337,7 @@ get_early_config(int argc, const char *argv[])
     int status;
     waitpid(worker, &status, 0);
 
-    // One of the parameters was taged as exit(0) in the child process
-    // so we need to check if we should exit here
+    // child exited via exit(0) for early-exit paths (e.g. --help, --version)
     if (!have_data && WIFEXITED(status) && WEXITSTATUS(status) == 0) {
       exit(0);
     }