FSAL async API
by Frank Filz
Daniel,
So I'm looking at the async API, I guess right now although the return is
passed by a callback, it's actually always synchronous, and called before
returning from read2 or write2?
So the callers need to be modified to be prepared for a true async callback.
If they can't actually be async, they need to do their own signaling (they
should be prepared for an inline callback or an async callback, for inline,
they need to just do what their callback will need to let the main function
know there was an inline response, otherwise, a non-async capable caller
needs to wait on a condition variable or some other sleep and wakeup
mechanism.
I'll start to model that with NFS v4 ops.
Frank
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: NFS4: Prepare nfs4_Compound.c for async compounds
by GerritHub
From Frank Filz <ffilzlnx(a)mindspring.com>:
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/420310
Change subject: NFS4: Prepare nfs4_Compound.c for async compounds
......................................................................
NFS4: Prepare nfs4_Compound.c for async compounds
Change-Id: Ia177bf78680a86256cd84e274d442e32a6effb7b
Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com>
---
M src/Protocols/NFS/nfs4_Compound.c
M src/include/nfs_proto_data.h
2 files changed, 319 insertions(+), 292 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/10/420310/1
--
To view, visit https://review.gerrithub.io/420310
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: Ia177bf78680a86256cd84e274d442e32a6effb7b
Gerrit-Change-Number: 420310
Gerrit-PatchSet: 1
Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com>
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: FSAL_CEPH: abort the connection before shutdown if server will be res...
by GerritHub
From Jeff Layton <jlayton(a)redhat.com>:
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420302
Change subject: FSAL_CEPH: abort the connection before shutdown if server will be restarting
......................................................................
FSAL_CEPH: abort the connection before shutdown if server will be restarting
If the server is still listed as a member of the cluster at shutdown,
abort the connection prior to tearing down the stateful objects. This
will ensure that caps will be held in reserve for the client while it's
rebooting.
Change-Id: I7111fbf3f0746bf488252d6bff6858680283deb3
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/CMakeLists.txt
M src/FSAL/FSAL_CEPH/export.c
M src/cmake/modules/FindCEPHFS.cmake
M src/include/config-h.in.cmake
4 files changed, 32 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/02/420302/1
--
To view, visit https://review.gerrithub.io/420302
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: I7111fbf3f0746bf488252d6bff6858680283deb3
Gerrit-Change-Number: 420302
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: SAL: add a new is_member operation to recovery ops
by GerritHub
From Jeff Layton <jlayton(a)redhat.com>:
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420301
Change subject: SAL: add a new is_member operation to recovery ops
......................................................................
SAL: add a new is_member operation to recovery ops
Change-Id: I2bc0148136ae7ebff54a4c1ee928e21674705909
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/SAL/nfs4_recovery.c
M src/SAL/recovery/recovery_rados_cluster.c
M src/include/sal_functions.h
3 files changed, 31 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/01/420301/1
--
To view, visit https://review.gerrithub.io/420301
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: I2bc0148136ae7ebff54a4c1ee928e21674705909
Gerrit-Change-Number: 420301
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: MDCACHE: add in the prepare_unexport passthrough
by GerritHub
From Jeff Layton <jlayton(a)redhat.com>:
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420300
Change subject: MDCACHE: add in the prepare_unexport passthrough
......................................................................
MDCACHE: add in the prepare_unexport passthrough
Change-Id: I37dd87b4637c45f30c20a55b56b61a10d5ede29b
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_export.c
1 file changed, 17 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/00/420300/1
--
To view, visit https://review.gerrithub.io/420300
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: I37dd87b4637c45f30c20a55b56b61a10d5ede29b
Gerrit-Change-Number: 420300
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: FSAL_CEPH: enable the readdir_plus boolean
by GerritHub
From Jeff Layton <jlayton(a)redhat.com>:
Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/420294
Change subject: FSAL_CEPH: enable the readdir_plus boolean
......................................................................
FSAL_CEPH: enable the readdir_plus boolean
libcephfs has had a readdirplus call for ages.
Change-Id: Icba2ed88419d3366e4de5e0138e3d09da9ab6089
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
M src/FSAL/FSAL_CEPH/main.c
1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/94/420294/1
--
To view, visit https://review.gerrithub.io/420294
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: Icba2ed88419d3366e4de5e0138e3d09da9ab6089
Gerrit-Change-Number: 420294
Gerrit-PatchSet: 1
Gerrit-Owner: Jeff Layton <jlayton(a)redhat.com>
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: GLUSTER - Enable new readdir_plus if backend support exists
by GerritHub
From Daniel Gryniewicz <dang(a)redhat.com>:
Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/420290
Change subject: GLUSTER - Enable new readdir_plus if backend support exists
......................................................................
GLUSTER - Enable new readdir_plus if backend support exists
Change-Id: I3141a951917a121772eddd8c852691ce57842e96
Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com>
---
M src/FSAL/FSAL_GLUSTER/main.c
1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/90/420290/1
--
To view, visit https://review.gerrithub.io/420290
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: I3141a951917a121772eddd8c852691ce57842e96
Gerrit-Change-Number: 420290
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: MDCACHE - Handle ESTALE for all attribute refreshes
by GerritHub
From Daniel Gryniewicz <dang(a)redhat.com>:
Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/420289
Change subject: MDCACHE - Handle ESTALE for all attribute refreshes
......................................................................
MDCACHE - Handle ESTALE for all attribute refreshes
Change-Id: I2f25617ec09cf4ccdd1398806c5c6a3df7439166
Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com>
---
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_file.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_handle.c
M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_int.h
3 files changed, 37 insertions(+), 31 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/89/420289/1
--
To view, visit https://review.gerrithub.io/420289
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: I2f25617ec09cf4ccdd1398806c5c6a3df7439166
Gerrit-Change-Number: 420289
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
5 years, 9 months
Change in ffilz/nfs-ganesha[next]: FSAL_MEM - Fix log level for readdir
by GerritHub
From Daniel Gryniewicz <dang(a)redhat.com>:
Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/420288
Change subject: FSAL_MEM - Fix log level for readdir
......................................................................
FSAL_MEM - Fix log level for readdir
Change-Id: Ibcf8e61a08c6e53ba7d0eb0ae6fb152681a4d08d
Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com>
---
M src/FSAL/FSAL_MEM/mem_handle.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/88/420288/1
--
To view, visit https://review.gerrithub.io/420288
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: Ibcf8e61a08c6e53ba7d0eb0ae6fb152681a4d08d
Gerrit-Change-Number: 420288
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com>
5 years, 9 months
Crash in mdcache_close() in Ganesah2.5
by Malahal Naineni
Here is a crash we got from a customer. The subhandle obj_ops point to
NULLs causing the function call
"entry->sub_handle->obj_ops.close(entry->sub_handle)" crash. Appreciate if
this kind of defect is fixed or known to be in Ganesha2.5 release.
Here is the backtrace:
(gdb) bt
#0 0x00007f8d8a2a154b in raise (sig=11) at
../nptl/sysdeps/unix/sysv/linux/pt-raise.c:37
#1 0x00000000004548cc in crash_handler (signo=11, info=0x7f8d86f04eb0,
ctx=0x7f8d86f04d80)
at
/usr/src/debug/nfs-ganesha-2.5.3-ibm021.00-0.1.1-Source/MainNFSD/nfs_init.c:225
#2 <signal handler called>
#3 0x0000000000000000 in ?? ()
#4 0x0000000000540fd4 in mdcache_close (obj_hdl=0x7f8a6c075718)
at
/usr/src/debug/nfs-ganesha-2.5.3-ibm021.00-0.1.1-Source/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_file.c:441
#5 0x00000000005307c4 in fsal_close (obj_hdl=0x7f8a6c075718)
at
/usr/src/debug/nfs-ganesha-2.5.3-ibm021.00-0.1.1-Source/include/fsal.h:432
#6 0x0000000000533fbf in lru_run_lane (lane=87, totalclosed=0x7f8d86f05460)
at
/usr/src/debug/nfs-ganesha-2.5.3-ibm021.00-0.1.1-Source/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c:1146
#7 0x00000000005348ac in lru_run (ctx=0x145d5b0)
at
/usr/src/debug/nfs-ganesha-2.5.3-ibm021.00-0.1.1-Source/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_lru.c:1328
#8 0x000000000050a5d0 in fridgethr_start_routine (arg=0x145d5b0)
at
/usr/src/debug/nfs-ganesha-2.5.3-ibm021.00-0.1.1-Source/support/fridgethr.c:550
#9 0x00007f8d8a299dd5 in start_thread (arg=0x7f8d86f06700) at
pthread_create.c:308
#10 0x00007f8d89954b3d in clone () at
../sysdeps/unix/sysv/linux/x86_64/clone.S:113
5 years, 9 months
