os/bluestore: narrow lock scope in _deferred_aio_finish

Signed-off-by: Sage Weil <sage@redhat.com>

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 3b5701ef470a81bfd76f35b778d9478a89786381..89244dc6fb6c158262001fbb1d0e1e14e7c51625 100644 (file)
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -8290,13 +8290,13 @@ void BlueStore::_deferred_aio_finish(OpSequencer *osr)
   DeferredBatch *b = osr->deferred_running;
   {
     std::lock_guard<std::mutex> l2(osr->qlock);
-    std::lock_guard<std::mutex> l(kv_lock);
     for (auto& i : b->txcs) {
       TransContext *txc = &i;
       txc->state = TransContext::STATE_DEFERRED_CLEANUP;
       txc->osr->qcond.notify_all();
       throttle_deferred_bytes.put(txc->cost);
     }
+    std::lock_guard<std::mutex> l(kv_lock);
     deferred_done_queue.emplace_back(b);
   }