From: Xiubo Li <xiubli@redhat.com>
Date: Fri, 7 Aug 2020 07:45:52 +0000 (+0800)
Subject: msg: throw a system error when center.init fails
X-Git-Tag: v14.2.22~28^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=df2d768dd7d0db320017c72a15b56aa25f738592;p=ceph.git

msg: throw a system error when center.init fails

In the libcephfs test case, it will run handreds of threads in
parallel, it will possibly reach the open files limit, but there
won't useful logs about what has happened.

This will just throw a system error, just like:

C++ exception with description "(24) Too many open files" thrown in the test body.

Fixes: https://tracker.ceph.com/issues/43039
Signed-off-by: Xiubo Li <xiubli@redhat.com>
(cherry picked from commit 6338050)

Conflicts:
      src/msg/async/Stack.cc
- nautilus uses plain "i" as the for loop counter variable, while
master has more fancy "worker_id"
---

diff --git a/src/msg/async/Stack.cc b/src/msg/async/Stack.cc
index 8976c3cc43a8..db3247ed1a84 100644
--- a/src/msg/async/Stack.cc
+++ b/src/msg/async/Stack.cc
@@ -117,7 +117,9 @@ NetworkStack::NetworkStack(CephContext *c, const string &t): type(t), started(fa
 
   for (unsigned i = 0; i < num_workers; ++i) {
     Worker *w = create_worker(cct, type, i);
-    w->center.init(InitEventNumber, i, type);
+    int ret = w->center.init(InitEventNumber, i, type);
+    if (ret)
+      throw std::system_error(-ret, std::generic_category());
     workers.push_back(w);
   }
 }