shaharhoch@google.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222249?usp=email ) Change subject: Fix notify_granted_completion for NFSv4.1 ...................................................................... Fix notify_granted_completion for NFSv4.1 This fixes an issue introduced in: https://github.com/nfs-ganesha/nfs-ganesha/commit/1cfef0bde73e0dca0a347df7e0... The patch changed it so up channel slot release is only called for NFSv4.1, but it used the clientid under the channel. In NFSv4.1 the clientid is not initialized in the channel. The channel keeps a union between the clientid (for NFSv4.0) and the session (for nfsv4.1). So, in NFSv4.1, the check looked at garbage, which means that in some cases the slot was never released. We change the check to look at the channel type, which is always initialized. Change-Id: I9565b148cfdd6a8d1e663e7d12f0d2705294899b Signed-off-by: Shahar Hochma <shaharhoch@google.com> --- M src/Protocols/NFS/nfs4_op_lock.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/49/1222249/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222249?usp=email To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I9565b148cfdd6a8d1e663e7d12f0d2705294899b Gerrit-Change-Number: 1222249 Gerrit-PatchSet: 1 Gerrit-Owner: shaharhoch@google.com