From: NitzanMordhai <nmordech@redhat.com>
Date: Thu, 20 Oct 2022 06:29:30 +0000 (+0000)
Subject: OSD: Remove shard lock that mClockScheduler use when updating conf
X-Git-Tag: v18.1.0~421^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=6450dae40120b0e3d9e46b4561b8212ef330470a;p=ceph-ci.git

OSD: Remove shard lock that mClockScheduler use when updating conf

When mClockScheduler uses bench to determine conf. values a deadlock
can occur if the update scheduler config run during consume_map already started.

Fixes: https://tracker.ceph.com/issues/57699
Signed-off-by: Nitzan Mordechai <nmordec@redhat.com>
---

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 04a1b3608aa..b13583cde3a 100644
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -10496,7 +10496,6 @@ void OSDShard::unprime_split_children(spg_t parent, unsigned old_pg_num)
 
 void OSDShard::update_scheduler_config()
 {
-  std::lock_guard l(shard_lock);
   scheduler->update_configuration();
 }