diff options
author | Andy Adamson <andros@netapp.com> | 2009-12-04 15:55:38 -0500 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2009-12-04 15:55:38 -0500 |
commit | ea028ac92541ac30bf202ed94cb53eec2ea0c9d6 (patch) | |
tree | f9b807cf0f870db922f96e72dab68ed37ba18cf0 /include/linux/nfs_fs_sb.h | |
parent | 05f0d2364726c92f6b870db654967088349379fe (diff) |
nfs41: nfs41: fix state manager deadlock in session reset
If the session is reset during state recovery, the state manager thread can
sleep on the slot_tbl_waitq causing a deadlock.
Add a completion framework to the session. Have the state manager thread set
a new session state (NFS4CLNT_SESSION_DRAINING) and wait for the session slot
table to drain.
Signal the state manager thread in nfs41_sequence_free_slot when the
NFS4CLNT_SESSION_DRAINING bit is set and the session is drained.
Reported-by: Trond Myklebust <trond@netapp.com>
Signed-off-by: Andy Adamson <andros@netapp.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'include/linux/nfs_fs_sb.h')
-rw-r--r-- | include/linux/nfs_fs_sb.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/linux/nfs_fs_sb.h b/include/linux/nfs_fs_sb.h index 320569eabe3b..34fc6be5bfcf 100644 --- a/include/linux/nfs_fs_sb.h +++ b/include/linux/nfs_fs_sb.h | |||
@@ -209,6 +209,7 @@ struct nfs4_session { | |||
209 | unsigned long session_state; | 209 | unsigned long session_state; |
210 | u32 hash_alg; | 210 | u32 hash_alg; |
211 | u32 ssv_len; | 211 | u32 ssv_len; |
212 | struct completion complete; | ||
212 | 213 | ||
213 | /* The fore and back channel */ | 214 | /* The fore and back channel */ |
214 | struct nfs4_channel_attrs fc_attrs; | 215 | struct nfs4_channel_attrs fc_attrs; |