* [PATCH] nfsd: print status when nfsd4_open fails to open file it just created
@ 2014-07-21 13:37 Jeff Layton
2014-07-21 20:13 ` J. Bruce Fields
2014-07-30 1:37 ` [PATCH resend] " Jeff Layton
0 siblings, 2 replies; 5+ messages in thread
From: Jeff Layton @ 2014-07-21 13:37 UTC (permalink / raw)
To: bfields; +Cc: linux-nfs
It's possible for nfsd to fail opening a file that it has just created.
When that happens, we throw a WARN but it doesn't include any info about
the error code. Print the status code to give us a bit more info.
Our QA group hit some of these warnings under some very heavy stress
testing. My suspicion is that they hit the file-max limit, but it's hard
to know for sure. Go ahead and add a -ENFILE mapping to
nfserr_serverfault to make the error more distinct (and correct).
Signed-off-by: Jeff Layton <jlayton@primarydata.com>
---
fs/nfsd/nfs4proc.c | 4 +++-
fs/nfsd/nfsproc.c | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
index 29a617ebe38c..8611585f739d 100644
--- a/fs/nfsd/nfs4proc.c
+++ b/fs/nfsd/nfs4proc.c
@@ -460,7 +460,9 @@ nfsd4_open(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
* set, (2) sets open->op_stateid, (3) sets open->op_delegation.
*/
status = nfsd4_process_open2(rqstp, resfh, open);
- WARN_ON(status && open->op_created);
+ WARN(status && open->op_created,
+ "nfsd4_process_open2 failed to open newly-created file! status=%u\n",
+ be32_to_cpu(status));
out:
if (resfh && resfh != &cstate->current_fh) {
fh_dup2(&cstate->current_fh, resfh);
diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c
index b19c7e8bf64c..b8680738f588 100644
--- a/fs/nfsd/nfsproc.c
+++ b/fs/nfsd/nfsproc.c
@@ -745,6 +745,7 @@ nfserrno (int errno)
{ nfserr_notsupp, -EOPNOTSUPP },
{ nfserr_toosmall, -ETOOSMALL },
{ nfserr_serverfault, -ESERVERFAULT },
+ { nfserr_serverfault, -ENFILE },
};
int i;
--
1.9.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] nfsd: print status when nfsd4_open fails to open file it just created
2014-07-21 13:37 [PATCH] nfsd: print status when nfsd4_open fails to open file it just created Jeff Layton
@ 2014-07-21 20:13 ` J. Bruce Fields
2014-07-21 20:17 ` Jeff Layton
2014-07-30 1:37 ` [PATCH resend] " Jeff Layton
1 sibling, 1 reply; 5+ messages in thread
From: J. Bruce Fields @ 2014-07-21 20:13 UTC (permalink / raw)
To: Jeff Layton; +Cc: linux-nfs
On Mon, Jul 21, 2014 at 09:37:30AM -0400, Jeff Layton wrote:
> It's possible for nfsd to fail opening a file that it has just created.
> When that happens, we throw a WARN but it doesn't include any info about
> the error code. Print the status code to give us a bit more info.
>
> Our QA group hit some of these warnings under some very heavy stress
> testing. My suspicion is that they hit the file-max limit, but it's hard
> to know for sure. Go ahead and add a -ENFILE mapping to
> nfserr_serverfault to make the error more distinct (and correct).
Thanks, applying.
We do need to fix this. The open code is complicated, but I think there
probably aren't too many failure cases after a successful create left.
--b.
>
> Signed-off-by: Jeff Layton <jlayton@primarydata.com>
> ---
> fs/nfsd/nfs4proc.c | 4 +++-
> fs/nfsd/nfsproc.c | 1 +
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
> index 29a617ebe38c..8611585f739d 100644
> --- a/fs/nfsd/nfs4proc.c
> +++ b/fs/nfsd/nfs4proc.c
> @@ -460,7 +460,9 @@ nfsd4_open(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
> * set, (2) sets open->op_stateid, (3) sets open->op_delegation.
> */
> status = nfsd4_process_open2(rqstp, resfh, open);
> - WARN_ON(status && open->op_created);
> + WARN(status && open->op_created,
> + "nfsd4_process_open2 failed to open newly-created file! status=%u\n",
> + be32_to_cpu(status));
> out:
> if (resfh && resfh != &cstate->current_fh) {
> fh_dup2(&cstate->current_fh, resfh);
> diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c
> index b19c7e8bf64c..b8680738f588 100644
> --- a/fs/nfsd/nfsproc.c
> +++ b/fs/nfsd/nfsproc.c
> @@ -745,6 +745,7 @@ nfserrno (int errno)
> { nfserr_notsupp, -EOPNOTSUPP },
> { nfserr_toosmall, -ETOOSMALL },
> { nfserr_serverfault, -ESERVERFAULT },
> + { nfserr_serverfault, -ENFILE },
> };
> int i;
>
> --
> 1.9.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] nfsd: print status when nfsd4_open fails to open file it just created
2014-07-21 20:13 ` J. Bruce Fields
@ 2014-07-21 20:17 ` Jeff Layton
0 siblings, 0 replies; 5+ messages in thread
From: Jeff Layton @ 2014-07-21 20:17 UTC (permalink / raw)
To: J. Bruce Fields; +Cc: linux-nfs
On Mon, 21 Jul 2014 16:13:20 -0400
"J. Bruce Fields" <bfields@fieldses.org> wrote:
> On Mon, Jul 21, 2014 at 09:37:30AM -0400, Jeff Layton wrote:
> > It's possible for nfsd to fail opening a file that it has just created.
> > When that happens, we throw a WARN but it doesn't include any info about
> > the error code. Print the status code to give us a bit more info.
> >
> > Our QA group hit some of these warnings under some very heavy stress
> > testing. My suspicion is that they hit the file-max limit, but it's hard
> > to know for sure. Go ahead and add a -ENFILE mapping to
> > nfserr_serverfault to make the error more distinct (and correct).
>
> Thanks, applying.
>
> We do need to fix this. The open code is complicated, but I think there
> probably aren't too many failure cases after a successful create left.
>
> --b.
>
Yeah, this is just an interim step to help us nail down some of the
failure scenarios. It doesn't really fix anything...
What we'll eventually need to do is reorganize the code so that the
creates and opens are atomic. That'll take some work to untangle though
since there are several steps between creating a file and opening it
now.
> >
> > Signed-off-by: Jeff Layton <jlayton@primarydata.com>
> > ---
> > fs/nfsd/nfs4proc.c | 4 +++-
> > fs/nfsd/nfsproc.c | 1 +
> > 2 files changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
> > index 29a617ebe38c..8611585f739d 100644
> > --- a/fs/nfsd/nfs4proc.c
> > +++ b/fs/nfsd/nfs4proc.c
> > @@ -460,7 +460,9 @@ nfsd4_open(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
> > * set, (2) sets open->op_stateid, (3) sets open->op_delegation.
> > */
> > status = nfsd4_process_open2(rqstp, resfh, open);
> > - WARN_ON(status && open->op_created);
> > + WARN(status && open->op_created,
> > + "nfsd4_process_open2 failed to open newly-created file! status=%u\n",
> > + be32_to_cpu(status));
> > out:
> > if (resfh && resfh != &cstate->current_fh) {
> > fh_dup2(&cstate->current_fh, resfh);
> > diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c
> > index b19c7e8bf64c..b8680738f588 100644
> > --- a/fs/nfsd/nfsproc.c
> > +++ b/fs/nfsd/nfsproc.c
> > @@ -745,6 +745,7 @@ nfserrno (int errno)
> > { nfserr_notsupp, -EOPNOTSUPP },
> > { nfserr_toosmall, -ETOOSMALL },
> > { nfserr_serverfault, -ESERVERFAULT },
> > + { nfserr_serverfault, -ENFILE },
> > };
> > int i;
> >
> > --
> > 1.9.3
> >
--
Jeff Layton <jlayton@primarydata.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH resend] nfsd: print status when nfsd4_open fails to open file it just created
2014-07-21 13:37 [PATCH] nfsd: print status when nfsd4_open fails to open file it just created Jeff Layton
2014-07-21 20:13 ` J. Bruce Fields
@ 2014-07-30 1:37 ` Jeff Layton
2014-07-30 13:28 ` J. Bruce Fields
1 sibling, 1 reply; 5+ messages in thread
From: Jeff Layton @ 2014-07-30 1:37 UTC (permalink / raw)
To: bfields; +Cc: linux-nfs
It's possible for nfsd to fail opening a file that it has just created.
When that happens, we throw a WARN but it doesn't include any info about
the error code. Print the status code to give us a bit more info.
Our QA group hit some of these warnings under some very heavy stress
testing. My suspicion is that they hit the file-max limit, but it's hard
to know for sure. Go ahead and add a -ENFILE mapping to
nfserr_serverfault to make the error more distinct (and correct).
Signed-off-by: Jeff Layton <jlayton@primarydata.com>
---
fs/nfsd/nfs4proc.c | 4 +++-
fs/nfsd/nfsproc.c | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
index 29a617ebe38c..8611585f739d 100644
--- a/fs/nfsd/nfs4proc.c
+++ b/fs/nfsd/nfs4proc.c
@@ -460,7 +460,9 @@ nfsd4_open(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
* set, (2) sets open->op_stateid, (3) sets open->op_delegation.
*/
status = nfsd4_process_open2(rqstp, resfh, open);
- WARN_ON(status && open->op_created);
+ WARN(status && open->op_created,
+ "nfsd4_process_open2 failed to open newly-created file! status=%u\n",
+ be32_to_cpu(status));
out:
if (resfh && resfh != &cstate->current_fh) {
fh_dup2(&cstate->current_fh, resfh);
diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c
index b19c7e8bf64c..b8680738f588 100644
--- a/fs/nfsd/nfsproc.c
+++ b/fs/nfsd/nfsproc.c
@@ -745,6 +745,7 @@ nfserrno (int errno)
{ nfserr_notsupp, -EOPNOTSUPP },
{ nfserr_toosmall, -ETOOSMALL },
{ nfserr_serverfault, -ESERVERFAULT },
+ { nfserr_serverfault, -ENFILE },
};
int i;
--
1.9.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH resend] nfsd: print status when nfsd4_open fails to open file it just created
2014-07-30 1:37 ` [PATCH resend] " Jeff Layton
@ 2014-07-30 13:28 ` J. Bruce Fields
0 siblings, 0 replies; 5+ messages in thread
From: J. Bruce Fields @ 2014-07-30 13:28 UTC (permalink / raw)
To: Jeff Layton; +Cc: linux-nfs
Thanks for the resend, applied.--b.
On Tue, Jul 29, 2014 at 09:37:44PM -0400, Jeff Layton wrote:
> It's possible for nfsd to fail opening a file that it has just created.
> When that happens, we throw a WARN but it doesn't include any info about
> the error code. Print the status code to give us a bit more info.
>
> Our QA group hit some of these warnings under some very heavy stress
> testing. My suspicion is that they hit the file-max limit, but it's hard
> to know for sure. Go ahead and add a -ENFILE mapping to
> nfserr_serverfault to make the error more distinct (and correct).
>
> Signed-off-by: Jeff Layton <jlayton@primarydata.com>
> ---
> fs/nfsd/nfs4proc.c | 4 +++-
> fs/nfsd/nfsproc.c | 1 +
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
> index 29a617ebe38c..8611585f739d 100644
> --- a/fs/nfsd/nfs4proc.c
> +++ b/fs/nfsd/nfs4proc.c
> @@ -460,7 +460,9 @@ nfsd4_open(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
> * set, (2) sets open->op_stateid, (3) sets open->op_delegation.
> */
> status = nfsd4_process_open2(rqstp, resfh, open);
> - WARN_ON(status && open->op_created);
> + WARN(status && open->op_created,
> + "nfsd4_process_open2 failed to open newly-created file! status=%u\n",
> + be32_to_cpu(status));
> out:
> if (resfh && resfh != &cstate->current_fh) {
> fh_dup2(&cstate->current_fh, resfh);
> diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c
> index b19c7e8bf64c..b8680738f588 100644
> --- a/fs/nfsd/nfsproc.c
> +++ b/fs/nfsd/nfsproc.c
> @@ -745,6 +745,7 @@ nfserrno (int errno)
> { nfserr_notsupp, -EOPNOTSUPP },
> { nfserr_toosmall, -ETOOSMALL },
> { nfserr_serverfault, -ESERVERFAULT },
> + { nfserr_serverfault, -ENFILE },
> };
> int i;
>
> --
> 1.9.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-07-30 13:28 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-21 13:37 [PATCH] nfsd: print status when nfsd4_open fails to open file it just created Jeff Layton
2014-07-21 20:13 ` J. Bruce Fields
2014-07-21 20:17 ` Jeff Layton
2014-07-30 1:37 ` [PATCH resend] " Jeff Layton
2014-07-30 13:28 ` J. Bruce Fields
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).