mbox series

[v3,0/6] NFSD: Improvements for the NFSv4.2 READ_PLUS operation

Message ID 20220715184433.838521-1-anna@kernel.org (mailing list archive)
Headers show
Series NFSD: Improvements for the NFSv4.2 READ_PLUS operation | expand

Message

Anna Schumaker July 15, 2022, 6:44 p.m. UTC 
From: Anna Schumaker <Anna.Schumaker@Netapp.com>

The main motivation for this patchset is fixing generic/091 and
generic/263 with READ_PLUS. These tests appear to be failing due to
files getting modified in the middle of reply encoding. Attempts to lock
the file for the entire encode result in a deadlock, since llseek() and
read() both need the file lock.

The solution is to read everything from disk at once, and then check if
each buffer page is all zeroes or not. As a bonus, this lets us support
READ_PLUS hole segments on filesystems that don't track sparse files.
Additionally, this also solves the performance issues I hit when testing
using btrfs on a virtual machine.

I created a wiki page with the results of my performance testing here:
    https://wiki.linux-nfs.org/wiki/index.php/Read_Plus_May_2022

These patches should probably go in before the related client changes
as the client will also be changed to make use of the
xdr_stream_move_subsegment() function.

Changed in v3:
  - Respond to as many of Chuck's comments as possible

Changed in v2:
  - Update to v5.19-rc6
  - Rename xdr_stream_move_segment() -> xdr_stream_move_subsegment()

Thoughts?
Anna


Anna Schumaker (6):
  SUNRPC: Introduce xdr_stream_move_subsegment()
  SUNRPC: Introduce xdr_encode_double()
  SUNRPC: Introduce xdr_buf_trim_head()
  SUNRPC: Introduce xdr_buf_nth_page_address()
  SUNRPC: Export xdr_buf_pagecount()
  NFSD: Repeal and replace the READ_PLUS implementation

 fs/nfsd/nfs4xdr.c          | 219 ++++++++++++++++++++-----------------
 include/linux/sunrpc/xdr.h |   6 +
 net/sunrpc/xdr.c           | 102 +++++++++++++++++
 3 files changed, 227 insertions(+), 100 deletions(-)