os/bluestore/ExtentFreelistManager: move default create out of parent

The only other instance of FreelistManager defines this
differently anyway.

Signed-off-by: Sage Weil <sage@redhat.com>

diff --git a/src/os/bluestore/ExtentFreelistManager.cc b/src/os/bluestore/ExtentFreelistManager.cc
index 97de391b289fae184c7a52656f3994fde326a4ff..3bc0ed9cc9771228df0d0808a579d92f11160e6d 100644 (file)
--- a/src/os/bluestore/ExtentFreelistManager.cc
+++ b/src/os/bluestore/ExtentFreelistManager.cc
@@ -11,6 +11,12 @@
 #undef dout_prefix
 #define dout_prefix *_dout << "freelist "
 
+int ExtentFreelistManager::create(uint64_t size, KeyValueDB::Transaction txn)
+{
+  release(0, size, txn);
+  return 0;
+}
+
 int ExtentFreelistManager::init()
 {
   dout(1) << __func__ << dendl;

diff --git a/src/os/bluestore/ExtentFreelistManager.h b/src/os/bluestore/ExtentFreelistManager.h
index 71a2ccc3d2a728a4b75f4f1d8e8307040d50240c..839a251817ece46f671034bf1d71da55b12acc2e 100644 (file)
--- a/src/os/bluestore/ExtentFreelistManager.h
+++ b/src/os/bluestore/ExtentFreelistManager.h
@@ -35,6 +35,8 @@ public:
     total_free(0) {
   }
 
+  int create(uint64_t size, KeyValueDB::Transaction txn) override;
+
   int init() override;
   void shutdown() override;
 

diff --git a/src/os/bluestore/FreelistManager.h b/src/os/bluestore/FreelistManager.h
index e037c0901cfc6f75b70b65aa1d206e3f4f517b69..b3ca24b7bf46907136e4aa2d910feb1edea96566 100644 (file)
--- a/src/os/bluestore/FreelistManager.h
+++ b/src/os/bluestore/FreelistManager.h
@@ -22,10 +22,7 @@ public:
 
   static void setup_merge_operators(KeyValueDB *db);
 
-  virtual int create(uint64_t size, KeyValueDB::Transaction txn) {
-    release(0, size, txn);
-    return 0;
-  }
+  virtual int create(uint64_t size, KeyValueDB::Transaction txn) = 0;
 
   virtual int init() = 0;
   virtual void shutdown() = 0;