os/bluestore: assert shared blob cache cleared on split

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

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 98c85d0b85366583477eecdf4e87ce166c350faf..f09b22faeaba3096f1f05a110b2439b1cbe5801e 100644 (file)
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -8215,7 +8215,9 @@ int BlueStore::_split_collection(TransContext *txc,
   // blow away the caches.  FIXME.
   c->onode_map.clear();
   d->onode_map.clear();
-#warning assert that shared_blob cache (and others?) are also cleared..
+
+  assert(c->shared_blob_set.empty());
+  assert(d->shared_blob_set.empty());
 
   c->cnode.bits = bits;
   assert(d->cnode.bits == bits);

diff --git a/src/os/bluestore/BlueStore.h b/src/os/bluestore/BlueStore.h
index 0c5590b7703fb33bd9b3417eaf50f44948a144d2..eac753e766b99b29468758e4b05f18ac058c0d3b 100644 (file)
--- a/src/os/bluestore/BlueStore.h
+++ b/src/os/bluestore/BlueStore.h
@@ -376,6 +376,10 @@ public:
       }
       return false;
     }
+
+    bool empty() {
+      return uset.empty();
+    }
   };
 
   /// in-memory blob metadata and associated cached buffers (if any)