April 2025 - Devel - Nfs Ganesha List Archives

[XS] Change in ...nfs-ganesha[next]: CID 542646: Fix for safer dereference of clientid reference counter

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/+/1213675?usp=email ) Change subject: CID 542646: Fix for safer dereference of clientid reference counter ...................................................................... CID 542646: Fix for safer dereference of clientid reference counter Change-Id: I7ea9ff16c76b6c0ec8dee9170b704eefb8a0e9ed Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> Signed-off-by: Sreedhat Agraharam <sragraha(a)redhat.com> --- M src/FSAL_UP/fsal_up_async.c 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/75/1213675/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213675?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: I7ea9ff16c76b6c0ec8dee9170b704eefb8a0e9ed Gerrit-Change-Number: 1213675 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 month, 2 weeks

1
0
0 / 0

[XL] Change in ...nfs-ganesha[next]: gRPC Infrastructure

by Name of user not set (GerritHub)

Avani.Rateria(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213670?usp=email ) Change subject: gRPC Infrastructure ...................................................................... gRPC Infrastructure gRPC (gRPC Remote Procedure Call) is an open-source framework that enables communication between services through remote procedure calls (RPCs). It is built on top of HTTP/2 and uses Protocol Buffers (protobuf) as the interface definition language (IDL). It allows you to define services and their methods, generate client and server code, and facilitate communication between distributed systems. The changes will facilitate both intra server communication (IPC) by replacing DBus and inter server communication. Currently, we have implemented three APIs to replace DBus: 1) Retrieve connected client IDs. 2) Determine whether NFS is in grace period. 3) Retrieve session IDs. Change-Id: I500c278f883697be0810985ae15e49eca79d230b Signed-off-by: Avani Rateria <Avani.Rateria(a)ibm.com> --- M src/CMakeLists.txt M src/MainNFSD/CMakeLists.txt M src/MainNFSD/nfs_init.c A src/grpc_server/CMakeLists.txt A src/grpc_server/GrpcServer.cc A src/grpc_server/nfsService.h A src/grpc_server/nfsServiceClient.cpp A src/grpc_server/nfsServiceServer.cpp M src/include/config-h.in.cmake A src/include/gRPC/GrpcServer.h M src/include/gsh_config.h A src/proto/CMakeLists.txt A src/proto/nfsService.proto M src/support/nfs_read_conf.c 14 files changed, 911 insertions(+), 146 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/70/1213670/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213670?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: I500c278f883697be0810985ae15e49eca79d230b Gerrit-Change-Number: 1213670 Gerrit-PatchSet: 1 Gerrit-Owner: Avani.Rateria(a)ibm.com

1 month, 2 weeks

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: CID 542662 : Intialized scalar variable of ceph_statx

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/+/1213668?usp=email ) Change subject: CID 542662 : Intialized scalar variable of ceph_statx ...................................................................... CID 542662 : Intialized scalar variable of ceph_statx Change-Id: Ifecddfddea54e748b92cbd99f5d62176eeaa6c3c Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com> Signed-off-by: Sreedhat Agraharam <sragraha(a)redhat.com> --- M src/FSAL/FSAL_CEPH/handle.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/68/1213668/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213668?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: Ifecddfddea54e748b92cbd99f5d62176eeaa6c3c Gerrit-Change-Number: 1213668 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 month, 2 weeks

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: Updated with new nomenclature of nfs prometneus metrics

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/+/1213666?usp=email ) Change subject: Updated with new nomenclature of nfs prometneus metrics ...................................................................... Updated with new nomenclature of nfs prometneus metrics Change-Id: Ic1c1b3aae234292613a6a0e3e4e9049ee55c3898 Signed-off-by: Sreedhat Agraharam <sragraha(a)redhat.com> --- M src/MainNFSD/nfs_metrics.c M src/RPCAL/connection_manager_metrics.c M src/SAL/sal_metrics.c M src/idmapper/idmapper_monitoring.c M src/monitoring/dynamic_metrics.cc M src/monitoring/prometheus_exposer.cc 6 files changed, 46 insertions(+), 45 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/66/1213666/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213666?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: Ic1c1b3aae234292613a6a0e3e4e9049ee55c3898 Gerrit-Change-Number: 1213666 Gerrit-PatchSet: 1 Gerrit-Owner: sragraha(a)redhat.com

1 month, 2 weeks

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Coverity 72124: Fix for file descriptor leak in nlm_send_async

by Name of user not set (GerritHub)

