July 2018 - Devel - NFS Ganesha List Archives

Fwd: Ganesha 2.5, crash /segfault while executing nlm4_Unlock
by Sachin Punadikar 19 Sep '18

19 Sep '18

---------- Forwarded message ---------- From: Sachin Punadikar <punadikar.sachin(a)gmail.com> Date: Tue, Jun 26, 2018 at 3:57 PM Subject: Ganesha 2.5, crash /segfault while executing nlm4_Unlock To: nfs-ganesha-devel <nfs-ganesha-devel(a)lists.sourceforge.net> Hi All, Recently a crash was reported by customer for Ganesha 2.5. (gdb) where #0 0x00007f475872900b in pthread_rwlock_wrlock () from /lib64/libpthread.so.0 #1 0x000000000041eac9 in fsal_obj_handle_fini (obj=0x7f4378028028) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL/commonlib.c:192 #2 0x000000000053180f in mdcache_lru_clean (entry=0x7f4378027ff0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c:589 #3 0x0000000000536587 in _mdcache_lru_unref (entry=0x7f4378027ff0, flags=0, func=0x5a9380 <__func__.23209> "cih_remove_checked", line=406) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c:1921 #4 0x0000000000543e91 in cih_remove_checked (entry=0x7f4378027ff0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_hash.h:406 #5 0x0000000000544b26 in mdc_clean_entry (entry=0x7f4378027ff0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c:235 #6 0x000000000053181e in mdcache_lru_clean (entry=0x7f4378027ff0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c:592 #7 0x0000000000536587 in _mdcache_lru_unref (entry=0x7f4378027ff0, flags=0, func=0x5a70af <__func__.23112> "mdcache_put", line=190) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c:1921 #8 0x0000000000539666 in mdcache_put (entry=0x7f4378027ff0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.h:190 #9 0x000000000053f062 in mdcache_put_ref (obj_hdl=0x7f4378028028) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/FSAL /Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c:1709 #10 0x000000000049bf0f in nlm4_Unlock (args=0x7f4294165830, req=0x7f4294165028, res=0x7f43f001e0e0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/Prot ocols/NLM/nlm_Unlock.c:128 #11 0x000000000044c719 in nfs_rpc_execute (reqdata=0x7f4294165000) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/Main NFSD/nfs_worker_thread.c:1290 #12 0x000000000044cf23 in worker_run (ctx=0x3c200e0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/Main NFSD/nfs_worker_thread.c:1562 #13 0x000000000050a3e7 in fridgethr_start_routine (arg=0x3c200e0) at /usr/src/debug/nfs-ganesha-2.5.3-ibm013.00-0.1.1-Source/supp ort/fridgethr.c:550 #14 0x00007f4758725dc5 in start_thread () from /lib64/libpthread.so.0 #15 0x00007f4757de673d in clone () from /lib64/libc.so.6 A closer look at the backtrace indicates that there was cyclic flow of execution as below: nlm4_Unlock -> mdcache_put_ref -> mdcache_put -> _mdcache_lru_unref -> mdcache_lru_clean -> fsal_obj_handle_fini and then mdc_clean_entry -> cih_remove_checked -> (purposely coping next flow on below line) -> _mdcache_lru_unref -> mdcache_lru_clean -> fsal_obj_handle_fini (currently crashing here) Do we see any code issue here ? Any hints on how to RCA this issue ? Thanks in advance. -- with regards, Sachin Punadikar -- with regards, Sachin Punadikar

4 7

Change in ffilz/nfs-ganesha[next]: ganesha-rados-grace: add a new --ns option
by GerritHub 31 Jul '18

31 Jul '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420913 Change subject: ganesha-rados-grace: add a new --ns option ...................................................................... ganesha-rados-grace: add a new --ns option Add a new --ns option so that we can set the namespace for the grace oid properly. Change-Id: I699d9f29b4e28aa240d07a410448cd7aa737b6ef Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/doc/man/ganesha-rados-grace.rst M src/tools/ganesha-rados-grace.c 2 files changed, 20 insertions(+), 8 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/13/420913/1 -- To view, visit https://review.gerrithub.io/420913 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: I699d9f29b4e28aa240d07a410448cd7aa737b6ef Gerrit-Change-Number: 420913 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: ganesha-rados-grace: rename --name option to --oid
by GerritHub 31 Jul '18

