Hi Christian, Steve,
Here are some miscellaneous fixes and changes for netfslib and cifs, if you
could consider pulling them. All the bugs fixed were observed in cifs, so
they should probably go through the cifs tree unless Christian would much
prefer for them to go through the VFS tree.
Many of these were found because a bug in Samba was causing smbd to crash
and restart after about 1-2s and this was vigorously and abruptly
exercising the netfslib retry paths.
Subsequent testing of the cifs RDMA support showed up some more bugs, for
which fixes are also included here.
First, there are some netfs fixes:
(1) Fix a hang due to missing case in final DIO read result collection
not breaking out of a loop if the request finished, but there were no
subrequests being processed and NETFS_RREQ_ALL_QUEUED wasn't yet set.
(2) Fix a double put of the netfs_io_request struct if completion happened
in the pause loop.
(3) Provide some helpers to abstract out NETFS_RREQ_IN_PROGRESS flag
wrangling.
(4) Fix infinite looping in netfs_wait_for_pause/request() which wa caused
by a loop waiting for NETFS_RREQ_ALL_QUEUED to get set - but which
wouldn't get set until the looping function returned. This uses patch
(3) above.
(5) Fix a ref leak on an extra subrequest inserted into a request's list
of subreqs because more subreq records were needed for retrying than
were needed for the original request (say, for instance, that the
amount of cifs credit available was reduced and, subsequently, the ops
had to be smaller).
Then a bunch of cifs fixes, some of which are from other people:
(6-8) cifs: Fix various RPC callbacks to set NETFS_SREQ_NEED_RETRY if a
subrequest fails retriably.
(9) Fix a regression with SMB symlinks.
(10) Fix a warning in the workqueue code when reconnecting a channel.
(11) Fix smbd_post_send_iter() to correctly respect the max_send_size and
to transmit all the data.
(12) Fix reading into an ITER_FOLIOQ from the smbdirect code.
(13) Fix the smbd_response slab to allow copy_to_iter() to be used on it
without incurring a bug from the usercopy hardening code.
(14) Fix a potential deadlock during channel reconnection.
And finally a couple of patches to improve tracing output, but that should
otherwise not affect functionality:
(15) Renumber the NETFS_RREQ_* flags to make the hex values easier to
interpret by eye, including moving the main status flags down to the
lowest bits, with IN_PROGRESS in bit 0.
(16) Update the tracepoints in a number of ways, including adding more
tracepoints into the cifs read/write RPC callback so that differend
MID_RESPONSE_* values can be differentiated.
Those last two could wait for the next merge window.
The patches can also be found here:
https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/log/?h=netfs-fixes
Thanks,
David
David Howells (9):
netfs: Fix hang due to missing case in final DIO read result
collection
netfs: Put double put of request
netfs: Provide helpers to perform NETFS_RREQ_IN_PROGRESS flag wangling
netfs: Fix looping in wait functions
netfs: Fix ref leak on inserted extra subreq in write retry
cifs: Fix reading into an ITER_FOLIOQ from the smbdirect code
cifs: Fix the smbd_reponse slab to allow usercopy
netfs: Renumber the NETFS_RREQ_* flags to make traces easier to read
netfs: Update tracepoints in a number of ways
Paulo Alcantara (6):
smb: client: set missing retry flag in smb2_writev_callback()
smb: client: set missing retry flag in cifs_readv_callback()
smb: client: set missing retry flag in cifs_writev_callback()
smb: client: fix regression with native SMB symlinks
smb: client: fix warning when reconnecting channel
smb: client: fix potential deadlock when reconnecting channels
Stefan Metzmacher (1):
smb: client: let smbd_post_send_iter() respect the peers max_send_size
and transmit all data
fs/netfs/direct_write.c | 1 -
fs/netfs/internal.h | 20 ++++-
fs/netfs/main.c | 6 +-
fs/netfs/misc.c | 50 +++++++----
fs/netfs/read_collect.c | 16 ++--
fs/netfs/write_collect.c | 8 +-
fs/netfs/write_retry.c | 3 +-
fs/smb/client/cifsglob.h | 2 +
fs/smb/client/cifssmb.c | 22 +++++
fs/smb/client/connect.c | 53 +++++++-----
fs/smb/client/reparse.c | 20 +----
fs/smb/client/smb2pdu.c | 37 +++++---
fs/smb/client/smbdirect.c | 163 +++++++++++++----------------------
include/linux/netfs.h | 20 ++---
include/trace/events/netfs.h | 29 +++++--
15 files changed, 245 insertions(+), 205 deletions(-)