Vidya.Thumukunta(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213658?usp=email ) Change subject: Coverity 72124: Fix for file descriptor leak in nlm_send_async ...................................................................... Coverity 72124: Fix for file descriptor leak in nlm_send_async Change-Id: I5e3ccf6c36f9cdefba9fdb815d8350afefb279b7 Signed-off-by: VidyaThumukunta <Vidya.Thumukunta(a)ibm.com> --- M src/Protocols/NLM/nlm_async.c 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/58/1213658/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213658?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: I5e3ccf6c36f9cdefba9fdb815d8350afefb279b7 Gerrit-Change-Number: 1213658 Gerrit-PatchSet: 1 Gerrit-Owner: Vidya.Thumukunta(a)ibm.com

1 month, 2 weeks

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Coverity 542657: Calling display_client_id_rec dereferences freed poi...

by Suhas Athani (GerritHub)

Suhas Athani has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213561?usp=email ) Change subject: Coverity 542657: Calling display_client_id_rec dereferences freed pointer clientid->cid_client_record. Fix Added. ...................................................................... Coverity 542657: Calling display_client_id_rec dereferences freed pointer clientid->cid_client_record. Fix Added. Change-Id: I9e756463a1a8c36a3943196e739c09e0fa6dfcf1 Signed-off-by: s-athani <suhas.athani(a)ibm.com> --- M src/SAL/nfs4_clientid.c 1 file changed, 3 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/61/1213561/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213561?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: I9e756463a1a8c36a3943196e739c09e0fa6dfcf1 Gerrit-Change-Number: 1213561 Gerrit-PatchSet: 1 Gerrit-Owner: Suhas Athani <Suhas.Athani(a)ibm.com>

1 month, 2 weeks

1
0
0 / 0

[L] Change in ...nfs-ganesha[next]: Refactored idmapping code paths, added logs and lttng traces refactor...

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213541?usp=email to review the following change. Change subject: Refactored idmapping code paths, added logs and lttng traces refactored and added metrics ...................................................................... Refactored idmapping code paths, added logs and lttng traces refactored and added metrics Change-Id: Ie15bc586549c96b2984aa9f4de1992465bea8a7d Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/Protocols/9P/9p_proto_tools.c M src/idmapper/idmapper.c M src/idmapper/idmapper_cache.c M src/idmapper/idmapper_monitoring.c M src/idmapper/idmapper_negative_cache.c M src/include/gsh_lttng/gsh_lttng.h M src/include/idmapper_monitoring.h M src/include/uid2grp.h M src/support/nfs_creds.c M src/support/uid2grp.c M src/support/uid2grp_cache.c 11 files changed, 517 insertions(+), 141 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/41/1213541/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213541?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: Ie15bc586549c96b2984aa9f4de1992465bea8a7d Gerrit-Change-Number: 1213541 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 month, 2 weeks

1
0
0 / 0

checkpatch.conf update pushed to next

by Frank Filz

Some folks have been tripping over the indented labels check with C++ code so I pushed a checkpatch.conf update to avoid that. If you are having this problem, rebase on next or manually make the config change. Thanks Frank

1 month, 2 weeks

1
0
0 / 0

[XS] Change in ...nfs-ganesha[next]: Fix Coverity - 542653, Variable buf going out of scope leaks the stor...

by Name of user not set (GerritHub)

Avani.Rateria(a)ibm.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213525?usp=email ) Change subject: Fix Coverity - 542653, Variable buf going out of scope leaks the storage it points to. Fix Added ...................................................................... Fix Coverity - 542653, Variable buf going out of scope leaks the storage it points to. Fix Added Change-Id: I66cf6eb6bc36880b17e2dfb6270085d4d9d99e87 Signed-off-by: Avani Rateria <Avani.Rateria(a)ibm.com> --- M src/FSAL/FSAL_GPFS/handle.c 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/25/1213525/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213525?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: I66cf6eb6bc36880b17e2dfb6270085d4d9d99e87 Gerrit-Change-Number: 1213525 Gerrit-PatchSet: 1 Gerrit-Owner: Avani.Rateria(a)ibm.com

1 month, 2 weeks

1
0
0 / 0

[M] Change in ...nfs-ganesha[next]: Add Dbus method for showing and purging negative cache

by Lior Suliman (GerritHub)

Attention is currently required from: Frank Filz. Hello Frank Filz, I'd like you to do a code review. Please visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213492?usp=email to review the following change. Change subject: Add Dbus method for showing and purging negative cache ...................................................................... Add Dbus method for showing and purging negative cache Change-Id: Ib85b8fba36da3fc87628711234df8ed3f09e46d7 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/FSAL/localfs.c M src/MainNFSD/nfs_admin_thread.c M src/idmapper/idmapper_negative_cache.c M src/include/idmapper.h 4 files changed, 171 insertions(+), 18 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/92/1213492/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1213492?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: Ib85b8fba36da3fc87628711234df8ed3f09e46d7 Gerrit-Change-Number: 1213492 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>

1 month, 2 weeks

1
0
0 / 0

2025

2024

2023

2022

2021

2020

2019

2018

Devel April 2025