April 2026 - Devel - NFS Ganesha List Archives

[M] Change in ...nfs-ganesha[next]: CONFIG/RGW fix open_memstream use auto after stack pop
by Name of user not set (GerritHub) 08 Apr '26

08 Apr '26

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235150?usp=email ) Change subject: CONFIG/RGW fix open_memstream use auto after stack pop ...................................................................... CONFIG/RGW fix open_memstream use auto after stack pop The routine open_memstream creates a string buffer that can be accessed by other stdio calls. The Gnu libc version documents that fclose on a descriptor created by this routine will update the "streambuf" and "streamsz" variables that were passed to open_memstream. But the stack frame in cu_rados_url_fetch which contained these variables is long gone, so this wins up being a reference to memory that is likely outside the current stack, but might be used by some other unsuspecting routine now or in the future. The changes here make it possible for url_fetch to return a callback that will later be used to release any storage associated when the stream when it is no longer needed. And the logic in conf_url_rados has been modified to take advantage of that to keep streambuf and streamsz in valid memory until after fclose. Change-Id: Ia419acbd8fa0e518f39438f1c5e34edd4ec5bf56 Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/config_parsing/conf_lex.l M src/config_parsing/conf_url.c M src/config_parsing/conf_url_rados.c M src/include/conf_url.h 4 files changed, 40 insertions(+), 22 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/50/1235150/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235150?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: Ia419acbd8fa0e518f39438f1c5e34edd4ec5bf56 Gerrit-Change-Number: 1235150 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 0

[XS] Change in ...nfs-ganesha[next]: CONFIG - fix for unknown block "rados_urls"
by Name of user not set (GerritHub) 08 Apr '26

08 Apr '26

mwatts(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235149?usp=email ) Change subject: CONFIG - fix for unknown block "rados_urls" ...................................................................... CONFIG - fix for unknown block "rados_urls" The logic in find_unused_blocks() to identify unused configuration nodes relies on the "found" flag in the node beign set, but but it's not possible for logic in conf_url_rados.c to set it. I've modified config_GetBlockNode to unconditionally set this, since there doesn't seem to be any valid reason to not set this flag right there. As it turns out, conf_url_rados is the only user of this routine. Change-Id: I4dd4bd5b3ffa6c44b6d2823885344fa662243519 Signed-off-by: Marcus Watts <mwatts(a)redhat.com> --- M src/config_parsing/config_parsing.c 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/49/1235149/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235149?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: I4dd4bd5b3ffa6c44b6d2823885344fa662243519 Gerrit-Change-Number: 1235149 Gerrit-PatchSet: 1 Gerrit-Owner: mwatts(a)redhat.com

1 0

[XS] Change in ...nfs-ganesha[next]: Return NFS4ERR_NOTSUPP for GET_DIR_DELEGATION (NFSv4.1 op 46) #1385
by Suhas Athani (GerritHub) 07 Apr '26

07 Apr '26

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235051?usp=email ) Change subject: Return NFS4ERR_NOTSUPP for GET_DIR_DELEGATION (NFSv4.1 op 46) #1385 ...................................................................... Return NFS4ERR_NOTSUPP for GET_DIR_DELEGATION (NFSv4.1 op 46) #1385 Map NFS4_OP_GET_DIR_DELEGATION to the standard nfs4_op_notsupp handler instead of nfs4_op_illegal, so Ganesha returns NFS4ERR_NOTSUPP for a recognized-but-unimplemented optional op. Change-Id: Ia219c3e46fbe7e575ccf63f892297114e369cc2f Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/Protocols/NFS/nfs4_Compound.c 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/51/1235051/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235051?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: Ia219c3e46fbe7e575ccf63f892297114e369cc2f Gerrit-Change-Number: 1235051 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[XS] Change in ...nfs-ganesha[next]: Accept RO as Delegations alias for read-only delegations
by Suhas Athani (GerritHub) 07 Apr '26

