commit af4ca715bd1820616e04fcc63ad4e64d64df4024
author Steven Moreland <smoreland@google.com> Mon May 24 23:22:08 2021 +0000
committer Steven Moreland <smoreland@google.com> Tue May 25 02:42:07 2021 +0000
tree c58f18b6142f954877e8486e2430b5cff15363c1
parent f51742734f476e7f9478f7cb56c18b9215ecad43

binderRpcTest: use waitpid

Actually reap child processes. This gives us stronger guarantees (that
everything can shut down) and it avoids 'kill'.

Bug: 186661301
Test: binderRpcTest
Change-Id: If10f00de845eb8097813b4edbf8e2b8ffdc90c5f

libs/binder/RpcSession.cpp

diff --git a/libs/binder/RpcSession.cpp b/libs/binder/RpcSession.cpp
index d5ce324..ac7544f 100644
--- a/libs/binder/RpcSession.cpp
+++ b/libs/binder/RpcSession.cpp
@@ -199,7 +199,8 @@
                 state()->getAndExecuteCommand(connection->fd, sp<RpcSession>::fromExisting(this));
 
         if (error != OK) {
-            ALOGI("Binder connection thread closing w/ status %s", statusToString(error).c_str());
+            LOG_RPC_DETAIL("Binder connection thread closing w/ status %s",
+                           statusToString(error).c_str());
             break;
         }
     }