September 2020 - Devel - NFS Ganesha List Archives

Change in ...nfs-ganesha[next]: RADOS_KV: fix wrong string size of cidstr
by sepia-liu (GerritHub) 27 Sep '20

27 Sep '20

sepia-liu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/503053 ) Change subject: RADOS_KV: fix wrong string size of cidstr ...................................................................... RADOS_KV: fix wrong string size of cidstr Change-Id: I26f3c379942bf79c3cac95bb859355853770bd3d Signed-off-by: sepia-liu <liuwei_coder(a)163.com> --- M src/SAL/recovery/recovery_rados_kv.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/53/503053/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/503053 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: I26f3c379942bf79c3cac95bb859355853770bd3d Gerrit-Change-Number: 503053 Gerrit-PatchSet: 1 Gerrit-Owner: sepia-liu <liuwei_coder(a)163.com> Gerrit-MessageType: newchange

1 0

Announce Push of V4-dev.35
by Frank Filz 25 Sep '20

25 Sep '20

Branch next Tag:V4-dev.35 Merge Highlights * Don't deadlock if bindresvport_sa fails. * Add FSAL_KVSFS * NLM: fix the bug of using null pointer Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: ce3b703 Frank S. Filz V4-dev.35 fffe4c0 sepia-liu NLM: fix the bug of using null pointer 7a9e1d1 Philippe DENIEL Update RPM specfile so that it can build a package for FSAL_KVSFS 08ceeba Philippe DENIEL Add FSAL_KVSFS bd1cf64 Solomon Boulos Don't deadlock if bindresvport_sa fails.

1 0

Change in ...nfs-ganesha[next]: NLM: fix the bug of using null pointer
by sepia-liu (GerritHub) 23 Sep '20

23 Sep '20

sepia-liu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502868 ) Change subject: NLM: fix the bug of using null pointer ...................................................................... NLM: fix the bug of using null pointer When set enable_NLM false, the uninitialized ht_nlm_client can not be used, it is NULL pointer. Signed-off-by: sepia-liu <liuwei_coder(a)163.com> Change-Id: I6e26452fee158be955e09ca4b41c59d62e325fc8 --- M src/SAL/nfs4_recovery.c 1 file changed, 3 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/68/502868/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502868 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: I6e26452fee158be955e09ca4b41c59d62e325fc8 Gerrit-Change-Number: 502868 Gerrit-PatchSet: 1 Gerrit-Owner: sepia-liu <liuwei_coder(a)163.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Don't deadlock if bindresvport_sa fails.
by Solomon Boulos (GerritHub) 22 Sep '20

22 Sep '20

Solomon Boulos has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502811 ) Change subject: Don't deadlock if bindresvport_sa fails. ...................................................................... Don't deadlock if bindresvport_sa fails. My workaround for the bindresvport_sa race ignored the error case, where I'd still hold onto rpcLock. Change-Id: Ie8a5786fb283d3f7d255affaa4f0312f1b21374a Signed-off-by: Solomon Boulos <boulos(a)google.com> --- M src/FSAL/FSAL_PROXY_V3/rpc.c 1 file changed, 9 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/502811/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502811 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: Ie8a5786fb283d3f7d255affaa4f0312f1b21374a Gerrit-Change-Number: 502811 Gerrit-PatchSet: 1 Gerrit-Owner: Solomon Boulos <boulos(a)google.com> Gerrit-MessageType: newchange

1 0

NFS Ganesha using OpenEBS volume in kubernetes
by krishna T 22 Sep '20

22 Sep '20

Has anyone tried this in production ?

2 1

Announce Push of V4-dev.34
by Frank Filz 18 Sep '20

18 Sep '20

Branch next Tag:V4-dev.34 Merge Highlights * PROXY_V3: work around bindresvport threadsafety. * Fix two clang build errors. * Move nfs_init_stats_time outside USE_DBUS. * In nfs_export_get_root_entry release ref if not dir also fix locking * Introduce fsal_lookup_path helper * Specify & for inline asm output operands * FSAL_CEPH: paper over -ENOTCONN return from close when shutting down * Handle error conversion for ENXIO for NFS V3 and V4. Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 5b5ab87 Frank S. Filz V4-dev.34 8b75a13 Gaurav Gangalwar Handle error conversion for ENXIO for NFS V3 and V4. 5af0950 Jeff Layton FSAL_CEPH: paper over -ENOTCONN return from close when shutting down 9a8489b Matthew DeVore Specify & for inline asm output operands 2acd535 Frank S. Filz Introduce fsal_lookup_path helper edb9d74 Frank S. Filz In nfs_export_get_root_entry release ref if not dir also fix locking cb72562 Solomon Boulos Move nfs_init_stats_time outside USE_DBUS. ef71584 Solomon Boulos Fix two clang build errors. 48c464b Solomon Boulos PROXY_V3: work around bindresvport threadsafety.

