February 2026 - Devel - NFS Ganesha List Archives

[S] Change in ...nfs-ganesha[next]: Add option for binding to interface.
by Arik Rinberg (GerritHub) 18 Feb '26

18 Feb '26

Arik Rinberg has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231604?usp=email ) Change subject: Add option for binding to interface. ...................................................................... Add option for binding to interface. When bringing up Ganesha on a system with multiple NICs, we want to be able to specify which interface to bind the Ganesha IP too. Change-Id: Iee0611fcb3bb51738a38eb27333e27adbc57718d Signed-off-by: Arik Rinberg <ArikRinberg(a)gmail.com> --- M src/MainNFSD/nfs_rpc_dispatcher_thread.c M src/doc/man/ganesha-core-config.rst M src/include/gsh_config.h M src/support/nfs_read_conf.c 4 files changed, 19 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/04/1231604/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231604?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: Iee0611fcb3bb51738a38eb27333e27adbc57718d Gerrit-Change-Number: 1231604 Gerrit-PatchSet: 1 Gerrit-Owner: Arik Rinberg <ArikRinberg(a)gmail.com>

1 0

dread patch breaking dsread
by marc eshel 17 Feb '26

17 Feb '26

HI Yanghh, It looks like your patch has broken dsread. commit a3fd7cbd98d911aa754442afe06499ea4f1c9e21 Author: Haihua Yang <yanghh(a)gmail.com> Date: Tue Aug 26 16:59:17 2025 +0000 Add RDMA support in dsread Change-Id: I435368449e048fc82cc27079a3e295dd4618f425 Signed-off-by: Haihua Yang <yanghh(a)gmail.com> The code in get_buffer_for_io_response() is under #ifdef _USE_NFS_RDMA and returns NULL if it is not set. The call from op_dsread() is notunder_USE_NFS_RDMA. There is no error checking for returning NULL. Marc.

3 2

[XL] Change in ...nfs-ganesha[next]: Switch TSM RPC to callback-based lifecycle with evchan registration
by Suhas Athani (GerritHub) 17 Feb '26

17 Feb '26

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231473?usp=email ) Change subject: Switch TSM RPC to callback-based lifecycle with evchan registration ...................................................................... Switch TSM RPC to callback-based lifecycle with evchan registration This change updates the TSM RPC send path to use the callback‑based client request flow. Requests are now queued with CLNT_CALL_BACK, completion is handled in tsm_rpc_call_process, and request memory is released only after the RPC layer signals completion. The client transport is registered on the event channel to make callback processing safe. Change-Id: Id638234dbbe8afe6308a010093760f1971c6c87d Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- A src/MainNFSD/transparent_recovery.c A src/include/transparent_recovery.h A src/include/tsm.h 3 files changed, 1,846 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/73/1231473/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231473?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: Id638234dbbe8afe6308a010093760f1971c6c87d Gerrit-Change-Number: 1231473 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[M] Change in ...nfs-ganesha[next]: Modify get_buffer_for_io_response to allocate buffer unless asked not to
by Frank Filz (GerritHub) 16 Feb '26

16 Feb '26

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231461?usp=email ) Change subject: Modify get_buffer_for_io_response to allocate buffer unless asked not to ...................................................................... Modify get_buffer_for_io_response to allocate buffer unless asked not to Originally get_buffer_for_io_response only allocated a buffer if RDMA was in use. This results in a confusing API. The new version allocates a buffer unless asked not to for zero copy purposes. An RDMA buffer is still allocated if required. Comments clarify that op_ctx->is_rdma_buff_used can be referenced to determine if the allocated buffer was an RDMA one or not. Change-Id: I579d65a7c517892886c8fe31cfb791f4c030171f Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/fsal_helper.c M src/Protocols/NFS/nfs3_readdir.c M src/Protocols/NFS/nfs3_readdirplus.c M src/Protocols/NFS/nfs4_op_read.c M src/Protocols/NFS/nfs4_op_readdir.c M src/RPCAL/rpc_tools.c M src/include/fsal.h M src/include/gsh_rpc.h 8 files changed, 94 insertions(+), 70 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/61/1231461/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231461?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: I579d65a7c517892886c8fe31cfb791f4c030171f Gerrit-Change-Number: 1231461 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

dread patch breaking dsread
by Marc Eshel 16 Feb '26

16 Feb '26

HI Yanghh, It looks like your patch has broken dsread. commit a3fd7cbd98d911aa754442afe06499ea4f1c9e21 Author: Haihua Yang <yanghh(a)gmail.com<mailto:yanghh@gmail.com>> Date: Tue Aug 26 16:59:17 2025 +0000 Add RDMA support in dsread Change-Id: I435368449e048fc82cc27079a3e295dd4618f425 Signed-off-by: Haihua Yang <yanghh(a)gmail.com<mailto:yanghh@gmail.com>> The code in get_buffer_for_io_response() is under #ifdef _USE_NFS_RDMA and returns NULL if it is not set. The call from op_dsread() is not under _USE_NFS_RDMA. There is no error checking for returning NULL. Marc.

1 0

[S] Change in ...nfs-ganesha[next]: CONFIG: Allow for static message buffer for config errors
by Frank Filz (GerritHub) 12 Feb '26

