]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mds: remove ScrubStack::scrubstack
authorYan, Zheng <zyan@redhat.com>
Sun, 3 May 2020 15:17:51 +0000 (23:17 +0800)
committerYan, Zheng <zyan@redhat.com>
Mon, 16 Nov 2020 01:02:16 +0000 (09:02 +0800)
comment says it's hack for dout. I don't see any reason it's needed.

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
src/mds/ScrubStack.cc
src/mds/ScrubStack.h

index b2e426757a47c40b9530ef8fd8fdbbfcd476f458..93f879307e5a2c463b531d080749c2050e7e3ca7 100644 (file)
@@ -23,7 +23,7 @@
 #define dout_context g_ceph_context
 #define dout_subsys ceph_subsys_mds
 #undef dout_prefix
-#define dout_prefix _prefix(_dout, scrubstack->mdcache->mds)
+#define dout_prefix _prefix(_dout, mdcache->mds)
 static ostream& _prefix(std::ostream *_dout, MDSRank *mds) {
   return *_dout << "mds." << mds->get_nodeid() << ".scrubstack ";
 }
index 127679b48126f74482b6e25fcd16bf17af6a9edf..7fedd37280129416099383bfc36772cfe133e61d 100644 (file)
@@ -34,7 +34,6 @@ public:
     clog(clog),
     finisher(finisher_),
     inode_stack(member_offset(CInode, item_scrub)),
-    scrubstack(this),
     scrub_kick(mdc, this) {}
   ~ScrubStack() {
     ceph_assert(inode_stack.empty());
@@ -141,7 +140,6 @@ protected:
   elist<CInode*> inode_stack;
   /// current number of dentries we're actually scrubbing
   int scrubs_in_progress = 0;
-  ScrubStack *scrubstack; // hack for dout
   int stack_size = 0;
 
   C_KickOffScrubs scrub_kick;