os/bluestore: remove zoned namespace support

Lately we've been adding a lot of commits that could've interfered with
smr support but since no one is actively reviewing/supporting smr in
bluestore, it doesn't make sense for us to mantain it.

Signed-off-by: Pere Diaz Bou <[email protected]>

Author: pereman2

src/common/options/global.yaml.in

@@ -4897,7 +4897,6 @@ options:
   - stupid
   - avl
   - hybrid
-  - zoned
   with_legacy: true
 - name: bluestore_freelist_blocks_per_key
   type: size
@@ -6326,7 +6325,6 @@ options:
   - aio
   - spdk
   - pmem
-  - hm_smr
 - name: bluestore_cleaner_sleep_interval
   type: float
   level: advanced

src/os/CMakeLists.txt

@@ -27,12 +27,6 @@ if(WITH_BLUESTORE)
   )
 endif(WITH_BLUESTORE)
 
-if(WITH_ZBD)
-  list(APPEND libos_srcs
-    bluestore/ZonedFreelistManager.cc
-    bluestore/ZonedAllocator.cc)
-endif()
-
 if(WITH_FUSE)
   list(APPEND libos_srcs
     FuseStore.cc)

src/os/bluestore/Allocator.cc

@@ -173,8 +173,6 @@ Allocator *Allocator::create(
   std::string_view type,
   int64_t size,
   int64_t block_size,
-  int64_t zone_size,
-  int64_t first_sequential_zone,
   std::string_view name)
 {
   Allocator* alloc = nullptr;
@@ -190,11 +188,6 @@ Allocator *Allocator::create(
     return new HybridAllocator(cct, size, block_size,
       cct->_conf.get_val<uint64_t>("bluestore_hybrid_alloc_mem_cap"),
       name);
-#ifdef HAVE_LIBZBD
-  } else if (type == "zoned") {
-    return new ZonedAllocator(cct, size, block_size, zone_size, first_sequential_zone,
-			      name);
-#endif
   }
   if (alloc == nullptr) {
     lderr(cct) << "Allocator::" << __func__ << " unknown alloc type "

src/os/bluestore/Allocator.h

@@ -72,8 +72,6 @@ class Allocator {
     std::string_view type,
     int64_t size,
     int64_t block_size,
-    int64_t zone_size = 0,
-    int64_t firs_sequential_zone = 0,
     const std::string_view name = ""
     );
 

src/os/bluestore/BitmapFreelistManager.cc

@@ -67,7 +67,6 @@ BitmapFreelistManager::BitmapFreelistManager(CephContext* cct,
 }
 
 int BitmapFreelistManager::create(uint64_t new_size, uint64_t granularity,
-				  uint64_t zone_size, uint64_t first_sequential_zone,
 				  KeyValueDB::Transaction txn)
 {
   bytes_per_block = granularity;

src/os/bluestore/BitmapFreelistManager.h

@@ -63,7 +63,6 @@ class BitmapFreelistManager : public FreelistManager {
   static void setup_merge_operator(KeyValueDB *db, std::string prefix);
 
   int create(uint64_t size, uint64_t granularity,
-	     uint64_t zone_size, uint64_t first_sequential_zone,
 	     KeyValueDB::Transaction txn) override;
 
   int init(KeyValueDB *kvdb, bool db_in_read_only,

src/os/bluestore/BlueFS.cc

@@ -749,7 +749,6 @@ void BlueFS::_init_alloc()
       alloc[id] = Allocator::create(cct, cct->_conf->bluefs_allocator,
 				    bdev[id]->get_size(),
 				    alloc_size[id],
-				    0, 0,
 				    name);
       alloc[id]->init_add_free(
         block_reserved[id],