July 2018 - Devel - NFS Ganesha List Archives

Change in ffilz/nfs-ganesha[next]: Fix various issues in GPFS_FSAL stats
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/420755 Change subject: Fix various issues in GPFS_FSAL stats ...................................................................... Fix various issues in GPFS_FSAL stats This patch fixes below issues in the existing code: 1. Do not display operations wich has 0 count 2. Avoid crash, in case FSAL stats counting is not enabled 3. Based on FSAL name it should display the stats 4. Avoid crash when FSAL stats are enabled but has 0 counts Change-Id: I0a3eada6ea1131e35c71e1923971d088177c8eea Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com> --- M src/FSAL/FSAL_GPFS/fsal_stats_gpfs.c M src/include/abstract_mem.h M src/include/server_stats_private.h M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py M src/support/export_mgr.c M src/support/server_stats.c 6 files changed, 89 insertions(+), 75 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/55/420755/1 -- To view, visit https://review.gerrithub.io/420755 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: I0a3eada6ea1131e35c71e1923971d088177c8eea Gerrit-Change-Number: 420755 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com>

1 0

Change in ffilz/nfs-ganesha[next]: FreeBSD: make sure we destroy all pthread objects
by GerritHub 30 Jul '18

30 Jul '18

From <fatih(a)gandi.net>: fatih(a)gandi.net has uploaded this change for review. ( https://review.gerrithub.io/420733 Change subject: FreeBSD: make sure we destroy all pthread objects ...................................................................... FreeBSD: make sure we destroy all pthread objects pthread objects are dynamically allocated on init whereas objects are static on Linux we have to destroy them in order to free memory Change-Id: I9ab72a951c3737e7f2f722f116b1bff6fdda05cd Signed-off-by: Fatih Acar <fatih.acar(a)gandi.net> --- M src/FSAL/FSAL_MEM/mem_export.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_hash.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_main.c M src/FSAL/commonlib.c M src/FSAL/fsal_manager.c M src/MainNFSD/9p_dispatcher.c M src/MainNFSD/nfs_init.c M src/include/gsh_wait_queue.h M src/support/client_mgr.c M src/support/delayed_exec.c M src/support/ds.c M src/support/export_mgr.c 13 files changed, 23 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/33/420733/1 -- To view, visit https://review.gerrithub.io/420733 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: I9ab72a951c3737e7f2f722f116b1bff6fdda05cd Gerrit-Change-Number: 420733 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <fatih(a)gandi.net>

1 0

Coverity covscan for 2018-07-28-13bcfa9d (master branch)
by staticanalysis＠gluster.org 28 Jul '18

28 Jul '18

NFS-Ganesha Coverity covscan results are available from http://download.nfs-ganesha.org/coverity/2018-07-28-13bcfa9d

1 0

No merge this week
by Frank Filz 27 Jul '18

27 Jul '18

I've run out of time and there's something really wrong spewing out lots of unexpected channel errors for pynfs4.1. I will figure out what the cause of it is on Monday and hopefully figure out what to put out for dev.22. Or maybe we'll be close enough with a few other patches to pull those in too and call it rc1. Frank

1 0

V2.7 entering rc1 next week
by Frank Filz 27 Jul '18

27 Jul '18

I decided to hold off on rc1 until next week. Daniel was going to give us a libnitprc pullup and I still need to update the FSAL API version. We also have some memory pool accounting coming in. Frank

1 0

FSAL cache cleanup - How to drop in-mem caches at nfs-ganesha
by sshaikh＠commvault.com 27 Jul '18

27 Jul '18

Hi, I have a requirement in my FSM to invalidate the cache. While doing it I want to nfs-ganesha to drop any in-memory cache for the given export. How can I inform nfs-ganesha to drop all in-memory caches for the given export? Regards, Shahid Shaikh.

2 3

Change in ffilz/nfs-ganesha[next]: NFS: disallow layout stateids for WRITE op
by GerritHub 26 Jul '18

26 Jul '18

From Jeff Layton <jlayton(a)redhat.com>: Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420531 Change subject: NFS: disallow layout stateids for WRITE op ...................................................................... NFS: disallow layout stateids for WRITE op RFC5661 says: Except when special stateids are used, the stateid value for a WRITE request represents a value returned from a previous byte-range LOCK or OPEN request or the stateid associated with a delegation. Layout stateids are not valid here and should be disallowed. Change-Id: Iaf2fd97c70ce7008718e74d310091045a8c13f6c Signed-off-by: Jeff Layton <jlayton(a)redhat.com> --- M src/Protocols/NFS/nfs4_op_write.c 1 file changed, 0 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/31/420531/1 -- To view, visit https://review.gerrithub.io/420531 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: Iaf2fd97c70ce7008718e74d310091045a8c13f6c Gerrit-Change-Number: 420531 Gerrit-PatchSet: 1 Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>

1 0

Async request processing at the top end
by Frank Filz 25 Jul '18

25 Jul '18

I have done some work for async processing in nfs4_Compound.c and nfs_worker_thread.c, but what we need to actually be able to return from the request processing call stack without completing the request is to have async support at the RPC layer. I'm investigating that code, but it's quite a maze of twisty passages all alike. Following the code through all the function vectors is quite a journey. I'm not sure if there is stuff we can bypass so we don't need to do work inside libntirpc. Anyone who is more familiar with that code structure that has ideas, please share. Fundamentally, in order to not leave a worker thread blocked on FSAL I/O, we need to be able to return all the way up the stack without actually retiring the request. Frank

2 3

Change in ffilz/nfs-ganesha[next]: NFS: Prepare nfs_worker_thread.c for async processing
by GerritHub 25 Jul '18

25 Jul '18

From Frank Filz <ffilzlnx(a)mindspring.com>: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/420460 Change subject: NFS: Prepare nfs_worker_thread.c for async processing ...................................................................... NFS: Prepare nfs_worker_thread.c for async processing Change-Id: I2dcd961268b569b3e3aad6c348a4164abcde0e52 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> --- M src/MainNFSD/nfs_worker_thread.c M src/include/nfs_proto_functions.h 2 files changed, 182 insertions(+), 131 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/60/420460/1 -- To view, visit https://review.gerrithub.io/420460 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: I2dcd961268b569b3e3aad6c348a4164abcde0e52 Gerrit-Change-Number: 420460 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>

1 0

Change in ffilz/nfs-ganesha[next]: Memory pool allocation counters extraction
by GerritHub 25 Jul '18

25 Jul '18

From Sachin Punadikar <psachin(a)in.ibm.com>: Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/420423 Change subject: Memory pool allocation counters extraction ...................................................................... Memory pool allocation counters extraction Code added to get allocation counters for various memory pools. This information is useful to figure out any memory leak in pool allocations. Change-Id: I6aaa64818a34f2325a0a02dc8866fd2689769711 Signed-off-by: Sachin Punadikar <psachin(a)in.ibm.com> --- M src/include/abstract_mem.h M src/include/server_stats_private.h M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py M src/scripts/ganeshactl/ganesha_stats.py M src/support/export_mgr.c 5 files changed, 113 insertions(+), 2 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/23/420423/1 -- To view, visit https://review.gerrithub.io/420423 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: I6aaa64818a34f2325a0a02dc8866fd2689769711 Gerrit-Change-Number: 420423 Gerrit-PatchSet: 1 Gerrit-Owner: Sachin Punadikar <psachin(a)in.ibm.com>

1 0