os/bluestore: assert shared blob cache cleared on split

Signed-off-by: Sage Weil <sage@redhat.com>
ceph · Sep 7, 2016 · fad3d99 · fad3d99
1 parent 2d8a145
commit fad3d99
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git 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
@@ -376,6 +376,10 @@ class BlueStore : public ObjectStore,
       }
       return false;
     }
+
+    bool empty() {
+      return uset.empty();
+    }
   };
 
   /// in-memory blob metadata and associated cached buffers (if any)