September 2025 - Devel - NFS Ganesha List Archives

[XS] Change in ...nfs-ganesha[next]: Fix notify_granted_completion for NFSv4.1
by Name of user not set (GerritHub) 09 Sep '25

09 Sep '25

shaharhoch(a)google.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222249?usp=email ) Change subject: Fix notify_granted_completion for NFSv4.1 ...................................................................... Fix notify_granted_completion for NFSv4.1 This fixes an issue introduced in: https://github.com/nfs-ganesha/nfs-ganesha/commit/1cfef0bde73e0dca0a347df7e… The patch changed it so up channel slot release is only called for NFSv4.1, but it used the clientid under the channel. In NFSv4.1 the clientid is not initialized in the channel. The channel keeps a union between the clientid (for NFSv4.0) and the session (for nfsv4.1). So, in NFSv4.1, the check looked at garbage, which means that in some cases the slot was never released. We change the check to look at the channel type, which is always initialized. Change-Id: I9565b148cfdd6a8d1e663e7d12f0d2705294899b Signed-off-by: Shahar Hochma <shaharhoch(a)google.com> --- M src/Protocols/NFS/nfs4_op_lock.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/49/1222249/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222249?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: I9565b148cfdd6a8d1e663e7d12f0d2705294899b Gerrit-Change-Number: 1222249 Gerrit-PatchSet: 1 Gerrit-Owner: shaharhoch(a)google.com

1 0

[XS] Change in ...nfs-ganesha[next]: SAL: Avoid extra logging during grace period
by Sachin Punadikar (GerritHub) 08 Sep '25

08 Sep '25

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222231?usp=email ) Change subject: SAL: Avoid extra logging during grace period ...................................................................... SAL: Avoid extra logging during grace period In grace period, status of client reclaim in the form of count, is getting logged. This logging should be done only when we have active clients in the recovery database. Change-Id: I1fbe93c8415c54b4501e53021f70c9da52a458b3 Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/SAL/nfs4_recovery.c 1 file changed, 5 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/31/1222231/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222231?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: I1fbe93c8415c54b4501e53021f70c9da52a458b3 Gerrit-Change-Number: 1222231 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 0

[M] Change in ...nfs-ganesha[next]: Support QoS DS read/write control
by Xueqian Hu (GerritHub) 08 Sep '25

08 Sep '25

Xueqian Hu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222226?usp=email ) Change subject: Support QoS DS read/write control ...................................................................... Support QoS DS read/write control QoS control on bandwidth/token does not work for DS read/write. This change add an option param to apply the bw/token limitaion on ds read/write. Change-Id: I238c9005fc26d3ca0420865a0ddc8685ffc7ed50 Signed-off-by: xueqian.hu <xh140312(a)outlook.com> --- M src/MainNFSD/nfs_qos.c M src/Protocols/NFS/nfs4_op_read.c M src/Protocols/NFS/nfs4_op_write.c M src/doc/man/ganesha-qos-config.rst M src/include/nfs_qos.h M src/support/exports.c M src/support/nfs_read_conf.c 7 files changed, 120 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/26/1222226/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222226?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: I238c9005fc26d3ca0420865a0ddc8685ffc7ed50 Gerrit-Change-Number: 1222226 Gerrit-PatchSet: 1 Gerrit-Owner: Xueqian Hu <xh140312(a)outlook.com>

1 0

[M] Change in ...nfs-ganesha[next]: Fix conditional builds for several things
by Daniel Gryniewicz (GerritHub) 08 Sep '25

08 Sep '25

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222224?usp=email ) Change subject: Fix conditional builds for several things ...................................................................... Fix conditional builds for several things - Make MONITORING conditional, and integrate with ntirpc - Pull up ntirpc to correct version - Fix RPM build for conditional monitoring, and rename it's package to match the naming convention for Ganesha - Fix a few CMake errors from deprecation (only showed up in RPM build) - Fix RPM conditional build for QOS and NFSIDMAP and ADMIN_TOOLS Change-Id: I4c75b3f3b5f1fab65150c923edd7f0a36a76f5db Signed-off-by: Daniel Gryniewicz <dang(a)fprintf.net> --- M src/CMakeLists.txt M src/MainNFSD/CMakeLists.txt M src/MainNFSD/nfs_main.c M src/cmake/modules/FindNTIRPC.cmake M src/libntirpc M src/nfs-ganesha.spec-in.cmake 6 files changed, 67 insertions(+), 19 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/24/1222224/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222224?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: I4c75b3f3b5f1fab65150c923edd7f0a36a76f5db Gerrit-Change-Number: 1222224 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

[XS] Change in ...nfs-ganesha[next]: Ignore .egg-info directories from python scripts
by Daniel Gryniewicz (GerritHub) 08 Sep '25

08 Sep '25

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222223?usp=email ) Change subject: Ignore .egg-info directories from python scripts ...................................................................... Ignore .egg-info directories from python scripts Change-Id: Ie37f0dd3834396cf8b2d9b896dc812b5262df7b9 Signed-off-by: Daniel Gryniewicz <dang(a)fprintf.net> --- M .gitignore 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/23/1222223/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222223?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: Ie37f0dd3834396cf8b2d9b896dc812b5262df7b9 Gerrit-Change-Number: 1222223 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

[S] Change in ...nfs-ganesha[next]: Fix unitialized errors
by Daniel Gryniewicz (GerritHub) 08 Sep '25

