]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
vfs: fix page locking deadlocks when deduping files
authorDarrick J. Wong <darrick.wong@oracle.com>
Sun, 11 Aug 2019 22:52:25 +0000 (15:52 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 10 Sep 2019 09:32:15 +0000 (10:32 +0100)
commit1aca2008e22f3735031b2630d289dfafec8abbf2
tree8cf47cc0f9b853269797571b8b0bb1d2c0ac3183
parentc62760f0c614642ec57bc71f32a4f825cffa98da
vfs: fix page locking deadlocks when deduping files

[ Upstream commit edc58dd0123b552453a74369bd0c8d890b497b4b ]

When dedupe wants to use the page cache to compare parts of two files
for dedupe, we must be very careful to handle locking correctly.  The
current code doesn't do this.  It must lock and unlock the page only
once if the two pages are the same, since the overlapping range check
doesn't catch this when blocksize < pagesize.  If the pages are distinct
but from the same file, we must observe page locking order and lock them
in order of increasing offset to avoid clashing with writeback locking.

Fixes: 876bec6f9bbfcb3 ("vfs: refactor clone/dedupe_file_range common functions")
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Bill O'Donnell <billodo@redhat.com>
Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
fs/read_write.c