commit 7c9d8958aaa963419a415fff863a6c322e38cfc2
author Stan Rokita <srok@google.com> Thu Aug 01 14:24:25 2019 -0700
committer Stan Rokita <srok@google.com> Mon Aug 05 13:48:51 2019 -0700
tree 45e0d8a4351ad49ea739b052641cf3bdfb31686a
parent 410180fcac53b1031f35ff71fec6ae8fc1f7de50

Skip SensorDevice disable sensors for HAL 2.0

SystemServerTimingAsync_StartSensorService_avg
Prefix -> 43.111111111111114O ms
Postfix -> 10.777777777777779 ms

Bug: 133859966
Test: Pushed libsensorservice onto local device. Observed that
code flow chose to skip disabling sensors for HAL 2.0 and framework
seemed to behave properly. Ran atest boottime and observed change in
sensor service start time.

Change-Id: I99f3d38b62e4bf73aeb76be55a1b6b8e72052215

services/sensorservice/SensorDevice.cpp

diff --git a/services/sensorservice/SensorDevice.cpp b/services/sensorservice/SensorDevice.cpp
index 717f317..c7a8f5b 100644
--- a/services/sensorservice/SensorDevice.cpp
+++ b/services/sensorservice/SensorDevice.cpp
@@ -134,7 +134,12 @@
 
                     mActivationCount.add(list[i].sensorHandle, model);
 
-                    checkReturn(mSensors->activate(list[i].sensorHandle, 0 /* enabled */));
+                    // Only disable all sensors on HAL 1.0 since HAL 2.0
+                    // handles this in its initialize method
+                    if (!mSensors->supportsMessageQueues()) {
+                        checkReturn(mSensors->activate(list[i].sensorHandle,
+                                    0 /* enabled */));
+                    }
                 }
             }));
 }