summaryrefslogtreecommitdiff
path: root/mm/backing-dev.c
diff options
context:
space:
mode:
authorOGAWA Hirofumi <hirofumi@mail.parknet.co.jp>2009-12-03 13:49:43 +0100
committerJens Axboe <jens.axboe@oracle.com>2009-12-03 13:49:43 +0100
commitbf7ec5bb6114b2f086e536e24486fdacd1c0d339 (patch)
tree8f42b40fed799332e94142a084fed158f56a4d1a /mm/backing-dev.c
parent220d0b1dbf78c6417a658c96e571415552d3abac (diff)
downloadconfigs-bf7ec5bb6114b2f086e536e24486fdacd1c0d339.tar.gz
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.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/backing-dev.c b/mm/backing-dev.c
index 67a33a5a1a9..0e8ca034770 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);
}
}