Linux v4.12.8
This commit is contained in:
parent
23b162e846
commit
8814d15255
|
@ -0,0 +1,83 @@
|
|||
From eafad73ed3851707fa6e3124a255fc049ff9545d Mon Sep 17 00:00:00 2001
|
||||
From: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
|
||||
Date: Sat, 12 Aug 2017 09:09:21 -0700
|
||||
Subject: iio: hid-sensor-trigger: Fix the race with user space powering up
|
||||
sensors
|
||||
|
||||
It has been reported for a while that with iio-sensor-proxy service the
|
||||
rotation only works after one suspend/resume cycle. This required a wait
|
||||
in the systemd unit file to avoid race. I found a Yoga 900 where I could
|
||||
reproduce this.
|
||||
|
||||
The problem scenerio is:
|
||||
- During sensor driver init, enable run time PM and also set a
|
||||
auto-suspend for 3 seconds.
|
||||
This result in one runtime resume. But there is a check to avoid
|
||||
a powerup in this sequence, but rpm is active
|
||||
- User space iio-sensor-proxy tries to power up the sensor. Since rpm is
|
||||
active it will simply return. But sensors were not actually
|
||||
powered up in the prior sequence, so actaully the sensors will not work
|
||||
- After 3 seconds the auto suspend kicks
|
||||
|
||||
If we add a wait in systemd service file to fire iio-sensor-proxy after
|
||||
3 seconds, then now everything will work as the runtime resume will
|
||||
actually powerup the sensor as this is a user request.
|
||||
|
||||
To avoid this:
|
||||
- Remove the check to match user requested state, this will cause a
|
||||
brief powerup, but if the iio-sensor-proxy starts immediately it will
|
||||
still work as the sensors are ON.
|
||||
- Also move the autosuspend delay to place when user requested turn off
|
||||
of sensors, like after user finished raw read or buffer disable
|
||||
|
||||
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
|
||||
Tested-by: Bastien Nocera <hadess@hadess.net>
|
||||
Cc: <Stable@vger.kernel.org>
|
||||
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
|
||||
---
|
||||
drivers/iio/common/hid-sensors/hid-sensor-trigger.c | 8 ++++----
|
||||
1 file changed, 4 insertions(+), 4 deletions(-)
|
||||
|
||||
diff --git a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
|
||||
index 16ade0a..0e4b379 100644
|
||||
--- a/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
|
||||
+++ b/drivers/iio/common/hid-sensors/hid-sensor-trigger.c
|
||||
@@ -111,8 +111,6 @@ static int _hid_sensor_power_state(struct hid_sensor_common *st, bool state)
|
||||
s32 poll_value = 0;
|
||||
|
||||
if (state) {
|
||||
- if (!atomic_read(&st->user_requested_state))
|
||||
- return 0;
|
||||
if (sensor_hub_device_open(st->hsdev))
|
||||
return -EIO;
|
||||
|
||||
@@ -161,6 +159,9 @@ static int _hid_sensor_power_state(struct hid_sensor_common *st, bool state)
|
||||
&report_val);
|
||||
}
|
||||
|
||||
+ pr_debug("HID_SENSOR %s set power_state %d report_state %d\n",
|
||||
+ st->pdev->name, state_val, report_val);
|
||||
+
|
||||
sensor_hub_get_feature(st->hsdev, st->power_state.report_id,
|
||||
st->power_state.index,
|
||||
sizeof(state_val), &state_val);
|
||||
@@ -182,6 +183,7 @@ int hid_sensor_power_state(struct hid_sensor_common *st, bool state)
|
||||
ret = pm_runtime_get_sync(&st->pdev->dev);
|
||||
else {
|
||||
pm_runtime_mark_last_busy(&st->pdev->dev);
|
||||
+ pm_runtime_use_autosuspend(&st->pdev->dev);
|
||||
ret = pm_runtime_put_autosuspend(&st->pdev->dev);
|
||||
}
|
||||
if (ret < 0) {
|
||||
@@ -285,8 +287,6 @@ int hid_sensor_setup_trigger(struct iio_dev *indio_dev, const char *name,
|
||||
/* Default to 3 seconds, but can be changed from sysfs */
|
||||
pm_runtime_set_autosuspend_delay(&attrb->pdev->dev,
|
||||
3000);
|
||||
- pm_runtime_use_autosuspend(&attrb->pdev->dev);
|
||||
-
|
||||
return ret;
|
||||
error_unreg_trigger:
|
||||
iio_trigger_unregister(trig);
|
||||
--
|
||||
cgit v1.1
|
||||
|
10
kernel.spec
10
kernel.spec
|
@ -54,7 +54,7 @@ Summary: The Linux kernel
|
|||
%if 0%{?released_kernel}
|
||||
|
||||
# Do we have a -stable update to apply?
|
||||
%define stable_update 7
|
||||
%define stable_update 8
|
||||
# Set rpm version accordingly
|
||||
%if 0%{?stable_update}
|
||||
%define stablerev %{stable_update}
|
||||
|
@ -678,6 +678,10 @@ Patch707: Back-out-qxl-atomic-delay.patch
|
|||
# CVE-2017-12134 rhbz 1477656 1481786
|
||||
Patch708: xsa229.patch
|
||||
|
||||
# request for bug fix
|
||||
Patch709: iio-race-fix.patch
|
||||
|
||||
|
||||
# END OF PATCH DEFINITIONS
|
||||
|
||||
%endif
|
||||
|
@ -2251,6 +2255,10 @@ fi
|
|||
#
|
||||
#
|
||||
%changelog
|
||||
* Thu Aug 17 2017 Justin M. Forbes <jforbes@fedoraproject.org> - 4.12.8-200
|
||||
- Linux v4.12.8
|
||||
- Fix for iio race
|
||||
|
||||
* Wed Aug 16 2017 Justin M. Forbes <jforbes@fedoraproject.org>
|
||||
- Fix xen CVE-2017-12134 (rhbz 1477656 1481786)
|
||||
|
||||
|
|
2
sources
2
sources
|
@ -1,3 +1,3 @@
|
|||
SHA512 (perf-man-4.12.tar.gz) = 4d3bbda1f520dba0007c351af46f45085fe4842074eb2e01aee736fd369df595f8f72ed6c1192715f1120bf3353279777f9dca1178fe93bffe5be2de700d409c
|
||||
SHA512 (linux-4.12.tar.xz) = 8e81b41b253e63233e92948941f44c6482acb52aa3a3fd172f03a38a86f2c35b2ad4fd407acd1bc3964673eba344fe104d3a03e3ff4bf9cd1f22bd44263bd728
|
||||
SHA512 (patch-4.12.7.xz) = 22d6b937796298e9bb83d216b5cfa8b6910c8efe7bf5c4628c5fac42f73f916a5ba29b519fed1007542faa033c39d34175961731dfae88cd36c29fc6177fddcf
|
||||
SHA512 (patch-4.12.8.xz) = 44e2417fe1379977b4891f74b227c2c6d6588f3726b598ad8c8a52ecaea32f56cf222b877230fc87c043583a6f230f3044fe3c9b293c67699e1743c0161004cf
|
||||
|
|
Loading…
Reference in New Issue