mds: drop useless virtual-ness for finish_scatter_gather_update

If/when we do this on dentries, _then_ we can use a virtual function, and
rewrite the single user (scatter_writebehind) to generalize to non-inodes.

diff --git a/src/mds/Locker.cc b/src/mds/Locker.cc
index f9a460c04838f72cce81c3f1fae2276e523c9828..fafda5ffa5f8e21b135534fcfde449d01fd62874 100644 (file)
--- a/src/mds/Locker.cc
+++ b/src/mds/Locker.cc
@@ -3136,7 +3136,7 @@ void Locker::scatter_writebehind(ScatterLock *lock)
   inode_t *pi = in->project_inode();
   pi->version = in->pre_dirty();
 
-  lock->get_parent()->finish_scatter_gather_update(lock->get_type());
+  in->finish_scatter_gather_update(lock->get_type());
   lock->start_flush();
 
   EUpdate *le = new EUpdate(mds->mdlog, "scatter_writebehind");

diff --git a/src/mds/mdstypes.h b/src/mds/mdstypes.h
index 0263df9942c81051a243fd42331a6168ff5b7f69..b37cdadeda45b074d626f7373274492fb4cda608 100644 (file)
--- a/src/mds/mdstypes.h
+++ b/src/mds/mdstypes.h
@@ -2025,7 +2025,6 @@ protected:
   virtual bool is_lock_waiting(int type, uint64_t mask) { assert(0); return false; }
 
   virtual void clear_dirty_scattered(int type) { assert(0); }
-  virtual void finish_scatter_gather_update(int type) { }
 
   // ---------------------------------------------
   // ordering