November 2018 - Devel - NFS Ganesha List Archives

Change in ffilz/nfs-ganesha[next]: NLM share reservation access check with owner_skip
by Suhrud Patankar (GerritHub) 28 Nov '18

28 Nov '18

Suhrud Patankar has uploaded this change for review. ( https://review.gerrithub.io/435279 Change subject: NLM share reservation access check with owner_skip ...................................................................... NLM share reservation access check with owner_skip Windows client tries to take write share on file with mode 555. Allow the owner of the file to take the reservation. The owner_skip flag must be used for access check. Change-Id: Ia647069f1d2c7e23a07c1ded75b7079de1894c3a Signed-off-by: Suhrud Patankar <suhrudpatankar(a)gmail.com> --- M src/SAL/state_share.c 1 file changed, 23 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/79/435279/1 -- To view, visit https://review.gerrithub.io/435279 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: Ia647069f1d2c7e23a07c1ded75b7079de1894c3a Gerrit-Change-Number: 435279 Gerrit-PatchSet: 1 Gerrit-Owner: Suhrud Patankar <suhrudpatankar(a)gmail.com>

1 0

Change in ffilz/nfs-ganesha[next]: FSAL_UP: add missing lock in async delegation recall
by Fatih Acar (GerritHub) 26 Nov '18

26 Nov '18

Fatih Acar has uploaded this change for review. ( https://review.gerrithub.io/434844 Change subject: FSAL_UP: add missing lock in async delegation recall ...................................................................... FSAL_UP: add missing lock in async delegation recall We must take the lock before calling delegrecall_one as it calls deleg_revoke which in turn calls state_del_locked. Change-Id: I99a59cc50565c8d96fe41816ce86b690e45e380a Signed-off-by: Fatih Acar <fatih.acar(a)gandi.net> --- M src/FSAL_UP/fsal_up_top.c 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/44/434844/1 -- To view, visit https://review.gerrithub.io/434844 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: I99a59cc50565c8d96fe41816ce86b690e45e380a Gerrit-Change-Number: 434844 Gerrit-PatchSet: 1 Gerrit-Owner: Fatih Acar <fatih(a)gandi.net>

1 0

Change in ffilz/nfs-ganesha[next]: Fix admin tools to read DisplayExport output
by Sriram Patil (GerritHub) 26 Nov '18

26 Nov '18

Sriram Patil has uploaded this change for review. ( https://review.gerrithub.io/434843 Change subject: Fix admin tools to read DisplayExport output ...................................................................... Fix admin tools to read DisplayExport output DisplayExport now returns the list of clients configured for the export. With this change, admin tool scripts read the output correctly and display the output in proper format. Change-Id: I3e62b04969a2a31a377296bec0be817f9ae53577 Signed-off-by: Sriram Patil <sriramp(a)vmware.com> --- M src/scripts/ganeshactl/Ganesha/ganesha_mgr_utils.py M src/scripts/ganeshactl/ganesha_mgr.py 2 files changed, 52 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/43/434843/1 -- To view, visit https://review.gerrithub.io/434843 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: I3e62b04969a2a31a377296bec0be817f9ae53577 Gerrit-Change-Number: 434843 Gerrit-PatchSet: 1 Gerrit-Owner: Sriram Patil <sriramp(a)vmware.com>

1 0

Change in ffilz/nfs-ganesha[next]: Fixed dereferencing null ctx_export on nfs_read
by Sergey Lysanov (GerritHub) 22 Nov '18

22 Nov '18

Sergey Lysanov has uploaded this change for review. ( https://review.gerrithub.io/434380 Change subject: Fixed dereferencing null ctx_export on nfs_read ...................................................................... Fixed dereferencing null ctx_export on nfs_read op_ctx->ctx_export may be null in case of DS read and no related mds export. Change-Id: If96ec4565bce1280bcc8209934fcc7c045048979 Signed-off-by: Sergey Lysanov <slysanov(a)virtuozzo.com> --- M src/Protocols/NFS/nfs4_op_read.c 1 file changed, 6 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/80/434380/1 -- To view, visit https://review.gerrithub.io/434380 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: If96ec4565bce1280bcc8209934fcc7c045048979 Gerrit-Change-Number: 434380 Gerrit-PatchSet: 1 Gerrit-Owner: Sergey Lysanov <slysanov(a)virtuozzo.com>

1 0

Change in ffilz/nfs-ganesha[next]: FSAL_VFS, FreeBSD: upstream support for d_off
by Jack Halford (GerritHub) 22 Nov '18

22 Nov '18

Jack Halford has uploaded this change for review. ( https://review.gerrithub.io/434351 Change subject: FSAL_VFS, FreeBSD: upstream support for d_off ...................................................................... FSAL_VFS, FreeBSD: upstream support for d_off read_dirents needed a FreeBSD-only hack because of the lack of d_off. Support for d_off was MFC'd into FreeBSD 12: https://reviews.freebsd.org/rS340431 Change-Id: I6c8146e47cfc7a46d95a96cfa65fdabb65b59acb Signed-off-by: Jack Halford <jack(a)crans.org> --- M src/FSAL/FSAL_VFS/handle.c M src/os/freebsd/subr.c 2 files changed, 9 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/51/434351/1 -- To view, visit https://review.gerrithub.io/434351 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: I6c8146e47cfc7a46d95a96cfa65fdabb65b59acb Gerrit-Change-Number: 434351 Gerrit-PatchSet: 1 Gerrit-Owner: Jack Halford <jack(a)crans.org>

1 0

Change in ffilz/nfs-ganesha[next]: Replaced a LogCrit() with LogEvent() in commonlib.c
by Anonymous Coward (GerritHub) 22 Nov '18

22 Nov '18

ntrishal(a)in.ibm.com has uploaded this change for review. ( https://review.gerrithub.io/434308 Change subject: Replaced a LogCrit() with LogEvent() in commonlib.c ...................................................................... Replaced a LogCrit() with LogEvent() in commonlib.c If stat fails in posix_get_fsid() log the message as an Event. Change-Id: Ib3fb0eac884a916c4c54d9073b6b7c9028d4a315 Signed-off-by: Trishali Nayar <ntrishal(a)in.ibm.com> --- M src/FSAL/commonlib.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/08/434308/1 -- To view, visit https://review.gerrithub.io/434308 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: Ib3fb0eac884a916c4c54d9073b6b7c9028d4a315 Gerrit-Change-Number: 434308 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <ntrishal(a)in.ibm.com>

1 0

Announce Push of V2.8-dev.7
by Frank Filz 21 Nov '18

21 Nov '18

Branch next Tag:V2.8-dev.7 NOTE: This merge includes an ntirpc pullup, please update your submodule. Release Highlights * Replaced a LogMajor() with LogEvent() in nlm_util.c * fix the bug of import config form rados url * RADOS_URLS: allow 3 component URLs that set namespace * Pullup ntirpc urcu changes Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 590e644 Frank S. Filz V2.8-dev.7 3e00d87 Daniel Gryniewicz Pullup ntirpc urcu changes 23c85f2 Jeff Layton RADOS_URLS: allow 3 component URLs that set namespace f1e53a1 minshenglin fix the bug of import config form rados url 0cc3e87 Trishali Nayar Replaced a LogMajor() with LogEvent() in nlm_util.c

1 0

Change in ffilz/nfs-ganesha[next]: WIP: vfs is_referral: add extended attr check for fs_location
by Jack Halford (GerritHub) 21 Nov '18

21 Nov '18

Jack Halford has uploaded this change for review. ( https://review.gerrithub.io/434210 Change subject: WIP: vfs is_referral: add extended attr check for fs_location ...................................................................... WIP: vfs is_referral: add extended attr check for fs_location When a directory has the sticky bit set, ganesha treats it as a referral, we need an additional check for the fs_locations in the extended attributes to determine wether it is a referral (see https://github.com/nfs-ganesha/nfs-ganesha/wiki/Referral) WIP: With this patch a directory with the sticky now works as expected, although I may have broken the referral feature so I'm still checking about that. Change-Id: I00608e628549efd36c0f6ce6ec237efade600113 Signed-off-by: Jack Halford <jack(a)gandi.net> --- M src/FSAL/commonlib.c 1 file changed, 4 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/10/434210/1 -- To view, visit https://review.gerrithub.io/434210 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: I00608e628549efd36c0f6ce6ec237efade600113 Gerrit-Change-Number: 434210 Gerrit-PatchSet: 1 Gerrit-Owner: Jack Halford <jack(a)crans.org>

1 0

Change in ffilz/nfs-ganesha[next]: VFS: Implement vfs_update_export
by Frank Filz (GerritHub) 19 Nov '18

19 Nov '18

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/433988 Change subject: VFS: Implement vfs_update_export ...................................................................... VFS: Implement vfs_update_export Just check that nothing has changed, currently no updateable options. Change-Id: I76b64fdb4ef72de626adfb361894488997076830 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/FSAL_VFS/export.c 1 file changed, 56 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/88/433988/1 -- To view, visit https://review.gerrithub.io/433988 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: I76b64fdb4ef72de626adfb361894488997076830 Gerrit-Change-Number: 433988 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: Add ability to update EXPORT FSAL parameters
by Frank Filz (GerritHub) 19 Nov '18

19 Nov '18

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/433987 Change subject: Add ability to update EXPORT FSAL parameters ...................................................................... Add ability to update EXPORT FSAL parameters Change-Id: I17044a6738aa09daf3cf814e3e430a8de1c0f983 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_main.c M src/FSAL/Stackable_FSALs/FSAL_NULL/export.c M src/FSAL/Stackable_FSALs/FSAL_NULL/main.c M src/FSAL/default_methods.c M src/include/FSAL/fsal_commonlib.h M src/include/fsal_api.h M src/include/mdcache.h M src/support/exports.c 8 files changed, 286 insertions(+), 33 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/433987/1 -- To view, visit https://review.gerrithub.io/433987 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: I17044a6738aa09daf3cf814e3e430a8de1c0f983 Gerrit-Change-Number: 433987 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0