diff options
| author | Amit Gud <amitgud@gmail.com> | 2009-04-02 17:01:27 -0400 |
|---|---|---|
| committer | Chris Mason <chris.mason@oracle.com> | 2009-04-02 17:01:27 -0400 |
| commit | b5555f77115bfba3630fefff2d4e68140b644171 (patch) | |
| tree | 5a2712105e9e5e493ebf01b9b382828d91e046b8 /fs/btrfs/async-thread.c | |
| parent | dccae99995089641fbac452ebc7f0cab18751ddb (diff) | |
| download | kernel_samsung_smdk4412-b5555f77115bfba3630fefff2d4e68140b644171.zip kernel_samsung_smdk4412-b5555f77115bfba3630fefff2d4e68140b644171.tar.gz kernel_samsung_smdk4412-b5555f77115bfba3630fefff2d4e68140b644171.tar.bz2 | |
Btrfs: fix race in worker_loop
Need to check kthread_should_stop after schedule_timeout() before calling
schedule(). This causes threads to sleep with potentially no one to wake them
up causing mount(2) to hang in btrfs_stop_workers waiting for threads to stop.
Signed-off-by: Amit Gud <gud@ksu.edu>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/async-thread.c')
| -rw-r--r-- | fs/btrfs/async-thread.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/btrfs/async-thread.c b/fs/btrfs/async-thread.c index c84ca1f..cba701d 100644 --- a/fs/btrfs/async-thread.c +++ b/fs/btrfs/async-thread.c @@ -195,6 +195,9 @@ again_locked: if (!list_empty(&worker->pending)) continue; + if (kthread_should_stop()) + break; + /* still no more work?, sleep for real */ spin_lock_irq(&worker->lock); set_current_state(TASK_INTERRUPTIBLE); @@ -208,7 +211,8 @@ again_locked: worker->working = 0; spin_unlock_irq(&worker->lock); - schedule(); + if (!kthread_should_stop()) + schedule(); } __set_current_state(TASK_RUNNING); } |