commit 19601aa29ec9dc950fc25d4473c2b006670f7b3a
author Miao Wang <miaowang@google.com> Thu Apr 21 20:36:57 2016 -0700
committer Miao Wang <miaowang@google.com> Thu Apr 21 20:39:59 2016 -0700
tree 7c8ae53370a2c1243d8e470a03678e1ee26959e5
parent f620fbf0cb0489d02323d615202d718c9b383ae6

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

Bug: 23109602
Change-Id: Ifb6cf75009c0533439b687108b20f100abdcfac0

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;