August 2026 - Devel - NFS Ganesha List Archives

[M] Change in ...nfs-ganesha[next]: Support export fencing via Server_Addrs
by Sachin Punadikar (GerritHub) 19 Aug '26

19 Aug '26

Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1246011?usp=email ) Change subject: Support export fencing via Server_Addrs ...................................................................... Support export fencing via Server_Addrs Restrict access to the export on basis of which IP address is used by NFS client to access the export. This feature is useful when Ganesha is listening on all available n/w interfaces and administrator wants to provide access to the export only when the export is being accessed using a specific IP. Change-Id: I19133c3861e8a70a1c76c74d28c7f5aa91ce470d Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com> --- M src/doc/man/ganesha-export-config.rst M src/include/export_mgr.h M src/support/export_mgr.c M src/support/exports.c 4 files changed, 105 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/1246011/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1246011?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: I19133c3861e8a70a1c76c74d28c7f5aa91ce470d Gerrit-Change-Number: 1246011 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>

1 0

[S] Change in ...nfs-ganesha[next]: QoS: fix NULL qos_block deref in qos_perexport_insert
by Suhas Athani (GerritHub) 19 Aug '26

19 Aug '26

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245955?usp=email ) Change subject: QoS: fix NULL qos_block deref in qos_perexport_insert ...................................................................... QoS: fix NULL qos_block deref in qos_perexport_insert qos_perexport_insert(export, NULL) is valid and means "use g_qos_config" via lqos_block. The USE_MONITORING path still used qos_block->qos_type, which SIGSEGV'd when EnableExportQosBwControl passed NULL. Use lqos_block for the Per_Export metrics check. Crash site: nfs_qos.c qos_perexport_insert(), qos_block == NULL. Change-Id: I04a982e8c3b5142c9f0c99e2a8d645e6a6ee98d3 Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/MainNFSD/nfs_qos.c 1 file changed, 11 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/55/1245955/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245955?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: I04a982e8c3b5142c9f0c99e2a8d645e6a6ee98d3 Gerrit-Change-Number: 1245955 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[L] Change in ...nfs-ganesha[next]: gRPC: add Per_Client QoS admin RPCs
by Suhas Athani (GerritHub) 19 Aug '26

19 Aug '26

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245950?usp=email ) Change subject: gRPC: add Per_Client QoS admin RPCs ...................................................................... gRPC: add Per_Client QoS admin RPCs Add gRPC equivalents of the Per_Client D-Bus QoS methods on org.ganesha.nfsd.qos (QosMgr). RPCs - GetClientBandwidth / SetClientBandwidth - GetClientTokens / SetClientTokens - GetClientIOPS / SetClientIOPS - EnableExportQosBwControl / DisableExportQosBwControl These land on qosService.QosMgr, with C helpers in nfs_qosmgr.c and handlers in nfsServiceServer.cc. Behavior matches the existing D-Bus methods: lookup the client by IP, then get/set rbucket/wbucket limits. Change-Id: I8b3a241a98f0b6835260efa4871f2346160f3c71 Signed-off-by: Suhas Athani <Suhas.Athani(a)ibm.com> --- M src/MainNFSD/nfs_qos.c M src/grpc_server/GrpcServer.cc M src/grpc_server/GrpcServer.h M src/grpc_server/nfsService.h M src/grpc_server/nfsServiceServer.cc A src/grpc_server/proto/qosService.proto A src/include/nfs_qos_grpc.h M src/support/nfs_qosmgr.c 8 files changed, 916 insertions(+), 24 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/50/1245950/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245950?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: I8b3a241a98f0b6835260efa4871f2346160f3c71 Gerrit-Change-Number: 1245950 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 0

[M] Change in ...nfs-ganesha[next]: Fix NULL ev_p Crash During Blocked Lock Grant
by Rajesh Prasad (GerritHub) 18 Aug '26

18 Aug '26

