Skip to content

NAS-135640 / 25.04.1 / NFSD: fix hang in nfsd4_shutdown_callback#215

Merged
yocalebo merged 1 commit into
stable/fangtoothfrom
NAS-135640
May 1, 2025
Merged

NAS-135640 / 25.04.1 / NFSD: fix hang in nfsd4_shutdown_callback#215
yocalebo merged 1 commit into
stable/fangtoothfrom
NAS-135640

Conversation

@ixhamza

@ixhamza ixhamza commented Apr 30, 2025

Copy link
Copy Markdown
Member

If nfs4_client is in courtesy state then there is no point to send the callback. This causes nfsd4_shutdown_callback to hang since cl_cb_inflight is not 0. This hang lasts about 15 minutes until TCP notifies NFSD that the connection was dropped.

This patch modifies nfsd4_run_cb_work to skip the RPC call if nfs4_client is in courtesy state.

Fixes: 66af257 ("NFSD: add courteous server support for thread with only delegation")
Cc: [email protected]
Reviewed-by: Jeff Layton [email protected]

Testing

If nfs4_client is in courtesy state then there is no point to send
the callback. This causes nfsd4_shutdown_callback to hang since
cl_cb_inflight is not 0. This hang lasts about 15 minutes until TCP
notifies NFSD that the connection was dropped.

This patch modifies nfsd4_run_cb_work to skip the RPC call if
nfs4_client is in courtesy state.

Signed-off-by: Dai Ngo <[email protected]>
Fixes: 66af257 ("NFSD: add courteous server support for thread with only delegation")
Cc: [email protected]
Reviewed-by: Jeff Layton <[email protected]>
Signed-off-by: Chuck Lever <[email protected]>
@ixhamza ixhamza requested review from amotin and mgrimesix April 30, 2025 23:08
@bugclerk

Copy link
Copy Markdown

@bugclerk bugclerk changed the title NFSD: fix hang in nfsd4_shutdown_callback NAS-135640 / 25.04.1 / NFSD: fix hang in nfsd4_shutdown_callback Apr 30, 2025
@yocalebo yocalebo merged commit 9a63150 into stable/fangtooth May 1, 2025
@yocalebo yocalebo deleted the NAS-135640 branch May 1, 2025 10:58
@bugclerk

bugclerk commented May 1, 2025

Copy link
Copy Markdown

JIRA ticket https://ixsystems.atlassian.net/browse/NAS-135640 is targeted to the following versions which have not received their corresponding PRs: 25.10

@bugclerk

bugclerk commented May 1, 2025

Copy link
Copy Markdown

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators May 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants