mds: drop obsolete hack for base inodes

We used to skip base inodes for scatter_writebehind.  But we can
journal these just like anything else, and it potentially breaks
try_to_expire if a base inode's lock is dirty, because the
completion queued on WAIT_STABLE by scatter_nudge never gets
completed.

diff --git a/src/mds/Locker.cc b/src/mds/Locker.cc
index e90af84c5d5e85f9e289c31a49014e2127d277eb..31d4b71a6dc9ca20c38ab5a24ddcbbe12cbc464c 100644 (file)
--- a/src/mds/Locker.cc
+++ b/src/mds/Locker.cc
@@ -2813,15 +2813,6 @@ void Locker::scatter_writebehind(ScatterLock *lock)
   CInode *in = (CInode*)lock->get_parent();
   dout(10) << "scatter_writebehind " << in->inode.mtime << " on " << *lock << " on " << *in << dendl;
 
-  // hack:
-  if (in->is_base()) {
-    dout(10) << "scatter_writebehind just clearing updated flag for base inode " << *in << dendl;
-    lock->clear_dirty();
-    if (!lock->is_stable())
-      eval_gather(lock);
-    return;
-  }
-
   // journal
   Mutation *mut = new Mutation;
   mut->ls = mds->mdlog->get_current_segment();