summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2023-03-15 21:49:07 +0000
committer Steven Moreland <smoreland@google.com> 2023-03-15 22:03:17 +0000
commit73aa6f88e40bb66945ac67a87cdc0f271d655133 (patch)
tree2ed7429898015826e436f0b96e6b0173a530f560
parent69e22e9ea51f01b63e06a621a8c5a977617214fa (diff)
binderRpcTest: undo ThreadPoolLimit optimization
This test started flaking when we reduced the time it took. It should be re-written to avoid needing to sleep so long, but for now, restoring it so it isn't breaking. Fixes: 272429574 Test: binderRpcTest Change-Id: Ieef2d585f6743a9a7ac79101d5f7724cff7936c3
-rw-r--r--libs/binder/tests/binderRpcTest.cpp8
1 files changed, 6 insertions, 2 deletions
diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp
index 5952c4172e..7cd34a3b5a 100644
--- a/libs/binder/tests/binderRpcTest.cpp
+++ b/libs/binder/tests/binderRpcTest.cpp
@@ -465,7 +465,9 @@ TEST_P(BinderRpc, ThreadPoolOverSaturated) {
constexpr size_t kNumThreads = 10;
constexpr size_t kNumCalls = kNumThreads + 3;
auto proc = createRpcTestSocketServerProcess({.numThreads = kNumThreads});
- testThreadPoolOverSaturated(proc.rootIface, kNumCalls, 250 /*ms*/);
+
+ // b/272429574 - below 500ms, the test fails
+ testThreadPoolOverSaturated(proc.rootIface, kNumCalls, 500 /*ms*/);
}
TEST_P(BinderRpc, ThreadPoolLimitOutgoing) {
@@ -478,7 +480,9 @@ TEST_P(BinderRpc, ThreadPoolLimitOutgoing) {
constexpr size_t kNumCalls = kNumOutgoingConnections + 3;
auto proc = createRpcTestSocketServerProcess(
{.numThreads = kNumThreads, .numOutgoingConnections = kNumOutgoingConnections});
- testThreadPoolOverSaturated(proc.rootIface, kNumCalls, 250 /*ms*/);
+
+ // b/272429574 - below 500ms, the test fails
+ testThreadPoolOverSaturated(proc.rootIface, kNumCalls, 500 /*ms*/);
}
TEST_P(BinderRpc, ThreadingStressTest) {