April 2020 - Devel - NFS Ganesha List Archives

Change in ...nfs-ganesha[next]: Merge req_op_context and root_op_context and normalize use
by Frank Filz (GerritHub) 30 Apr '20

30 Apr '20

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491399 ) Change subject: Merge req_op_context and root_op_context and normalize use ...................................................................... Merge req_op_context and root_op_context and normalize use Change-Id: I55c6d2dfbd26164e7a895cf5f1a34d1a9ed8514d Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_CEPH/export.c M src/FSAL/FSAL_CEPH/handle.c M src/FSAL/FSAL_CEPH/main.c M src/FSAL/FSAL_GLUSTER/ds.c M src/FSAL/FSAL_GLUSTER/handle.c M src/FSAL/FSAL_GLUSTER/mds.c M src/FSAL/FSAL_GPFS/export.c M src/FSAL/FSAL_GPFS/file.c M src/FSAL/FSAL_GPFS/fsal_attrs.c M src/FSAL/FSAL_GPFS/fsal_create.c M src/FSAL/FSAL_GPFS/fsal_fileop.c M src/FSAL/FSAL_GPFS/fsal_internal.c M src/FSAL/FSAL_GPFS/fsal_symlinks.c M src/FSAL/FSAL_GPFS/fsal_up.c M src/FSAL/FSAL_GPFS/gpfs_methods.h M src/FSAL/FSAL_LIZARDFS/ds.c M src/FSAL/FSAL_LIZARDFS/lzfs_internal.c M src/FSAL/FSAL_MEM/mem_handle.c M src/FSAL/FSAL_PROXY_V3/main.c M src/FSAL/FSAL_PROXY_V3/nlm.c M src/FSAL/FSAL_PROXY_V4/handle.c M src/FSAL/FSAL_PSEUDO/handle.c M src/FSAL/FSAL_RGW/handle.c M src/FSAL/FSAL_VFS/export.c M src/FSAL/FSAL_VFS/file.c M src/FSAL/FSAL_VFS/handle.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_up.c M src/FSAL/access_check.c M src/FSAL/commonlib.c M src/FSAL/default_methods.c M src/FSAL/fsal_helper.c M src/FSAL_UP/fsal_up_top.c M src/MainNFSD/9p_dispatcher.c M src/MainNFSD/libganesha_nfsd.ver M src/MainNFSD/nfs_worker_thread.c M src/Protocols/9P/9p_attach.c M src/Protocols/9P/9p_interpreter.c M src/Protocols/9P/9p_lcreate.c M src/Protocols/9P/9p_link.c M src/Protocols/9P/9p_mkdir.c M src/Protocols/9P/9p_mknod.c M src/Protocols/9P/9p_proto_tools.c M src/Protocols/9P/9p_remove.c M src/Protocols/9P/9p_rename.c M src/Protocols/9P/9p_renameat.c M src/Protocols/9P/9p_setattr.c M src/Protocols/9P/9p_symlink.c M src/Protocols/9P/9p_unlinkat.c M src/Protocols/9P/9p_write.c M src/Protocols/9P/9p_xattrcreate.c M src/Protocols/NFS/mnt_Export.c M src/Protocols/NFS/mnt_Mnt.c M src/Protocols/NFS/nfs3_read.c M src/Protocols/NFS/nfs3_write.c M src/Protocols/NFS/nfs4_Compound.c M src/Protocols/NFS/nfs4_op_layoutreturn.c M src/Protocols/NFS/nfs4_op_open.c M src/Protocols/NFS/nfs4_op_readdir.c M src/Protocols/NFS/nfs4_op_restorefh.c M src/Protocols/NFS/nfs4_op_savefh.c M src/Protocols/NFS/nfs4_op_secinfo.c M src/Protocols/NFS/nfs4_op_secinfo_no_name.c M src/Protocols/NFS/nfs4_pseudo.c M src/Protocols/NLM/nlm_util.c M src/Protocols/RQUOTA/rquota_getquota.c M src/Protocols/RQUOTA/rquota_setquota.c M src/SAL/nfs4_clientid.c M src/SAL/nfs4_state.c M src/SAL/state_async.c M src/SAL/state_deleg.c M src/SAL/state_lock.c M src/SAL/state_misc.c M src/gtest/gtest.hh M src/gtest/gtest_nfs4.hh M src/gtest/test_ci_hash_dist1.cc M src/include/fsal.h M src/include/fsal_api.h M src/include/nfs_core.h M src/include/nfs_proto_data.h M src/support/ds.c M src/support/export_mgr.c M src/support/exports.c M src/support/nfs_creds.c 86 files changed, 751 insertions(+), 878 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/99/491399/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491399 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I55c6d2dfbd26164e7a895cf5f1a34d1a9ed8514d Gerrit-Change-Number: 491399 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Remove broken Copy export that was intended to restore on update failure
by Frank Filz (GerritHub) 30 Apr '20

