summaryrefslogtreecommitdiff
path: root/include/linux
diff options
context:
space:
mode:
authorDavid Howells <dhowells@redhat.com>2024-12-13 13:50:09 +0000
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2025-01-17 13:40:35 +0100
commit43b8d3249b0b71bad239d42dbe08ce6c938ba000 (patch)
treed20641ca2a4c9489549d35038c3a965da6ec453a /include/linux
parent2c3348864ab73f596f37ed85b9600616c992a8ee (diff)
downloadlinux-43b8d3249b0b71bad239d42dbe08ce6c938ba000.tar.gz
linux-43b8d3249b0b71bad239d42dbe08ce6c938ba000.tar.bz2
linux-43b8d3249b0b71bad239d42dbe08ce6c938ba000.zip
netfs: Fix ceph copy to cache on write-begin
[ Upstream commit 38cf8e945721ffe708fa675507465da7f4f2a9f7 ] At the end of netfs_unlock_read_folio() in which folios are marked appropriately for copying to the cache (either with by being marked dirty and having their private data set or by having PG_private_2 set) and then unlocked, the folio_queue struct has the entry pointing to the folio cleared. This presents a problem for netfs_pgpriv2_write_to_the_cache(), which is used to write folios marked with PG_private_2 to the cache as it expects to be able to trawl the folio_queue list thereafter to find the relevant folios, leading to a hang. Fix this by not clearing the folio_queue entry if we're going to do the deprecated copy-to-cache. The clearance will be done instead as the folios are written to the cache. This can be reproduced by starting cachefiles, mounting a ceph filesystem with "-o fsc" and writing to it. Fixes: 796a4049640b ("netfs: In readahead, put the folio refs as soon extracted") Reported-by: Max Kellermann <max.kellermann@ionos.com> Closes: https://lore.kernel.org/r/CAKPOu+_4m80thNy5_fvROoxBm689YtA0dZ-=gcmkzwYSY4syqw@mail.gmail.com/ Signed-off-by: David Howells <dhowells@redhat.com> Link: https://lore.kernel.org/r/20241213135013.2964079-10-dhowells@redhat.com Fixes: ee4cdf7ba857 ("netfs: Speed up buffered reading") cc: Jeff Layton <jlayton@kernel.org> cc: Ilya Dryomov <idryomov@gmail.com> cc: Xiubo Li <xiubli@redhat.com> cc: netfs@lists.linux.dev cc: ceph-devel@vger.kernel.org cc: linux-fsdevel@vger.kernel.org Signed-off-by: Christian Brauner <brauner@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions