Commit 74da9d88 authored by Andrew Morton's avatar Andrew Morton Committed by David S. Miller

[BLUETOOTH] rfcomm_worker(): fix wakeup race

Set TASK_INTERRUPTIBLE prior to testing the flag to avoid missed wakeups.
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Acked-by: default avatarMarcel Holtmann <marcel@holtmann.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 9198d222
...@@ -1851,18 +1851,18 @@ static void rfcomm_worker(void) ...@@ -1851,18 +1851,18 @@ static void rfcomm_worker(void)
BT_DBG(""); BT_DBG("");
while (!atomic_read(&terminate)) { while (!atomic_read(&terminate)) {
set_current_state(TASK_INTERRUPTIBLE);
if (!test_bit(RFCOMM_SCHED_WAKEUP, &rfcomm_event)) { if (!test_bit(RFCOMM_SCHED_WAKEUP, &rfcomm_event)) {
/* No pending events. Let's sleep. /* No pending events. Let's sleep.
* Incoming connections and data will wake us up. */ * Incoming connections and data will wake us up. */
set_current_state(TASK_INTERRUPTIBLE);
schedule(); schedule();
} }
set_current_state(TASK_RUNNING);
/* Process stuff */ /* Process stuff */
clear_bit(RFCOMM_SCHED_WAKEUP, &rfcomm_event); clear_bit(RFCOMM_SCHED_WAKEUP, &rfcomm_event);
rfcomm_process_sessions(); rfcomm_process_sessions();
} }
set_current_state(TASK_RUNNING);
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment