os/bluestore: set min compressed blob size floor at 2x min_alloc_size

Otherwise there's not point in compressing--we won't be able to allocation a smaller space to store it. Signed-off-by: Sage Weil <[email protected]>
zmc · Sep 30, 2016 · beed433 · beed433
1 parent 87f6d23
commit beed433
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
@@ -7687,6 +7687,13 @@ int BlueStore::_do_write(
       wctx.target_blob_size > g_conf->bluestore_max_blob_size) {
     wctx.target_blob_size = g_conf->bluestore_max_blob_size;
   }
+  // set the min blob size floor at 2x the min_alloc_size, or else we
+  // won't be able to allocate a smaller extent for the compressed
+  // data.
+  if (wctx.compress &&
+      wctx.target_blob_size < min_alloc_size * 2) {
+    wctx.target_blob_size = min_alloc_size * 2;
+  }
 
   dout(20) << __func__ << " prefer csum_order " << wctx.csum_order
 	   << " target_blob_size 0x" << std::hex << wctx.target_blob_size