From: Samuel Just <sjust@redhat.com>
Date: Tue, 15 Apr 2025 22:01:43 +0000 (-0700)
Subject: crimson/.../osd_operation: remove max_in_progress shortcut in with_throttle
X-Git-Tag: testing/wip-rishabh-testing-20250422.155049-debug~5^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=1c009f4f24fc3594e876db2c49dae294f51c0bad;p=ceph-ci.git

crimson/.../osd_operation: remove max_in_progress shortcut in with_throttle

This made sense prior to the addition of the scheduler.  Now, we need
to go through the scheduler whether there's a max or not.

Signed-off-by: Samuel Just <sjust@redhat.com>
---

diff --git a/src/crimson/osd/osd_operation.h b/src/crimson/osd/osd_operation.h
index 6ae64b59c7d..03b6aa6c4be 100644
--- a/src/crimson/osd/osd_operation.h
+++ b/src/crimson/osd/osd_operation.h
@@ -314,7 +314,6 @@ public:
   auto with_throttle(
     crimson::osd::scheduler::params_t params,
     F &&f) {
-    if (!max_in_progress) return f();
     return acquire_throttle(params)
       .then(std::forward<F>(f))
       .finally([this] {