From a1dc9a28e1bbb829266f3cf5710abc8e7c2bec4b Mon Sep 17 00:00:00 2001 From: Radoslaw Zarzynski Date: Wed, 14 Sep 2022 11:25:07 +0000 Subject: [PATCH] crimson/osd: PGAdvanceMap ops complete in order For the analysis refer to the bug tracker. Dropping the `handle.exit()` looks like the simplest solution, far simpler than e.g. introducing extra future-promise. Fixes: https://tracker.ceph.com/issues/57536 Signed-off-by: Radoslaw Zarzynski --- src/crimson/osd/osd_operations/pg_advance_map.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/src/crimson/osd/osd_operations/pg_advance_map.cc b/src/crimson/osd/osd_operations/pg_advance_map.cc index 51b279c7edb..c7fcacb315d 100644 --- a/src/crimson/osd/osd_operations/pg_advance_map.cc +++ b/src/crimson/osd/osd_operations/pg_advance_map.cc @@ -74,7 +74,6 @@ seastar::future<> PGAdvanceMap::start() }); }).then([this] { pg->handle_activate_map(rctx); - handle.exit(); if (do_init) { shard_manager.pg_created(pg->get_pgid(), pg); shard_manager.get_shard_services().inc_pg_num(); -- 2.39.5