xiaowei.hu at oracle.com
2014-Jan-24 06:11 UTC
[Ocfs2-devel] [PATCH] make ocfs2 do not log ENOENT error
From: "Xiaowei.Hu" <xiaowei.hu at oracle.com> suppress log message like this: (open_delete,8328,0):ocfs2_unlink:951 ERROR: status = -2 Orabug:17445485 --- fs/ocfs2/namei.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/ocfs2/namei.c b/fs/ocfs2/namei.c index 0ba1cf0..ca6c5ba 100644 --- a/fs/ocfs2/namei.c +++ b/fs/ocfs2/namei.c @@ -947,7 +947,7 @@ leave: ocfs2_free_dir_lookup_result(&orphan_insert); ocfs2_free_dir_lookup_result(&lookup); - if (status && (status != -ENOTEMPTY)) + if (status && (status != -ENOTEMPTY) && (status != -ENOENT)) mlog_errno(status); return status; -- 1.8.4.2
Andrew Morton
2014-Jan-24 20:34 UTC
[Ocfs2-devel] [PATCH] make ocfs2 do not log ENOENT error
On Fri, 24 Jan 2014 14:11:09 +0800 xiaowei.hu at oracle.com wrote:> From: "Xiaowei.Hu" <xiaowei.hu at oracle.com> > > suppress log message like this: > (open_delete,8328,0):ocfs2_unlink:951 ERROR: status = -2 > > Orabug:17445485 > --- > fs/ocfs2/namei.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/ocfs2/namei.c b/fs/ocfs2/namei.c > index 0ba1cf0..ca6c5ba 100644 > --- a/fs/ocfs2/namei.c > +++ b/fs/ocfs2/namei.c > @@ -947,7 +947,7 @@ leave: > ocfs2_free_dir_lookup_result(&orphan_insert); > ocfs2_free_dir_lookup_result(&lookup); > > - if (status && (status != -ENOTEMPTY)) > + if (status && (status != -ENOTEMPTY) && (status != -ENOENT)) > mlog_errno(status); > > return status;Please send a signed-off-by: for this patch.
Sorry for missing that. signed-off-by: xiaowei.hu at oracle.com Thanks On 01/25/2014 04:34 AM, Andrew Morton wrote:> On Fri, 24 Jan 2014 14:11:09 +0800 xiaowei.hu at oracle.com wrote: > >> From: "Xiaowei.Hu" <xiaowei.hu at oracle.com> >> >> suppress log message like this: >> (open_delete,8328,0):ocfs2_unlink:951 ERROR: status = -2 >> >> Orabug:17445485 >> --- >> fs/ocfs2/namei.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/fs/ocfs2/namei.c b/fs/ocfs2/namei.c >> index 0ba1cf0..ca6c5ba 100644 >> --- a/fs/ocfs2/namei.c >> +++ b/fs/ocfs2/namei.c >> @@ -947,7 +947,7 @@ leave: >> ocfs2_free_dir_lookup_result(&orphan_insert); >> ocfs2_free_dir_lookup_result(&lookup); >> >> - if (status && (status != -ENOTEMPTY)) >> + if (status && (status != -ENOTEMPTY) && (status != -ENOENT)) >> mlog_errno(status); >> >> return status; > Please send a signed-off-by: for this patch.