October 2018 - Devel - NFS Ganesha List Archives

Change in ffilz/nfs-ganesha[next]: SAL: add a new get_nodeid recovery backend operation
by GerritHub 11 Oct '18

11 Oct '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/428918 Change subject: SAL: add a new get_nodeid recovery backend operation ...................................................................... SAL: add a new get_nodeid recovery backend operation Allow other layers to call down into the recovery backend to find out what the local cluster nodeid is (if any). If the recovery backend doesn't provide that operation, then just use the hostname. Change-Id: I3aa20b82309b0863d79e25840be7903e9bcf1e3b Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/SAL/nfs4_recovery.c M src/include/sal_functions.h 2 files changed, 34 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/18/428918/1 -- To view, visit https://review.gerrithub.io/428918 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: I3aa20b82309b0863d79e25840be7903e9bcf1e3b Gerrit-Change-Number: 428918 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: FSAL_GLUSTER: APIs to recall delegation from backend
by GerritHub 11 Oct '18

11 Oct '18

From Soumya <skoduri(a)redhat.com>: Soumya has uploaded this change for review. ( https://review.gerrithub.io/428750 Change subject: FSAL_GLUSTER: APIs to recall delegation from backend ...................................................................... FSAL_GLUSTER: APIs to recall delegation from backend Use UPCALL APIs provided by glusterfs/gfapi to recall delegation when requested from the backend. Change-Id: I2cc6b8b3a512e47425d623a5c8a103939f79a9a9 Signed-off-by: Soumya Koduri <skoduri(a)redhat.com> --- M src/FSAL/FSAL_GLUSTER/ds.c M src/FSAL/FSAL_GLUSTER/fsal_up.c M src/FSAL/FSAL_GLUSTER/gluster_internal.h 3 files changed, 90 insertions(+), 15 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/50/428750/1 -- To view, visit https://review.gerrithub.io/428750 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: I2cc6b8b3a512e47425d623a5c8a103939f79a9a9 Gerrit-Change-Number: 428750 Gerrit-PatchSet: 1 Gerrit-Owner: Soumya <skoduri(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: WIP: Enable delegations for FSAL_GLUSTER
by GerritHub 11 Oct '18

11 Oct '18

From Soumya <skoduri(a)redhat.com>: Soumya has uploaded this change for review. ( https://review.gerrithub.io/428749 Change subject: WIP: Enable delegations for FSAL_GLUSTER ...................................................................... WIP: Enable delegations for FSAL_GLUSTER Patch not be merged yet till delegations are properly tested on FSAL_GLUSTER Change-Id: I77696d49252c2377c8df4a2e1a4d1da13926eae5 Signed-off-by: Soumya Koduri <skoduri(a)redhat.com> --- M src/CMakeLists.txt M src/FSAL/FSAL_GLUSTER/main.c M src/include/config-h.in.cmake 3 files changed, 9 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/49/428749/1 -- To view, visit https://review.gerrithub.io/428749 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: I77696d49252c2377c8df4a2e1a4d1da13926eae5 Gerrit-Change-Number: 428749 Gerrit-PatchSet: 1 Gerrit-Owner: Soumya <skoduri(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: Without a shared queueing we no longer need a shared request_data_t
by GerritHub 10 Oct '18

10 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/428782 Change subject: Without a shared queueing we no longer need a shared request_data_t ...................................................................... Without a shared queueing we no longer need a shared request_data_t Use nfs_request_t for NFS request structures. With this, we also get rid of the request_data pool (which didn't really do anything other than maybe obscure where things were allocated, and due to the request_data_t being a union of three different structures, possibly contribute to over allocation of memory. Change-Id: I866d3987bc13565e3abe7278278f8bf5e76061f3 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/MainNFSD/nfs_init.c M src/MainNFSD/nfs_rpc_callback.c M src/MainNFSD/nfs_rpc_dispatcher_thread.c M src/MainNFSD/nfs_worker_thread.c M src/include/gsh_lttng/nfs_rpc.h M src/include/nfs_core.h M src/include/nfs_init.h M src/include/nfs_proto_functions.h M src/include/server_stats.h M src/support/server_stats.c 10 files changed, 182 insertions(+), 202 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/82/428782/1 -- To view, visit https://review.gerrithub.io/428782 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: I866d3987bc13565e3abe7278278f8bf5e76061f3 Gerrit-Change-Number: 428782 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: Remove vestiges of 9P work queue from NFS code and rename functions
by GerritHub 10 Oct '18

10 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/428781 Change subject: Remove vestiges of 9P work queue from NFS code and rename functions ...................................................................... Remove vestiges of 9P work queue from NFS code and rename functions The work queue is only used for 9P, so move all of that stuff into exclusively 9P code and rename things to reflect that it is 9p not nfs. Change-Id: I3521df2ce7ef5aa3f5ce5a56a36694ac99dfd417 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/MainNFSD/9p_dispatcher.c M src/MainNFSD/9p_rdma_callbacks.c M src/include/9p.h R src/include/9p_req_queue.h M src/include/fridgethr.h M src/include/nfs_core.h M src/scripts/runcp.sh 7 files changed, 102 insertions(+), 183 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/81/428781/1 -- To view, visit https://review.gerrithub.io/428781 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: I3521df2ce7ef5aa3f5ce5a56a36694ac99dfd417 Gerrit-Change-Number: 428781 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: Remove rpc_call_t from request_data_t
by GerritHub 10 Oct '18

10 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/428780 Change subject: Remove rpc_call_t from request_data_t ...................................................................... Remove rpc_call_t from request_data_t We no longer use request_data_t as a universal queued request, so no need for it to do tripple duty. Change-Id: I42662ad18eaa1134fba98e50246ea23e876bd67c Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/MainNFSD/nfs_rpc_callback.c M src/include/nfs_core.h M src/libntirpc 3 files changed, 6 insertions(+), 9 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/80/428780/1 -- To view, visit https://review.gerrithub.io/428780 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: I42662ad18eaa1134fba98e50246ea23e876bd67c Gerrit-Change-Number: 428780 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: (mostly) remove queue_wait
by GerritHub 10 Oct '18

10 Oct '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/428779 Change subject: (mostly) remove queue_wait ...................................................................... (mostly) remove queue_wait Since the queueing changes, queue_wait and qwait_time are meaningless so remove them. In order to not mess up DBUS tools, output a hard coded 0 in the queue_wait position. Change-Id: I580b45801719877b6203320bd0271b905a35f8b5 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/MainNFSD/nfs_worker_thread.c M src/include/fsal_api.h M src/support/server_stats.c 3 files changed, 44 insertions(+), 76 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/79/428779/1 -- To view, visit https://review.gerrithub.io/428779 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: I580b45801719877b6203320bd0271b905a35f8b5 Gerrit-Change-Number: 428779 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: tools: make sure ganesha-rados-grace is installed if built
by GerritHub 10 Oct '18

10 Oct '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/428619 Change subject: tools: make sure ganesha-rados-grace is installed if built ...................................................................... tools: make sure ganesha-rados-grace is installed if built Change-Id: Id22de08a14312349917ff0e87138593a8e842038 Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/CMakeLists.txt M src/tools/CMakeLists.txt 2 files changed, 2 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/19/428619/1 -- To view, visit https://review.gerrithub.io/428619 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: Id22de08a14312349917ff0e87138593a8e842038 Gerrit-Change-Number: 428619 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

ganesha-rados-grace tool not build
by Stefan Kooman 10 Oct '18

10 Oct '18

Hi, I'm building a deb package based on the gluster PPA debs [1]. The build flag "USE_RADOS_RECOV=ON" is enabled. According to CMakelists.txt file of the "tools" directory this would trigger the build of the ganesha-rados-grace binary: if (USE_RADOS_RECOV) add_executable(ganesha-rados-grace ganesha-rados-grace.c) target_link_libraries(ganesha-rados-grace rados_grace ${RADOS_LIBRARIES}) endif(USE_RADOS_RECOV) ########### install files ############### if(USE_TOOL_MULTILOCK) add_subdirectory(multilock) endif(USE_TOOL_MULTILOCK) It does not get build however. It's also not available in the cluster PPA, altough USE_RADOS_RECOV=ON is enabled by default in 2.7 and all needed libraries (librados-dev / librados-dev) are available. ... When I remove the "if" statements the tool does get build (just not installed because there no install script for it in the deb package, yet). Any idea why the if statement might not work here? Gr. Stefan [1]: https://launchpad.net/~gluster/ -- | BIT BV http://www.bit.nl/ Kamer van Koophandel 09090351 | GPG: 0xD14839C6 +31 318 648 688 / info(a)bit.nl

3 2

Ganesha 2.5.4 - usage of device major, minor
by krishna.harathi＠storagecraft.com 09 Oct '18

09 Oct '18

We are using Ganesha 2.5.4 VFS FSAL with FUSE based filesystem. During our testing of deleting existing exports and creating new ones, found that if a device major and minor is reused, clients get ESTALE for accessing a newly created export (nfs2 below). This seems to cause the following log entry, and explains the ESTALE response. 04/10/2018 T15:16:59.769027-0700 : nfs-ganesha-26627[sigmgr] 1595 :claim_posix_filesystems :FSAL :INFO :Root fs for export /exports/nfs1 is /exports/nfs2 We use our own Exportid and unique FSID configured for each export in the configuration file. I would like to know more about the intent and purpose of the usage of device major and minor of an export in this context. Any help in fixing this reuse issue is also appreciated. Thanks. Regards. Krishna Harathi

1 0