Removed haspeer check and changed small method default limit
This check appears unecessary and is causing monkey failures. The race condition
this is supposed to work around doesn't appear to exist in this instance as the
thread_list_lock_ is held. This might be related to b/8050919. Tested on a Nexus
4: phone booted, multiple apps tested.
Also, updated the small method limit size to a value determined by shmooing a well
known benchmark.
Bug: 8050919
Change-Id: I0d98624926b830953af7136f43f0e2b57d930a99
diff --git a/src/thread_list.cc b/src/thread_list.cc
index ebb63dd..eacd848 100644
--- a/src/thread_list.cc
+++ b/src/thread_list.cc
@@ -466,10 +466,8 @@
all_threads_are_daemons = true;
MutexLock mu(self, *Locks::thread_list_lock_);
for (It it = list_.begin(), end = list_.end(); it != end; ++it) {
- // TODO: there's a race here with thread exit that's being worked around by checking if the
- // thread has a peer.
Thread* thread = *it;
- if (thread != self && thread->HasPeer() && !thread->IsDaemon()) {
+ if (thread != self && !thread->IsDaemon()) {
all_threads_are_daemons = false;
break;
}