February 2024 - Devel - NFS Ganesha List Archives

[M] Change in ...nfs-ganesha[next]: idmapping: Move DomainName from NFSv4 to new DIRECTORY_SERVICES confi...
by Dipit Grover (GerritHub) 12 Feb '24

12 Feb '24

Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176766?usp=email ) Change subject: idmapping: Move DomainName from NFSv4 to new DIRECTORY_SERVICES config section ...................................................................... idmapping: Move DomainName from NFSv4 to new DIRECTORY_SERVICES config section Change-Id: Iccb2de63e2e12287516d00db58621f7913351339 Signed-off-by: Dipit Grover <dipit(a)google.com> --- M src/MainNFSD/nfs_init.c M src/config_samples/config.txt M src/config_samples/gpfs.ganesha.main.conf M src/doc/man/ganesha-core-config.rst M src/idmapper/idmapper.c M src/include/gsh_config.h M src/include/nfs_core.h M src/support/nfs_read_conf.c 8 files changed, 57 insertions(+), 12 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/66/1176766/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176766?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: Iccb2de63e2e12287516d00db58621f7913351339 Gerrit-Change-Number: 1176766 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover(a)gmail.com>

1 0

[M] Change in ...nfs-ganesha[next]: idmapper: Extract owner-domain set and clear as separate functions
by Dipit Grover (GerritHub) 12 Feb '24

12 Feb '24

Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176765?usp=email ) Change subject: idmapper: Extract owner-domain set and clear as separate functions ...................................................................... idmapper: Extract owner-domain set and clear as separate functions Change-Id: I8a2089bd1b031f62cd4bb5b19efee1aa7ad3e44d Signed-off-by: Dipit Grover <dipit(a)google.com> --- M src/idmapper/idmapper.c 1 file changed, 69 insertions(+), 22 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/65/1176765/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176765?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: I8a2089bd1b031f62cd4bb5b19efee1aa7ad3e44d Gerrit-Change-Number: 1176765 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover(a)gmail.com>

1 0

[M] Change in ...nfs-ganesha[next]: idmapping: Extract user,group cache addition to separate functions
by Dipit Grover (GerritHub) 12 Feb '24

12 Feb '24

Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176764?usp=email ) Change subject: idmapping: Extract user,group cache addition to separate functions ...................................................................... idmapping: Extract user,group cache addition to separate functions Change-Id: Ifb7002c9e739ef4f2681604cd6ac64f59b8245c8 Signed-off-by: Dipit Grover <dipit(a)google.com> --- M src/idmapper/idmapper.c 1 file changed, 49 insertions(+), 33 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/64/1176764/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176764?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: Ifb7002c9e739ef4f2681604cd6ac64f59b8245c8 Gerrit-Change-Number: 1176764 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover(a)gmail.com>

1 0

[M] Change in ...nfs-ganesha[next]: uid2grp: Handle ERANGE return code when calling pw-functions
by Dipit Grover (GerritHub) 12 Feb '24

12 Feb '24

Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176763?usp=email ) Change subject: uid2grp: Handle ERANGE return code when calling pw-functions ...................................................................... uid2grp: Handle ERANGE return code when calling pw-functions The pw-functions accept a buffer for holding the strings in the fetched passwd entry. The buffer's size is derived from the system config, which should be sufficient in most cases. To handle the cases where this buffer size is not sufficient, this CL calls these functions iteratively, while increasing the buffer size in each iteration. Change-Id: I5f049684f16d26cb795fc9c42dd23c0593fb5504 Signed-off-by: Dipit Grover <dipit(a)google.com> --- M src/idmapper/idmapper.c M src/include/idmapper.h M src/support/uid2grp.c 3 files changed, 55 insertions(+), 26 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/63/1176763/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176763?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: I5f049684f16d26cb795fc9c42dd23c0593fb5504 Gerrit-Change-Number: 1176763 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover(a)gmail.com>

1 0

Announce Push of V6-dev.4
by Frank Filz 09 Feb '24

