writeback: bdi_writeback_task() must set task state before calling schedule()
authorJens Axboe <jens.axboe@oracle.com>
Tue, 18 May 2010 12:31:45 +0000 (14:31 +0200)
committerJens Axboe <jens.axboe@oracle.com>
Fri, 21 May 2010 18:00:25 +0000 (20:00 +0200)
Calling schedule without setting the task state to non-running will
return immediately, so ensure that we set it properly and check our
sleep conditions after doing so.

This is a fixup for commit 69b62d01.

Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
fs/fs-writeback.c

index 76f546d..437a743 100644 (file)
@@ -978,8 +978,13 @@ int bdi_writeback_task(struct bdi_writeback *wb)
                if (dirty_writeback_interval) {
                        wait_jiffies = msecs_to_jiffies(dirty_writeback_interval * 10);
                        schedule_timeout_interruptible(wait_jiffies);
-               } else
-                       schedule();
+               } else {
+                       set_current_state(TASK_INTERRUPTIBLE);
+                       if (list_empty_careful(&wb->bdi->work_list) &&
+                           !kthread_should_stop())
+                               schedule();
+                       __set_current_state(TASK_RUNNING);
+               }
 
                try_to_freeze();
        }