Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/526271 ) Change subject: nsm client always has a caller_name, don't use ssc_client in hash ...................................................................... nsm client always has a caller_name, don't use ssc_client in hash Clarify the caller_name is always present and also clarify that a deep copy is done of it. Since when hashing by IP address, caller_name is the string form of the IP address, so just hash and search by that and not the actual IP address. Because of this, it is also OK if there is no op_ctx->client since we don't reference it. NOTE: There WILL always be an op_ctx->client any time we are establishing a new nsm client since a lock request needs to come from a client. This allowance is made for SM_NOTIFY which always comes from loop back and not a real client. Change-Id: I00ec438e4661dd1fe0c41a7a64e1fca292b80a58 Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com> --- M src/SAL/nlm_owner.c M src/include/sal_data.h 2 files changed, 47 insertions(+), 34 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/71/526271/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/526271 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I00ec438e4661dd1fe0c41a7a64e1fca292b80a58 Gerrit-Change-Number: 526271 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx@mindspring.com> Gerrit-MessageType: newchange