From: Kefu Chai <k.chai@proxmox.com>
Date: Sun, 29 Mar 2026 11:41:48 +0000 (+0800)
Subject: crimson/osd: remove unnecessary named string for --smp value
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=cf8eafb6851346dae8879b09b62fbdffa832d9bc;p=ceph.git

crimson/osd: remove unnecessary named string for --smp value

The local variable `smp` is used only in the two immediately following
statements. Inline the fmt::format() call into emplace_back() and pass
reactor_num directly to the logger.

Signed-off-by: Kefu Chai <k.chai@proxmox.com>
---

diff --git a/src/crimson/osd/main_config_bootstrap_helpers.cc b/src/crimson/osd/main_config_bootstrap_helpers.cc
index 614e61773e38..b701247ae7ff 100644
--- a/src/crimson/osd/main_config_bootstrap_helpers.cc
+++ b/src/crimson/osd/main_config_bootstrap_helpers.cc
@@ -224,13 +224,12 @@ _get_early_config(int argc, const char *argv[])
 	                     " or crimson_cpu_num must be set");
 	      ceph_abort();
 	    }
-	    std::string smp = fmt::format("{}", reactor_num);
 	    ret.early_args.emplace_back("--smp");
-	    ret.early_args.emplace_back(smp);
+	    ret.early_args.emplace_back(fmt::format("{}", reactor_num));
 	    ret.early_args.emplace_back("--thread-affinity");
 	    ret.early_args.emplace_back("0");
 	    logger().info("get_early_config: set --thread-affinity 0 --smp {}",
-	                  smp);
+	                  reactor_num);
 
 	  }
 	} else {