From cf8eafb6851346dae8879b09b62fbdffa832d9bc Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Sun, 29 Mar 2026 19:41:48 +0800 Subject: [PATCH] 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 --- src/crimson/osd/main_config_bootstrap_helpers.cc | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) 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 { -- 2.47.3