1 0

Change in ...nfs-ganesha[next]: FSAL_CEPH: paper over -ENOTCONN return from close when shutting down
by Jeff Layton (GerritHub) 18 Sep '20

18 Sep '20

Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502619 ) Change subject: FSAL_CEPH: paper over -ENOTCONN return from close when shutting down ...................................................................... FSAL_CEPH: paper over -ENOTCONN return from close when shutting down When we're shutting down the server, we'll usually abort the connection first. The mdcache will then try to clean up entries and issue a ->close to each, and FSAL_CEPH ends up returning -ENOTCONN in that situation which causes a lot of log spam. Fix this by just ignoring -ENOTCONN errors in ceph_close_my_fd when ganesha is shutting down. Change-Id: If8231998a61e759be4d044f102ae6dbcebfdc975 Reported-and-Tested-by: liuwei <liuwei_coder(a)163.com> Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/FSAL/FSAL_CEPH/handle.c 1 file changed, 11 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/19/502619/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502619 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: If8231998a61e759be4d044f102ae6dbcebfdc975 Gerrit-Change-Number: 502619 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Introduce fsal_lookup_path helper
by Frank Filz (GerritHub) 17 Sep '20

17 Sep '20

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502536 ) Change subject: Introduce fsal_lookup_path helper ...................................................................... Introduce fsal_lookup_path helper Instead of using FSAL API lookup_path method to accomplish subdir mounts (and 9P attach), do a path walk in fsal_helper.c using the normal lookup method. This brings that path walk into common code applicable to all FSALs and works if mount_path_pseudo is true (in which case the mount path may not be able to be looked up in the backend). The helper function checks that each component is not ".." and is a directory. This prevents security hacking by using ".." to escape the export or using a symlink to escape the export. It also neatly assures we never allow mounting a single file. We also allow extra '/' in the path beyond the export root and allow "." components. Change-Id: I301a7cf34185fd7f57c73c21a069366d92abd25f Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/FSAL/fsal_helper.c M src/Protocols/9P/9p_attach.c M src/Protocols/NFS/mnt_Mnt.c M src/include/fsal.h 4 files changed, 181 insertions(+), 42 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/36/502536/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502536 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: I301a7cf34185fd7f57c73c21a069366d92abd25f Gerrit-Change-Number: 502536 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: In nfs_export_get_root_entry release ref if not dir also fix locking
by Frank Filz (GerritHub) 17 Sep '20

17 Sep '20

Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502535 ) Change subject: In nfs_export_get_root_entry release ref if not dir also fix locking ...................................................................... In nfs_export_get_root_entry release ref if not dir also fix locking Change-Id: Ifccd7138991e493ac96e3764c4c54cc8025e3987 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/support/exports.c 1 file changed, 6 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/35/502535/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502535 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: Ifccd7138991e493ac96e3764c4c54cc8025e3987 Gerrit-Change-Number: 502535 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Proxy V3: Support nested (subdirectory) mounts.
by Solomon Boulos (GerritHub) 16 Sep '20

16 Sep '20

Solomon Boulos has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502442 ) Change subject: Proxy V3: Support nested (subdirectory) mounts. ...................................................................... Proxy V3: Support nested (subdirectory) mounts. There's been an @todo in lookup_path since my first version of this code with "oh yeah, go through and look for forward slashes". I decided to finally handle that :). I tested both by hand: sudo mount -t nfs -o proto=vers=3 ${SERVER}:/export/nested /mnt/sub and with a cthon run of the same nested export. Change-Id: Ib4cff04b1e3a49114b8b19c4e69257e8004d0bfe Signed-off-by: Solomon Boulos <boulos(a)google.com> --- M src/FSAL/FSAL_PROXY_V3/main.c 1 file changed, 61 insertions(+), 16 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/42/502442/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/502442 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: Ib4cff04b1e3a49114b8b19c4e69257e8004d0bfe Gerrit-Change-Number: 502442 Gerrit-PatchSet: 1 Gerrit-Owner: Solomon Boulos <boulos(a)google.com> Gerrit-MessageType: newchange

1 0