[S] Change in ...nfs-ganesha[next]: Turn on more compilation warnings and resolve existing ones
by Lior Suliman (GerritHub)
Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201316?usp=email )
Change subject: Turn on more compilation warnings and resolve existing ones
......................................................................
Turn on more compilation warnings and resolve existing ones
Signed-off-by: Lior Suliman <liorsu(a)gmail.com>
Change-Id: I05413e76442077161e752da7ba57756a6da7a33e
---
M src/CMakeLists.txt
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_avl.c
M src/FSAL/fsal_helper.c
M src/Protocols/NFS/nfs_proto_tools.c
M src/SAL/nfs4_recovery.c
M src/SAL/recovery/recovery_fs.c
M src/include/fsal_api.h
M src/support/client_mgr.c
M src/support/fridgethr.c
9 files changed, 14 insertions(+), 14 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/16/1201316/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201316?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: I05413e76442077161e752da7ba57756a6da7a33e
Gerrit-Change-Number: 1201316
Gerrit-PatchSet: 1
Gerrit-Owner: Lior Suliman <liorsu(a)google.com>
2 days, 14 hours
[S] Change in ...nfs-ganesha[next]: Change NFSv3 proxy FSAL to return delay instead of invalid val or ser...
by Lior Suliman (GerritHub)
Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201315?usp=email )
Change subject: Change NFSv3 proxy FSAL to return delay instead of invalid val or server fault to the client [b/234604886]
......................................................................
Change NFSv3 proxy FSAL to return delay instead of invalid val or server fault to the client [b/234604886]
If there is an error in sending / receiving a response to / from the NFSv3 server, return the
client with NFS4ERR_DELAY that will cause it to retry issuing that operation
https://datatracker.ietf.org/doc/html/rfc5661#section-15.1.1.3
For this not to be an issue, we also need to change the current way the XID is generated by the NFSv3 proxy FSAL
so that two requests that are the same will yield with the same XID, otherwise it can be executed more than once
by the NFSv3 back-end server as they will be recognized as different request
Signed-off-by: Lior Suliman <liorsu(a)gmail.com>
Change-Id: If94c1485ec0ea8fca4e9ad2afc8128eaa0baaa77
---
M src/FSAL/FSAL_PROXY_V3/main.c
M src/FSAL/FSAL_PROXY_V3/nlm.c
2 files changed, 13 insertions(+), 13 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/15/1201315/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201315?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: If94c1485ec0ea8fca4e9ad2afc8128eaa0baaa77
Gerrit-Change-Number: 1201315
Gerrit-PatchSet: 1
Gerrit-Owner: Lior Suliman <liorsu(a)google.com>
2 days, 14 hours
[S] Change in ...nfs-ganesha[next]: Change NFSv3 proxy FSAL to return delay instead of invalid val or ser...
by Lior Suliman (GerritHub)
Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201312?usp=email )
Change subject: Change NFSv3 proxy FSAL to return delay instead of invalid val or server fault to the client [b/234604886]
......................................................................
Change NFSv3 proxy FSAL to return delay instead of invalid val or server fault to the client [b/234604886]
If there is an error in sending / receiving a response to / from the NFSv3 server, return the
client with NFS4ERR_DELAY that will cause it to retry issuing that operation
https://datatracker.ietf.org/doc/html/rfc5661#section-15.1.1.3
For this not to be an issue, we also need to change the current way the XID is generated by the NFSv3 proxy FSAL
so that two requests that are the same will yield with the same XID, otherwise it can be executed more than once
by the NFSv3 back-end server as they will be recognized as different request
Change-Id: I7f6947898d208bd5ae61768bf710e6e6592d31dd
Signed-off-by: Lior Suliman <liorsu(a)gmail.com>
---
M src/FSAL/FSAL_PROXY_V3/main.c
M src/FSAL/FSAL_PROXY_V3/nlm.c
2 files changed, 13 insertions(+), 13 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/12/1201312/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201312?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: I7f6947898d208bd5ae61768bf710e6e6592d31dd
Gerrit-Change-Number: 1201312
Gerrit-PatchSet: 1
Gerrit-Owner: Lior Suliman <liorsu(a)google.com>
2 days, 14 hours
[M] Change in ...nfs-ganesha[next]: Added nfs request to op context if a request is originated from nfs c...
by Lior Suliman (GerritHub)
Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201309?usp=email )
Change subject: Added nfs request to op context if a request is originated from nfs client
......................................................................
Added nfs request to op context if a request is originated from nfs client
Change-Id: Ia37abd5402005aed3efa8be7cdd90bcf24b108b4
Signed-off-by: Lior Suliman <liorsu(a)gmail.com>
---
M src/FSAL/commonlib.c
M src/MainNFSD/9p_dispatcher.c
M src/MainNFSD/nfs_worker_thread.c
M src/Protocols/9P/9p_proto_tools.c
M src/Protocols/NFS/nfs4_pseudo.c
M src/Protocols/NLM/nlm_util.c
M src/SAL/state_async.c
M src/SAL/state_lock.c
M src/include/fsal.h
M src/include/fsal_api.h
M src/include/fsal_types.h
M src/support/export_mgr.c
12 files changed, 34 insertions(+), 25 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/09/1201309/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201309?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: Ia37abd5402005aed3efa8be7cdd90bcf24b108b4
Gerrit-Change-Number: 1201309
Gerrit-PatchSet: 1
Gerrit-Owner: Lior Suliman <liorsu(a)google.com>
2 days, 14 hours