crimson/os: reduce the scope of lock

we just need to synchronize accesses to `stopping`, so only need to
guard the mutation of it.

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/crimson/os/alienstore/thread_pool.h b/src/crimson/os/alienstore/thread_pool.h
index fd22c9743db577bcac7e78633747bcf696cba587..25bd782d7014700010bf30da49675996c210971d 100644 (file)
--- a/src/crimson/os/alienstore/thread_pool.h
+++ b/src/crimson/os/alienstore/thread_pool.h
@@ -90,8 +90,10 @@ public:
     return work_item;
   }
   void stop() {
-    std::unique_lock lock{mutex};
-    stopping = true;
+    {
+      std::unique_lock lock{mutex};
+      stopping = true;
+    }
     cond.notify_all();
   }
   void push_back(WorkItem* work_item) {