From cc42cbf896c0788aa1dd9b2810e8e89c37944760 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Fri, 12 Aug 2016 11:13:46 -0400 Subject: [PATCH] os/bluestore/ExtentFreelistManager: move default create out of parent The only other instance of FreelistManager defines this differently anyway. Signed-off-by: Sage Weil --- src/os/bluestore/ExtentFreelistManager.cc | 6 ++++++ src/os/bluestore/ExtentFreelistManager.h | 2 ++ src/os/bluestore/FreelistManager.h | 5 +---- 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/src/os/bluestore/ExtentFreelistManager.cc b/src/os/bluestore/ExtentFreelistManager.cc index 97de391b289fa..3bc0ed9cc9771 100644 --- 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 71a2ccc3d2a72..839a251817ece 100644 --- 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 e037c0901cfc6..b3ca24b7bf469 100644 --- 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; -- 2.39.5