From: Yingxin Cheng <yingxin.cheng@intel.com>
Date: Thu, 21 Mar 2024 01:43:34 +0000 (+0800)
Subject: crimson/common/tri_mutex: drop the unused greedy param
X-Git-Tag: v20.0.0~2079^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=9cdb67fa19ea7d912a5f3e7a14092a205dee80ac;p=ceph.git

crimson/common/tri_mutex: drop the unused greedy param

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
---

diff --git a/src/crimson/common/tri_mutex.cc b/src/crimson/common/tri_mutex.cc
index e4b18128053e3..633be55f98c18 100644
--- a/src/crimson/common/tri_mutex.cc
+++ b/src/crimson/common/tri_mutex.cc
@@ -15,7 +15,7 @@ void read_lock::unlock()
 
 seastar::future<> write_lock::lock()
 {
-  return static_cast<tri_mutex*>(this)->lock_for_write(false);
+  return static_cast<tri_mutex*>(this)->lock_for_write();
 }
 
 void write_lock::unlock()
@@ -110,19 +110,19 @@ void tri_mutex::demote_to_read()
   ++readers;
 }
 
-seastar::future<> tri_mutex::lock_for_write(bool greedy)
+seastar::future<> tri_mutex::lock_for_write()
 {
-  if (try_lock_for_write(greedy)) {
+  if (try_lock_for_write()) {
     return seastar::make_ready_future<>();
   }
   waiters.emplace_back(seastar::promise<>(), type_t::write);
   return waiters.back().pr.get_future();
 }
 
-bool tri_mutex::try_lock_for_write(bool greedy) noexcept
+bool tri_mutex::try_lock_for_write() noexcept
 {
   if (!readers && !exclusively_used) {
-    if (greedy || waiters.empty()) {
+    if (waiters.empty()) {
       ++writers;
       return true;
     }
diff --git a/src/crimson/common/tri_mutex.h b/src/crimson/common/tri_mutex.h
index 0533f3539d970..369a8802cf067 100644
--- a/src/crimson/common/tri_mutex.h
+++ b/src/crimson/common/tri_mutex.h
@@ -99,8 +99,8 @@ public:
   }
 
   // for shared writers
-  seastar::future<> lock_for_write(bool greedy);
-  bool try_lock_for_write(bool greedy) noexcept;
+  seastar::future<> lock_for_write();
+  bool try_lock_for_write() noexcept;
   void unlock_for_write();
   void promote_from_write();
   void demote_to_write();