mds: xlock_finish should only do_issue in certain cases.

We accidentally (we think) initialized this variable to true when
we want it to be false: we should only do_issue if there aren't
any remaining locks, not in all cases.

Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>

diff --git a/src/mds/Locker.cc b/src/mds/Locker.cc
index 8b40bafede3f09fca54203a9914d5db57cc7a6d4..bac5b1027570b166ac978599669d992d4d276281 100644 (file)
--- a/src/mds/Locker.cc
+++ b/src/mds/Locker.cc
@@ -1124,7 +1124,7 @@ void Locker::xlock_finish(SimpleLock *lock, Mutation *mut)
   mut->xlocks.erase(lock);
   mut->locks.erase(lock);
   
-  bool do_issue = true;
+  bool do_issue = false;
 
   // remote xlock?
   if (!lock->get_parent()->is_auth()) {