From: Radoslaw Zarzynski <rzarzyns@redhat.com>
Date: Sun, 11 May 2025 11:29:18 +0000 (+0000)
Subject: crimson/osd: tackle the get_write_plan() problem of ECBackend
X-Git-Tag: v21.0.0~3^2~34
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=88737b1c85f1a1e209fdddb73802ed8ab8be016e;p=ceph.git

crimson/osd: tackle the get_write_plan() problem of ECBackend

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
---

diff --git a/src/crimson/osd/ec_backend.cc b/src/crimson/osd/ec_backend.cc
index 333acaf14ae3..4ca9f3bf75ec 100644
--- a/src/crimson/osd/ec_backend.cc
+++ b/src/crimson/osd/ec_backend.cc
@@ -263,22 +263,6 @@ struct ECCrimsonOp : ECCommon::RMWPipeline::Op {
       &temp_cleared,
       dpp,
       require_osd_release);
-#endif
-  }
-
-  template <typename F>
-  static ECTransaction::WritePlan get_write_plan(
-    const ECUtil::stripe_info_t &sinfo,
-    PGTransaction& t,
-    F &&get_hinfo,
-    DoutPrefixProvider *dpp)
-  {
-#if 1
-    return ECTransaction::get_write_plan(
-      sinfo,
-      t,
-      std::forward<F>(get_hinfo),
-      dpp);
 #endif
   }
 };
@@ -325,9 +309,11 @@ ECBackend::submit_transaction(const std::set<pg_shard_t> &pg_shards,
   //if (client_op) {
   //  op->trace = client_op->pg_trace;
   //}
-  op->plan = op->get_write_plan(
+  op->plan = ECCommon::get_write_plan(
     sinfo,
     *(op->t),
+    read_pipeline,
+    rmw_pipeline,
     &dpp);
   logger().info("{}: op {} starting", "_submit_transaction", ""); //*op);
   rmw_pipeline.start_rmw(std::move(op));