4:48 a.m.
Hi,
I found many error info in the ganesha.log when stop nfs-ganesha.service, as follows.
ganesha.log:
：ganesha.nfsd-1550083[Admin] mdcache_lru_clean INODE:F_DBG:Trusting op_ctx export id 2
：ganesha.nfsd-1550083[Admin] posix2fsal_error:FSAL:CRIT:Default case mapping Transport
endpoint is not connected (107) to ERR_FSAL_SERVERFAULT
：ganesha.nfsd-1550083[Admin]
mdcache_Iru_clean:INODE LRU:CRIT:Error closing file in cleanup:Undefined server error
My version info: Ganesha-3.3+FSAL_CEPH(ceph version 14.2.10)；
I went through the relevant code and still didn't understand the intention of the
code.
static void release_export(struct gsh_export *export)
{
export->fsal_export->exp_ops.prepare_unexport(export->fsal_export)；
//mdcache_prepare_unexport
/* Release state belonging to this export */
state_release_export(export);
/* Flush FSAL-specific state */
export->fsal_export->exp_ops.unexport(export->fsal_export,obj);
//mdcache_unexport
}
The first thing to declare is that
USE_FSAL_CEPH_ABORT_CONN has been opened, so calling ceph_abort_conn() in
ceph_prepare_unexport will abort the connection and umount.
However, the following operation will still need export->cmount and the connect, such
as fsal_close() in
mdcache_Iru _clean().
I think that is the reason for the error printing. To sum up, I have two questions:
1、Isn't it reasonable to disconnect in
prepare_unexport before mdcache_lru_clean?
2、What is the intention of doing so? What will happen if I turn off
USE_FSAL_CEPH_ABORT_CONN?
| |
liuwei_coder
|
|
邮箱：liuwei_coder(a)163.com
|
签名由 网易邮箱大师 定制
6:26 a.m.
On Mon, 2020-09-14 at 19:32 +0800, liuwei wrote:
Hi,
I found many error info in the ganesha.log when stop nfs-ganesha.service, as follows.
ganesha.log:
：ganesha.nfsd-1550083[Admin] mdcache_lru_clean INODE:F_DBG:Trusting op_ctx export id 2
：ganesha.nfsd-1550083[Admin] posix2fsal_error:FSAL:CRIT:Default case mapping Transport
endpoint is not connected (107) to ERR_FSAL_SERVERFAULT
：ganesha.nfsd-1550083[Admin]
mdcache_Iru_clean:INODE LRU:CRIT:Error closing file in cleanup:Undefined server error
My version info: Ganesha-3.3+FSAL_CEPH(ceph version 14.2.10)；
I went through the relevant code and still didn't understand the intention of the
code.
static void release_export(struct gsh_export *export)
{
export->fsal_export->exp_ops.prepare_unexport(export->fsal_export)；
//mdcache_prepare_unexport
/* Release state belonging to this export */
state_release_export(export);
/* Flush FSAL-specific state */
export->fsal_export->exp_ops.unexport(export->fsal_export,obj);
//mdcache_unexport
}
The first thing to declare is that
USE_FSAL_CEPH_ABORT_CONN has been opened, so calling ceph_abort_conn() in
ceph_prepare_unexport will abort the connection and umount.
No, it just aborts the connection. The "mount" is still there, it's just
that attempting to send anything to the MDS after that point will result
in an error (like you're seeing on close). That error should be harmless
as the server is going down anyway and has stopped talking to clients at
that point.
However, the following operation will still need export->cmount
and the connect, such as fsal_close() in
mdcache_Iru _clean().
I think that is the reason for the error printing. To sum up, I have two questions:
1、Isn't it reasonable to disconnect in
prepare_unexport before mdcache_lru_clean?
2、What is the intention of doing so? What will happen if I turn off
USE_FSAL_CEPH_ABORT_CONN?
Turning off USE_FSAL_CEPH_ABORT_CONN will open up a race window that
would allow other ceph clients to acquire state that was still being
held by the server that is restarting.
FSAL_CEPH depends on the MDS preserving the state of ganesha's ceph
client if a server head has to be restarted. If we just tear down the
connection in this situation (as in a normal umount), the MDS would
release all of the state held by that ganesha, and other active/active
server heads could sneak in and steal the locks that it had previously
held.
Aborting the connection ensures that ganesha will have no further
communication with the MDS until the server comes back. To the MDS, it
looks like the ganesha client just dropped off the net. The MDS will
then keep the state held by that client until it comes back or times
out. Once the client does come back, it then ensures that the other
ganesha servers are enforcing the grace period, and then it will ask the
MDS to kill off the old session.
--
Jeff Layton <jlayton(a)redhat.com>
9:54 a.m.
On Mon, 2020-09-14 at 09:26 -0400, Jeff Layton wrote:
On Mon, 2020-09-14 at 19:32 +0800, liuwei wrote:
> Hi,
> I found many error info in the ganesha.log when stop nfs-ganesha.service, as
follows.
> ganesha.log:
> ：ganesha.nfsd-1550083[Admin] mdcache_lru_clean INODE:F_DBG:Trusting op_ctx export id
2
> ：ganesha.nfsd-1550083[Admin] posix2fsal_error:FSAL:CRIT:Default case mapping
Transport endpoint is not connected (107) to ERR_FSAL_SERVERFAULT
> ：ganesha.nfsd-1550083[Admin]
> mdcache_Iru_clean:INODE LRU:CRIT:Error closing file in cleanup:Undefined server
error
> My version info: Ganesha-3.3+FSAL_CEPH(ceph version 14.2.10)；
>
We should probably add a patch similar to this. I don't have the cycles
to test this at the moment. liuwei, would you be able to do so?
---------------------------8<----------------------------
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-by: liuwei <liuwei_coder(a)163.com>
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
src/FSAL/FSAL_CEPH/handle.c | 14 +++++++++++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/src/FSAL/FSAL_CEPH/handle.c b/src/FSAL/FSAL_CEPH/handle.c
index 464db598e6c8..c5891894bd01 100644
--- a/src/FSAL/FSAL_CEPH/handle.c
+++ b/src/FSAL/FSAL_CEPH/handle.c
@@ -45,6 +45,7 @@
#include "nfs_exports.h"
#include "sal_data.h"
#include "statx_compat.h"
+#include "nfs_core.h"
#include "linux/falloc.h"
/**
@@ -863,15 +864,22 @@ static fsal_status_t ceph_open_my_fd(struct ceph_handle *myself,
static fsal_status_t ceph_close_my_fd(struct ceph_fd *my_fd)
{
- int rc = 0;
fsal_status_t status = fsalstat(ERR_FSAL_NO_ERROR, 0);
struct ceph_export *export =
container_of(op_ctx->fsal_export, struct ceph_export, export);
if (my_fd->fd != NULL && my_fd->openflags != FSAL_O_CLOSED) {
- rc = ceph_ll_close(export->cmount, my_fd->fd);
- if (rc < 0)
+ int rc = ceph_ll_close(export->cmount, my_fd->fd);
+
+ if (rc < 0) {
+ /*
+ * We expect -ENOTCONN errors on shutdown. Ignore
+ * them so we don't spam the logs.
+ */
+ if (rc == -ENOTCONN && admin_shutdown)
+ rc = 0;
status = ceph2fsal_error(rc);
+ }
my_fd->fd = NULL;
my_fd->openflags = FSAL_O_CLOSED;
}
--
2.26.2
7:25 a.m.
On Tue, 2020-09-15 at 22:08 +0800, liuwei wrote:
Thanks for your kind interpretation, Jeff.
This patch looks good, and validation works. But I still have some doubts, I want to ask
you, thank you again for your help.
FSAL_CEPH depends on the MDS preserving the state of ganesha's ceph
client if a server head has to be restarted. If we just tear down the
connection in this situation (as in a normal umount), the MDS would
release all of the state held by that ganesha, and other active/active
server heads could sneak in and steal the locks that it had previously
held.
Aborting the connection ensures that ganesha will have no further
communication with the MDS until the server comes back. To the MDS, it
looks like the ganesha client just dropped off the net. The MDS will
then keep the state held by that client until it comes back or times
out. Once the client does come back, it then ensures that the other
ganesha servers are enforcing the grace period, and then it will ask the
MDS to kill off the old session.
Does MDS kill thd old session when calling ceph_reclaim_start in ganesha start process?
After the client restarts, it will ask the MDS to kill off any old
sessions that are still being held on its behalf in reclaim_reset().
Why was it added to the osd blacklist?
It shouldn't be added to the blacklist. If that happened, then it may be
that the client took too long to come back. Note that we do extend the
session timeout for these clients to 5m (again, see reclaim_reset() in
FSAL_CEPH code), but if the server took longer than that to come back up
then it could be blacklisted in the meantime.
For active/active ganesha server, what will they do when other
ganesha enter grace period, or what is the effect of keeping the previous state here?
I still don't quite understand this process. I hope you can tell me more.
That's a rather broad question, but basically when a server restarts,
the other servers will see that it has requested a grace period and will
begin grace period enforcement themselves.
Here are the slides for the talk I gave at the bakeathon in 2018, which
cover some of this:
http://www.nfsv4bat.org/Documents/BakeAThon/2018/Active_Active%20NFS%20Se...
Let me know if you have more specific questions.
At 2020-09-15 00:54:31, "Jeff Layton"
<jlayton(a)redhat.com> wrote:
>On Mon, 2020-09-14 at 09:26 -0400, Jeff Layton wrote:
>> On Mon, 2020-09-14 at 19:32 +0800, liuwei wrote:
>> > Hi,
>> > I found many error info in the ganesha.log when stop nfs-ganesha.service,
as follows.
>> > ganesha.log:
>> > ：ganesha.nfsd-1550083[Admin] mdcache_lru_clean INODE:F_DBG:Trusting op_ctx
export id 2
>> > ：ganesha.nfsd-1550083[Admin] posix2fsal_error:FSAL:CRIT:Default case
mapping Transport endpoint is not connected (107) to ERR_FSAL_SERVERFAULT
>> > ：ganesha.nfsd-1550083[Admin]
>> > mdcache_Iru_clean:INODE LRU:CRIT:Error closing file in cleanup:Undefined
server error
>> > My version info: Ganesha-3.3+FSAL_CEPH(ceph version 14.2.10)；
>> >
>
>We should probably add a patch similar to this. I don't have the cycles
>to test this at the moment. liuwei, would you be able to do so?
>
>---------------------------8<----------------------------
>
>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-by: liuwei <liuwei_coder(a)163.com>
>Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
>---
> src/FSAL/FSAL_CEPH/handle.c | 14 +++++++++++---
> 1 file changed, 11 insertions(+), 3 deletions(-)
>
>diff --git a/src/FSAL/FSAL_CEPH/handle.c b/src/FSAL/FSAL_CEPH/handle.c
>index 464db598e6c8..c5891894bd01 100644
>--- a/src/FSAL/FSAL_CEPH/handle.c
>+++ b/src/FSAL/FSAL_CEPH/handle.c
>@@ -45,6 +45,7 @@
> #include "nfs_exports.h"
> #include "sal_data.h"
> #include "statx_compat.h"
>+#include "nfs_core.h"
> #include "linux/falloc.h"
>
> /**
>@@ -863,15 +864,22 @@ static fsal_status_t ceph_open_my_fd(struct ceph_handle
*myself,
>
> static fsal_status_t ceph_close_my_fd(struct ceph_fd *my_fd)
> {
>- int rc = 0;
> fsal_status_t status = fsalstat(ERR_FSAL_NO_ERROR, 0);
> struct ceph_export *export =
> container_of(op_ctx->fsal_export, struct ceph_export, export);
>
> if (my_fd->fd != NULL && my_fd->openflags != FSAL_O_CLOSED) {
>- rc = ceph_ll_close(export->cmount, my_fd->fd);
>- if (rc < 0)
>+ int rc = ceph_ll_close(export->cmount, my_fd->fd);
>+
>+ if (rc < 0) {
>+ /*
>+ * We expect -ENOTCONN errors on shutdown. Ignore
>+ * them so we don't spam the logs.
>+ */
>+ if (rc == -ENOTCONN && admin_shutdown)
>+ rc = 0;
> status = ceph2fsal_error(rc);
>+ }
> my_fd->fd = NULL;
> my_fd->openflags = FSAL_O_CLOSED;
> }
>--
>2.26.2
>
>_______________________________________________
>Devel mailing list -- devel(a)lists.nfs-ganesha.org
>To unsubscribe send an email to devel-leave(a)lists.nfs-ganesha.org
--
Jeff Layton <jlayton(a)redhat.com>
8:08 p.m.
Thanks for your kind interpretation, Jeff.
This patch looks good, and validation works. But I still have some doubts, I want to ask
you, thank you again for your help.
FSAL_CEPH depends on the MDS preserving the state of ganesha's ceph
client if a server head has to be restarted. If we just tear down the
connection in this situation (as in a normal umount), the MDS would
release all of the state held by that ganesha, and other active/active
server heads could sneak in and steal the locks that it had previously
held.
Aborting the connection ensures that ganesha will have no further
communication with the MDS until the server comes back. To the MDS, it
looks like the ganesha client just dropped off the net. The MDS will
then keep the state held by that client until it comes back or times
out. Once the client does come back, it then ensures that the other
ganesha servers are enforcing the grace period, and then it will ask the
MDS to kill off the old session.
Does MDS kill thd old session when calling ceph_reclaim_start in ganesha start process?
Why was it added to the osd blacklist?
For active/active ganesha server, what will they do when other ganesha enter grace period,
or what is the effect of keeping the previous state here?
I still don't quite understand this process. I hope you can tell me more.
At 2020-09-15 00:54:31, "Jeff Layton" <jlayton(a)redhat.com> wrote:
On Mon, 2020-09-14 at 09:26 -0400, Jeff Layton wrote:
> On Mon, 2020-09-14 at 19:32 +0800, liuwei wrote:
> > Hi,
> > I found many error info in the ganesha.log when stop nfs-ganesha.service, as
follows.
> > ganesha.log:
> > ：ganesha.nfsd-1550083[Admin] mdcache_lru_clean INODE:F_DBG:Trusting op_ctx
export id 2
> > ：ganesha.nfsd-1550083[Admin] posix2fsal_error:FSAL:CRIT:Default case mapping
Transport endpoint is not connected (107) to ERR_FSAL_SERVERFAULT
> > ：ganesha.nfsd-1550083[Admin]
> > mdcache_Iru_clean:INODE LRU:CRIT:Error closing file in cleanup:Undefined server
error
> > My version info: Ganesha-3.3+FSAL_CEPH(ceph version 14.2.10)；
> >
We should probably add a patch similar to this. I don't have the cycles
to test this at the moment. liuwei, would you be able to do so?
---------------------------8<----------------------------
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-by: liuwei <liuwei_coder(a)163.com>
Signed-off-by: Jeff Layton <jlayton(a)redhat.com>
---
src/FSAL/FSAL_CEPH/handle.c | 14 +++++++++++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/src/FSAL/FSAL_CEPH/handle.c b/src/FSAL/FSAL_CEPH/handle.c
index 464db598e6c8..c5891894bd01 100644
--- a/src/FSAL/FSAL_CEPH/handle.c
+++ b/src/FSAL/FSAL_CEPH/handle.c
@@ -45,6 +45,7 @@
#include "nfs_exports.h"
#include "sal_data.h"
#include "statx_compat.h"
+#include "nfs_core.h"
#include "linux/falloc.h"
/**
@@ -863,15 +864,22 @@ static fsal_status_t ceph_open_my_fd(struct ceph_handle *myself,
static fsal_status_t ceph_close_my_fd(struct ceph_fd *my_fd)
{
- int rc = 0;
fsal_status_t status = fsalstat(ERR_FSAL_NO_ERROR, 0);
struct ceph_export *export =
container_of(op_ctx->fsal_export, struct ceph_export, export);
if (my_fd->fd != NULL && my_fd->openflags != FSAL_O_CLOSED) {
- rc = ceph_ll_close(export->cmount, my_fd->fd);
- if (rc < 0)
+ int rc = ceph_ll_close(export->cmount, my_fd->fd);
+
+ if (rc < 0) {
+ /*
+ * We expect -ENOTCONN errors on shutdown. Ignore
+ * them so we don't spam the logs.
+ */
+ if (rc == -ENOTCONN && admin_shutdown)
+ rc = 0;
status = ceph2fsal_error(rc);
+ }
my_fd->fd = NULL;
my_fd->openflags = FSAL_O_CLOSED;
}
--
2.26.2
_______________________________________________
Devel mailing list -- devel(a)lists.nfs-ganesha.org
To unsubscribe send an email to devel-leave(a)lists.nfs-ganesha.org
1337
days inactive
1339
days old
4 comments
2 participants
participants (2)
-
Jeff Layton -
liuwei