From: Radosław Zarzyński <rzarzyns@redhat.com>
Date: Thu, 28 Sep 2023 16:21:15 +0000 (+0200)
Subject: crimson/osd: wire MOSDECSubOpWriteReply up with ECBackend
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=f6637bd5c4c7d7dabd34c21512dac46aae1ba995;p=ceph-ci.git

crimson/osd: wire MOSDECSubOpWriteReply up with ECBackend

Signed-off-by: Radosław Zarzyński <rzarzyns@redhat.com>
---

diff --git a/src/crimson/osd/ec_backend.cc b/src/crimson/osd/ec_backend.cc
index 9d06eea6361..07af5a25852 100644
--- a/src/crimson/osd/ec_backend.cc
+++ b/src/crimson/osd/ec_backend.cc
@@ -154,8 +154,33 @@ ECBackend::handle_rep_write_op(
 }
 
 ECBackend::write_iertr::future<>
-ECBackend::handle_rep_write_reply(Ref<MOSDECSubOpWriteReply>)
+ECBackend::handle_rep_write_reply(Ref<MOSDECSubOpWriteReply> m)
 {
+  const auto& op = m->op;
+  assert(rmw_pipeline.tid_to_op_map.contains(op.tid));
+  const auto& from = op.from;
+  auto& wop = *rmw_pipeline.tid_to_op_map.at(op.tid);
+  if (op.committed) {
+    // TODO: trace.event("sub write committed");
+    ceph_assert(wop.pending_commit.count(from));
+    wop.pending_commit.erase(from);
+  }
+  if (op.applied) {
+    // TODO: trace.event("sub write applied");
+    ceph_assert(wop.pending_apply.count(from));
+    wop.pending_apply.erase(from);
+  }
+
+  if (wop.pending_commit.empty() &&
+      wop.on_all_commit &&
+      // also wait for apply, to preserve ordering with luminous peers.
+      wop.pending_apply.empty()) {
+    logger().info("{}: calling on_all_commit on {}", __func__, wop);
+    wop.on_all_commit->complete(0);
+    wop.on_all_commit = 0;
+    // TODO: wop.trace.event("ec write all committed");
+  }
+  rmw_pipeline.check_ops();
   return write_iertr::now();
 }
 
diff --git a/src/crimson/osd/osd_operations/ecrep_request.cc b/src/crimson/osd/osd_operations/ecrep_request.cc
index 3b0b8fb2f6d..5d28b197aea 100644
--- a/src/crimson/osd/osd_operations/ecrep_request.cc
+++ b/src/crimson/osd/osd_operations/ecrep_request.cc
@@ -70,10 +70,8 @@ seastar::future<> ECRepRequest::with_pg(
       [pg] (Ref<MOSDECSubOpWrite> concrete_req) {
         return pg->handle_rep_write_op(std::move(concrete_req));
       },
-      [ec_backend] (Ref<MOSDECSubOpWriteReply> concrete_req) {
-        return ec_backend->handle_rep_write_reply(
-	  std::move(concrete_req)
-	).handle_error_interruptible(crimson::ct_error::assert_all{});
+      [pg] (Ref<MOSDECSubOpWriteReply> concrete_req) {
+        return pg->handle_rep_write_reply(std::move(concrete_req));
       },
       [pg] (Ref<MOSDECSubOpRead> concrete_req) {
         return pg->handle_rep_read_op(std::move(concrete_req));
diff --git a/src/crimson/osd/pg.cc b/src/crimson/osd/pg.cc
index 9c6b192e45e..6a785a880fc 100644
--- a/src/crimson/osd/pg.cc
+++ b/src/crimson/osd/pg.cc
@@ -1464,6 +1464,21 @@ PG::interruptible_future<> PG::handle_rep_write_op(Ref<MOSDECSubOpWrite> m)
   }).handle_error_interruptible(crimson::ct_error::assert_all{});
 }
 
+PG::interruptible_future<> PG::handle_rep_write_reply(Ref<MOSDECSubOpWriteReply> m)
+{
+  if (const auto& op = m->op; op.committed) {
+    // TODO: trace.event("sub write committed");
+    if (op.from != pg_whoami) {
+      peering_state.update_peer_last_complete_ondisk(op.from, op.last_complete);
+    }
+  }
+  auto* ec_backend=dynamic_cast<::ECBackend*>(&get_backend());
+  assert(ec_backend);
+  return ec_backend->handle_rep_write_reply(
+    std::move(m)
+  ).handle_error_interruptible(crimson::ct_error::assert_all{});
+}
+
 PG::interruptible_future<> PG::handle_rep_read_op(Ref<MOSDECSubOpRead> m)
 {
   auto* ec_backend=dynamic_cast<::ECBackend*>(&get_backend());
diff --git a/src/crimson/osd/pg.h b/src/crimson/osd/pg.h
index e8729981656..1bfb8e52f67 100644
--- a/src/crimson/osd/pg.h
+++ b/src/crimson/osd/pg.h
@@ -774,6 +774,7 @@ public:
   seastar::future<> clear_temp_objects();
 
   interruptible_future<> handle_rep_write_op(Ref<MOSDECSubOpWrite>);
+  interruptible_future<> handle_rep_write_reply(Ref<MOSDECSubOpWriteReply>);
   interruptible_future<> handle_rep_read_op(Ref<MOSDECSubOpRead>);
 
 private: