writeback: Fix sync livelock due to b_dirty_time processing
authorJan Kara <jack@suse.cz>
Fri, 29 May 2020 14:08:58 +0000 (16:08 +0200)
committerJan Kara <jack@suse.cz>
Mon, 15 Jun 2020 07:18:45 +0000 (09:18 +0200)
commitf9cae926f35e8230330f28c7b743ad088611a8de
treeda547e89b99f9b31f8eb1db11ba82a00da9809b8
parent5afced3bf28100d81fb2fe7e98918632a08feaf5
writeback: Fix sync livelock due to b_dirty_time processing

When we are processing writeback for sync(2), move_expired_inodes()
didn't set any inode expiry value (older_than_this). This can result in
writeback never completing if there's steady stream of inodes added to
b_dirty_time list as writeback rechecks dirty lists after each writeback
round whether there's more work to be done. Fix the problem by using
sync(2) start time is inode expiry value when processing b_dirty_time
list similarly as for ordinarily dirtied inodes. This requires some
refactoring of older_than_this handling which simplifies the code
noticeably as a bonus.

Fixes: 0ae45f63d4ef ("vfs: add support for a lazytime mount option")
CC: stable@vger.kernel.org
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jan Kara <jack@suse.cz>
fs/fs-writeback.c
include/trace/events/writeback.h