diff options
author | Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> | 2023-09-15 03:05:59 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2023-09-15 03:05:59 +0000 |
commit | 624d91447d4d9839429cff011e7c731e139e44b1 (patch) | |
tree | b6996949fa06cd751913185fd4020355127d38c2 | |
parent | a86c114dd34a3b08a3c5201ce352bfc6c025c63c (diff) | |
parent | a081c2ffd08a131279da20a1d4d4cc38354ab243 (diff) | |
download | libhardware-624d91447d4d9839429cff011e7c731e139e44b1.tar.gz |
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>
-rw-r--r-- | modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp | 12 | ||||
-rw-r--r-- | modules/sensors/dynamic_sensor/HidRawSensorDevice.h | 1 |
2 files changed, 1 insertions, 12 deletions
diff --git a/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp b/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp index 5bc4abc2..8aa4cf92 100644 --- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp +++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.cpp @@ -35,15 +35,6 @@ 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 @@ -83,8 +74,7 @@ HidRawSensorDevice::HidRawSensorDevice(const std::string &devName) return; } - run("HidRawSensor", PRIORITY_URGENT_DISPLAY); - enableSchedFifoMode(); + run("HidRawSensor"); mValid = true; } diff --git a/modules/sensors/dynamic_sensor/HidRawSensorDevice.h b/modules/sensors/dynamic_sensor/HidRawSensorDevice.h index 7818bf16..06d435e7 100644 --- a/modules/sensors/dynamic_sensor/HidRawSensorDevice.h +++ b/modules/sensors/dynamic_sensor/HidRawSensorDevice.h @@ -41,7 +41,6 @@ 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; |