]> www.infradead.org Git - users/jedix/linux-maple.git/commit
ext4: fix deadlock during page writeback
authorJan Kara <jack@suse.cz>
Mon, 4 Jul 2016 14:14:01 +0000 (10:14 -0400)
committerDhaval Giani <dhaval.giani@oracle.com>
Fri, 20 Jan 2017 22:22:06 +0000 (17:22 -0500)
commit4454919238de885938859f214f7d5b486aaaf2e8
tree9945c2c538bb38ba7d3ca14399b9e555e4ae230d
parentb70626101448ab749ff1bcd99a30ceec70c5b850
ext4: fix deadlock during page writeback

Orabug: 25308117

[ Upstream commit 646caa9c8e196880b41cd3e3d33a2ebc752bdb85 ]

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>
Signed-off-by: Sasha Levin <alexander.levin@verizon.com>
(cherry picked from commit 906d6f4d9cdc8509c505f29f6146ec627fef2f06)
Signed-off-by: Dhaval Giani <dhaval.giani@oracle.com>
fs/ext4/inode.c