common/Finisher: only queue empty only wake up waiter.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

diff --git a/src/common/Finisher.h b/src/common/Finisher.h
index 4b6d2d3c8a136da2896df3e5264b0ee10ca8b366..94607ea21a679aece473fa1a77c83956e251b661 100644 (file)
--- a/src/common/Finisher.h
+++ b/src/common/Finisher.h
@@ -193,18 +193,22 @@ public:
   ContextQueue(Mutex& mut, Cond& con) : mutex(mut), cond(con) {}
 
   void queue(list<Context *>& ls) {
+    bool empty = false;
     {
       std::scoped_lock l(q_mutex);
       if (q.empty()) {
        q.swap(ls);
+       empty = true;
       } else {
        q.insert(q.end(), ls.begin(), ls.end());
       }
     }
 
-    mutex.Lock();
-    cond.SignalOne();
-    mutex.Unlock();
+    if (empty) {
+      mutex.Lock();
+      cond.SignalOne();
+      mutex.Unlock();
+    }
 
     ls.clear();
   }