writeback: Fix periodic writeback after fs mount
authorJan Kara <jack@suse.cz>
Fri, 28 Jun 2013 14:04:02 +0000 (16:04 +0200)
committerBen Hutchings <ben@decadent.org.uk>
Sat, 27 Jul 2013 04:34:22 +0000 (05:34 +0100)
commit5ba9a2e746033d2886d56d21b4298f5da7178d12
treed0bc6f15b01580084c749ce1cc2cc0936e03bb86
parente0a59ed79b08eea4ce3d88ddf8ef781265301b90
writeback: Fix periodic writeback after fs mount

commit a5faeaf9109578e65e1a32e2a3e76c8b47e7dcb6 upstream.

Code in blkdev.c moves a device inode to default_backing_dev_info when
the last reference to the device is put and moves the device inode back
to its bdi when the first reference is acquired. This includes moving to
wb.b_dirty list if the device inode is dirty. The code however doesn't
setup timer to wake corresponding flusher thread and while wb.b_dirty
list is non-empty __mark_inode_dirty() will not set it up either. Thus
periodic writeback is effectively disabled until a sync(2) call which can
lead to unexpected data loss in case of crash or power failure.

Fix the problem by setting up a timer for periodic writeback in case we
add the first dirty inode to wb.b_dirty list in bdev_inode_switch_bdi().

Reported-by: Bert De Jonghe <Bert.DeJonghe@amplidata.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
fs/block_dev.c