os/bluestore: drop unreferenced spanning blobs

Otherwise we can "leak" these and keep them around indefinitely. Signed-off-by: Sage Weil <sage@redhat.com>
ceph · Sep 23, 2016 · 8f40caf · 8f40caf
1 parent c621146
commit 8f40caf
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
@@ -7380,6 +7380,13 @@ void BlueStore::_wctx_finish(
       if (blob.is_compressed()) {
 	txc->statfs_delta.compressed() -= blob.get_compressed_payload_length();
       }
+      if (b->id >= 0) {
+	// explicitly remove from spanning map
+	dout(20) << __func__ << "  spanning_blob_map removing empty " << *b
+		 << dendl;
+	auto it = o->extent_map.spanning_blob_map.iterator_to(*b);
+	o->extent_map.spanning_blob_map.erase(it);
+      }
     }
     txc->statfs_delta.stored() -= lo.length;
     if (blob.is_compressed()) {