]> git.itanic.dy.fi Git - linux-stable/commit
Revert "NFSD: Reschedule CB operations when backchannel rpc_clnt is shut down"
authorChuck Lever <chuck.lever@oracle.com>
Tue, 23 Apr 2024 21:48:43 +0000 (17:48 -0400)
committerChuck Lever <chuck.lever@oracle.com>
Wed, 24 Apr 2024 00:12:34 +0000 (20:12 -0400)
commit9c8ecb9308d8013ff9ac9d36fdd8ae746033b93c
tree7afca23c40cfe857f610e40ca7c61b661fc867c0
parent32cf5a4eda464d76d553ee3f1b06c4d33d796c52
Revert "NFSD: Reschedule CB operations when backchannel rpc_clnt is shut down"

The reverted commit attempted to enable NFSD to retransmit pending
callback operations if an NFS client disconnects, but
unintentionally introduces a hazardous behavior regression if the
client becomes permanently unreachable while callback operations are
still pending.

A disconnect can occur due to network partition or if the NFS server
needs to force the NFS client to retransmit (for example, if a GSS
window under-run occurs).

Reverting the commit will make NFSD behave the same as it did in
v6.8 and before. Pending callback operations are permanently lost if
the client connection is terminated before the client receives them.

For some callback operations, this loss is not harmful.

However, for CB_RECALL, the loss means a delegation might be revoked
unnecessarily. For CB_OFFLOAD, pending COPY operations will never
complete unless the NFS client subsequently sends an OFFLOAD_STATUS
operation, which the Linux NFS client does not currently implement.

These issues still need to be addressed somehow.

Reported-by: Dai Ngo <dai.ngo@oracle.com>
Link: https://bugzilla.kernel.org/show_bug.cgi?id=218735
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4callback.c