12 Feb '26

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231196?usp=email ) Change subject: CONFIG: Allow for static message buffer for config errors ...................................................................... CONFIG: Allow for static message buffer for config errors There are some situations like DBUS commands that use config parsing functions. These functions take a complex set of structures that allow collecting a stream of error messages. For those cases where only a single error is expected, allow for a static buffer to be passed. The new init_error_type_static() function initializes a struct config_error_type with a static buffer instead of a file stream and a dynamic buffer. The fp is then set to NULL. The config_proc_error() function will check for a valid buffer and NULL fp and just print into that buffer. It won't display the filename and linenumber (so cnode can be NULL) which is appropriate for a DBUS command. Change-Id: I0365850ecbfe91b9af7cd028a026bde12f882b56 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/config_parsing/config_parsing.c M src/include/config_parsing.h 2 files changed, 29 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/96/1231196/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231196?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: I0365850ecbfe91b9af7cd028a026bde12f882b56 Gerrit-Change-Number: 1231196 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

[S] Change in ...nfs-ganesha[next]: NLM: Add config to not unmonitor on clean shutdown
by Frank Filz (GerritHub) 11 Feb '26

11 Feb '26

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231088?usp=email ) Change subject: NLM: Add config to not unmonitor on clean shutdown ...................................................................... NLM: Add config to not unmonitor on clean shutdown On a clean shutdown, Ganesha unmonitors all NLM clients because a refcount goes to zero. This can be avoided by terminating Ganesha with SIGKILL, but it may be desireable to allow a clean shutdown without unmonitoring. Note that this works with external or internal statd. With internal statd, the monitored NLM clients are listed in the recovery database and SM_NOTIFY will be sent on restart. With external rpc.statd, scripting will be required to cause SM_NOTIFY to be sent. Change-Id: Ie87688d574a5b91613544bab090e65b184d1ac67 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/Protocols/NLM/nsm.c M src/config_samples/config.txt M src/doc/man/ganesha-core-config.rst M src/include/gsh_config.h M src/support/nfs_read_conf.c 5 files changed, 22 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/88/1231088/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1231088?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: Ie87688d574a5b91613544bab090e65b184d1ac67 Gerrit-Change-Number: 1231088 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

[XS] Change in ...nfs-ganesha[next]: Fix segfault when Enable_Dynamic_Metrics is false
by Suhas Athani (GerritHub) 08 Feb '26

08 Feb '26

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230840?usp=email ) Change subject: Fix segfault when Enable_Dynamic_Metrics is false ...................................................................... Fix segfault when Enable_Dynamic_Metrics is false Added a null check for dynamic_metrics in dynamic_metrics__mem_info to prevent dereferencing a null pointer when dynamic metrics are disabled but monitoring is enabled. Change-Id: I5ffc17ba90287d1f934ccdbc0decff35ef77beaf Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/monitoring/dynamic_metrics.cc 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/40/1230840/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230840?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: I5ffc17ba90287d1f934ccdbc0decff35ef77beaf Gerrit-Change-Number: 1230840 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[L] Change in ...nfs-ganesha[next]: [Do not Review]: [DNM]: Test PR: revert test
by Name of user not set (GerritHub) 08 Feb '26

08 Feb '26

manim(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230825?usp=email ) Change subject: [Do not Review]: [DNM]: Test PR: revert test ...................................................................... [Do not Review]: [DNM]: Test PR: revert test Change-Id: Ib67675218d2431fbcd0e7bf5a119addd976d24cb Signed-off-by: Manimaran M <manim(a)redhat.com> --- M src/MainNFSD/libganesha_nfsd.ver M src/SAL/nfs4_recovery.c M src/SAL/recovery/recovery_fs.c M src/SAL/recovery/recovery_fs.h M src/SAL/recovery/recovery_rados.h M src/SAL/recovery/recovery_rados_cluster.c M src/SAL/recovery/recovery_rados_kv.c M src/SAL/recovery/recovery_rados_ng.c M src/include/sal_functions.h 9 files changed, 458 insertions(+), 321 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/25/1230825/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230825?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: Ib67675218d2431fbcd0e7bf5a119addd976d24cb Gerrit-Change-Number: 1230825 Gerrit-PatchSet: 1 Gerrit-Owner: manim(a)redhat.com

1 0

Announce Push of V9.5
by Frank Filz 06 Feb '26

06 Feb '26

Branch next Tag:V9.5 Merge Highlights * FSAL_CEPH: Avoid OPEN_DOWNGRADE hang by fixing async deleg * FSAL_CEPH: Allow runtime configuration of Async I/O for write op * Coverity 638299: Fix for uninitialized scalar variable. * Deleg option parsing for FSALs that don't support * Fix squash handling corruption or race conditions * CEPH: Use root creds for reopen call * Add Content-Type header to Prometheus metrics response * Add IP-based client owner separation for NFSv4.1+. * Allow read locks on exported RO filesystems. Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 144f8b447 Frank S. Filz V9.5 eada9e9a3 Pradeep Allow read locks on exported RO filesystems. bf49da609 Lior Suliman Add IP-based client owner separation for NFSv4.1+. 3f0ee7321 Sanyam Add Content-Type header to Prometheus metrics response 85a2ce071 Sachin Punadikar CEPH: Use root creds for reopen call c35c6f9be Sachin Punadikar Fix squash handling corruption or race conditions 0617e7a36 Animesh Javali Deleg option parsing for FSALs that don't support 26749adfc Animesh Javali Coverity 638299: Fix for uninitialized scalar variable. e717b4553 Suhas Athani FSAL_CEPH: Allow runtime configuration of Async I/O for write op 271aa3824 Suhas Athani FSAL_CEPH: Avoid OPEN_DOWNGRADE hang by fixing async deleg recall handling.

1 0