October 2026 - Devel - NFS Ganesha List Archives

[M] Change in ...nfs-ganesha[next]: MDCACHE: survive directory entries removed outside Ganesha
by Name of user not set (GerritHub) 21 Oct '26

21 Oct '26

4h8qdp6jx9(a)privaterelay.appleid.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255431?usp=email ) Change subject: MDCACHE: survive directory entries removed outside Ganesha ...................................................................... MDCACHE: survive directory entries removed outside Ganesha After a file was removed outside Ganesha, three bugs kept a cached directory broken: - READDIR failed with NFS4ERR_NOENT when looking up an evicted entry again. Drop the entry instead. - mdcache_is_referral() refreshed expired attributes without the mtime check that flushes cached entries, so the cache was trusted forever. - A cached READDIR never revalidated the directory, so it missed the change unless a GETATTR came first. Change-Id: I0527fdee179a13868e5242e8ab313eb822fa2a25 Signed-off-by: Evan Champion <4h8qdp6jx9(a)privaterelay.appleid.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c 2 files changed, 52 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/31/1255431/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255431?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: I0527fdee179a13868e5242e8ab313eb822fa2a25 Gerrit-Change-Number: 1255431 Gerrit-PatchSet: 1 Gerrit-Owner: 4h8qdp6jx9(a)privaterelay.appleid.com

1 0

[M] Change in ...nfs-ganesha[next]: FSAL_GPFS: Use setfsuid/setfsgid for credential impersonation to pres...
by Name of user not set (GerritHub) 21 Oct '26

21 Oct '26

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255426?usp=email ) Change subject: FSAL_GPFS: Use setfsuid/setfsgid for credential impersonation to preserve capabilities ...................................................................... FSAL_GPFS: Use setfsuid/setfsgid for credential impersonation to preserve capabilities Non-root NFS client operations (create, mkdir, chmod, link, rename, etc.) fail with EPERM over NFS-Ganesha with GPFS backend, but work fine directly on the GPFS filesystem. `fsal_set_credentials()` uses `setresuid()` to impersonate NFS client users. This changes the effective UID from 0 to non-zero, which causes the Linux kernel to drop all capabilities including `CAP_SYS_ADMIN`. The GPFS ioctl interface requires `CAP_SYS_ADMIN`, so all subsequent GPFS calls fail with EPERM. Introduce `gpfs_set_credentials()` / `gpfs_restore_ganesha_credentials()` using `setfsuid()` / `setfsgid()` instead. These change only the filesystem UID/GID without touching the effective UID, so capabilities are preserved and the GPFS ioctl succeeds. pjdfstest POSIX suite over NFS with GPFS backend: - Before: 45 test scripts failing - After: 4 residual failures (unrelated to credentials) Fixes: IBMCEPH-13445 Change-Id: Ic0ff12a2d51032262e34d5f53fa62554e137a71c Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> --- M src/FSAL/FSAL_GPFS/export.c M src/FSAL/FSAL_GPFS/file.c M src/FSAL/FSAL_GPFS/fsal_create.c M src/FSAL/FSAL_GPFS/fsal_fileop.c M src/FSAL/FSAL_GPFS/fsal_internal.c M src/FSAL/FSAL_GPFS/fsal_internal.h M src/FSAL/FSAL_GPFS/fsal_symlinks.c 7 files changed, 96 insertions(+), 40 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/26/1255426/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255426?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: Ic0ff12a2d51032262e34d5f53fa62554e137a71c Gerrit-Change-Number: 1255426 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 0

[L] Change in ...nfs-ganesha[next]: idmapper: use NUL terminated strings for user and group names
by Suhas Athani (GerritHub) 21 Oct '26

21 Oct '26

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255418?usp=email ) Change subject: idmapper: use NUL terminated strings for user and group names ...................................................................... idmapper: use NUL terminated strings for user and group names Part of github issue #1404. The idmapper, negative cache and uid2grp APIs took names as struct gsh_buffdesc, and cached names were stored without a NUL terminator even though the DBus dump code printed them with "%s". Change name2uid(), name2gid(), uname2grp() and the idmapper, negative cache and uid2grp cache interfaces to take const char *. Cached names are now stored NUL terminated and compared with strcmp(). Also fix decode_group(), which built a terminated copy of the owner_group string but passed the raw XDR buffer to name2gid(), letting strtoul() read past the end of the name when idmapping is disabled. Change-Id: I4c7de01d7e77e8545d3baccbbad6a77b8729b625 Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/Protocols/NFS/nfs_proto_tools.c M src/idmapper/idmapper.c M src/idmapper/idmapper_cache.c M src/idmapper/idmapper_negative_cache.c M src/include/idmapper.h M src/include/uid2grp.h M src/support/uid2grp.c M src/support/uid2grp_cache.c 8 files changed, 160 insertions(+), 205 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/18/1255418/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255418?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: I4c7de01d7e77e8545d3baccbbad6a77b8729b625 Gerrit-Change-Number: 1255418 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[S] Change in ...nfs-ganesha[next]: MDCACHE: force attr refresh after setattr when delegated
by Xueqian Hu (GerritHub) 20 Oct '26

