September 2025 - Devel - Nfs Ganesha List Archives

[M] Change in ...nfs-ganesha[next]: Add Ganesha resource usage metrics for Prometheus

by Name of user not set (GerritHub)

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222981?usp=email ) Change subject: Add Ganesha resource usage metrics for Prometheus ...................................................................... Add Ganesha resource usage metrics for Prometheus nfs_memory_resident_ram_size: Resident set size (physical RAM used by the process), in MB nfs_virtual_ram_size: Virtual memory usage of the process, in GB nfs_memory_swap_size: Swap memory used by the process, in KB nfs_cpu_utilization: Average CPU utilization by the Ganesha service, expressed as a percent Change-Id: I961d4e1d5ea1a638a27ca80d399effdcb35a633c Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> --- M src/monitoring/CMakeLists.txt M src/monitoring/dynamic_metrics.cc M src/monitoring/include/dynamic_metrics.h M src/monitoring/include/prometheus_exposer.h M src/monitoring/prometheus_exposer.cc 5 files changed, 82 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/81/1222981/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222981?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: I961d4e1d5ea1a638a27ca80d399effdcb35a633c Gerrit-Change-Number: 1222981 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 hour, 24 minutes

1
0
0 / 0

[L] Change in ...nfs-ganesha[next]: Added new metrics support QOS feature

by Name of user not set (GerritHub)

sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222927?usp=email ) Change subject: Added new metrics support QOS feature ...................................................................... Added new metrics support QOS feature This PR introduces structured support for QoS metrics across three categories: Per_Export Per_Client Per_Export_Per_Client Each category supports 6 metrics, covering: Bandwidth (BW) IOPS Tokens Example: Metrics for Per_Export Bandwidth: QoS_per_export_readbucket_bw_info QoS_per_export_writebucket_bw_info IOPS: QoS_per_export_readbucket_iops_info QoS_per_export_writebucket_iops_info Tokens: QoS_per_export_readbucket_tokens_info QoS_per_export_writebucket_tokens_info The same 6-metric structure applies to: Per_Client (e.g., QoS_per_client_*) Per_Export_Per_Client (e.g., QoS_per_pepc_*) Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> Change-Id: Ib66c7643aae2e89b76fff47ac5468b665b995d8b --- M src/MainNFSD/nfs_qos.c M src/include/nfs_qos.h 2 files changed, 293 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/27/1222927/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222927?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: Ib66c7643aae2e89b76fff47ac5468b665b995d8b Gerrit-Change-Number: 1222927 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

2 days, 16 hours

1
0
0 / 0

[XL] Change in ...nfs-ganesha[next]: [DNM]: Test PR to check the too many characters

by Name of user not set (GerritHub)

manim(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222628?usp=email ) Change subject: [DNM]: Test PR to check the too many characters ...................................................................... [DNM]: Test PR to check the too many characters Change-Id: I9f652a79ef71631d689121cfe0a3ba0554b79ddf Signed-off-by: Manimaran M <manim(a)redhat.com> Signed-off-by: Naresh Chillarege <naresh.chillarege(a)ibm.com> --- M src/CMakeLists.txt M src/MainNFSD/CMakeLists.txt A src/MainNFSD/nfs_cluster_qos.c M src/MainNFSD/nfs_init.c M src/MainNFSD/nfs_qos.c M src/MainNFSD/nfs_rpc_dispatcher_thread.c M src/MainNFSD/nfs_worker_thread.c M src/Protocols/CMakeLists.txt A src/Protocols/CQOS/CMakeLists.txt A src/Protocols/CQOS/cqos_Null.c A src/Protocols/CQOS/cqos_pubsub.c M src/Protocols/XDR/CMakeLists.txt A src/Protocols/XDR/cqos.x A src/Protocols/XDR/cqos_xdr.c A src/Protocols/XDR/xdr_cqos.c M src/include/client_mgr.h M src/include/config-h.in.cmake A src/include/cqos.h M src/include/gsh_config.h A src/include/nfs_cluster_qos.h M src/include/nfs_init.h M src/include/nfs_proto_data.h M src/include/nfs_proto_functions.h M src/include/nfs_qos.h M src/nfs-ganesha.spec-in.cmake M src/support/client_mgr.c M src/support/nfs_read_conf.c 27 files changed, 2,404 insertions(+), 35 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/28/1222628/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222628?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: I9f652a79ef71631d689121cfe0a3ba0554b79ddf Gerrit-Change-Number: 1222628 Gerrit-PatchSet: 1 Gerrit-Owner: manim(a)redhat.com

6 days, 1 hour

1
0
0 / 0

[XL] Change in ...nfs-ganesha[next]: Cluster QoS module implementation

by Name of user not set (GerritHub)

