From: Venky Shankar <vshankar@redhat.com>
Date: Tue, 19 Dec 2023 05:19:39 +0000 (-0500)
Subject: mds: func name fixup, plus, and additional log message
X-Git-Tag: v19.3.0~213^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=31fb87e32649adf2d51f37ff562248ed0e6f5a3e;p=ceph.git

mds: func name fixup, plus, and additional log message

... when tid isn't found in pending_notifies.

Found this by code reading when trying to RCA

        https://tracker.ceph.com/issues/54741

Signed-off-by: Venky Shankar <vshankar@redhat.com>
---

diff --git a/src/mds/MDSTableServer.cc b/src/mds/MDSTableServer.cc
index 5514f1e78478..c5cb0b817e02 100644
--- a/src/mds/MDSTableServer.cc
+++ b/src/mds/MDSTableServer.cc
@@ -71,7 +71,7 @@ void MDSTableServer::handle_prepare(const cref_t<MMDSTableRequest> &req)
 
 void MDSTableServer::_prepare_logged(const cref_t<MMDSTableRequest> &req, version_t tid)
 {
-  dout(7) << "_create_logged " << *req << " tid " << tid << dendl;
+  dout(7) << __func__ << ": req=" << *req << " tid " << tid << dendl;
   mds_rank_t from = mds_rank_t(req->get_source().num());
 
   ceph_assert(g_conf()->mds_kill_mdstable_at != 2);
@@ -114,6 +114,8 @@ void MDSTableServer::handle_notify_ack(const cref_t<MMDSTableRequest> &m)
       dout(0) << "got unexpected notify ack for tid " <<  tid << " from mds." << from << dendl;
     }
   } else {
+    dout(0) << __func__ << ": tid=" << tid << " from mds." << from
+	    << " not tracked in pending notifies" << dendl;
   }
 }