09 Feb '24

Branch next Tag:V6-dev.4 NOTE: This merge includes an ntirpc pullup, make sure to update your submodule. Merge Highlights * Improve session handling including back channels Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 20fd703bb Frank S. Filz V6-dev.4 c9677878f Dipit Grover nfs41_session: Replace connections array by a linked-list 4ee4a1ed5 Dipit Grover nfs_tcp_dispatch: Register callback to un-reference user-data during xprt destroy db731888d Dipit Grover xprt_handler: Add function to un-reference xprt client-data 2111931d0 Dipit Grover nfs41_session: Add function to destroy the session backchannel for a xprt 355dc6bf7 Dipit Grover nfs41_session: Add function to remove a xprt from the session 4c22e07b5 Dipit Grover svc_xprt: Maintain mapping between nfs41_session and connection 857d81606 Dipit Grover nfs41_session: Update nfs41_Session_Del function to accept session 4e46d8f38 Dipit Grover bind_conn_to_session: Validate check/add connection to session output e7bb9affd Dipit Grover destroy-channel: Expose function that does not lock channel 417dd296b Dipit Grover logging-component: Add new COMPONENT_XPRT and remove unused COMPONENT_RPC 4e644b45f Dipit Grover bind_conn_to_session: Fix backchannel implementation b263e06c0 Dipit Grover gsh_rpc: Add method to display xprt's socket-address 75d60d8aa Dipit Grover libntirpc: Update submodule b853d6d9b Dipit Grover nfs41_session: Store security params for backchannel creation 4d048f9b7 Dipit Grover nfs_client_id: Store state protection mechanism for the client

1 0

[S] Change in ...nfs-ganesha[next]: idmapper_cache: Do not destroy the cache locks when clearing the caches
by Dipit Grover (GerritHub) 09 Feb '24

09 Feb '24

Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176637?usp=email ) Change subject: idmapper_cache: Do not destroy the cache locks when clearing the caches ...................................................................... idmapper_cache: Do not destroy the cache locks when clearing the caches This fixes a bug where calling cache-clear will also destroy the cache locks, hence causing the caches to become un-usable. We extract cache-cleanup into a separate function, which will clear the caches and additionally also destroy the cache locks. The original cache-clear function will do only clearing of the caches. Change-Id: I531cf293be093f31a0a3d89db67808286c5a6a49 Signed-off-by: Dipit Grover <dipit(a)google.com> --- M src/idmapper/idmapper.c M src/idmapper/idmapper_cache.c M src/include/idmapper.h 3 files changed, 11 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/37/1176637/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176637?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: I531cf293be093f31a0a3d89db67808286c5a6a49 Gerrit-Change-Number: 1176637 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover(a)gmail.com>

1 0

[S] Change in ...nfs-ganesha[next]: uid2grp: Extract add-user-groups-to-cache to a separate function
by Dipit Grover (GerritHub) 09 Feb '24

09 Feb '24

Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176635?usp=email ) Change subject: uid2grp: Extract add-user-groups-to-cache to a separate function ...................................................................... uid2grp: Extract add-user-groups-to-cache to a separate function Change-Id: I89894f3560aac3625ac8d3e57ec90ca2a7af50bc Signed-off-by: Dipit Grover <dipit(a)google.com> --- M src/support/uid2grp.c 1 file changed, 19 insertions(+), 16 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/35/1176635/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176635?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: I89894f3560aac3625ac8d3e57ec90ca2a7af50bc Gerrit-Change-Number: 1176635 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover(a)gmail.com>

1 0

[M] Change in ...nfs-ganesha[next]: Enable libunwind in ganesha
by Xueqian Hu (GerritHub) 08 Feb '24

08 Feb '24

