commit a4c39654219a7ed7ee6bb3ef5acbb1afd32a4a48
author Pawin Vongmasa <pawin@google.com> Tue Jun 30 09:30:58 2020 -0700
committer Pawin Vongmasa <pawin@google.com> Tue Jun 30 16:47:24 2020 +0000
tree c6f8ea77b418a0971d6252dc2948b0961024a835
parent ae87d21ce7b5a6b76a88fdefbe80ba0e3120b16a

mediaserver: add more hwbinder threads

Since we're now using HIDL IGraphicBufferSource in CCodec, the
existence of a callstack with mixed binder and hwbinder calls can cause
a stall when all threads are in use. (b/35283480) The recommended
workaround was to increase the number of threads. This CL does that.

Test: atest CtsMediaTestCases
Test: atest CtsCameraTestCases

Bug: 153828976
Change-Id: I37f02b6237ad032e311a8219b209713d2e4463a6

media/mediaserver/main_mediaserver.cpp

diff --git a/media/mediaserver/main_mediaserver.cpp b/media/mediaserver/main_mediaserver.cpp
index 7b22b05..316732b 100644
--- a/media/mediaserver/main_mediaserver.cpp
+++ b/media/mediaserver/main_mediaserver.cpp
@@ -22,6 +22,7 @@
 #include <binder/IPCThreadState.h>
 #include <binder/ProcessState.h>
 #include <binder/IServiceManager.h>
+#include <hidl/HidlTransportSupport.h>
 #include <utils/Log.h>
 #include "RegisterExtensions.h"
 
@@ -42,6 +43,8 @@
     MediaPlayerService::instantiate();
     ResourceManagerService::instantiate();
     registerExtensions();
+    ::android::hardware::configureRpcThreadpool(16, false);
     ProcessState::self()->startThreadPool();
     IPCThreadState::self()->joinThreadPool();
+    ::android::hardware::joinRpcThreadpool();
 }