commit 4a5400da3144775b6596d511e3e39a28765bd27d
author Miao Wang <miaowang@google.com> Fri Apr 22 17:00:05 2016 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Apr 22 17:00:05 2016 +0000
tree 35c96bb8ac77783fead17718e9dcb2e1d01a95fd
parent f0b048cb756781cd7cb9fcfa95166845eb1d59bc
parent 19601aa29ec9dc950fc25d4473c2b006670f7b3a

Merge "Add TODO for proper fix of the race condition during CPU reference driver teardown."

cpu_ref/rsCpuCore.cpp

diff --git a/cpu_ref/rsCpuCore.cpp b/cpu_ref/rsCpuCore.cpp
index 011b8e3..7e47a03 100644
--- a/cpu_ref/rsCpuCore.cpp
+++ b/cpu_ref/rsCpuCore.cpp
@@ -316,7 +316,10 @@
     for (uint32_t ct = 0; ct < mWorkers.mCount; ct++) {
         pthread_join(mWorkers.mThreadId[ct], &res);
     }
-    rsAssert(__sync_fetch_and_or(&mWorkers.mRunningCount, 0) == 0);
+    // b/23109602
+    // TODO: Refactor the implementation with threadpool to
+    // fix the race condition in the destuctor.
+    // rsAssert(__sync_fetch_and_or(&mWorkers.mRunningCount, 0) == 0);
     free(mWorkers.mThreadId);
     free(mWorkers.mNativeThreadId);
     delete[] mWorkers.mLaunchSignals;