30 Apr '20

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491398 ) Change subject: Remove broken Copy export that was intended to restore on update failure ...................................................................... Remove broken Copy export that was intended to restore on update failure Just remove this code for now, when all the pseudofs update is complete, failure will be handled differently (by LogFatal...). Change-Id: I1f51a144b6a33007d221bd45adb4897d25709c5d Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/support/exports.c 1 file changed, 0 insertions(+), 8 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/98/491398/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491398 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I1f51a144b6a33007d221bd45adb4897d25709c5d Gerrit-Change-Number: 491398 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Remove req_op_context parameters from FSAL API and other functions
by Frank Filz (GerritHub) 30 Apr '20

30 Apr '20

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491377 ) Change subject: Remove req_op_context parameters from FSAL API and other functions ...................................................................... Remove req_op_context parameters from FSAL API and other functions Instead if passing req_op_context as a parameter, just use the thread local variable op_ctx. Change-Id: I492680ba537a4488afdec018494c4ba192187e71 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_CEPH/ds.c M src/FSAL/FSAL_CEPH/mds.c M src/FSAL/FSAL_GLUSTER/ds.c M src/FSAL/FSAL_GLUSTER/mds.c M src/FSAL/FSAL_GPFS/file.c M src/FSAL/FSAL_GPFS/fsal_attrs.c M src/FSAL/FSAL_GPFS/fsal_create.c M src/FSAL/FSAL_GPFS/fsal_ds.c M src/FSAL/FSAL_GPFS/fsal_fileop.c M src/FSAL/FSAL_GPFS/fsal_internal.c M src/FSAL/FSAL_GPFS/fsal_internal.h M src/FSAL/FSAL_GPFS/fsal_lookup.c M src/FSAL/FSAL_GPFS/fsal_mds.c M src/FSAL/FSAL_GPFS/fsal_rename.c M src/FSAL/FSAL_GPFS/fsal_symlinks.c M src/FSAL/FSAL_GPFS/fsal_unlink.c M src/FSAL/FSAL_GPFS/handle.c M src/FSAL/FSAL_LIZARDFS/ds.c M src/FSAL/FSAL_LIZARDFS/mds_handle.c M src/FSAL/FSAL_PROXY_V4/proxyv4_fsal_methods.h M src/FSAL/FSAL_PROXY_V4/xattrs.c M src/FSAL/FSAL_VFS/panfs/mds.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c M src/FSAL/default_methods.c M src/Protocols/NFS/nfs4_op_commit.c M src/Protocols/NFS/nfs4_op_layoutcommit.c M src/Protocols/NFS/nfs4_op_layoutget.c M src/Protocols/NFS/nfs4_op_layoutreturn.c M src/Protocols/NFS/nfs4_op_read.c M src/Protocols/NFS/nfs4_op_write.c M src/include/fsal_api.h 31 files changed, 95 insertions(+), 234 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/77/491377/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491377 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I492680ba537a4488afdec018494c4ba192187e71 Gerrit-Change-Number: 491377 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: selinux: additional policy
by Kaleb KEITHLEY (GerritHub) 30 Apr '20

30 Apr '20

Kaleb KEITHLEY has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491370 ) Change subject: selinux: additional policy ...................................................................... selinux: additional policy whack-a-mole Change-Id: I18bc0dbb2e6f21a40bb2d851dc4d3925caa4f44d Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com> --- M src/selinux/ganesha.te 1 file changed, 4 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/70/491370/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491370 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I18bc0dbb2e6f21a40bb2d851dc4d3925caa4f44d Gerrit-Change-Number: 491370 Gerrit-PatchSet: 1 Gerrit-Owner: Kaleb KEITHLEY <kaleb(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Report STATE_ESTALE error with LogEventLimited() in state_lock(), sta...
by Madhu Thorat (GerritHub) 30 Apr '20

30 Apr '20

Madhu Thorat has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491361 ) Change subject: Report STATE_ESTALE error with LogEventLimited() in state_lock(), state_test() ...................................................................... Report STATE_ESTALE error with LogEventLimited() in state_lock(), state_test() Currently in state_lock() and state_test(), STATE_ESTALE error is reported with LogMajor(). Used LogEventLimited() to report STATE_ESTALE in state_lock() and state_test(). Change-Id: I21b9be47238c1818c55b445cb8f4071665c53dd0 Signed-off-by: Madhu Thorat <madhu.punjabi(a)in.ibm.com> --- M src/SAL/state_lock.c 1 file changed, 12 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/61/491361/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491361 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I21b9be47238c1818c55b445cb8f4071665c53dd0 Gerrit-Change-Number: 491361 Gerrit-PatchSet: 1 Gerrit-Owner: Madhu Thorat <madhu.punjabi(a)in.ibm.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Adding xattr support for FSAL_GLUSTER
by Arjun Sharma (GerritHub) 30 Apr '20

