diff options
author | Tomas Winkler <tomas.winkler@intel.com> | 2016-02-07 23:35:30 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-02-07 14:47:20 -0800 |
commit | a1f9ae2bd264e3aed95aacd0102bd22a0422b8d1 (patch) | |
tree | 9d089dc38c60b17344042bfc16bbc4eccd9a67f4 /drivers/misc/mei/client.c | |
parent | b74d883138911b110cf8f5b7b94790e16bb56795 (diff) | |
download | op-kernel-dev-a1f9ae2bd264e3aed95aacd0102bd22a0422b8d1.zip op-kernel-dev-a1f9ae2bd264e3aed95aacd0102bd22a0422b8d1.tar.gz |
mei: bus: fix RX event scheduling
In this particular case this more correct and safer to check if the RX
event is set in the event mask rather than query waitqueue_active
Since the check is already performed in the mei_cl_bus_rx_event
function, it is just required to check for its return value.
Second, since we don't have exclusive waiter wake_up_interruptible_all
is not used correctly here.
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/misc/mei/client.c')
-rw-r--r-- | drivers/misc/mei/client.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/misc/mei/client.c b/drivers/misc/mei/client.c index 3299168..574f739 100644 --- a/drivers/misc/mei/client.c +++ b/drivers/misc/mei/client.c @@ -1735,10 +1735,8 @@ void mei_cl_complete(struct mei_cl *cl, struct mei_cl_cb *cb) case MEI_FOP_READ: list_add_tail(&cb->list, &cl->rd_completed); - if (waitqueue_active(&cl->rx_wait)) - wake_up_interruptible_all(&cl->rx_wait); - else - mei_cl_bus_rx_event(cl); + if (!mei_cl_bus_rx_event(cl)) + wake_up_interruptible(&cl->rx_wait); break; case MEI_FOP_CONNECT: |