aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRick Macklem <rmacklem@FreeBSD.org>2022-05-27 21:32:46 +0000
committerRick Macklem <rmacklem@FreeBSD.org>2022-05-27 21:32:46 +0000
commita7bb120f8b8735bc4c417b3c8fc2308c3d2964aa (patch)
tree40596151dcd98e9f0df127126867f694114efcda
parent425e5c739bcf190265330b79e85f89eb4d7b5f25 (diff)
downloadsrc-a7bb120f8b8735bc4c417b3c8fc2308c3d2964aa.tar.gz
src-a7bb120f8b8735bc4c417b3c8fc2308c3d2964aa.zip
nfscl: Add a diagnostic printf() for a "should never happen" case
When a NFSv4.1/4.2 session to the NFS server (not a pNFS DS) is replaced, the old session should always be marked defunct by nfsess_defunct being set non-zero. However, the hang reported by the PR suggests that this might be the case. This patch adds a printf() to indicate this has somehow happened. PR: 260011 MFC after: 2 weeks
-rw-r--r--sys/fs/nfsclient/nfs_clrpcops.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/fs/nfsclient/nfs_clrpcops.c b/sys/fs/nfsclient/nfs_clrpcops.c
index b1d3ee423194..b785446b1ae6 100644
--- a/sys/fs/nfsclient/nfs_clrpcops.c
+++ b/sys/fs/nfsclient/nfs_clrpcops.c
@@ -1062,8 +1062,12 @@ nfsrpc_setclient(struct nfsmount *nmp, struct nfsclclient *clp, int reclaim,
* in-progress RPCs.
*/
tsep = NULL;
- if (TAILQ_FIRST(&nmp->nm_sess) != NULL)
+ if (TAILQ_FIRST(&nmp->nm_sess) != NULL) {
tsep = NFSMNT_MDSSESSION(nmp);
+ if (tsep->nfsess_defunct == 0)
+ printf("nfsrpc_setclient: "
+ "nfsess_defunct not set\n");
+ }
TAILQ_INSERT_HEAD(&nmp->nm_sess, dsp,
nfsclds_list);
/*