]> www.infradead.org Git - users/jedix/linux-maple.git/commit
cachefiles, netfs: Fix write to partial block at EOF
authorDavid Howells <dhowells@redhat.com>
Fri, 12 Jul 2024 11:44:30 +0000 (12:44 +0100)
committerChristian Brauner <brauner@kernel.org>
Thu, 12 Sep 2024 10:20:41 +0000 (12:20 +0200)
commitc4f1450ecccc5311db87f806998eda1c824c4e35
tree2e88bc87b79074ba2cf481d5209850818750b568
parent86b374d061ee0dc1cf15b56659ab13542518770a
cachefiles, netfs: Fix write to partial block at EOF

Because it uses DIO writes, cachefiles is unable to make a write to the
backing file if that write is not aligned to and sized according to the
backing file's DIO block alignment.  This makes it tricky to handle a write
to the cache where the EOF on the network file is not correctly aligned.

To get around this, netfslib attempts to tell the driver it is calling how
much more data there is available beyond the EOF that it can use to pad the
write (netfslib preclears the part of the folio above the EOF).  However,
it tries to tell the cache what the maximum length is, but doesn't
calculate this correctly; and, in any case, cachefiles actually ignores the
value and just skips the block.

Fix this by:

 (1) Change the value passed to indicate the amount of extra data that can
     be added to the operation (now ->submit_extendable_to).  This is much
     simpler to calculate as it's just the end of the folio minus the top
     of the data within the folio - rather than having to account for data
     spread over multiple folios.

 (2) Make cachefiles add some of this data if the subrequest it is given
     ends at the network file's i_size if the extra data is sufficient to
     pad out to a whole block.

Signed-off-by: David Howells <dhowells@redhat.com>
cc: Jeff Layton <jlayton@kernel.org>
cc: netfs@lists.linux.dev
cc: linux-fsdevel@vger.kernel.org
Link: https://lore.kernel.org/r/20240814203850.2240469-22-dhowells@redhat.com/
Signed-off-by: Christian Brauner <brauner@kernel.org>
fs/cachefiles/io.c
fs/netfs/read_pgpriv2.c
fs/netfs/write_issue.c
include/linux/netfs.h