From: Samuel Just <sjust@redhat.com>
Date: Mon, 13 Jul 2020 22:45:58 +0000 (-0700)
Subject: crimson/os/seastore: actually use LBAManager::complete_transaction
X-Git-Tag: v16.1.0~1513^2~5
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0f7e8973e1b0081366bdae94c27d860291a0862a;p=ceph.git

crimson/os/seastore: actually use LBAManager::complete_transaction

Signed-off-by: Samuel Just <sjust@redhat.com>
---

diff --git a/src/crimson/os/seastore/lba_manager.h b/src/crimson/os/seastore/lba_manager.h
index fc5d4a271402..e329eb551722 100644
--- a/src/crimson/os/seastore/lba_manager.h
+++ b/src/crimson/os/seastore/lba_manager.h
@@ -111,7 +111,6 @@ public:
     Transaction &t,
     laddr_t addr) = 0;
 
-  // TODO: probably unused, removed
   using complete_transaction_ertr = crimson::errorator<
     crimson::ct_error::input_output_error>;
   using complete_transaction_ret = complete_transaction_ertr::future<>;
diff --git a/src/crimson/os/seastore/transaction_manager.cc b/src/crimson/os/seastore/transaction_manager.cc
index 8b9591869667..c41dec000cf6 100644
--- a/src/crimson/os/seastore/transaction_manager.cc
+++ b/src/crimson/os/seastore/transaction_manager.cc
@@ -134,6 +134,7 @@ TransactionManager::submit_transaction(
   return journal.submit_record(std::move(*record)).safe_then(
     [this, t=std::move(t)](paddr_t addr) mutable {
       cache.complete_commit(*t, addr);
+      lba_manager.complete_transaction(*t);
     },
     submit_transaction_ertr::pass_further{},
     crimson::ct_error::all_same_way([](auto e) {