From 2c66e9b7fad4eec017cecc6bd6005522cc0fb264 Mon Sep 17 00:00:00 2001 From: Casey Bodley Date: Tue, 11 Oct 2016 15:23:16 -0400 Subject: [PATCH] rgw: clean up RGWShardedOmapCRManager on early return ShardedOmapCRManager is spawning coroutines that sleep. if we don't finish them before trying to drain_all(), they'll lead to deadlock Fixes: http://tracker.ceph.com/issues/17571 Signed-off-by: Casey Bodley --- src/rgw/rgw_sync.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/rgw/rgw_sync.cc b/src/rgw/rgw_sync.cc index 947be259ff6f..b34e7fe68916 100644 --- a/src/rgw/rgw_sync.cc +++ b/src/rgw/rgw_sync.cc @@ -814,6 +814,7 @@ public: } if (get_ret_status() < 0) { ldout(cct, 0) << "ERROR: failed to fetch metadata sections" << dendl; + yield entries_index->finish(); yield lease_cr->go_down(); drain_all(); return set_cr_error(get_ret_status()); @@ -829,6 +830,7 @@ public: } if (get_ret_status() < 0) { ldout(cct, 0) << "ERROR: failed to fetch metadata section: " << *sections_iter << dendl; + yield entries_index->finish(); yield lease_cr->go_down(); drain_all(); return set_cr_error(get_ret_status()); -- 2.47.3