commit 1092bf3003b4bc83f2cf613b769a72c13c6e3bca
author Mathieu Chartier <mathieuc@google.com> Tue Sep 10 12:39:55 2019 -0700
committer Mathieu Chartier <mathieuc@google.com> Tue Sep 10 12:39:55 2019 -0700
tree 9d7738d96439362462aaa1aab65b318b8e17266d
parent 5c93af3cc8894deefbc9cba69ea69526c45842aa

Avoid creating runtime thread pool for system server

The thread pool is not required and would never be deleted since
there is NotifyStartupCompleted is not called.

Bug: 140777418
Test: TH
Change-Id: Ic6a83812a1d389730423796ddc01ee49e13cd91b

runtime/runtime.cc

diff --git a/runtime/runtime.cc b/runtime/runtime.cc
index 766782d..e7ce33a 100644
--- a/runtime/runtime.cc
+++ b/runtime/runtime.cc
@@ -1002,7 +1002,9 @@
 
   // Create the thread pools.
   heap_->CreateThreadPool();
-  {
+  // Avoid creating the runtime thread pool for system server since it will not be used and would
+  // waste memory.
+  if (!is_system_server) {
     ScopedTrace timing("CreateThreadPool");
     constexpr size_t kStackSize = 64 * KB;
     constexpr size_t kMaxRuntimeWorkers = 4u;