David Herrmann 0e0d752006 HID: uhid: simplify report-cb shutdown
The report-query is blocking, so when user-space destroys a device we have
to wake up any blocking kernel context that is currently in the report-cb.
We used some broken correlation between @report_done and @running so far.
Replace it by a much more obvious use.

We now wake up the report-cb if either @report_done or @running is set.
wake_up() and wait_event() serve as implicit barriers (as they always do)
so no need to use smp_rmb/wmb directly.

Note that @report_done is never reset by anyone but the report-cb, thus
it cannot flip twice while we wait for it. And whenever we set @running,
we afterwards synchronously remove the HID device. Therefore, we wait for
all report-cbs to finish before we return. This way, @running can never
flip to true while we wait for it.

Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
2014-08-25 03:28:05 -05:00
..
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2014-08-21 10:43:28 -05:00
2013-01-03 10:27:31 +01:00
2014-07-29 11:37:38 +02:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2013-04-24 16:32:27 +02:00
2014-08-21 10:43:28 -05:00
2013-01-03 10:27:31 +01:00
2014-08-21 10:43:28 -05:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2014-06-27 00:33:44 +02:00
2013-01-03 10:27:31 +01:00
2014-08-21 10:43:28 -05:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2013-01-03 10:27:31 +01:00
2014-08-25 03:28:05 -05:00
2014-08-06 23:36:12 -07:00