aboutsummaryrefslogtreecommitdiffstats
path: root/fs/fs-writeback.c
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2011-01-13 18:45:47 -0500
committerLinus Torvalds <torvalds@linux-foundation.org>2011-01-13 20:32:32 -0500
commitaa373cf550994623efb5d49a4d8775bafd10bbc1 (patch)
tree64b14d1f5491f436a8b3ea5c32edecde5c255285 /fs/fs-writeback.c
parent71927e84e0aebfbe5a91565c3b207af25a4e9162 (diff)
writeback: stop background/kupdate works from livelocking other works
Background writeback is easily livelockable in a loop in wb_writeback() by a process continuously re-dirtying pages (or continuously appending to a file). This is in fact intended as the target of background writeback is to write dirty pages it can find as long as we are over dirty_background_threshold. But the above behavior gets inconvenient at times because no other work queued in the flusher thread's queue gets processed. In particular, since e.g. sync(1) relies on flusher thread to do all the IO for it, sync(1) can hang forever waiting for flusher thread to do the work. Generally, when a flusher thread has some work queued, someone submitted the work to achieve a goal more specific than what background writeback does. Moreover by working on the specific work, we also reduce amount of dirty pages which is exactly the target of background writeout. So it makes sense to give specific work a priority over a generic page cleaning. Thus we interrupt background writeback if there is some other work to do. We return to the background writeback after completing all the queued work. This may delay the writeback of expired inodes for a while, however the expired inodes will eventually be flushed to disk as long as the other works won't livelock. [fengguang.wu@intel.com: update comment] Signed-off-by: Jan Kara <jack@suse.cz> Signed-off-by: Wu Fengguang <fengguang.wu@intel.com> Cc: Johannes Weiner <hannes@cmpxchg.org> Cc: Dave Chinner <david@fromorbit.com> Cc: Christoph Hellwig <hch@lst.de> Cc: Jan Engelhardt <jengelh@medozas.de> Cc: Jens Axboe <axboe@kernel.dk> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/fs-writeback.c')
-rw-r--r--fs/fs-writeback.c10
1 files changed, 10 insertions, 0 deletions
diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c
index 482de0a92ca7..9e72d04e706e 100644
--- a/fs/fs-writeback.c
+++ b/fs/fs-writeback.c
@@ -651,6 +651,16 @@ static long wb_writeback(struct bdi_writeback *wb,
651 break; 651 break;
652 652
653 /* 653 /*
654 * Background writeout and kupdate-style writeback may
655 * run forever. Stop them if there is other work to do
656 * so that e.g. sync can proceed. They'll be restarted
657 * after the other works are all done.
658 */
659 if ((work->for_background || work->for_kupdate) &&
660 !list_empty(&wb->bdi->work_list))
661 break;
662
663 /*
654 * For background writeout, stop when we are below the 664 * For background writeout, stop when we are below the
655 * background dirty threshold 665 * background dirty threshold
656 */ 666 */