[M] Change in ...nfs-ganesha[next]: nfs41_session: Replace connections array by a linked-list
Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1173663?usp=email ) Change subject: nfs41_session: Replace connections array by a linked-list ...................................................................... nfs41_session: Replace connections array by a linked-list In nfs41_Session, the connections array has an upper cap of 16 entries. This can be a problem in scenarios where a client is mounted with nconnect as 16, and undergoes a situation (VM migration, client IP tables update, etc) which leads to disconnection of the existing connections from client's end. Now if the client connects using a fresh set of connections, the server will not add these new connections to the client's existing session, until the server discovers that the older connections are terminated. Since there are certain NFS ops (eg: DESTROY_SESSION in current Ganesha's implementation) that rely on the request being received on the session's connection, those ops will fail under this scenario. Change-Id: I41a9bdbf6db952ca8b36fa2c9f157709ae76ba79 Signed-off-by: Dipit Grover <dipit@google.com> --- M src/Protocols/NFS/nfs4_op_create_session.c M src/SAL/nfs41_session_id.c M src/include/sal_data.h 3 files changed, 54 insertions(+), 38 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/63/1173663/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1173663?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: I41a9bdbf6db952ca8b36fa2c9f157709ae76ba79 Gerrit-Change-Number: 1173663 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover@gmail.com>
participants (1)
-
Dipit Grover (GerritHub)