From: Jianpeng Ma <jianpeng.ma@intel.com>
Date: Tue, 7 Sep 2021 02:19:55 +0000 (+0800)
Subject: librbd/cache/pwl/ssd: Remove unused parameter.
X-Git-Tag: v16.2.7~50^2~9
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ec058625f6f1e71aecba2245849bc7922f183c43;p=ceph.git

librbd/cache/pwl/ssd: Remove unused parameter.

Met the following compiler warning message:
>[38/80] Building CXX object
src/librbd/CMakeFiles/librbd_plugin_pwl_cache.dir/cache/pwl/ssd/WriteLog.cc.o
>../src/librbd/cache/pwl/ssd/WriteLog.cc:37:25: warning: unused variable
'ops_appended_together' [-Wunused-const-variable]
>const unsigned long int ops_appended_together = MAX_WRITES_PER_SYNC_POINT;

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
(cherry picked from commit 40dad4c30c3b46ed3ac06961ebe740f6d82a6bc1)
---

diff --git a/src/librbd/cache/pwl/ssd/WriteLog.cc b/src/librbd/cache/pwl/ssd/WriteLog.cc
index e0d9a13199a..962accc6c4a 100644
--- a/src/librbd/cache/pwl/ssd/WriteLog.cc
+++ b/src/librbd/cache/pwl/ssd/WriteLog.cc
@@ -42,16 +42,6 @@ static bool is_valid_pool_root(const WriteLogPoolRoot& root) {
          root.first_free_entry % MIN_WRITE_ALLOC_SSD_SIZE == 0;
 }
 
-static bool is_valid_pool_root(const WriteLogPoolRoot& root) {
-  return root.pool_size % MIN_WRITE_ALLOC_SSD_SIZE == 0 &&
-         root.first_valid_entry >= DATA_RING_BUFFER_OFFSET &&
-         root.first_valid_entry < root.pool_size &&
-         root.first_valid_entry % MIN_WRITE_ALLOC_SSD_SIZE == 0 &&
-         root.first_free_entry >= DATA_RING_BUFFER_OFFSET &&
-         root.first_free_entry < root.pool_size &&
-         root.first_free_entry % MIN_WRITE_ALLOC_SSD_SIZE == 0;
-}
-
 template <typename I>
 Builder<AbstractWriteLog<I>>* WriteLog<I>::create_builder() {
   m_builderobj = new Builder<This>();