Rajesh Prasad has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245847?usp=email ) Change subject: Fix NULL ev_p Crash During Blocked Lock Grant ...................................................................... Fix NULL ev_p Crash During Blocked Lock Grant Bug: Ganesha crashes with a NULL dereference in `svc_rqst_expire_insert` when granting a blocked NFSv4 lock to a client that disconnected before the grant was attempted. Event Sequence: Client A - Acquires lock on file F Client B - Requests same lock; placed in blocked-lock queue (`STATE_BLOCKING`) Client B - Disconnect — `svc_rqst_unreg()` clears `rec->ev_p = NULL Client A - Issues `LOCKU` — lock released Ganesha - Calls `grant_blocked_locks()` — Client B's entry is the first candidate Ganesha - `nfsv4_granted_callback()` → `nfs_rpc_cb_single()` → `nfs_rpc_call()` → `CLNT_CALL_BACK()` → `clnt_req_callback()` → `svc_rqst_expire_insert()` CRASH - `sr_rec = cx->cx_rec->ev_p` is `NULL` → `mutex_lock(sr_rec->)` → SIGSEGV Fix: Before building the `CB_COMPOUND` payload, verify the client still has a working back channel. NFSv4.0 — read the `cb_chan_down` flag NFSv4.1/4.2 — scan `cb_session_list` under `cid_mutex` for any session that has `session_bc_up` set Handle RPC failure case - Covers the narrow race where the transport is destroyed after the pre-check passes but before (or during) the RPC send. If RPC failed, remove entry, try next. Used `STATE_ESTALE` to cleans up correctly - `try_to_grant_lock()` handles the return value of the callback: ``` STATE_LOCK_BLOCKED → restore sle_blocked, leave in list (will retry) STATE_SUCCESS → return (lock granted, entry promoted) anything else → fall through to remove_from_locklist() ``` `STATE_ESTALE` is "anything else", so the entry is removed. `grant_blocked_locks()` uses `glist_for_each_safe`, so after the entry is removed the loop automatically advances to the next blocked waiter. Change-Id: I119d51ff60fb20022a1e38e3c1c510da8f45dcb2 Signed-off-by: Rajesh Prasad <raprasad(a)redhat.com> --- M src/Protocols/NFS/nfs4_op_lock.c 1 file changed, 69 insertions(+), 7 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/47/1245847/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245847?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: I119d51ff60fb20022a1e38e3c1c510da8f45dcb2 Gerrit-Change-Number: 1245847 Gerrit-PatchSet: 1 Gerrit-Owner: Rajesh Prasad <raprasad(a)redhat.com>

1 0

[M] Change in ...nfs-ganesha[next]: Add NFS_CORE_PARAM Mem_Stats_Disable for memory statistics capture
by Nishant Puri (GerritHub) 18 Aug '26

18 Aug '26

Nishant Puri has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245787?usp=email ) Change subject: Add NFS_CORE_PARAM Mem_Stats_Disable for memory statistics capture ...................................................................... Add NFS_CORE_PARAM Mem_Stats_Disable for memory statistics capture Introduce Mem_Stats_Disable (default false) to skip gsh_malloc/gsh_free per-component counter updates when operators do not need mem_stats. When disabled, ganesha_stats mem_stats show reports zero for all counters; mem_stats status reports active/inactive via new GetMemoryStatsStatus D-Bus method. Log enabled/disabled once at init (COMPONENT_INIT). Skip shutdown gsh_log_mem_stats() when capture is inactive. Option is read at daemon startup only; SIGHUP config reload does not change it. Document the option in ganesha-core-config.rst. Change-Id: I2e6ff0f43014472e4e8eecaa8a70948af657741a Signed-off-by: Nishant Puri <npuri(a)redhat.com> --- M src/MainNFSD/nfs_init.c M src/doc/man/ganesha-core-config.rst M src/include/gsh_config.h M src/include/server_stats.h M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py M src/scripts/ganeshactl/ganesha_stats.py M src/support/nfs_read_conf.c M src/support/server_stats.c 8 files changed, 171 insertions(+), 36 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/1245787/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245787?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: I2e6ff0f43014472e4e8eecaa8a70948af657741a Gerrit-Change-Number: 1245787 Gerrit-PatchSet: 1 Gerrit-Owner: Nishant Puri <npuri(a)redhat.com>

1 0

[M] Change in ...nfs-ganesha[next]: Enhance client ID diagnostics, IP logging, and reaper trace control 1...
by Name of user not set (GerritHub) 17 Aug '26

17 Aug '26

inneek(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245716?usp=email ) Change subject: Enhance client ID diagnostics, IP logging, and reaper trace control 1. Add dump_client_owners_and_states() to log client owners and state IDs when extra references block client expiration. 2. Log a warning in EXCHANGE_ID when same client owner name connects from different IP addresses. 3. Suppress LTTng trace spam when the client reaper loops >100 times waiting for state owner refcounts to drop. ...................................................................... Enhance client ID diagnostics, IP logging, and reaper trace control 1. Add dump_client_owners_and_states() to log client owners and state IDs when extra references block client expiration. 2. Log a warning in EXCHANGE_ID when same client owner name connects from different IP addresses. 3. Suppress LTTng trace spam when the client reaper loops >100 times waiting for state owner refcounts to drop. Change-Id: I5e758ba124fa554188f2ae961b7ca27b5e4e04bc Signed-off-by: Inna Batenkov <inneek(a)gmail.com> --- M src/Protocols/NFS/nfs4_op_exchange_id.c M src/SAL/nfs4_clientid.c M src/SAL/state_misc.c M src/include/sal_functions.h 4 files changed, 201 insertions(+), 13 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/16/1245716/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245716?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: I5e758ba124fa554188f2ae961b7ca27b5e4e04bc Gerrit-Change-Number: 1245716 Gerrit-PatchSet: 1 Gerrit-Owner: inneek(a)gmail.com

1 0

[S] Change in ...nfs-ganesha[next]: Reorder operations in fsal_test_access to eliminate redundant getattr...
by Name of user not set (GerritHub) 17 Aug '26

17 Aug '26

inneek(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245714?usp=email ) Change subject: Reorder operations in fsal_test_access to eliminate redundant getattrs calls. ...................................................................... Reorder operations in fsal_test_access to eliminate redundant getattrs calls. When access_type is 0 ("nothing requested"), fsal_test_access previously called getattrs on the object handle before checking access_type. Moving the access_type check to the beginning of fsal_test_access avoids executing an unnecessary getattrs RPC query when access_type == 0 Change-Id: I791a175e3a92f2fb449f856a9c3f7918258dffdd Signed-off-by: Inna Batenkov <inneek(a)gmail.com> --- M src/FSAL/access_check.c 1 file changed, 17 insertions(+), 8 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/14/1245714/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245714?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: I791a175e3a92f2fb449f856a9c3f7918258dffdd Gerrit-Change-Number: 1245714 Gerrit-PatchSet: 1 Gerrit-Owner: inneek(a)gmail.com

1 0

[M] Change in ...nfs-ganesha[next]: FSAL_VFS: implement NFSv4.2 READ_PLUS
by Name of user not set (GerritHub) 16 Aug '26

16 Aug '26

taaahaaa75(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245611?usp=email ) Change subject: FSAL_VFS: implement NFSv4.2 READ_PLUS ...................................................................... FSAL_VFS: implement NFSv4.2 READ_PLUS Use SEEK_DATA and SEEK_HOLE to report sparse file extents through NFSv4.2 READ_PLUS. Return short data extents at hole boundaries and avoid retaining or transmitting buffers for hole responses. Also clear the data pointer for zero-length READ_PLUS results to avoid retaining a released buffer. Fixes: https://github.com/nfs-ganesha/nfs-ganesha/issues/1344 Related-to: https://github.com/nfs-ganesha/nfs-ganesha/issues/1188 Change-Id: I85202481f987b547c244184e2666f8c00bea2d11 Signed-off-by: t.hosseini <taaahaaa75(a)gmail.com> --- M src/FSAL/FSAL_VFS/file.c M src/Protocols/NFS/nfs4_op_read.c 2 files changed, 100 insertions(+), 22 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/1245611/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245611?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: I85202481f987b547c244184e2666f8c00bea2d11 Gerrit-Change-Number: 1245611 Gerrit-PatchSet: 1 Gerrit-Owner: taaahaaa75(a)gmail.com

1 0

Announce Push of V15.1
by Frank Filz 14 Aug '26

14 Aug '26

Branch next Tag:V15.1 This merge includes some bug fixes and some enhancements to gRPC. Merge Highlights * Use GPFS-specific logrotate config for Ganesha packages * Fix malformed legacy Python install bcond generation * Fix gRPC build for two-arg gsh_free and C++ linkage of mem stats * gRPC: Add gRPC FakeRecall API for CBSIM fake_recall * Fix Coverity leak in ceph_register_nfs_service(Coverity: 651219) * Github issue #1128 - Reuse common synchronization for UpdateExport * Grpc: Extend ShowExports response with protocol summary * memstats: fix comp-tag mismatches, and bpftrace audit * FSAL/CEPH: Release invalidated dentry inode Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: f35046ff9 Frank S. Filz V15.1 3a0dcc393 Sheng Li FSAL/CEPH: Release invalidated dentry inode 84093c189 Nishant Puri memstats: fix comp-tag mismatches, and bpftrace audit 28a7f28a7 VidyaThumukunta Grpc: Extend ShowExports response with protocol summary dcb5934ea VidyaThumukunta Github issue #1128 - Reuse common synchronization for UpdateExport 2a8abe7ea Suhas Athani Fix Coverity leak in ceph_register_nfs_service(Coverity: 651219) bcd37c533 Suhas Athani gRPC: Add gRPC FakeRecall API for CBSIM fake_recall fafff0966 Suhas Athani Fix gRPC build for two-arg gsh_free and C++ linkage of mem stats 0353f0953 Prabhu Murugesan Fix malformed legacy Python install bcond generation 624586f78 Prabhu Murugesan Use GPFS-specific logrotate config for Ganesha packages

1 0

[M] Change in ...nfs-ganesha[next]: memstats: fix comp-tag mismatches, and bpftrace audit
by Nishant Puri (GerritHub) 13 Aug '26

13 Aug '26

Nishant Puri has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245460?usp=email ) Change subject: memstats: fix comp-tag mismatches, and bpftrace audit ...................................................................... memstats: fix comp-tag mismatches, and bpftrace audit Fix READDIR vio_base free tag (IO_BUFFER) and utf8string XDR_FREE via free_utf8string() so PROTOCOL allocs are not freed as LIBNTIRPC. Improve audit file: delete per-pointer BPF keys on free, larger hash maps, and silence bpftrace delete() warnings. Change-Id: I2eba1ca8ec9c91872d1836c926ad493d7932eb9c Signed-off-by: Nishant Puri <npuri(a)redhat.com> --- M src/Protocols/NFS/nfs4_op_readdir.c M src/include/nfsv41.h M src/scripts/bpftrace/README.md M src/scripts/bpftrace/mem_comp_audit.bt 4 files changed, 68 insertions(+), 22 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/60/1245460/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1245460?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: I2eba1ca8ec9c91872d1836c926ad493d7932eb9c Gerrit-Change-Number: 1245460 Gerrit-PatchSet: 1 Gerrit-Owner: Nishant Puri <npuri(a)redhat.com>

1 0