September 2025 - Devel - NFS Ganesha List Archives

Announce Push of V7.1
by Frank Filz 04 Sep '25

04 Sep '25

Branch next Tag:V7.1 NOTE: This merge includes an ntirpc pullup. Please update your submodule. NOTE: Prometheus is now run-time disabled by default - if you want Prometheus active, you must change configuration. Merge Highlights * CEPH: Make statfs report quota inforation for handle provided rather than cmount * NFSv4.0: fix crash in release_cb_slot() due to uninitialized mutex. * Fix resource leak in gsh_libunwind (Coverity CID 583559) * GPFS FSAL use correct verifier pointer. * Fix missing return type for prometheus_exposer__stop to resolve compilation error * Fix: Configured based enabling of monitoring/prometheus module. * dbus: release export ref if attempt to acquire export lock fails * Remove unnecessary lookup from cih_remove_checked * mdc_up_invalidate to release acl in entry->attrs * STATELOCK_lock should only be called by REGULAR_FILE * fix layout state ref_count * updating layout.granting should be atomic * CEPH: Log ceph uuid as event * CEPH : Add dentry invalidation callback * Fix coverity issues * Coverity 583540: Fix for handling of sysconf negative return * NLM: Claim fsal object handle lock while cancelling all nlm blocked list * make: fix conditional syntax to avoid premature variable expansion * Pullup ntirpc 7.1 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 683de56bd Frank S. Filz V7.1 1b5420deb Frank S. Filz Pullup ntirpc 7.1 52cd040f2 Prabhu Murugesan cmake: fix conditional syntax to avoid premature variable expansion f2b9208f2 Nishant Puri NLM: Claim fsal object handle lock while cancelling all nlm blocked list 6516a5617 VidyaThumukunta Coverity 583540: Fix for handling of sysconf negative return cf04c1ad9 Sachin Punadikar Fix coverity issues 75e9d90d2 Sachin Punadikar CEPH : Add dentry invalidation callback 3dd469d06 Sachin Punadikar CEPH: Log ceph uuid as event 145e49e88 Haihua Yang updating layout.granting should be atomic 029854a7f Haihua Yang fix layout state ref_count 326022d93 xueqian.hu STATELOCK_lock should only be called by REGULAR_FILE f1aa9d6b1 Xueqian Hu mdc_up_invalidate to release acl in entry->attrs e5e8d6100 Deepak Arumugam Sankara Subramanian Remove unnecessary lookup from cih_remove_checked 951ea9ab7 Pradeep dbus: release export ref if attempt to acquire export lock fails d8cfe5b66 Sreedhar Agraharam Fix: Configured based enabling of monitoring/prometheus module. 449c7e2c8 Sreedhar Agraharam Fix missing return type for prometheus_exposer__stop to resolve compilation error 49040f70a Marc Eshel GPFS FSAL use correct verifier pointer. 98b3ed5fa Suhas Athani Fix resource leak in gsh_libunwind (Coverity CID 583559) 1cfef0bde Deeraj.Patil NFSv4.0: fix crash in release_cb_slot() due to uninitialized mutex. 400986591 Deeraj.Patil QOS : Fixes 668fd5e2d Frank S. Filz CEPH: Make statfs report quota inforation for handle provided rather than cmount

1 0

[M] Change in ...nfs-ganesha[next]: Changes include:
by Animesh Javali (GerritHub) 04 Sep '25

04 Sep '25

Animesh Javali has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222001?usp=email ) Change subject: Changes include: ...................................................................... Changes include: - Closing the used global FD using `ceph_close_my_fd()` during delegreturn. (Otherwise subsequent WRITE & READ would hang after a delegreturn.) - Enabling Read and Write file delegations by switching delegation option from `FSAL_OPTION_FILE_READ_DELEG` to `FSAL_OPTION_FILE_DELEGATIONS`. - In `nfs4_op_open`, store the associated open state key (`stateid_other`) within the delegation state to open the files using this information later via fsal_start_io. - Code changes to allow delegation option parsing from EXPORT_DEFAULT, EXPORT, CLIENT blocks - Added FSAL_O_ANY flag for write delegation and delegreturn cases. - Removed exit label in ceph_fsal_lease_op2 to avoid returning success whenever fsal_start_io returns failure. Change-Id: I1c6830f97a89567957057dd66070dc423490b236 Signed-off-by: Animesh Javali <Animesh.Javali(a)ibm.com> --- M src/FSAL/FSAL_CEPH/handle.c M src/FSAL/FSAL_CEPH/main.c M src/MainNFSD/libganesha_nfsd.ver M src/Protocols/NFS/nfs4_op_open.c M src/include/export_mgr.h M src/include/nfs_exports.h M src/support/exports.c 7 files changed, 76 insertions(+), 6 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/01/1222001/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222001?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: I1c6830f97a89567957057dd66070dc423490b236 Gerrit-Change-Number: 1222001 Gerrit-PatchSet: 1 Gerrit-Owner: Animesh Javali <Animesh.Javali(a)ibm.com>