31 Jul '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420912 Change subject: ganesha-rados-grace: rename --name option to --oid ...................................................................... ganesha-rados-grace: rename --name option to --oid In a later patch, we're going to be adding a --ns option to allow the tool to manipulate the object in a particular ns. Rename the --name option to --oid to make this less confusing. While we're in there too, change all instances of "hostname" in the manpage to "nodeid" since that's more correct. Change-Id: I1fcb52fe0ae7d16f0b40ef917782bef7c24eac3f Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/doc/man/ganesha-rados-grace.rst M src/tools/ganesha-rados-grace.c 2 files changed, 22 insertions(+), 22 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/12/420912/1 -- To view, visit https://review.gerrithub.io/420912 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: I1fcb52fe0ae7d16f0b40ef917782bef7c24eac3f Gerrit-Change-Number: 420912 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: SAL: add a namespace parameter to RADOS_KV block
by GerritHub 31 Jul '18

31 Jul '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420911 Change subject: SAL: add a namespace parameter to RADOS_KV block ...................................................................... SAL: add a namespace parameter to RADOS_KV block It's desirable to be able to further segment a pool into multiple namespaces and allow ganesha to set its objects within its own ns. Add an option for that to the RADOS_KV block. Change-Id: I759f9f248599fceb4755791dd428f0b0e34b4d9f Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/SAL/recovery/recovery_rados.h M src/SAL/recovery/recovery_rados_cluster.c M src/SAL/recovery/recovery_rados_kv.c M src/SAL/recovery/recovery_rados_ng.c 4 files changed, 14 insertions(+), 5 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/11/420911/1 -- To view, visit https://review.gerrithub.io/420911 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: I759f9f248599fceb4755791dd428f0b0e34b4d9f Gerrit-Change-Number: 420911 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Newbie question / help with installation
by Imam Toufique 30 Jul '18

30 Jul '18

Hi , I am very new to NFS Ganesha, trying to understand how this whole setup works. So, I need setup an NFS server in CentOS 7.5 with a ZFS pool. As I read the documentation, it looks like ZFS FSal have been discontinued from NFS Ganesha source code , it is available via libzfswrap. Documentation suggests that source should be in the contrib directory, but there is no contrib directory when I clone the latest 2.6 repo. Can you please help understand where do I go about getting libzfswrap? Also, are there rpm packages for CentOS 7 for the latest 2.6 release? If there is a step by step guide on setting up an NFS server using ZFS , that would also be very helpful. Thanks

3 8

Announce Push of V2.7-dev.22
by Frank Filz 30 Jul '18

30 Jul '18

Branch next Tag:V2.7-dev.22 Release Highlights * Enable readdirplus option for GLUSTER and CEPH * MDCACHE - Handle ESTALE for all attribute refreshes * Add fallocate FSAL op and use it for CEPH and GPFS * exports: add new prepare_unexport operation * SAL: add a new is_member operation to recovery ops * FSAL_CEPH: abort the connection before shutdown if server will be restarting Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: c2bd01f Frank S. Filz V2.7-dev.22 8559102 Jeff Layton FSAL_CEPH: abort the connection before shutdown if server will be restarting 9fad1ff Jeff Layton SAL: add a new is_member operation to recovery ops 470eade Jeff Layton exports: add new prepare_unexport operation 1586e54 Jeff Layton GPFS: wire up GPFS to new fallocate operation f2e8b59 Jeff Layton FSAL_CEPH: add an fallocate operation 2751f03 Jeff Layton NFS: rework nfs4_op_allocate/deallocate to dispatch through fallocate op 5129bfd Jeff Layton FSAL: add a new fallocate fsal operation 3d82f66 Jeff Layton NFS: disallow layout stateids for WRITE op 52b2593 Jeff Layton FSAL_CEPH: enable the readdir_plus boolean d8f011f Jeff Layton NFS: nfs4_write should be checking BLOCKS for quota 0366d7c Daniel Gryniewicz GLUSTER - Enable new readdir_plus if backend support exists 9c5c419 Daniel Gryniewicz MDCACHE - Handle ESTALE for all attribute refreshes 0494e71 Daniel Gryniewicz FSAL_MEM - Fix log level for readdir 8df0b47 Daniel Gryniewicz gtest - convert readdir_correctness to obj_ops pointer

