diff options
author | OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> | 2009-12-03 13:49:43 +0100 |
---|---|---|
committer | Gary King <gking@nvidia.com> | 2010-06-11 09:43:30 -0700 |
commit | 57f14c05b3cc857af5ccca9af698cfaafb2b831a (patch) | |
tree | 3ab590999058d72105ca23534d1dca3fdb9df103 | |
parent | 18362ebbdb53cca48ec201bece4e3a31680671e4 (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>
Change-Id: Ifcd4aa521a7bb1f3b263ac4dbb3c483deb935ae8
Reviewed-on: http://git-master/r/2423
Reviewed-by: Gary King <gking@nvidia.com>
Tested-by: Gary King <gking@nvidia.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) * 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); } } |