30 Apr '20

Arjun Sharma has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491311 ) Change subject: Adding xattr support for FSAL_GLUSTER ...................................................................... Adding xattr support for FSAL_GLUSTER Implementing functions getxattr(), setxattr() removexattr() and listxattr() in fsal_gluster Change-Id: I358042f63d28f2e305e2bdc319e99c8a55ffe3af Signed-off-by: Arjun <Arjun(a)localhost.localdomain> --- M src/FSAL/FSAL_GLUSTER/gluster_internal.h M src/FSAL/FSAL_GLUSTER/handle.c M src/include/fsal_api.h 3 files changed, 200 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/491311/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491311 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I358042f63d28f2e305e2bdc319e99c8a55ffe3af Gerrit-Change-Number: 491311 Gerrit-PatchSet: 1 Gerrit-Owner: Arjun Sharma <arjsharm(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: [GPFS] Don't do permission check after file has been created
by Madhu Thorat (GerritHub) 29 Apr '20

29 Apr '20

Madhu Thorat has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491245 ) Change subject: [GPFS] Don't do permission check after file has been created ...................................................................... [GPFS] Don't do permission check after file has been created In gpfs_open2(), we create a file with the caller's credentials active, so as such permission check is done. So we let the caller of gpfs_open2() know the permission check has already been done by setting *caller_check_perm=false. Change-Id: I74031d74fd7c027654c54c9c73a7318818037a9b Signed-off-by: Madhu Thorat <madhu.punjabi(a)in.ibm.com> --- M src/FSAL/FSAL_GPFS/file.c 1 file changed, 7 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/45/491245/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491245 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I74031d74fd7c027654c54c9c73a7318818037a9b Gerrit-Change-Number: 491245 Gerrit-PatchSet: 1 Gerrit-Owner: Madhu Thorat <madhu.punjabi(a)in.ibm.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Pull up NTIRPC fixes
by Daniel Gryniewicz (GerritHub) 28 Apr '20

28 Apr '20

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491155 ) Change subject: Pull up NTIRPC fixes ...................................................................... Pull up NTIRPC fixes Change-Id: Ie11fc5152345fd59f5f3efa7a7f36a8cbefe50ce Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/libntirpc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/55/491155/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491155 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ie11fc5152345fd59f5f3efa7a7f36a8cbefe50ce Gerrit-Change-Number: 491155 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Proposed 2.8.4
by Daniel Gryniewicz 27 Apr '20

27 Apr '20

This branch has my proposed 2.8.4 in it: https://github.com/dang/nfs-ganesha/tree/V2.8-stable Please look it over, and test if possible. Daniel

1 1

Change in ...nfs-ganesha[next]: Fix ganesha_stats v3_full|v4_full fails.
by xijinyu (GerritHub) 27 Apr '20

27 Apr '20

xijinyu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491038 ) Change subject: Fix ganesha_stats v3_full|v4_full fails. ...................................................................... Fix ganesha_stats v3_full|v4_full fails. From: https://github.com/nfs-ganesha/nfs-ganesha/issues/532 Signed-off-by: Xi Jinyu <xijinyu(a)cmss.chinamobile.com> Change-Id: I1c7afca944fe994f967cd2a7daa61a4e0389ac33 --- M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py 1 file changed, 13 insertions(+), 13 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/38/491038/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/491038 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I1c7afca944fe994f967cd2a7daa61a4e0389ac33 Gerrit-Change-Number: 491038 Gerrit-PatchSet: 1 Gerrit-Owner: xijinyu <xijinyu(a)cmss.chinamobile.com> Gerrit-MessageType: newchange

1 0