os/bluestore: use vector instead of set for zombies

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit ef34d2dc8f3dd40c8426b7373d13dae84f90ff73)

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 9dea4e235a7852521014d6f3b6310581904955b0..e205baff3d61d5b8c69fbf007d6610022ca8a4f7 100644 (file)
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -8792,7 +8792,7 @@ void BlueStore::_osr_drain_all()
   dout(10) << __func__ << dendl;
 
   set<OpSequencerRef> s;
-  set<OpSequencerRef> zombies;
+  vector<OpSequencerRef> zombies;
   {
     RWLock::RLocker l(coll_lock);
     for (auto& i : coll_map) {
@@ -8803,7 +8803,7 @@ void BlueStore::_osr_drain_all()
     std::lock_guard<std::mutex> l(zombie_osr_lock);
     for (auto& i : zombie_osr_set) {
       s.insert(i.second);
-      zombies.insert(i.second);
+      zombies.push_back(i.second);
     }
   }
   dout(20) << __func__ << " osr_set " << s << dendl;