August 2025 - Devel - NFS Ganesha List Archives

[XS] Change in ...nfs-ganesha[next]: Fix resource leak in gsh_libunwind (Coverity CID 583559)
by Suhas Athani (GerritHub) 14 Aug '25

14 Aug '25

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220426?usp=email ) Change subject: Fix resource leak in gsh_libunwind (Coverity CID 583559) ...................................................................... Fix resource leak in gsh_libunwind (Coverity CID 583559) A file descriptor resource leak in gsh_libunwind.The fd obtained via open() when writing backtrace output to a log file was not being closed before returning from the function. The fix ensures that fd is closed in all exit paths, including normal and error paths. Change-Id: I81f153d495391137a0f392b5903dd4f787f9cc88 Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/log/log_functions.c 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/26/1220426/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220426?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: I81f153d495391137a0f392b5903dd4f787f9cc88 Gerrit-Change-Number: 1220426 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[XS] Change in ...nfs-ganesha[next]: dbus: release export ref if attempt to acquire export lock fails
by Name of user not set (GerritHub) 13 Aug '25

13 Aug '25

pradeepthomas(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220421?usp=email ) Change subject: dbus: release export ref if attempt to acquire export lock fails ...................................................................... dbus: release export ref if attempt to acquire export lock fails Change-Id: I31c1b6b636d93bc1e044bed77d8407f6e7156f62 Signed-off-by: Pradeep <pradeepthomas(a)gmail.com> --- M src/support/export_mgr.c 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/21/1220421/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220421?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: I31c1b6b636d93bc1e044bed77d8407f6e7156f62 Gerrit-Change-Number: 1220421 Gerrit-PatchSet: 1 Gerrit-Owner: pradeepthomas(a)gmail.com

1 0

[M] Change in ...nfs-ganesha[next]: Get number of lanes and chunk lru lanes from config
by Name of user not set (GerritHub) 12 Aug '25

12 Aug '25

deepakarumugam.s(a)nutanix.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220341?usp=email ) Change subject: Get number of lanes and chunk lru lanes from config ...................................................................... Get number of lanes and chunk lru lanes from config Add a config option to dynamically change number of lanes in LRU and number of lanes in CHUNK LRU. We tune it in my company to get better parallelism during reaping Change-Id: I024224c65fc03584861cfa1cda48edeb4df171a7 Signed-off-by: Deepak Arumugam Sankara Subramanian <deepakarumugam.s(a)nutanix.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_ext.h M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.h M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_read_conf.c 4 files changed, 66 insertions(+), 44 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/41/1220341/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220341?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: I024224c65fc03584861cfa1cda48edeb4df171a7 Gerrit-Change-Number: 1220341 Gerrit-PatchSet: 1 Gerrit-Owner: deepakarumugam.s(a)nutanix.com

1 0

[S] Change in ...nfs-ganesha[next]: Remove unnecessary lookup from cih_remove_checked
by Name of user not set (GerritHub) 12 Aug '25

12 Aug '25

deepakarumugam.s(a)nutanix.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220340?usp=email ) Change subject: Remove unnecessary lookup from cih_remove_checked ...................................................................... Remove unnecessary lookup from cih_remove_checked If the entry is not in the avl theres no need to do a lookup of the entry in the avl. In my company, we try to release more entries during reaping. From flamegraph, the reaper thread spends a lot of time reaping a non existent entry from the cache. This becomes a big bottleneck when we have to reap large number of entries. With this change we got 30% improvement in reaping latencies. Change-Id: Ied9b02789c5736b80dbce4dc226dd585ce6d1161 Signed-off-by: Deepak Arumugam Sankara Subramanian <deepakarumugam.s(a)nutanix.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_hash.h 1 file changed, 18 insertions(+), 13 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/40/1220340/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220340?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: Ied9b02789c5736b80dbce4dc226dd585ce6d1161 Gerrit-Change-Number: 1220340 Gerrit-PatchSet: 1 Gerrit-Owner: deepakarumugam.s(a)nutanix.com

1 0

[S] Change in ...nfs-ganesha[next]: Modifed ganesha build info metric with ganesha_uptime_minutes info
by Name of user not set (GerritHub) 12 Aug '25

12 Aug '25

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220324?usp=email ) Change subject: Modifed ganesha build info metric with ganesha_uptime_minutes info ...................................................................... Modifed ganesha build info metric with ganesha_uptime_minutes info Change-Id: I6ebb808121f225a0038eb76fa00d8a77831e88ee Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> --- M src/MainNFSD/nfs_metrics.c 1 file changed, 26 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/24/1220324/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1220324?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: I6ebb808121f225a0038eb76fa00d8a77831e88ee Gerrit-Change-Number: 1220324 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 0

[L] Change in ...nfs-ganesha[next]: [TLS] : NFS Ganesha - TLS support. TLS entertained via AUTH_T...
by Name of user not set (GerritHub) 06 Aug '25

06 Aug '25

