commit d8625365ec20038cb8fd0fc093b88dd461529d98
author Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> Tue Sep 19 01:52:30 2023 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Tue Sep 19 01:52:30 2023 +0000
tree 041e9e6dfdc6892044c81e6ead5f247cbb017e1e
parent 6c52b7903aae21151018f45bf2c56ea16eb98dae
parent e8ed3f29dd8bc7dadd1f89ea3fd5408c5515916c

Merge "Revert "Revert "Increase HidRawSensor thread priority to reduce latency""" into main am: 095c6960f8 am: 60b7e3a576 am: 5f6bfea312 am: 7baa741ae2 am: e8ed3f29dd

Original change: https://android-review.googlesource.com/c/platform/hardware/libhardware/+/2755006

Change-Id: I12557277ac90d151bed7d4e6a0c954957a447e33
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp

diff --git a/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp b/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp
index 8aa4cf9..5bc4abc 100644
--- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp
+++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp
@@ -35,6 +35,15 @@
 const std::unordered_set<unsigned int> HidRawSensorDevice::sInterested{
         ACCELEROMETER_3D, GYROMETER_3D, COMPASS_3D, CUSTOM};
 
+void HidRawSensorDevice::enableSchedFifoMode() {
+  constexpr int kHidRawSensorPriority = 10;  // Matches with sensor service priority
+  struct sched_param param = {0};
+  param.sched_priority = kHidRawSensorPriority;
+  if (sched_setscheduler(getTid(), SCHED_FIFO | SCHED_RESET_ON_FORK, &param) != 0) {
+    ALOGE("Couldn't set SCHED_FIFO for HidRawSensor thread: %s", strerror(errno));
+  }
+}
+
 sp<HidRawSensorDevice> HidRawSensorDevice::create(const std::string &devName) {
     sp<HidRawSensorDevice> device(new HidRawSensorDevice(devName));
     // offset +1 strong count added by constructor
@@ -74,7 +83,8 @@
         return;
     }
 
-    run("HidRawSensor");
+    run("HidRawSensor", PRIORITY_URGENT_DISPLAY);
+    enableSchedFifoMode();
     mValid = true;
 }
 

modules/sensors/dynamic_sensor/HidRawSensorDevice.h

diff --git a/modules/sensors/dynamic_sensor/HidRawSensorDevice.h b/modules/sensors/dynamic_sensor/HidRawSensorDevice.h
index 06d435e..7818bf1 100644
--- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.h
+++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.h
@@ -41,6 +41,7 @@
 
     // constructor will result in +1 strong count
     explicit HidRawSensorDevice(const std::string &devName);
+    void enableSchedFifoMode();
     // implement function of Thread
     virtual bool threadLoop() override;
     std::unordered_map<unsigned int/*reportId*/, sp<HidRawSensor>> mSensors;