Naresh.Chillarege(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222624?usp=email ) Change subject: Cluster QoS module implementation ...................................................................... Cluster QoS module implementation The Cluster QoS feature does bandwidth rate-limiting and iops rate-limiting for across all nodes of cluster. It makes sure that aggregate bandwidth and iops of all nodes is with in the configured value. Change-Id: I985895068b54ae9404b5e9eb3cb1b69b8560849c Signed-off-by: Naresh Chillarege <naresh.chillarege(a)ibm.com> --- M src/CMakeLists.txt M src/MainNFSD/CMakeLists.txt A src/MainNFSD/nfs_cluster_qos.c M src/MainNFSD/nfs_init.c M src/MainNFSD/nfs_qos.c M src/MainNFSD/nfs_rpc_dispatcher_thread.c M src/MainNFSD/nfs_worker_thread.c M src/Protocols/CMakeLists.txt A src/Protocols/CQOS/CMakeLists.txt A src/Protocols/CQOS/cqos_Null.c A src/Protocols/CQOS/cqos_pubsub.c M src/Protocols/XDR/CMakeLists.txt A src/Protocols/XDR/cqos.x A src/Protocols/XDR/cqos_xdr.c A src/Protocols/XDR/xdr_cqos.c M src/include/client_mgr.h M src/include/config-h.in.cmake A src/include/cqos.h M src/include/gsh_config.h A src/include/nfs_cluster_qos.h M src/include/nfs_init.h M src/include/nfs_proto_data.h M src/include/nfs_proto_functions.h M src/include/nfs_qos.h M src/nfs-ganesha.spec-in.cmake M src/support/client_mgr.c M src/support/nfs_read_conf.c 27 files changed, 2,404 insertions(+), 35 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/24/1222624/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222624?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: I985895068b54ae9404b5e9eb3cb1b69b8560849c Gerrit-Change-Number: 1222624 Gerrit-PatchSet: 1 Gerrit-Owner: Naresh.Chillarege(a)ibm.com

6 days, 3 hours

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Fix up the protocol number for SCTP in FSAL_encode_ipv4_netaddr

by Peter Schwenke (GerritHub)

Peter Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222435?usp=email ) Change subject: Fix up the protocol number for SCTP in FSAL_encode_ipv4_netaddr ...................................................................... Fix up the protocol number for SCTP in FSAL_encode_ipv4_netaddr The protocol number was 123. According to /etc/protocols /usr/include/linux/in.h, /usr/include/glib-2.0/gio/gioenums.h, /usr/include/netinet/in.h and /usr/include/wireshark/epan/ipproto.h it should be 132. There is a 123 definition in the Internal Socket Options at /usr/include/linux/sctp.h:#define SCTP_STREAM_SCHEDULER 123. Change-Id: Ida3788b730500a3c00071c8a16e3ecabe3b5cb26 Signed-off-by: Peter Schwenke <pschwenke(a)ddn.com> --- M src/FSAL/common_pnfs.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/35/1222435/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222435?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: Ida3788b730500a3c00071c8a16e3ecabe3b5cb26 Gerrit-Change-Number: 1222435 Gerrit-PatchSet: 1 Gerrit-Owner: Peter Schwenke <pschwenke(a)ddn.com>

1 week, 3 days

1
0
0 / 0

[L] Change in ...nfs-ganesha[next]: Move IP related functions from rpc_tools. to ip_utils.c

by Peter Schwenke (GerritHub)

Peter Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222434?usp=email ) Change subject: Move IP related functions from rpc_tools. to ip_utils.c ...................................................................... Move IP related functions from rpc_tools. to ip_utils.c Some of the functions do require RPC_VSOCK and rpc-ish data members. However, for the most part, they fit better as general ip_utils. Change-Id: Ia8c845cf1e5776973e69f589ec9774d150c1ee66 Signed-off-by: Peter Schwenke <pschwenke(a)ddn.com> --- M src/RPCAL/rpc_tools.c M src/include/display.h M src/include/gsh_rpc.h M src/include/ip_utils.h M src/support/ip_utils.c 5 files changed, 463 insertions(+), 436 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/34/1222434/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222434?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: Ia8c845cf1e5776973e69f589ec9774d150c1ee66 Gerrit-Change-Number: 1222434 Gerrit-PatchSet: 1 Gerrit-Owner: Peter Schwenke <pschwenke(a)ddn.com>

1 week, 3 days

1
0
0 / 0

[XL] Change in ...nfs-ganesha[next]: Remove third party library libcidr

by Peter Schwenke (GerritHub)

Peter Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222433?usp=email ) Change subject: Remove third party library libcidr ...................................................................... Remove third party library libcidr The functionality is replaced by cidr_* functions in ip_utils.c Change-Id: I670ebc8462cf76a9d799ebc18b20b34d99efbda6 Signed-off-by: Peter Schwenke <pschwenke(a)ddn.com> --- M src/CMakeLists.txt D src/cidr/CMakeLists.txt D src/cidr/LICENSE D src/cidr/README.reindent D src/cidr/cidr_addr.c D src/cidr/cidr_compare.c D src/cidr/cidr_from_str.c D src/cidr/cidr_get.c D src/cidr/cidr_inaddr.c D src/cidr/cidr_mem.c D src/cidr/cidr_misc.c D src/cidr/cidr_net.c D src/cidr/cidr_num.c D src/cidr/cidr_pow2_p.h D src/cidr/cidr_to_str.c D src/include/cidr.h 16 files changed, 0 insertions(+), 2,830 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/33/1222433/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222433?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: I670ebc8462cf76a9d799ebc18b20b34d99efbda6 Gerrit-Change-Number: 1222433 Gerrit-PatchSet: 1 Gerrit-Owner: Peter Schwenke <pschwenke(a)ddn.com>