1 0

Change in ffilz/nfs-ganesha[next]: FSAL_MEM - Don't try to merge a handle with itself
by GerritHub 30 Jul '18

30 Jul '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/420777 Change subject: FSAL_MEM - Don't try to merge a handle with itself ...................................................................... FSAL_MEM - Don't try to merge a handle with itself Change-Id: I4efcab726c7d7c1651252febf0da212a63bd204e Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/FSAL_MEM/mem_handle.c 1 file changed, 5 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/77/420777/1 -- To view, visit https://review.gerrithub.io/420777 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: I4efcab726c7d7c1651252febf0da212a63bd204e Gerrit-Change-Number: 420777 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: Disallow client to create dirents with an empty name
by GerritHub 30 Jul '18

30 Jul '18

From Daniel Gryniewicz <dang(a)redhat.com>: Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/420770 Change subject: Disallow client to create dirents with an empty name ...................................................................... Disallow client to create dirents with an empty name NFSv4 spec forbids this, and Linux doesn't support it on local filesystems, and it's caused problems with Gluster (which returns the parent directory for an empty dirent name), so just protect against it. Also, NFSv4 spec requires the error to be NFS4ERR_INVAL, not NFS4ERR_BADNAME, so fix that as well. Change-Id: Iebcc589ad3f131fd2a4ea500110550741eeb6d07 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c M src/FSAL/fsal_helper.c 2 files changed, 10 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/70/420770/1 -- To view, visit https://review.gerrithub.io/420770 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: Iebcc589ad3f131fd2a4ea500110550741eeb6d07 Gerrit-Change-Number: 420770 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>

1 0

Change in ffilz/nfs-ganesha[next]: Event log for statistics activity
by GerritHub 30 Jul '18

30 Jul '18

From Sachin Punadikar <psachin(a)in.ibm.com>: Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/420760 Change subject: Event log for statistics activity ...................................................................... Event log for statistics activity Using "ganesha_stats" one can enable and disable statistics counting for the NFS server and for the FSAL. This activity should be logged as an event in the Ganesha log file. Change-Id: I569554b87426bd1d5ecc0fb2583d983b0005b36c Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com> --- M src/support/export_mgr.c 1 file changed, 24 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/60/420760/1 -- To view, visit https://review.gerrithub.io/420760 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: I569554b87426bd1d5ecc0fb2583d983b0005b36c Gerrit-Change-Number: 420760 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com>

1 0

Change in ffilz/nfs-ganesha[next]: Display GPFS FSAL response time in milliseconds
by GerritHub 30 Jul '18

30 Jul '18

From Sachin Punadikar <psachin(a)in.ibm.com>: Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/420759 Change subject: Display GPFS FSAL response time in milliseconds ...................................................................... Display GPFS FSAL response time in milliseconds "ganesha_stats" displays GPFS FSAL response time in microseconds. Make it more readable by converting into milliseconds. Change-Id: Ie30577cefafa66216a96c059623742e1fab91099 Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com> --- M src/FSAL/FSAL_GPFS/fsal_stats_gpfs.c M src/include/server_stats_private.h M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py 3 files changed, 17 insertions(+), 16 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/59/420759/1 -- To view, visit https://review.gerrithub.io/420759 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: Ie30577cefafa66216a96c059623742e1fab91099 Gerrit-Change-Number: 420759 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com>

1 0