07 Apr '26

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235049?usp=email ) Change subject: Accept RO as Delegations alias for read-only delegations ...................................................................... Accept RO as Delegations alias for read-only delegations "ro"/"RO", which was previously not recognized by Ganesha’s config parser, resulting in read delegations not being enabled even though I/O continued to work. With this change, ro/RO now maps to read-only delegations (same behavior as Read/R). Change-Id: I3caf6942ae351186a324fb51ac71ea23ce878d42 Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/support/exports.c 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/49/1235049/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235049?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: I3caf6942ae351186a324fb51ac71ea23ce878d42 Gerrit-Change-Number: 1235049 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

Announce Push of V9.10
by Frank Filz 06 Apr '26

06 Apr '26

Branch next Tag:V9.10 Merge Highlights * Fix handling of WRITE and READ deleg with ATTRS * NFS ACL: Synthesize ACLs based on mode bits * LOG: Adding new component "RECOVERY" for debug logging * Improved Usability: Added EVENT Logs for RDMA Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: b366b0541 Frank S. Filz V9.10 d0937a924 Rajesh Prasad Improved Usability: Added EVENT Logs for RDMA a687031a5 Sachin Punadikar LOG: Adding new component "RECOVERY" for debug logging 9645b570b Sachin Punadikar NFS ACL: Synthesize ACLs based on mode bits 06b5ff533 Animesh Javali Fix handling of WRITE and READ deleg with ATTRS

1 0

[L] Change in ...nfs-ganesha[next]: LOG: Adding new component "RECOVERY" for debug logging
by Sachin Punadikar (GerritHub) 06 Apr '26

06 Apr '26

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1234902?usp=email ) Change subject: LOG: Adding new component "RECOVERY" for debug logging ...................................................................... LOG: Adding new component "RECOVERY" for debug logging To gather debug logs related to Ganesha recovery on startup introducing new component "RECOVERY". Earlier this functionality was covered by CLIENTID, STATE component logging. This generates extra logging which may not be required. Change-Id: I58188d42b49a012834a6ebc77ea3db9ac13ba3ab Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/SAL/nfs4_recovery.c M src/SAL/recovery/recovery_fs.c M src/SAL/recovery/recovery_fs_ng.c 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/log_common.h M src/log/log_functions.c 8 files changed, 396 insertions(+), 362 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/02/1234902/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1234902?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: I58188d42b49a012834a6ebc77ea3db9ac13ba3ab Gerrit-Change-Number: 1234902 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 0

[XS] Change in ...nfs-ganesha[next]: Fix: IBMCEPH-13719
by Animesh Javali (GerritHub) 06 Apr '26

06 Apr '26

Animesh Javali has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235022?usp=email ) Change subject: Fix: IBMCEPH-13719 ...................................................................... Fix: IBMCEPH-13719 Added code to release object ref and owner in queue_deleg_transition_handler(). Change-Id: I0806525132ad447a435440e2f36039cb15953cd6 Signed-off-by: Animesh Javali <Animesh.Javali(a)ibm.com> --- M src/support/export_mgr.c 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/22/1235022/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1235022?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: I0806525132ad447a435440e2f36039cb15953cd6 Gerrit-Change-Number: 1235022 Gerrit-PatchSet: 1 Gerrit-Owner: Animesh Javali <Animesh.Javali(a)ibm.com>

1 0

[S] Change in ...nfs-ganesha[next]: NFS ACL: Synthesize ACLs based on mode bits
by Sachin Punadikar (GerritHub) 01 Apr '26

01 Apr '26

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1234617?usp=email ) Change subject: NFS ACL: Synthesize ACLs based on mode bits ...................................................................... NFS ACL: Synthesize ACLs based on mode bits Do not skip FATTR4_ACL when ATTR_ACL is missing but ATTR_MODE is present, so the second pass still schedules ACL encoding and encode_acl() can synthesize from mode bits. Change-Id: Ic3c677cce92b5a278b4b959d2aa3eca817013234 Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/Protocols/NFS/nfs_proto_tools.c 1 file changed, 12 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/17/1234617/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1234617?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: Ic3c677cce92b5a278b4b959d2aa3eca817013234 Gerrit-Change-Number: 1234617 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 0