* [PATCH] HID: usbhid: fix "always poll" quirk
@ 2017-07-13 0:14 Dmitry Torokhov
2017-07-21 9:26 ` Benjamin Tissoires
2017-07-21 16:15 ` Jiri Kosina
0 siblings, 2 replies; 3+ messages in thread
From: Dmitry Torokhov @ 2017-07-13 0:14 UTC (permalink / raw)
To: Jiri Kosina
Cc: Benjamin Tissoires, Olof Johansson, linux-usb, linux-input, linux-kernel
Even though the IO for devices with "always poll" quirk is already running,
we still need to set HID_OPENED bit in usbhid->iofl so the interrupt
handler does not ignore the data coming from the device.
Reported-by: Olof Johansson <olof@lixom.net>
Tested-by: Olof Johansson <olof@lixom.net>
Fixes: e399396a6b0 ("HID: usbhid: remove custom locking from usbhid_open...")
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
drivers/hid/usbhid/hid-core.c | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/drivers/hid/usbhid/hid-core.c b/drivers/hid/usbhid/hid-core.c
index 76013eb5cb7f..c008847e0b20 100644
--- a/drivers/hid/usbhid/hid-core.c
+++ b/drivers/hid/usbhid/hid-core.c
@@ -680,18 +680,21 @@ static int usbhid_open(struct hid_device *hid)
struct usbhid_device *usbhid = hid->driver_data;
int res;
+ set_bit(HID_OPENED, &usbhid->iofl);
+
if (hid->quirks & HID_QUIRK_ALWAYS_POLL)
return 0;
res = usb_autopm_get_interface(usbhid->intf);
/* the device must be awake to reliably request remote wakeup */
- if (res < 0)
+ if (res < 0) {
+ clear_bit(HID_OPENED, &usbhid->iofl);
return -EIO;
+ }
usbhid->intf->needs_remote_wakeup = 1;
set_bit(HID_RESUME_RUNNING, &usbhid->iofl);
- set_bit(HID_OPENED, &usbhid->iofl);
set_bit(HID_IN_POLLING, &usbhid->iofl);
res = hid_start_in(hid);
@@ -727,19 +730,20 @@ static void usbhid_close(struct hid_device *hid)
{
struct usbhid_device *usbhid = hid->driver_data;
- if (hid->quirks & HID_QUIRK_ALWAYS_POLL)
- return;
-
/*
* Make sure we don't restart data acquisition due to
* a resumption we no longer care about by avoiding racing
* with hid_start_in().
*/
spin_lock_irq(&usbhid->lock);
- clear_bit(HID_IN_POLLING, &usbhid->iofl);
clear_bit(HID_OPENED, &usbhid->iofl);
+ if (!(hid->quirks & HID_QUIRK_ALWAYS_POLL))
+ clear_bit(HID_IN_POLLING, &usbhid->iofl);
spin_unlock_irq(&usbhid->lock);
+ if (hid->quirks & HID_QUIRK_ALWAYS_POLL)
+ return;
+
hid_cancel_delayed_stuff(usbhid);
usb_kill_urb(usbhid->urbin);
usbhid->intf->needs_remote_wakeup = 0;
--
2.13.2.932.g7449e964c-goog
--
Dmitry
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] HID: usbhid: fix "always poll" quirk
2017-07-13 0:14 [PATCH] HID: usbhid: fix "always poll" quirk Dmitry Torokhov
@ 2017-07-21 9:26 ` Benjamin Tissoires
2017-07-21 16:15 ` Jiri Kosina
1 sibling, 0 replies; 3+ messages in thread
From: Benjamin Tissoires @ 2017-07-21 9:26 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Jiri Kosina, Olof Johansson, linux-usb, linux-input, linux-kernel
On Jul 12 2017 or thereabouts, Dmitry Torokhov wrote:
> Even though the IO for devices with "always poll" quirk is already running,
> we still need to set HID_OPENED bit in usbhid->iofl so the interrupt
> handler does not ignore the data coming from the device.
>
> Reported-by: Olof Johansson <olof@lixom.net>
> Tested-by: Olof Johansson <olof@lixom.net>
> Fixes: e399396a6b0 ("HID: usbhid: remove custom locking from usbhid_open...")
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> ---
slightly reviewed and
Acked-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Cheers,
Benjamin
> drivers/hid/usbhid/hid-core.c | 16 ++++++++++------
> 1 file changed, 10 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/hid/usbhid/hid-core.c b/drivers/hid/usbhid/hid-core.c
> index 76013eb5cb7f..c008847e0b20 100644
> --- a/drivers/hid/usbhid/hid-core.c
> +++ b/drivers/hid/usbhid/hid-core.c
> @@ -680,18 +680,21 @@ static int usbhid_open(struct hid_device *hid)
> struct usbhid_device *usbhid = hid->driver_data;
> int res;
>
> + set_bit(HID_OPENED, &usbhid->iofl);
> +
> if (hid->quirks & HID_QUIRK_ALWAYS_POLL)
> return 0;
>
> res = usb_autopm_get_interface(usbhid->intf);
> /* the device must be awake to reliably request remote wakeup */
> - if (res < 0)
> + if (res < 0) {
> + clear_bit(HID_OPENED, &usbhid->iofl);
> return -EIO;
> + }
>
> usbhid->intf->needs_remote_wakeup = 1;
>
> set_bit(HID_RESUME_RUNNING, &usbhid->iofl);
> - set_bit(HID_OPENED, &usbhid->iofl);
> set_bit(HID_IN_POLLING, &usbhid->iofl);
>
> res = hid_start_in(hid);
> @@ -727,19 +730,20 @@ static void usbhid_close(struct hid_device *hid)
> {
> struct usbhid_device *usbhid = hid->driver_data;
>
> - if (hid->quirks & HID_QUIRK_ALWAYS_POLL)
> - return;
> -
> /*
> * Make sure we don't restart data acquisition due to
> * a resumption we no longer care about by avoiding racing
> * with hid_start_in().
> */
> spin_lock_irq(&usbhid->lock);
> - clear_bit(HID_IN_POLLING, &usbhid->iofl);
> clear_bit(HID_OPENED, &usbhid->iofl);
> + if (!(hid->quirks & HID_QUIRK_ALWAYS_POLL))
> + clear_bit(HID_IN_POLLING, &usbhid->iofl);
> spin_unlock_irq(&usbhid->lock);
>
> + if (hid->quirks & HID_QUIRK_ALWAYS_POLL)
> + return;
> +
> hid_cancel_delayed_stuff(usbhid);
> usb_kill_urb(usbhid->urbin);
> usbhid->intf->needs_remote_wakeup = 0;
> --
> 2.13.2.932.g7449e964c-goog
>
>
> --
> Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] HID: usbhid: fix "always poll" quirk
2017-07-13 0:14 [PATCH] HID: usbhid: fix "always poll" quirk Dmitry Torokhov
2017-07-21 9:26 ` Benjamin Tissoires
@ 2017-07-21 16:15 ` Jiri Kosina
1 sibling, 0 replies; 3+ messages in thread
From: Jiri Kosina @ 2017-07-21 16:15 UTC (permalink / raw)
To: Dmitry Torokhov
Cc: Benjamin Tissoires, Olof Johansson, linux-usb, linux-input, linux-kernel
On Wed, 12 Jul 2017, Dmitry Torokhov wrote:
> Even though the IO for devices with "always poll" quirk is already running,
> we still need to set HID_OPENED bit in usbhid->iofl so the interrupt
> handler does not ignore the data coming from the device.
>
> Reported-by: Olof Johansson <olof@lixom.net>
> Tested-by: Olof Johansson <olof@lixom.net>
> Fixes: e399396a6b0 ("HID: usbhid: remove custom locking from usbhid_open...")
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Ah, pitty I've missed this one for the yesterday pull request, sorry for
that. Applied now to for-4.13/upstream-fixes and will be sending upstream
next week.
Thanks,
--
Jiri Kosina
SUSE Labs
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-07-21 16:15 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-13 0:14 [PATCH] HID: usbhid: fix "always poll" quirk Dmitry Torokhov
2017-07-21 9:26 ` Benjamin Tissoires
2017-07-21 16:15 ` Jiri Kosina
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).