diff options
author | Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> | 2023-09-19 01:52:30 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2023-09-19 01:52:30 +0000 |
commit | d8625365ec20038cb8fd0fc093b88dd461529d98 (patch) | |
tree | 041e9e6dfdc6892044c81e6ead5f247cbb017e1e | |
parent | 6c52b7903aae21151018f45bf2c56ea16eb98dae (diff) | |
parent | e8ed3f29dd8bc7dadd1f89ea3fd5408c5515916c (diff) | |
download | libhardware-d8625365ec20038cb8fd0fc093b88dd461529d98.tar.gz |
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>
-rw-r--r-- | modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp | 12 | ||||
-rw-r--r-- | modules/sensors/dynamic_sensor/HidRawSensorDevice.h | 1 |
2 files changed, 12 insertions, 1 deletions
diff --git a/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp b/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp index 8aa4cf92..5bc4abc2 100644 --- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp +++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp @@ -35,6 +35,15 @@ using namespace Hid::Sensor::PropertyUsage; 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, ¶m) != 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 @@ HidRawSensorDevice::HidRawSensorDevice(const std::string &devName) return; } - run("HidRawSensor"); + run("HidRawSensor", PRIORITY_URGENT_DISPLAY); + enableSchedFifoMode(); mValid = true; } diff --git a/modules/sensors/dynamic_sensor/HidRawSensorDevice.h b/modules/sensors/dynamic_sensor/HidRawSensorDevice.h index 06d435e7..7818bf16 100644 --- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.h +++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.h @@ -41,6 +41,7 @@ private: // 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; |