]> git.baikalelectronics.ru Git - kernel.git/commit
btrfs: skip waiting on ordered range for special files
authorJeff Mahoney <jeffm@suse.com>
Sat, 12 Sep 2015 01:44:17 +0000 (21:44 -0400)
committerFilipe Manana <fdmanana@suse.com>
Tue, 15 Sep 2015 01:21:08 +0000 (02:21 +0100)
commitf51c02a0230b1f8df5a28e3c198e8f8747ac95ed
tree274db6c41241c782ba3cdd9895c7e4f529797bac
parent534293c17354b839b6624648791dbf3029822668
btrfs: skip waiting on ordered range for special files

In btrfs_evict_inode, we properly truncate the page cache for evicted
inodes but then we call btrfs_wait_ordered_range for every inode as well.
It's the right thing to do for regular files but results in incorrect
behavior for device inodes for block devices.

filemap_fdatawrite_range gets called with inode->i_mapping which gets
resolved to the block device inode before getting passed to
wbc_attach_fdatawrite_inode and ultimately to inode_to_bdi.  What happens
next depends on whether there's an open file handle associated with the
inode.  If there is, we write to the block device, which is unexpected
behavior.  If there isn't, we through normally and inode->i_data is used.
We can also end up racing against open/close which can result in crashes
when i_mapping points to a block device inode that has been closed.

Since there can't be any page cache associated with special file inodes,
it's safe to skip the btrfs_wait_ordered_range call entirely and avoid
the problem.

Cc: <stable@vger.kernel.org>
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=100911
Tested-by: Christoph Biedl <linux-kernel.bfrz@manchmal.in-ulm.de>
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Reviewed-by: Filipe Manana <fdmanana@suse.com>
fs/btrfs/inode.c