From: Yan, Zheng <zyan@redhat.com>
Date: Tue, 19 Mar 2019 12:45:42 +0000 (+0800)
Subject: mds: remove superfluous error in StrayManager::advance_delayed()
X-Git-Tag: v13.2.7~55^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=7e0802008337c53f95204c200fe3f3f30d23da82;p=ceph.git

mds: remove superfluous error in StrayManager::advance_delayed()

Fixes: http://tracker.ceph.com/issues/38679
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit f51857772f4320020211f3628ba9f5d5bfb4f9e7)
---

diff --git a/src/mds/StrayManager.cc b/src/mds/StrayManager.cc
index d997f51dfa5..ae31bf495d5 100644
--- a/src/mds/StrayManager.cc
+++ b/src/mds/StrayManager.cc
@@ -357,17 +357,7 @@ void StrayManager::advance_delayed()
       continue;
     }
 
-    const bool purging = eval_stray(dn);
-    if (!purging) {
-      derr << "Dentry " << *dn << " was purgeable but no longer is!" << dendl;
-      /*
-       * This can happen if a stray is purgeable, but has gained an extra
-       * reference by virtue of having its backtrace updated.
-       * FIXME perhaps we could simplify this further by
-       * avoiding writing the backtrace of purge-ready strays, so
-       * that this code could be more rigid?
-       */
-    }
+    eval_stray(dn);
   }
   logger->set(l_mdc_num_strays_delayed, num_strays_delayed);
 }