From fad3d9985353295d4540b12979e9485103dd536a Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Wed, 7 Sep 2016 10:32:50 -0400 Subject: [PATCH] os/bluestore: assert shared blob cache cleared on split Signed-off-by: Sage Weil --- src/os/bluestore/BlueStore.cc | 4 +++- src/os/bluestore/BlueStore.h | 4 ++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc index 98c85d0b85366..f09b22faeaba3 100644 --- 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 0c5590b7703fb..eac753e766b99 100644 --- 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) -- 2.39.5