From: Casey Bodley <cbodley@redhat.com>
Date: Mon, 14 Oct 2024 18:18:15 +0000 (-0400)
Subject: common/async: Completion removes unnecessary bind_executor()
X-Git-Tag: v20.0.0~574^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=56e48d4ba7021cc2a2bd0928c3e8a2ecbca8e203;p=ceph.git

common/async: Completion removes unnecessary bind_executor()

destroy_defer() has already called bind_and_forward() to bind the
handler's associated executor

Signed-off-by: Casey Bodley <cbodley@redhat.com>
---

diff --git a/src/common/async/completion.h b/src/common/async/completion.h
index 3c310d7f8330..6cdfaaa63b73 100644
--- a/src/common/async/completion.h
+++ b/src/common/async/completion.h
@@ -202,7 +202,7 @@ class CompletionImpl final : public Completion<void(Args...), T> {
     auto f = bind_and_forward(ex2, std::move(handler), std::move(args));
     RebindTraits2::destroy(alloc2, this);
     RebindTraits2::deallocate(alloc2, this, 1);
-    boost::asio::defer(boost::asio::bind_executor(ex2, std::move(f)));
+    boost::asio::defer(std::move(f));
   }
   void destroy_dispatch(std::tuple<Args...>&& args) override {
     auto w = std::move(work);