mds: dump metareq_t instead of full op

This avoids infinite recursion when dumping locks with xlock_by
non-null.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 79c7ddce8a9c9deaf277aa27ca402aa0415420de)

diff --git a/src/mds/Mutation.cc b/src/mds/Mutation.cc
index e51348e638d5f133d247a4cc9de77a2e149b380c..3c94dfd22e366f87d50ce6d419378eef79123d9e 100644 (file)
--- a/src/mds/Mutation.cc
+++ b/src/mds/Mutation.cc
@@ -478,7 +478,7 @@ void MDRequestImpl::_dump(Formatter *f) const
 {
   std::lock_guard l(lock);
   f->dump_string("flag_point", _get_state_string());
-  f->dump_stream("reqid") << reqid;
+  f->dump_object("reqid", reqid);
   if (client_request) {
     f->dump_string("op_type", "client_request");
     f->open_object_section("client_info");

diff --git a/src/mds/SimpleLock.cc b/src/mds/SimpleLock.cc
index 4262c1f4856c2c93d1b48c7717ec9b6838e594a8..b23915f945219bbbe17c278ffb108547df301cd7 100644 (file)
--- a/src/mds/SimpleLock.cc
+++ b/src/mds/SimpleLock.cc
@@ -37,8 +37,8 @@ void SimpleLock::dump(ceph::Formatter *f) const {
   f->dump_int("num_wrlocks", get_num_wrlocks());
   f->dump_int("num_xlocks", get_num_xlocks());
   f->open_object_section("xlock_by");
-  if (get_xlock_by()) {
-    get_xlock_by()->dump(f);
+  if (auto mut = get_xlock_by(); mut) {
+    f->dump_object("reqid", mut->reqid);
   }
   f->close_section();
 }

diff --git a/src/mds/mdstypes.cc b/src/mds/mdstypes.cc
index 1588c1acf9eddcfd95346161f1742d8284b40653..2b1e037a6dd78748482d4f4a5cd3893f0947bc0b 100644 (file)
--- a/src/mds/mdstypes.cc
+++ b/src/mds/mdstypes.cc
@@ -770,6 +770,10 @@ void mds_table_pending_t::generate_test_instances(std::list<mds_table_pending_t*
   ls.back()->tid = 35434;
 }
 
+void metareqid_t::dump(ceph::Formatter* f) const {
+  f->dump_object("entity", name);
+  f->dump_unsigned("tid", tid);
+}
 
 /*
  * inode_load_vec_t

diff --git a/src/mds/mdstypes.h b/src/mds/mdstypes.h
index bfb27910846b2ed6c7d87d4e4f833dc206b6af9f..b20e28bc54f02e9237691911b19937fe3e2edcfb 100644 (file)
--- a/src/mds/mdstypes.h
+++ b/src/mds/mdstypes.h
@@ -1483,6 +1483,7 @@ struct metareqid_t {
     decode(name, p);
     decode(tid, p);
   }
+  void dump(ceph::Formatter *f) const;
 
   entity_name_t name;
   uint64_t tid = 0;