mds: verify is_stable() before calling simple_eval()

The finish_waiters may change the state again.

diff --git a/src/mds/Locker.cc b/src/mds/Locker.cc
index c4d282c62a2fa02e32d62703d5da167c4a3d48c1..9dc987631a40c601b56b96849124180af18b5cf6 100644 (file)
--- a/src/mds/Locker.cc
+++ b/src/mds/Locker.cc
@@ -1742,7 +1742,8 @@ void Locker::simple_eval_gather(SimpleLock *lock)
       lock->get_parent()->auth_unpin(lock);
 
       // re-eval?
-      simple_eval(lock);
+      if (lock->is_stable())
+       simple_eval(lock);
     }
   }
 
@@ -1759,7 +1760,8 @@ void Locker::simple_eval_gather(SimpleLock *lock)
     lock->get_parent()->auth_unpin(lock);
 
     // re-eval?
-    simple_eval(lock);
+    if (lock->is_stable())
+      simple_eval(lock);
   }
 }