commit 94aaec95445ac06ed5ec065a7e21ddd752652cee
author Vladimir Marko <vmarko@google.com> Wed Jul 06 12:12:28 2016 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed Jul 06 12:12:28 2016 +0000
tree 61913b1a2ba67d47887b1c4860df878557b064c6
parent ed22d481c12f1e03ea2ce78a0ce71f9d219892ee
parent d778cd68c5d093c2fb95f91608deb36fea8e8bad

Merge "Expect spurious wake-up from futex(2)/FUTEX_WAIT call."

runtime/thread_list.cc

diff --git a/runtime/thread_list.cc b/runtime/thread_list.cc
index 97bcb7d..16ef0ff 100644
--- a/runtime/thread_list.cc
+++ b/runtime/thread_list.cc
@@ -613,11 +613,7 @@
             PLOG(FATAL) << "futex wait failed for SuspendAllInternal()";
           }
         }
-      } else {
-        cur_val = pending_threads.LoadRelaxed();
-        CHECK_EQ(cur_val, 0);
-        break;
-      }
+      }  // else re-check pending_threads in the next iteration (this may be a spurious wake-up).
 #else
       // Spin wait. This is likely to be slow, but on most architecture ART_USE_FUTEXES is set.
 #endif