September 2024 - Devel - NFS Ganesha List Archives

[XS] Change in ...nfs-ganesha[next]: Fixing a false positive warning from old compiler on buildvm
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201447?usp=email ) Change subject: Fixing a false positive warning from old compiler on buildvm ...................................................................... Fixing a false positive warning from old compiler on buildvm We are getting the following warning [-Werror=free-nonheap-object]. Change-Id: I892861dfb4bdbd74b3d5128ac2b22ccd7d7b1f05 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/include/nfsv41.h 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/47/1201447/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201447?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: I892861dfb4bdbd74b3d5128ac2b22ccd7d7b1f05 Gerrit-Change-Number: 1201447 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-CC: gauravin(a)google.com

1 0

[XS] Change in ...nfs-ganesha[next]: nfs_lib.c: Free allocated strings
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201446?usp=email ) Change subject: nfs_lib.c: Free allocated strings ...................................................................... nfs_lib.c: Free allocated strings Change-Id: I4fb5e74dcdd6303b026242c1a302701cd4b7ea9c Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/MainNFSD/nfs_lib.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/46/1201446/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201446?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: I4fb5e74dcdd6303b026242c1a302701cd4b7ea9c Gerrit-Change-Number: 1201446 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-CC: ofir vainshtein <ofirvins(a)google.com>

1 0

[XS] Change in ...nfs-ganesha[next]: src/include/sal_functions.h: Avoid using export reserved keyword so t...
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201445?usp=email ) Change subject: src/include/sal_functions.h: Avoid using export reserved keyword so this file can be included in C++. ...................................................................... src/include/sal_functions.h: Avoid using export reserved keyword so this file can be included in C++. Change-Id: I0d474062ec2c5171e06c997ccca902b1b044627f Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/include/sal_functions.h 1 file changed, 4 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/45/1201445/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201445?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: I0d474062ec2c5171e06c997ccca902b1b044627f Gerrit-Change-Number: 1201445 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-CC: Yoni Couriel <yonic(a)google.com>

1 0

[S] Change in ...nfs-ganesha[next]: Fixing ganesha monitoring C++ code compilation errors
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201444?usp=email ) Change subject: Fixing ganesha monitoring C++ code compilation errors ...................................................................... Fixing ganesha monitoring C++ code compilation errors Rename identifier new which is a C++ keyword to make compilation work for OS installation with older userspace-rcu library Adding header for transform stl container function Change-Id: If7be9a7eaab77f1f81ae19a946f24c541cc4363c Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/include/gsh_refstr.h M src/monitoring/monitoring.cc 2 files changed, 7 insertions(+), 6 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/44/1201444/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201444?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: If7be9a7eaab77f1f81ae19a946f24c541cc4363c Gerrit-Change-Number: 1201444 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com>

1 0

[XS] Change in ...nfs-ganesha[next]: nfs-krb5 config: set active_krb5 default value as false
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201431?usp=email ) Change subject: nfs-krb5 config: set active_krb5 default value as false ...................................................................... nfs-krb5 config: set active_krb5 default value as false Change-Id: I14b4f7339ebf231acaa6350e5b5fd60804b10b37 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/support/nfs_read_conf.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/31/1201431/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201431?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: I14b4f7339ebf231acaa6350e5b5fd60804b10b37 Gerrit-Change-Number: 1201431 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-CC: Dipit Grover <dipitgrover(a)gmail.com>

1 0

[S] Change in ...nfs-ganesha[next]: Add dbus method reread_config
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201427?usp=email ) Change subject: Add dbus method reread_config ...................................................................... Add dbus method reread_config Call example: dbus-send --system --dest=org.ganesha.nfsd --type=method_call /org/ganesha/nfsd/admin org.ganesha.nfsd.admin.reread_config Change-Id: I12061e8382549fd608b686b47aee976b749e57fb Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/MainNFSD/nfs_admin_thread.c M src/MainNFSD/nfs_init.c 2 files changed, 31 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/27/1201427/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201427?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: I12061e8382549fd608b686b47aee976b749e57fb Gerrit-Change-Number: 1201427 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com>

1 0

[XS] Change in ...nfs-ganesha[next]: src/idmapper/idmapper: Fixed a bug to measure the correct time it tak...
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201426?usp=email ) Change subject: src/idmapper/idmapper: Fixed a bug to measure the correct time it takes to resolve with libnfsidmap ...................................................................... src/idmapper/idmapper: Fixed a bug to measure the correct time it takes to resolve with libnfsidmap Change-Id: I79ec940201ad5e2972ae1ade2a378bc10e4be416 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/idmapper/idmapper.c 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/26/1201426/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201426?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: I79ec940201ad5e2972ae1ade2a378bc10e4be416 Gerrit-Change-Number: 1201426 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com>

1 0

[XS] Change in ...nfs-ganesha[next]: src/idmapper/idmapper: Refactored log lines to print the function name
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201425?usp=email ) Change subject: src/idmapper/idmapper: Refactored log lines to print the function name ...................................................................... src/idmapper/idmapper: Refactored log lines to print the function name Change-Id: I02eb864a4eea48aa1fd6e6df1c77e5ce56ebb577 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/idmapper/idmapper.c 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/25/1201425/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201425?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: I02eb864a4eea48aa1fd6e6df1c77e5ce56ebb577 Gerrit-Change-Number: 1201425 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com>

1 0

[XS] Change in ...nfs-ganesha[next]: fridgether.c: After cancel we need to join, before freeing thread res...
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201424?usp=email ) Change subject: fridgether.c: After cancel we need to join, before freeing thread resources ...................................................................... fridgether.c: After cancel we need to join, before freeing thread resources Change-Id: I96d0c8bddd854e274875c9dc2919e18e83e429d2 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/support/fridgethr.c 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/24/1201424/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201424?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: I96d0c8bddd854e274875c9dc2919e18e83e429d2 Gerrit-Change-Number: 1201424 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-CC: ofir vainshtein <ofirvins(a)google.com>

1 0

[S] Change in ...nfs-ganesha[next]: idmapping cache cleanup: log number of entries removed
by Lior Suliman (GerritHub) 18 Sep '24

18 Sep '24

Lior Suliman has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201423?usp=email ) Change subject: idmapping cache cleanup: log number of entries removed ...................................................................... idmapping cache cleanup: log number of entries removed Change-Id: Ib0943e286b828a5f5cb9173c1d904501b62bce88 Signed-off-by: Lior Suliman <liorsu(a)gmail.com> --- M src/idmapper/idmapper_cache.c M src/support/uid2grp_cache.c 2 files changed, 17 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/23/1201423/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1201423?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: Ib0943e286b828a5f5cb9173c1d904501b62bce88 Gerrit-Change-Number: 1201423 Gerrit-PatchSet: 1 Gerrit-Owner: Lior Suliman <liorsu(a)google.com> Gerrit-CC: Dipit Grover <dipitgrover(a)gmail.com>

1 0