crimson/os/seastore/cache: improve logs for transaction conflict/reset

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>

diff --git a/src/crimson/os/seastore/cache.cc b/src/crimson/os/seastore/cache.cc
index fdf88d9f27bf9758c10ad58f74f239ed7394ef08..f0929fd276c3988f1e66d0a7790f1f70ed49f833 100644 (file)
--- a/src/crimson/os/seastore/cache.cc
+++ b/src/crimson/os/seastore/cache.cc
@@ -351,7 +351,7 @@ void Cache::invalidate(CachedExtent &extent) {
   DEBUG("invalidate begin -- extent {}", extent);
   for (auto &&i: extent.transactions) {
     if (!i.t->conflicted) {
-      DEBUGT("", i.t);
+      DEBUGT("set conflict", *i.t);
       i.t->conflicted = true;
       assert(!i.t->is_weak());
       auto m_key = std::make_pair(i.t->get_src(), extent.get_type());

diff --git a/src/crimson/os/seastore/cache.h b/src/crimson/os/seastore/cache.h
index 452fff7dccb12cdfaae104dbddb26cbed6ff5416..19432f429020b5c9bfceb372832ca75c112fc28b 100644 (file)
--- a/src/crimson/os/seastore/cache.h
+++ b/src/crimson/os/seastore/cache.h
@@ -134,10 +134,12 @@ public:
 
   /// Resets transaction preserving
   void reset_transaction_preserve_handle(Transaction &t) {
+    LOG_PREFIX(Cache::reset_transaction_preserve_handle);
     if (t.did_reset()) {
       ++(get_counter(stats.trans_created_by_src, t.get_src()));
     }
     t.reset_preserve_handle(last_commit);
+    DEBUGT("reset", t);
   }
 
   /**