diff options
author | Jan Kara <jack@suse.cz> | 2016-07-04 10:14:01 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2016-07-04 10:14:01 -0400 |
commit | 646caa9c8e196880b41cd3e3d33a2ebc752bdb85 (patch) | |
tree | 28d3c6dcf43b1ba8e8d1b5eff334b9c07b098db2 /kernel/bpf/arraymap.c | |
parent | fa96454069b85a7e5d10f38b7d95edcd5dc64b9a (diff) |
ext4: fix deadlock during page writeback
Commit 06bd3c36a733 (ext4: fix data exposure after a crash) uncovered a
deadlock in ext4_writepages() which was previously much harder to hit.
After this commit xfstest generic/130 reproduces the deadlock on small
filesystems.
The problem happens when ext4_do_update_inode() sets LARGE_FILE feature
and marks current inode handle as synchronous. That subsequently results
in ext4_journal_stop() called from ext4_writepages() to block waiting for
transaction commit while still holding page locks, reference to io_end,
and some prepared bio in mpd structure each of which can possibly block
transaction commit from completing and thus results in deadlock.
Fix the problem by releasing page locks, io_end reference, and
submitting prepared bio before calling ext4_journal_stop().
[ Changed to defer the call to ext4_journal_stop() only if the handle
is synchronous. --tytso ]
Reported-and-tested-by: Eryu Guan <eguan@redhat.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
CC: stable@vger.kernel.org
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'kernel/bpf/arraymap.c')
0 files changed, 0 insertions, 0 deletions