1 week, 3 days

1
0
0 / 0

[L] Change in ...nfs-ganesha[next]: Replace libcidr

by Peter Schwenke (GerritHub)

Peter Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222432?usp=email ) Change subject: Replace libcidr ...................................................................... Replace libcidr libcidr has been used by NFS Ganesha for quite some time. Only a handful of functions from it were used and a whole directory of source code exists for it. Maintenance of it gets difficult things like clang format changes make it difficult if fixes were made upstream (unlikely). CIDR has been made an opaque type, because there aren't reasons for it to be changed by other code. These new functions are simpler and make use of the IP parsing and conversion functions currently in rpc_tools.c. The second arg of cidr_contains is just a sockaddr_t because there was no need to implement the mask on that argument. NFS Ganesha doesn't require that. The function has been renamed to cidr_contains_ip. The unused flags arg dropped from cidr_to_str(). Add test program test_cidr.c This required some of the ip_utils.c:cidr_* functions to be added to MainNFSD/libganesha_nfsd.ver so they can be linked to the test program. The plan is to move the IP-related functions from rpc_tools to here. We will then have the functions in a more easily found file. In export_mgr.c, the behaviour for DisplayExport was kept the same, in case, someone is parsing the output or something. One byte is send on dbus from the old style character arrays for ip addres and mask. Similarly, for the protocol 1 was sent for IPv4 and 2 for IPv6. Similarly for the CIDR version. That behaviour has been mimicked for now. It is probably worth cleaning that up at some point. Change-Id: I77103440d67f1a3318f0492122e9e2a25d535aa5 Signed-off-by: Peter Schwenke <pschwenke(a)ddn.com> --- M src/MainNFSD/CMakeLists.txt M src/MainNFSD/libganesha_nfsd.ver M src/Protocols/NFS/mnt_Export.c M src/include/client_mgr.h A src/include/ip_utils.h M src/scripts/runcp.sh M src/support/CMakeLists.txt M src/support/client_mgr.c M src/support/export_mgr.c M src/support/exports.c A src/support/ip_utils.c M src/test/CMakeLists.txt A src/test/test_cidr.c 13 files changed, 724 insertions(+), 42 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/32/1222432/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222432?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: I77103440d67f1a3318f0492122e9e2a25d535aa5 Gerrit-Change-Number: 1222432 Gerrit-PatchSet: 1 Gerrit-Owner: Peter Schwenke <pschwenke(a)ddn.com>

1 week, 3 days

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: Move 2 comments that were causing bizarre clang-format behaviour

by Peter Schwenke (GerritHub)

Peter Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222431?usp=email ) Change subject: Move 2 comments that were causing bizarre clang-format behaviour ...................................................................... Move 2 comments that were causing bizarre clang-format behaviour The 2 "NOTE: the Client sub-block..." comments have been moved to after ifdefs. They should be attached to the client sub-block line anyway. Otherwise, the above line would be clanged to column 1. Change-Id: I3f0278a517ce4dd11eeaf89339a3dc17e0669f84 Signed-off-by: Peter Schwenke <pschwenke(a)ddn.com> --- M src/support/exports.c 1 file changed, 8 insertions(+), 8 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/31/1222431/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222431?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: I3f0278a517ce4dd11eeaf89339a3dc17e0669f84 Gerrit-Change-Number: 1222431 Gerrit-PatchSet: 1 Gerrit-Owner: Peter Schwenke <pschwenke(a)ddn.com>

1 week, 3 days

1
0
0 / 0

[S] Change in ...nfs-ganesha[next]: Enable callback channel setup for RDMA. Tirpc PR https://github.com/n...

by Gaurav (GerritHub)

Gaurav has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222344?usp=email ) Change subject: Enable callback channel setup for RDMA. Tirpc PR https://github.com/nfs-ganesha/ntirpc/pull/353 ...................................................................... Enable callback channel setup for RDMA. Tirpc PR https://github.com/nfs-ganesha/ntirpc/pull/353 Change-Id: Ib0bc0fa844a7fe58d49ced1fb95e3911c3def167 Signed-off-by: Gaurav Gangalwar <gaurav.gangalwar(a)gmail.com> --- M src/MainNFSD/nfs_rpc_callback.c 1 file changed, 8 insertions(+), 9 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/44/1222344/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1222344?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: Ib0bc0fa844a7fe58d49ced1fb95e3911c3def167 Gerrit-Change-Number: 1222344 Gerrit-PatchSet: 1 Gerrit-Owner: Gaurav <gaurav.gangalwar(a)gmail.com>

1 week, 4 days

1
0
0 / 0

2025

2024

2023

2022

2021

2020

2019

2018

Devel September 2025