20 Oct '26

Xueqian Hu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255384?usp=email ) Change subject: MDCACHE: force attr refresh after setattr when delegated ...................................................................... MDCACHE: force attr refresh after setattr when delegated mdcache_refresh_attrs() skips the backend getattr while a file is delegated, assuming attributes cannot change without a recall. Same-client write-class ops no longer recall, so setattr can complete while a delegation is still held. The following GETATTR then returns stale attributes from mdcache. Add a force flag to mdcache_refresh_attrs() and use it from mdcache_setattr2() so setattr always refreshes the attr cache. Change-Id: Ib792a563049e580e92c7c7e5cfac6fecd2f73154 Signed-off-by: xueqian.hu <xh140312(a)outlook.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h 2 files changed, 10 insertions(+), 6 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/84/1255384/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255384?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: Ib792a563049e580e92c7c7e5cfac6fecd2f73154 Gerrit-Change-Number: 1255384 Gerrit-PatchSet: 1 Gerrit-Owner: Xueqian Hu <xh140312(a)outlook.com>

1 0

[S] Change in ...nfs-ganesha[next]: [rdma-bind-addr] RDMA: honor Bind_Addr when creating the listener
by Kirill Gibelgaus (GerritHub) 20 Oct '26

20 Oct '26

Kirill Gibelgaus has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255306?usp=email ) Change subject: [rdma-bind-addr] RDMA: honor Bind_Addr when creating the listener ...................................................................... [rdma-bind-addr] RDMA: honor Bind_Addr when creating the listener The NFS/RDMA listener always uses 0.0.0.0, even when Bind_Addr specifies a particular local address. The TCP and UDP listeners already use Bind_Addr. A wildcard RDMA listener prevents instances bound to separate local IPs from using the same RDMA port in one network namespace. Use the configured bind address when creating the RDMA transport. Normalize IPv4-mapped IPv6 addresses with convert_ipv6_to_ipv4(), then format the address with sprint_sockip(). Keep the formatted address in static storage because ntirpc retains its pointer. A local C probe compiling the actual Create_RDMA() and address helpers with transport creation captured reproduced the failure before the fix and passed 10 IPv4, mapped IPv4, native IPv6, wildcard, and invalid-address cases after the fix under AddressSanitizer and UndefinedBehaviorSanitizer. clang-format, checkpatch, and git diff --check also passed. A full Linux/RDMA build and hardware-backed RDMA connections have not been tested. An omitted Bind_Addr normally initializes to ::, which is now passed through instead of the previously hardcoded IPv4 wildcard. Change-Id: I1505ee1099a38fdb040030d7d0cc55fdb597990f Signed-off-by: Kirill Gibelgaus <kgibelgaus(a)ozon.ru> --- M src/MainNFSD/nfs_rpc_dispatcher_thread.c 1 file changed, 11 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/06/1255306/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255306?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: I1505ee1099a38fdb040030d7d0cc55fdb597990f Gerrit-Change-Number: 1255306 Gerrit-PatchSet: 1 Gerrit-Owner: Kirill Gibelgaus <ksgibelgaus(a)edu.hse.ru>

1 0

[XS] Change in ...nfs-ganesha[next]: Prevent Input/Outpur error from a failed idmapping. If decode for ACL...
by Michael Diederich (GerritHub) 20 Oct '26

20 Oct '26

Michael Diederich has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255237?usp=email ) Change subject: Prevent Input/Outpur error from a failed idmapping. If decode for ACL fattr during setattr fails - log message "Decode attr FAILED:" due to idmap, the call is failed with FATTR_XDR_FAILED that the client will show as IO-Error. The patch will change the error to NFS4ERR_BADOWNER instead. ...................................................................... Prevent Input/Outpur error from a failed idmapping. If decode for ACL fattr during setattr fails - log message "Decode attr FAILED:" due to idmap, the call is failed with FATTR_XDR_FAILED that the client will show as IO-Error. The patch will change the error to NFS4ERR_BADOWNER instead. Change-Id: I40b3229598c667f1a0a1dcce5307289fecbbcfc0 Signed-off-by: Michael Diederich <diederich(a)de.ibm.com> --- M src/Protocols/NFS/nfs_proto_tools.c 1 file changed, 6 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/37/1255237/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255237?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: I40b3229598c667f1a0a1dcce5307289fecbbcfc0 Gerrit-Change-Number: 1255237 Gerrit-PatchSet: 1 Gerrit-Owner: Michael Diederich <diederich(a)de.ibm.com>

