diff options
author | OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> | 2009-12-03 07:49:43 -0500 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2009-12-03 07:49:43 -0500 |
commit | bf7ec5bb6114b2f086e536e24486fdacd1c0d339 (patch) | |
tree | 8f42b40fed799332e94142a084fed158f56a4d1a | |
parent | 220d0b1dbf78c6417a658c96e571415552d3abac (diff) |
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>
-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) | |||
609 | * it would never exet if it is currently stuck in the refrigerator. | 609 | * it would never exet if it is currently stuck in the refrigerator. |
610 | */ | 610 | */ |
611 | list_for_each_entry(wb, &bdi->wb_list, list) { | 611 | list_for_each_entry(wb, &bdi->wb_list, list) { |
612 | wb->task->flags &= ~PF_FROZEN; | 612 | thaw_process(wb->task); |
613 | kthread_stop(wb->task); | 613 | kthread_stop(wb->task); |
614 | } | 614 | } |
615 | } | 615 | } |