Xueqian Hu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176609?usp=email ) Change subject: Enable libunwind in ganesha ...................................................................... Enable libunwind in ganesha Currently, Ganesha uses backtrace during crashes, which may not provide sufficient useful information: 07/02/2024 19:09:18: epoch 65c3d55b: xueqian-hu: nfs-ganesha-39008[main] gsh_backtrace: NFS STARTUP: MAJ: stack backtrace follows: /lib64/libganesha_nfsd.so.5.6(+0x83ba1)[0x7f64d9172ba1] /lib64/libganesha_nfsd.so.5.6(+0x5d618)[0x7f64d914c618] /lib64/libpthread.so.0(+0xf630)[0x7f64d7d1a630] /lib64/libpthread.so.0(pthread_join+0xa7)[0x7f64d7d14017] /lib64/libganesha_nfsd.so.5.6(nfs_start+0x323)[0x7f64d914eff3] ganesha.nfsd(main+0xbef)[0x4025cf] /lib64/libc.so.6(__libc_start_main+0xf5)[0x7f64d753d555] ganesha.nfsd[0x402663] This patch introduces an option to use libunwind, which can display function information: 08/02/2024 20:06:04: epoch 65c53429: xueqian-hu: nfs-ganesha-99814[main] gsh_libunwind: NFS STARTUP: MAJ: BACKTRACE: Change-Id: I194853e25ec0c89c7f552dc4872c37a420ea8d4a Signed-off-by: Xueqian Hu <xh140312(a)outlook.com> --- M src/CMakeLists.txt M src/MainNFSD/nfs_init.c A src/cmake/modules/FindUnwind.cmake M src/include/config-h.in.cmake M src/include/log.h M src/log/log_functions.c M src/nfs-ganesha.spec-in.cmake 7 files changed, 160 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/09/1176609/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176609?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: I194853e25ec0c89c7f552dc4872c37a420ea8d4a Gerrit-Change-Number: 1176609 Gerrit-PatchSet: 1 Gerrit-Owner: Xueqian Hu <xh140312(a)outlook.com>

1 0

[M] Change in ...nfs-ganesha[next]: FSAL_PROXY_V3: Implement atomic pre/post attrs
by Name of user not set (GerritHub) 08 Feb '24

08 Feb '24

shaharhoch(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176603?usp=email ) Change subject: FSAL_PROXY_V3: Implement atomic pre/post attrs ...................................................................... FSAL_PROXY_V3: Implement atomic pre/post attrs Change-Id: Ibcef4b7bc41fa154d49562525f520c984daed17e Signed-off-by: Shahar Hochma <shaharhoch(a)gmail.com> --- M src/FSAL/FSAL_PROXY_V3/main.c M src/FSAL/FSAL_PROXY_V3/proxyv3_fsal_methods.h M src/FSAL/FSAL_PROXY_V3/utils.c 3 files changed, 133 insertions(+), 10 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/03/1176603/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176603?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: Ibcef4b7bc41fa154d49562525f520c984daed17e Gerrit-Change-Number: 1176603 Gerrit-PatchSet: 1 Gerrit-Owner: shaharhoch(a)gmail.com

1 0

[M] Change in ...nfs-ganesha[next]: nfs_creds: Apply minor refactoring to rpcsec_gss creds evaluation
by Dipit Grover (GerritHub) 07 Feb '24

07 Feb '24

Dipit Grover has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176546?usp=email ) Change subject: nfs_creds: Apply minor refactoring to rpcsec_gss creds evaluation ...................................................................... nfs_creds: Apply minor refactoring to rpcsec_gss creds evaluation This CL inverts an if condition to reduce nesting, and improve readability. Change-Id: Icef1214870beb259859dab09760a8f7932da8531 Signed-off-by: Dipit Grover <dipit(a)google.com> --- M src/support/nfs_creds.c 1 file changed, 43 insertions(+), 44 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/46/1176546/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1176546?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: Icef1214870beb259859dab09760a8f7932da8531 Gerrit-Change-Number: 1176546 Gerrit-PatchSet: 1 Gerrit-Owner: Dipit Grover <dipitgrover(a)gmail.com>

1 0