From bf6cd9f0d0d74a5a794bbce368dc5b36fdd0aacb Mon Sep 17 00:00:00 2001 From: "Yan, Zheng" Date: Tue, 19 Mar 2019 20:45:42 +0800 Subject: [PATCH] mds: remove superfluous error in StrayManager::advance_delayed() Fixes: http://tracker.ceph.com/issues/38679 Signed-off-by: "Yan, Zheng" (cherry picked from commit f51857772f4320020211f3628ba9f5d5bfb4f9e7) --- src/mds/StrayManager.cc | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/src/mds/StrayManager.cc b/src/mds/StrayManager.cc index f331722ba4d3..3d92ab71162f 100644 --- a/src/mds/StrayManager.cc +++ b/src/mds/StrayManager.cc @@ -356,17 +356,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); } -- 2.47.3