1 0

[S] Change in ...nfs-ganesha[next]: FSAL: Add delegation-aware FD reuse to fsal_start_io()
by Suhas Athani (GerritHub) 03 Sep '25

03 Sep '25

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221975?usp=email ) Change subject: FSAL: Add delegation-aware FD reuse to fsal_start_io() ...................................................................... FSAL: Add delegation-aware FD reuse to fsal_start_io() When a WRITE delegation is granted (DELEG_GRANTED), this change allows fsal_start_io() to reuse the associated file descriptor from the original open state, avoiding unnecessary reopen logic. This is done by resolving the delegated state to its openstate and attempting to start I/O on the related FD. Reference counting for openstate is properly maintained, and logs are added for both successful reuse and fallback paths. Change-Id: I2510d276db8f531a553362573896c194ba6bf610 Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/FSAL/commonlib.c M src/include/sal_data.h 2 files changed, 34 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/75/1221975/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221975?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: I2510d276db8f531a553362573896c194ba6bf610 Gerrit-Change-Number: 1221975 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[S] Change in ...nfs-ganesha[next]: Fix ip_str_to_sockaddr for IPv6
by Peter Schwenke (GerritHub) 03 Sep '25

03 Sep '25

Peter Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221963?usp=email ) Change subject: Fix ip_str_to_sockaddr for IPv6 ...................................................................... Fix ip_str_to_sockaddr for IPv6 The version of this function I committed previously is wrong. The calls to inet_pton() should fill in the argument sp (of type sockadd_t). If an IP address is the V6 equivalent of a V4 address it is converted to an IP address and sp is updated accordingly. Change-Id: I14f1f0445bb5089946ac5ba5cde3bba57d1b3d0c Signed-off-by: Peter Schwenke <pschwenke(a)ddn.com> --- M src/RPCAL/rpc_tools.c 1 file changed, 8 insertions(+), 13 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/63/1221963/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221963?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: I14f1f0445bb5089946ac5ba5cde3bba57d1b3d0c Gerrit-Change-Number: 1221963 Gerrit-PatchSet: 1 Gerrit-Owner: Peter Schwenke <pschwenke(a)ddn.com>

1 0

[S] Change in ...nfs-ganesha[next]: Fix coverity issues
by Sachin Punadikar (GerritHub) 03 Sep '25

03 Sep '25

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221887?usp=email ) Change subject: Fix coverity issues ...................................................................... Fix coverity issues Fixing 2 high impact coverity issues: 583531 583539 Change-Id: Ic3df5487fe1e8336bd83910259b1f1c24ebfe73a Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/SAL/recovery/recovery_fs.c M src/support/nfs_creds.c 2 files changed, 23 insertions(+), 22 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/1221887/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221887?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: Ic3df5487fe1e8336bd83910259b1f1c24ebfe73a Gerrit-Change-Number: 1221887 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 0

[XS] Change in ...nfs-ganesha[next]: NLM: Claim fsal object handle lock while cancelling all nlm blocked list
by Nishant Puri (GerritHub) 02 Sep '25

02 Sep '25

Nishant Puri has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221757?usp=email ) Change subject: NLM: Claim fsal object handle lock while cancelling all nlm blocked list ...................................................................... NLM: Claim fsal object handle lock while cancelling all nlm blocked list The lock was not being taken before accessing blocked lock list on fsal object handle. So after processing any element in the list, we should release the state lock. Change-Id: I271a99904512832b8f5b561990718dbb1b48eeaf Signed-off-by: Nishant Puri <npuri(a)redhat.com> --- M src/SAL/state_lock.c 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/57/1221757/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221757?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: I271a99904512832b8f5b561990718dbb1b48eeaf Gerrit-Change-Number: 1221757 Gerrit-PatchSet: 1 Gerrit-Owner: Nishant Puri <npuri(a)redhat.com>

1 0

[M] Change in ...nfs-ganesha[next]: CEPH : Add dentry validation callback
by Sachin Punadikar (GerritHub) 31 Aug '25

31 Aug '25

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221617?usp=email ) Change subject: CEPH : Add dentry validation callback ...................................................................... CEPH : Add dentry validation callback CEPH clint provides a mechanism to indicate a dentry has been removed. Ganesha need to register the callback to receive and process this event. Change-Id: I9dffc5ee9b46db8fae1bccc10b5134228815572f Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/FSAL/FSAL_CEPH/main.c 1 file changed, 53 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/17/1221617/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1221617?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: I9dffc5ee9b46db8fae1bccc10b5134228815572f Gerrit-Change-Number: 1221617 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 0