1 0

[S] Change in ...nfs-ganesha[next]: fix memleak or double free client callback state
by haihuayang (GerritHub) 19 Oct '26

19 Oct '26

haihuayang has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255188?usp=email ) Change subject: fix memleak or double free client callback state ...................................................................... fix memleak or double free client callback state Change-Id: Ida284ddc49d4de6404c9becf63da9ffd7395fe80 Signed-off-by: Haihua Yang <yanghh(a)gmail.com> --- M src/SAL/nfs4_clientid.c 1 file changed, 41 insertions(+), 6 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/88/1255188/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255188?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: Ida284ddc49d4de6404c9becf63da9ffd7395fe80 Gerrit-Change-Number: 1255188 Gerrit-PatchSet: 1 Gerrit-Owner: haihuayang <yanghh(a)gmail.com>

1 0

[XS] Change in ...nfs-ganesha[next]: Enable xattr only for GPFS version 5 and up.
by marc eshel (GerritHub) 19 Oct '26

19 Oct '26

marc eshel has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255156?usp=email ) Change subject: Enable xattr only for GPFS version 5 and up. ...................................................................... Enable xattr only for GPFS version 5 and up. Change-Id: I4023619e01f9f71aed2d90920d85bcc315474dfe Signed-off-by: marc eshel <eshel(a)us.ibm.com> --- M src/FSAL/FSAL_GPFS/export.c 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/56/1255156/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255156?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: I4023619e01f9f71aed2d90920d85bcc315474dfe Gerrit-Change-Number: 1255156 Gerrit-PatchSet: 1 Gerrit-Owner: marc eshel <eshel(a)us.ibm.com>

1 0

[XS] Change in ...nfs-ganesha[next]: Fixes #1423
by Vinícius Ferrão (GerritHub) 18 Oct '26

18 Oct '26

Vinícius Ferrão has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255089?usp=email ) Change subject: Fixes #1423 ...................................................................... Fixes #1423 Change-Id: I37a0673e17fb347dd1d34deaf90cb0c9e7ab65db Signed-off-by: Vinícius Ferrão <ferrao(a)versatushpc.com.br> --- M src/SAL/recovery/recovery_fs.c 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/89/1255089/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1255089?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: I37a0673e17fb347dd1d34deaf90cb0c9e7ab65db Gerrit-Change-Number: 1255089 Gerrit-PatchSet: 1 Gerrit-Owner: Vinícius Ferrão <vinicius(a)ferrao.net.br>

1 0

Announce Push of V15.9
by Frank Filz 16 Oct '26

16 Oct '26

Branch next Tag:V15.9 Merge Highlights * Removal of 9P code - 9P is not used to my knowledge and has not been maintained * Fix: skip Prometheus inode metrics when FSAL reports unsupported values (CephFS) * cmake: Fix case mismatch in FindTcMalloc.cmake * simple fix in handling LAYOUTERROR and LAYOUTSTATS * FSAL/CEPH: Reject handles from another filesystem * XDR: reject empty NLM caller_name and lock owner * export_mgr: Fix gsh_refstr memory leak in update_export_metadata_metric * nfs4_op_sequence: Fix session use-after-free on ca_maxrequestsize error Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 42bc41b62 Frank S. Filz V15.9 e4089a45e Shahar Hochma nfs4_op_sequence: Fix session use-after-free on ca_maxrequestsize error dc481c94a Shahar Hochma export_mgr: Fix gsh_refstr memory leak in update_export_metadata_metric 27792fbb4 Suhas Athani XDR: reject empty NLM caller_name and lock owner 45cc5f513 Sheng Li FSAL/CEPH: Reject handles from another filesystem 1c08fbefc Haihua Yang simple fix in handling LAYOUTERROR and LAYOUTSTATS f0dbd5849 bjfhdhhaha cmake: Fix case mismatch in FindTcMalloc.cmake 898d398a3 Sreedhar Agraharam Fix: skip Prometheus inode metrics when FSAL reports unsupported values (CephFS) 83c0658b4 Frank S. Filz Remove 9P support

1 0