commit 624d91447d4d9839429cff011e7c731e139e44b1
author Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> Fri Sep 15 03:05:59 2023 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Sep 15 03:05:59 2023 +0000
tree b6996949fa06cd751913185fd4020355127d38c2
parent a86c114dd34a3b08a3c5201ce352bfc6c025c63c
parent a081c2ffd08a131279da20a1d4d4cc38354ab243

Merge "Revert "Increase HidRawSensor thread priority to reduce latency"" into main am: 0d62798e8e am: a081c2ffd0

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

Change-Id: I0c11d7539a02b2dfc3bc7fffb7aa648a0b24c892
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 5bc4abc..8aa4cf9 100644
--- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp
+++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp
@@ -35,15 +35,6 @@
 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
@@ -83,8 +74,7 @@
         return;
     }
 
-    run("HidRawSensor", PRIORITY_URGENT_DISPLAY);
-    enableSchedFifoMode();
+    run("HidRawSensor");
     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 7818bf1..06d435e 100644
--- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.h
+++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.h
@@ -41,7 +41,6 @@
 
     // 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;