]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
msg/async: dec(l_msgr_active_connections) when unregister anonymous connection
authorJianpeng Ma <jianpeng.ma@intel.com>
Tue, 31 Dec 2019 02:20:16 +0000 (10:20 +0800)
committerJianpeng Ma <jianpeng.ma@intel.com>
Tue, 31 Dec 2019 02:20:16 +0000 (10:20 +0800)
When do all in one test,  l_msgr_active_connection don't decrease when
client shutdown.
This bug introduced by 5b41e12fdb445.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
src/msg/async/AsyncMessenger.h

index cd8ad3c8b831f822d672e0a56b8a1fc90371c418..6e18281f1b9c43211607cbae7f794b6ac11b4a4a 100644 (file)
@@ -397,7 +397,7 @@ public:
    */
   void unregister_conn(const AsyncConnectionRef& conn) {
     std::lock_guard l{deleted_lock};
-    if (!accepting_conns.count(conn))
+    if (!accepting_conns.count(conn) || anon_conns.count(conn))
       conn->get_perf_counter()->dec(l_msgr_active_connections);
     deleted_conns.emplace(std::move(conn));
     conn->unregister();