August 2018 - Devel - NFS Ganesha List Archives

Change in ffilz/nfs-ganesha[next]: FSAL_GLUSTER: copy garry from tmp2_fd when resuing open state fd
by GerritHub 30 Aug '18

30 Aug '18

From Kinglong Mee <kinglongmee(a)gmail.com>: Kinglong Mee has uploaded this change for review. ( https://review.gerrithub.io/424069 Change subject: FSAL_GLUSTER: copy garry from tmp2_fd when resuing open state fd ...................................................................... FSAL_GLUSTER: copy garry from tmp2_fd when resuing open state fd Also fix using bad my_fd->creds.caller_glen (zero) as the gsh_malloc length. Change-Id: Id729d0a873a2740af9e55dc096e06397edf25b02 Signed-off-by: Kinglong Mee <kinglongmee(a)gmail.com> --- M src/FSAL/FSAL_GLUSTER/handle.c 1 file changed, 3 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/69/424069/1 -- To view, visit https://review.gerrithub.io/424069 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Id729d0a873a2740af9e55dc096e06397edf25b02 Gerrit-Change-Number: 424069 Gerrit-PatchSet: 1 Gerrit-Owner: Kinglong Mee <kinglongmee(a)gmail.com>

1 0

Change in ffilz/nfs-ganesha[next]: SAL: recovery_backend API cleanups
by GerritHub 29 Aug '18

29 Aug '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/423943 Change subject: SAL: recovery_backend API cleanups ...................................................................... SAL: recovery_backend API cleanups Remove prototype for simple_try_lift_grace, as it no longer exists. Rename nfs4_end_grace to nfs_end_grace for better parity with related functions. Reorganize the header file a bit to co-locate related API functions. Clean up the definitions of the global grace-period tracking variables, and add some comments to clarify what the grace_mutex protects. Change-Id: I549844d0d3b5acd7decf6fe25e694a5fd5b79aa6 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/SAL/nfs4_recovery.c M src/include/sal_functions.h 2 files changed, 18 insertions(+), 10 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/43/423943/1 -- To view, visit https://review.gerrithub.io/423943 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I549844d0d3b5acd7decf6fe25e694a5fd5b79aa6 Gerrit-Change-Number: 423943 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: Set op_ctx in dec_nlm_state_ref
by GerritHub 29 Aug '18

29 Aug '18

From Sachin Punadikar <psachin(a)in.ibm.com>: Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/423882 Change subject: Set op_ctx in dec_nlm_state_ref ...................................................................... Set op_ctx in dec_nlm_state_ref While executing asynchronous lock notification, NFS Ganesha crashes in dec_nlm_state_ref due to non-availability of op_ctx. Change-Id: Ie60e8849707b415b22549f0c52bd261229ceeb3b Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com> --- M src/SAL/nlm_state.c 1 file changed, 12 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/82/423882/1 -- To view, visit https://review.gerrithub.io/423882 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Ie60e8849707b415b22549f0c52bd261229ceeb3b Gerrit-Change-Number: 423882 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com>

1 0

Change in ffilz/nfs-ganesha[next]: Fix bad assign logical caused use after freed
by GerritHub 29 Aug '18

29 Aug '18

From Kinglong Mee <kinglongmee(a)gmail.com>: Kinglong Mee has uploaded this change for review. ( https://review.gerrithub.io/423878 Change subject: Fix bad assign logical caused use after freed ...................................................................... Fix bad assign logical caused use after freed A case of entry->attrs contains an old acl, but the new attrs request_mask without ATTR_ACL. Mdcache assign the acl from entry->attrs to attrs at mdc_update_attr_cache, but following fsal_copy_attrs assign it back, after that, both entry->attrs and attrs have the same acl, only one refrence. After that, the acl will be freed when release attrs, the entry->attrs->acl will be used after freed. This patch make sure the entry->attrs->acl is clean. Change-Id: Ibdda58703e9a4a0fecd5af17ba41a003ff26616c Signed-off-by: Kinglong Mee <kinglongmee(a)gmail.com> --- M src/include/fsal.h 1 file changed, 10 insertions(+), 15 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/78/423878/1 -- To view, visit https://review.gerrithub.io/423878 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Ibdda58703e9a4a0fecd5af17ba41a003ff26616c Gerrit-Change-Number: 423878 Gerrit-PatchSet: 1 Gerrit-Owner: Kinglong Mee <kinglongmee(a)gmail.com>

1 0

Change in ffilz/nfs-ganesha[next]: Fix uinitialized end_of_file/fsal_stable at nfs
by GerritHub 29 Aug '18

29 Aug '18

From Kinglong Mee <kinglongmee(a)gmail.com>: Kinglong Mee has uploaded this change for review. ( https://review.gerrithub.io/423877 Change subject: Fix uinitialized end_of_file/fsal_stable at nfs ...................................................................... Fix uinitialized end_of_file/fsal_stable at nfs The subsals may not set end_of_file and fsal_stable, so initialize them at nfs. Change-Id: Ic5ee826944951bd45faa8a550207a37b01c443ce Signed-off-by: Kinglong Mee <kinglongmee(a)gmail.com> --- M src/Protocols/NFS/nfs3_read.c M src/Protocols/NFS/nfs3_write.c M src/Protocols/NFS/nfs4_op_read.c 3 files changed, 4 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/77/423877/1 -- To view, visit https://review.gerrithub.io/423877 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: Ic5ee826944951bd45faa8a550207a37b01c443ce Gerrit-Change-Number: 423877 Gerrit-PatchSet: 1 Gerrit-Owner: Kinglong Mee <kinglongmee(a)gmail.com>

1 0

Change in ffilz/nfs-ganesha[next]: Fix uinitialized xdr_attrs_args when meeting error at nfs4 readdir ca...
by GerritHub 29 Aug '18

29 Aug '18

From Kinglong Mee <kinglongmee(a)gmail.com>: Kinglong Mee has uploaded this change for review. ( https://review.gerrithub.io/423876 Change subject: Fix uinitialized xdr_attrs_args when meeting error at nfs4 readdir callback ...................................................................... Fix uinitialized xdr_attrs_args when meeting error at nfs4 readdir callback The goto skip condition skips the xdr)attrs_args initialize, avoid it, initialize it earlier. Change-Id: I432c968a0d01b6cfacb8fda3cc5e713804219258 Signed-off-by: Kinglong Mee <kinglongmee(a)gmail.com> --- M src/Protocols/NFS/nfs4_op_readdir.c 1 file changed, 7 insertions(+), 8 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/76/423876/1 -- To view, visit https://review.gerrithub.io/423876 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I432c968a0d01b6cfacb8fda3cc5e713804219258 Gerrit-Change-Number: 423876 Gerrit-PatchSet: 1 Gerrit-Owner: Kinglong Mee <kinglongmee(a)gmail.com>

1 0

Ganesha 2.6.3 / ntirpc 1.6.3
by Daniel Gryniewicz 29 Aug '18

29 Aug '18

Hi, Everyone. A week ago, I tagged and released ntirpc 1.6.3 and Ganesha 2.6.3. This Ganesha release has almost all the bug fixes that have gone into 2.7. Sorry for the late notice, I mentioned it in a call but forgot to send the email. Daniel

2 1

V2.5-stable + vmware
by Supriti Singh 29 Aug '18

29 Aug '18

Hello, I am testing if v2.5 for FSAL Ceph work with vmware client. For version 3.0, it works. But for version 4.1, there is no tcp packets exchanged from server to client and connection always timeout. Is it a known issue somehow? Is 4.1 supported for v2.5 and FSAL ceph. Thanks, Supriti ------ Supriti Singh SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)

2 1

Change in ffilz/nfs-ganesha[next]: FreeBSD: fix compilation on FreeBSD 12
by GerritHub 28 Aug '18

28 Aug '18

From <fatih(a)gandi.net>: fatih(a)gandi.net has uploaded this change for review. ( https://review.gerrithub.io/423684 Change subject: FreeBSD: fix compilation on FreeBSD 12 ...................................................................... FreeBSD: fix compilation on FreeBSD 12 EXTATTR_MAXNAMELEN is already defined on FreeBSD 12. It was not the case on 11. Change-Id: I6b5680f7a8a43196a3f73576a161e3b8efa8f741 Signed-off-by: Baptiste Daroussin <bapt(a)FreeBSD.org> Signed-off-by: Fatih Acar <fatih.acar(a)gandi.net> --- M src/os/freebsd/xattr.c 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/84/423684/1 -- To view, visit https://review.gerrithub.io/423684 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I6b5680f7a8a43196a3f73576a161e3b8efa8f741 Gerrit-Change-Number: 423684 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <fatih(a)gandi.net>

1 0

Coverity covscan for 2018-08-25-32e8a246 (master branch)
by staticanalysis＠gluster.org 25 Aug '18

25 Aug '18

NFS-Ganesha Coverity covscan results are available from http://download.nfs-ganesha.org/coverity/2018-08-25-32e8a246

1 0