diff options
author | OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> | 2009-12-03 13:49:43 +0100 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2009-12-03 13:49:43 +0100 |
commit | bf7ec5bb6114b2f086e536e24486fdacd1c0d339 (patch) | |
tree | 8f42b40fed799332e94142a084fed158f56a4d1a /mm/backing-dev.c | |
parent | 220d0b1dbf78c6417a658c96e571415552d3abac (diff) | |
download | lwn-bf7ec5bb6114b2f086e536e24486fdacd1c0d339.tar.gz lwn-bf7ec5bb6114b2f086e536e24486fdacd1c0d339.zip |
flusher: Fix PF_FROZEN race
To touch task->flags directly is racy. thaw_process() still has race
(changing non_current->flags, but this is another issue) though, I think
it's much better off.
So, use thaw_process() instead.
Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'mm/backing-dev.c')
-rw-r--r-- | mm/backing-dev.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/backing-dev.c b/mm/backing-dev.c index 67a33a5a1a93..0e8ca0347707 100644 --- a/mm/backing-dev.c +++ b/mm/backing-dev.c @@ -609,7 +609,7 @@ static void bdi_wb_shutdown(struct backing_dev_info *bdi) * it would never exet if it is currently stuck in the refrigerator. */ list_for_each_entry(wb, &bdi->wb_list, list) { - wb->task->flags &= ~PF_FROZEN; + thaw_process(wb->task); kthread_stop(wb->task); } } |