Commit 060559ed authored by Chuck Lever's avatar Chuck Lever Committed by Li Lingfeng
Browse files

NFSD: Reset cb_seq_status after NFS4ERR_DELAY 

mainline inclusion
from mainline-v6.9-rc1
commit 961b4b5e86bf56a2e4b567f81682defa5cba957e
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/IATOAX

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=961b4b5e86bf56a2e4b567f81682defa5cba957e



--------------------------------

I noticed that once an NFSv4.1 callback operation gets a
NFS4ERR_DELAY status on CB_SEQUENCE and then the connection is lost,
the callback client loops, resending it indefinitely.

The switch arm in nfsd4_cb_sequence_done() that handles
NFS4ERR_DELAY uses rpc_restart_call() to rearm the RPC state machine
for the retransmit, but that path does not call the rpc_prepare_call
callback again. Thus cb_seq_status is set to -10008 by the first
NFS4ERR_DELAY result, but is never set back to 1 for the retransmits.

nfsd4_cb_sequence_done() thinks it's getting nothing but a
long series of CB_SEQUENCE NFS4ERR_DELAY replies.

Fixes: 7ba6cad6 ("nfsd: New helper nfsd4_cb_sequence_done() for processing more cb errors")
Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
Reviewed-by: default avatarBenjamin Coddington <bcodding@redhat.com>
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarLi Lingfeng <lilingfeng3@huawei.com>
parent a7d98ce7

fs/nfsd/nfs4callback.c

+1 −0
Original line number Diff line number Diff line
@@ -1126,6 +1126,7 @@ static bool nfsd4_cb_sequence_done(struct rpc_task *task, struct nfsd4_callback 
		ret = false;

		break;

	case -NFS4ERR_DELAY:

		cb->cb_seq_status = 1;

		if (!rpc_restart_call(task))

			goto out;