08 Sep '25

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222222?usp=email ) Change subject: Fix unitialized errors ...................................................................... Fix unitialized errors Change-Id: Ia85973d46fd9819fea41312af69572a938c201cb Signed-off-by: Daniel Gryniewicz <dang(a)fprintf.net> --- M src/FSAL/FSAL_GPFS/fsal_ds.c M src/Protocols/NFS/nfs4_op_bind_conn.c M src/Protocols/NFS/nfs4_op_lock.c M src/Protocols/NFS/nfs4_op_savefh.c M src/Protocols/NFS/nfs_proto_tools.c 5 files changed, 6 insertions(+), 6 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/22/1222222/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222222?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: Ia85973d46fd9819fea41312af69572a938c201cb Gerrit-Change-Number: 1222222 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

[S] Change in ...nfs-ganesha[next]: Fix: Race condition between socket flush and close
by Name of user not set (GerritHub) 08 Sep '25

08 Sep '25

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222190?usp=email ) Change subject: Fix: Race condition between socket flush and close ...................................................................... Fix: Race condition between socket flush and close Change-Id: I15a913c8f235b55e72fa4b423a80dc0bca93583a Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> --- M src/cmake/modules/FindCEPHFS.cmake M src/monitoring/prometheus_exposer.cc 2 files changed, 14 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/90/1222190/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222190?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: I15a913c8f235b55e72fa4b423a80dc0bca93583a Gerrit-Change-Number: 1222190 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 0

[S] Change in ...nfs-ganesha[next]: Block signals immediately at the beginning of the main function.
by Name of user not set (GerritHub) 07 Sep '25

07 Sep '25

daian(a)google.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222179?usp=email ) Change subject: Block signals immediately at the beginning of the main function. ...................................................................... Block signals immediately at the beginning of the main function. Threads inherit the blocked signal mask from their creating thread. Thus, if a thread has been created before the signals where blocked, the signals are not blocked for that thread, meaning that when the signal is sent to the process, the (probably default) signal handler will be called on that thread, and the process may get terminated as a result. Indeed, when the logging library gets initialized and the binary is compiled with the USE_UNWIND_ENRICHED_BT macro defined, a new thread is spawned before the signals are masked, causing the SIGHUP signal to terminate the process instead of reloading the configuration. Change-Id: I8158c4d977b92fc2d2d62701c702ef569500e94f Signed-off-by: Eli Daian <daian(a)google.com> --- M src/MainNFSD/nfs_main.c 1 file changed, 23 insertions(+), 13 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/79/1222179/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222179?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: I8158c4d977b92fc2d2d62701c702ef569500e94f Gerrit-Change-Number: 1222179 Gerrit-PatchSet: 1 Gerrit-Owner: daian(a)google.com

1 0

[M] Change in ...nfs-ganesha[next]: Allow updating the UTF8 enforcement on a dynamic configuration change.
by Name of user not set (GerritHub) 07 Sep '25

07 Sep '25

daian(a)google.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222178?usp=email ) Change subject: Allow updating the UTF8 enforcement on a dynamic configuration change. ...................................................................... Allow updating the UTF8 enforcement on a dynamic configuration change. For this purpose, this change introduces the NFSv4_Dynamic configuration section. Right now, this section only contains the Enforce_UTF8_Validation field, but it may contain in the future more fields of the NFSv4 section that could be dynamically updated. For backward compatibility, the Enforce_UTF8_Validation field still exists in the NFSv4 section. If users set it to true, they would get a deprecation warning, but this value will be used. Otherwise, the value from NFSv4_Dynamic is used. Change-Id: I6fae54c6e24c643fd19384e115a896e4f07d888f Signed-off-by: Eli Daian <daian(a)google.com> --- M src/MainNFSD/nfs_init.c M src/config_samples/config.txt M src/doc/man/ganesha-core-config.rst M src/include/gsh_config.h M src/include/nfs_core.h M src/include/nfs_proto_tools.h M src/support/nfs_read_conf.c 7 files changed, 89 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/78/1222178/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222178?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: I6fae54c6e24c643fd19384e115a896e4f07d888f Gerrit-Change-Number: 1222178 Gerrit-PatchSet: 1 Gerrit-Owner: daian(a)google.com

1 0

FW: [Reminder] The Fall 2025 NFS Bake-a-thon in Raleigh, North Carolina, US
by Frank Filz 04 Sep '25

04 Sep '25

-----Original Message----- From: Steve Dickson [mailto:steved@redhat.com] Sent: Thursday, September 4, 2025 4:04 AM To: Linux NFS Mailing list <linux-nfs(a)vger.kernel.org> Cc: bakeathon-announce(a)googlegroups.com; NFSv4 <nfsv4(a)ietf.org> Subject: [Reminder] The Fall 2025 NFS Bake-a-thon in Raleigh, North Carolina, US [ Just another quick reminder the hotel block ] [ expires next Monday (Mon Sept 8th ] [ I hope to see there in-person or remotely ] Hello, Red Hat is pleased to sponsor the Fall 2025 NFS Bake-a-thon event, to be held *Mon Oct 6 - Fri Oct 10* in Raleigh, North Carolina, US at the Red Hat Towers office. This is an in-person event that includes secure remote access to the test network via VPN, enabling virtual participation. Event registration and network, hotel, and venue info: http://www.nfsv4bat.org/Events/2025/Oct/BAT/index.html Questions? Please send them to bakeathon-contact(a)googlegroups.com steved.

1 0