commit 574ac3e232b96a5639f29b1161d8643ae2c5c0d1
author Vladimir Komsiyski <vladokom@google.com> Fri Jul 28 07:13:46 2023 +0000
committer Vladimir Komsiyski <vladokom@google.com> Fri Jul 28 07:14:10 2023 +0000
tree 25263ba70e8c97e7724b0b1040f3ef868df8396b
parent 5df985132ea483627687c65e5f1c6b526d9e9720

Revert "Handle runtime sensor events even if there are no real ones."

This reverts commit 3b9574a68afb242d573f062ce8e0a99a452d5842.

Reason for revert: potential culprit for b/291461814

Bug: 291461814
Change-Id: Ib92a05135a05978d8d4e6a938df1c6fb5b8169e2
Test: m

services/sensorservice/SensorService.cpp

diff --git a/services/sensorservice/SensorService.cpp b/services/sensorservice/SensorService.cpp
index 90d7541..398d602 100644
--- a/services/sensorservice/SensorService.cpp
+++ b/services/sensorservice/SensorService.cpp
@@ -1055,12 +1055,7 @@
         if (count < 0) {
             if(count == DEAD_OBJECT && device.isReconnecting()) {
                 device.reconnect();
-                // There are no "real" events at this point, but do not skip the rest of the loop
-                // if there are pending runtime events.
-                Mutex::Autolock _l(&mLock);
-                if (mRuntimeSensorEventQueue.empty()) {
-                    continue;
-                }
+                continue;
             } else {
                 ALOGE("sensor poll failed (%s)", strerror(-count));
                 break;