From f3b61e68922a6038a6f1a3b31f3989ae4cb15e90 Mon Sep 17 00:00:00 2001 From: "Yan, Zheng" Date: Mon, 15 Dec 2014 09:51:52 +0800 Subject: [PATCH] mds: don't mark dirfrag dirty after dropping stale dentries now MDS use omap to store dirfrag, and always does partial commit. marking dirfrag dirty after dropping stale dentries is useless. Signed-off-by: Yan, Zheng --- src/mds/CDir.cc | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/mds/CDir.cc b/src/mds/CDir.cc index 056b1af0826a7..595f2d7e42a2d 100644 --- a/src/mds/CDir.cc +++ b/src/mds/CDir.cc @@ -1586,7 +1586,6 @@ void CDir::_omap_fetched(bufferlist& hdrbl, map& omap, // purge stale snaps? // only if we have past_parents open! - bool purged_any = false; const set *snaps = NULL; SnapRealm *realm = inode->find_snaprealm(); if (!realm->have_past_parents_open()) { @@ -1627,7 +1626,6 @@ void CDir::_omap_fetched(bufferlist& hdrbl, map& omap, if (p == snaps->end() || *p > last) { dout(10) << " skipping stale dentry on [" << first << "," << last << "]" << dendl; stale = true; - purged_any = true; } } @@ -1787,9 +1785,6 @@ void CDir::_omap_fetched(bufferlist& hdrbl, map& omap, //cache->mds->logger->inc("newin", num_new_inodes_loaded); - if (purged_any) - log_mark_dirty(); - // mark complete, !fetching mark_complete(); state_clear(STATE_FETCHING); -- 2.39.5