deeraj.patil(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1219994?usp=email ) Change subject: [TLS] : NFS Ganesha - TLS support. TLS entertained via AUTH_TLS and non-AUTH_TLS. KTLS and non KTLS based send and recv. export option introduced : Sectype= ,mtls/tls Client mount supported matrix is updated in man page. Can control export specific TLS required or not. ...................................................................... [TLS] : NFS Ganesha - TLS support. TLS entertained via AUTH_TLS and non-AUTH_TLS. KTLS and non KTLS based send and recv. export option introduced : Sectype= ,mtls/tls Client mount supported matrix is updated in man page. Can control export specific TLS required or not. Libntirpc changes : backend TLS lib : Openssl, GnuTLS. Compile time option for enabling : USE_TLS, USE_OPENSSL, USE_GNUTLS. NOTE : Still code cleanup is pending. Change-Id: I97f93258bc72e57ce5f0683be3b78a31915befaa Signed-off-by: Deeraj.Patil <Deeraj.Patil(a)ibm.com> --- M src/CMakeLists.txt M src/MainNFSD/nfs_init.c M src/MainNFSD/nfs_worker_thread.c M src/Protocols/NFS/mnt_Mnt.c A src/cmake/modules/FindTLS.cmake M src/config_samples/config.txt M src/doc/man/ganesha-config.rst M src/doc/man/ganesha-export-config.rst A src/doc/man/ganesha-tls-config.rst M src/doc/man/index.rst M src/include/config-h.in.cmake M src/include/nfs_exports.h M src/nfs-ganesha.spec-in.cmake M src/support/exports.c M src/support/nfs_creds.c M src/support/nfs_read_conf.c 16 files changed, 594 insertions(+), 17 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/94/1219994/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1219994?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: I97f93258bc72e57ce5f0683be3b78a31915befaa Gerrit-Change-Number: 1219994 Gerrit-PatchSet: 1 Gerrit-Owner: deeraj.patil(a)ibm.com

1 0

[S] Change in ...nfs-ganesha[next]: Added export_path label to export specific metrics
by Name of user not set (GerritHub) 06 Aug '25

06 Aug '25

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1219947?usp=email ) Change subject: Added export_path label to export specific metrics ...................................................................... Added export_path label to export specific metrics Change-Id: Ic5eab3fc3a0564c8d0db18a351de5f2c06f7c1a4 Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> --- M src/monitoring/dynamic_metrics.cc M src/monitoring/include/dynamic_metrics.h M src/support/server_stats.c 3 files changed, 17 insertions(+), 7 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/47/1219947/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1219947?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: Ic5eab3fc3a0564c8d0db18a351de5f2c06f7c1a4 Gerrit-Change-Number: 1219947 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 0

Invitation: Alternate NFS Ganesha Community Meeting @ Weekly from 4pm to 5pm on Tuesday (PDT) (devel@lists.nfs-ganesha.org)
by Frank Filz 04 Aug '25

04 Aug '25

Alternate NFS Ganesha Community Meeting Weekly from 4pm to 5pm on Tuesday Pacific Time - Los Angeles Join with Google Meet https://meet.google.com/kct-xfvw-yan?hs=224 Join by phone (US) +1 347-508-5969 PIN: 576193000 More joining options https://tel.meet/kct-xfvw-yan?pin=2004287696818&hs=0 Join using SIP 2004287696818(a)gmeet.redhat.com ID: 2004287696818 This is an alternate to the community meeting that is scheduled at 7:00 AM Pacific Time. This 4:00 PM Pacific Time scheduling will be more favorable to Australia, New Zealand, and Eastern Asia. Organizer Frank Filz ffilz(a)redhat.com Guests Frank Filz - organizer devel(a)lists.nfs-ganesha.org View all guest info https://calendar.google.com/calendar/event?action=VIEW&eid=NjNkYWNwcTdhbHA2… Reply for devel(a)lists.nfs-ganesha.org and view more details https://calendar.google.com/calendar/event?action=VIEW&eid=NjNkYWNwcTdhbHA2… Your attendance is optional. ~~//~~ Invitation from Google Calendar: https://calendar.google.com/calendar/ You are receiving this email because you are an attendee on the event. Forwarding this invitation could allow any recipient to send a response to the organizer, be added to the guest list, invite others regardless of their own invitation status, or modify your RSVP. Learn more https://support.google.com/calendar/answer/37135#forwarding

1 0

[XS] Change in ...nfs-ganesha[next]: Config Parsing: Allow 3 digit mask for IPv6 Export CIDRs
by Peter Schwenke (GerritHub) 04 Aug '25

04 Aug '25

Peter Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1219764?usp=email ) Change subject: Config Parsing: Allow 3 digit mask for IPv6 Export CIDRs ...................................................................... Config Parsing: Allow 3 digit mask for IPv6 Export CIDRs Allow CIDRs such as ::ffff:192.168.10.0/120 ( which is equivalent to 192.168.10.0/24) and fe80::5a4d:7416:a595:bd6f/120. CIDRs such as that will currently fail parsing. Change-Id: I42d8be0d5b7143b8e53e63fbdddf0cd696bdd4c8 Signed-off-by: Peter Schwenke <pschwenke(a)ddn.com> --- M src/config_parsing/conf_lex.l 1 file changed, 4 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/64/1219764/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1219764?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: I42d8be0d5b7143b8e53e63fbdddf0cd696bdd4c8 Gerrit-Change-Number: 1219764 Gerrit-PatchSet: 1 Gerrit-Owner: Peter Schwenke <pschwenke(a)ddn.com>

1 0