From: Radoslaw Zarzynski <rzarzyns@redhat.com>
Date: Wed, 8 Sep 2021 19:30:17 +0000 (+0000)
Subject: crimson/net: decouple listen_ertr from bind_ertr.
X-Git-Tag: v17.1.0~883^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=0f3779e2ac2eac320e60eef812148896f50d3e28;p=ceph.git

crimson/net: decouple listen_ertr from bind_ertr.

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
---

diff --git a/src/crimson/net/SocketMessenger.cc b/src/crimson/net/SocketMessenger.cc
index 81c12f54b2b0..eff85e1778ea 100644
--- a/src/crimson/net/SocketMessenger.cc
+++ b/src/crimson/net/SocketMessenger.cc
@@ -53,7 +53,8 @@ seastar::future<> SocketMessenger::set_myaddrs(const entity_addrvec_t& addrs)
   return Messenger::set_myaddrs(my_addrs);
 }
 
-SocketMessenger::bind_ertr::future<> SocketMessenger::do_listen(const entity_addrvec_t& addrs)
+crimson::net::listen_ertr::future<>
+SocketMessenger::do_listen(const entity_addrvec_t& addrs)
 {
   assert(seastar::this_shard_id() == master_sid);
   ceph_assert(addrs.front().get_family() == AF_INET);
@@ -65,12 +66,12 @@ SocketMessenger::bind_ertr::future<> SocketMessenger::do_listen(const entity_add
     } else {
       return seastar::now();
     }
-  }).then([this] () -> bind_ertr::future<> {
+  }).then([this] () -> listen_ertr::future<> {
     const entity_addr_t listen_addr = get_myaddr();
     logger().debug("{} do_listen: try listen {}...", *this, listen_addr);
     if (!listener) {
       logger().warn("{} do_listen: listener doesn't exist", *this);
-      return bind_ertr::now();
+      return listen_ertr::now();
     }
     return listener->listen(listen_addr);
   });
@@ -105,7 +106,7 @@ SocketMessenger::try_bind(const entity_addrvec_t& addrs,
         logger().info("{} try_bind: done", *this);
         return seastar::make_ready_future<std::optional<bool>>(
             std::make_optional<bool>(true));
-      }, bind_ertr::all_same_way([this, max_port, &port]
+      }, listen_ertr::all_same_way([this, max_port, &port]
                                  (const std::error_code& e) mutable
                                  -> seastar::future<std::optional<bool>> {
         assert(e == std::errc::address_in_use);
diff --git a/src/crimson/net/SocketMessenger.h b/src/crimson/net/SocketMessenger.h
index f9a225f54fd6..74bcac45494e 100644
--- a/src/crimson/net/SocketMessenger.h
+++ b/src/crimson/net/SocketMessenger.h
@@ -48,7 +48,7 @@ class SocketMessenger final : public Messenger {
   uint32_t global_seq = 0;
   bool started = false;
 
-  bind_ertr::future<> do_listen(const entity_addrvec_t& addr);
+  listen_ertr::future<> do_listen(const entity_addrvec_t& addr);
 
  public:
   SocketMessenger(const entity_name_t& myname,