From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761978AbcJ1WXg (ORCPT ); Fri, 28 Oct 2016 18:23:36 -0400 Received: from mout02.posteo.de ([185.67.36.142]:41597 "EHLO mout02.posteo.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755086AbcJ1WXd (ORCPT ); Fri, 28 Oct 2016 18:23:33 -0400 Message-ID: <1477693395.31471.1.camel@embedded.rocks> Subject: Re: [PATCH] ubifs: Fix regression in ubifs_readdir() From: =?ISO-8859-1?Q?J=F6rg?= Krause To: Richard Weinberger , linux-mtd@lists.infradead.org Cc: dedekind1@gmail.com, linux-kernel@vger.kernel.org, stable@vger.kernel.org, adrian.hunter@intel.com, ralph.sennhauser@gmail.com, peda@axentia.se Date: Sat, 29 Oct 2016 00:23:15 +0200 In-Reply-To: References: <1477648432-9543-1-git-send-email-richard@nod.at> <1477671544.8927.1.camel@embedded.rocks> Organization: Embedded Rocks Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.2 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Richard, On Fri, 2016-10-28 at 19:07 +0200, Richard Weinberger wrote: > Jörg, > > On 28.10.2016 18:19, Jörg Krause wrote: > > Hi, > > > > On Fri, 2016-10-28 at 11:53 +0200, Richard Weinberger wrote: > > > Commit c83ed4c9dbb35 ("ubifs: Abort readdir upon error") broke > > > overlayfs support because the fix exposed an internal error > > > code to VFS. > > > > > > Reported-by: Peter Rosin > > > Tested-by: Peter Rosin > > > Reported-by: Ralph Sennhauser > > > Fixes: c83ed4c9dbb35 ("ubifs: Abort readdir upon error") > > > Cc: stable@vger.kernel.org > > > Signed-off-by: Richard Weinberger > > > --- > > >  fs/ubifs/dir.c | 8 ++++++++ > > >  1 file changed, 8 insertions(+) > > > > > > diff --git a/fs/ubifs/dir.c b/fs/ubifs/dir.c > > > index bd4a5e8ce441..ca16c5d7bab1 100644 > > > --- a/fs/ubifs/dir.c > > > +++ b/fs/ubifs/dir.c > > > @@ -543,6 +543,14 @@ static int ubifs_readdir(struct file *file, > > > struct dir_context *ctx) > > >   > > >   if (err != -ENOENT) > > >   ubifs_err(c, "cannot find next direntry, error > > > %d", > > > err); > > > + else > > > + /* > > > +  * -ENOENT is a non-fatal error in this context, > > > the > > > TNC uses > > > +  * it to indicate that the cursor moved past the > > > current directory > > > +  * and readdir() has to stop. > > > +  */ > > > + err = 0; > > > + > > >   > > >   /* 2 is a special value indicating that there are no > > > more > > > direntries */ > > >   ctx->pos = 2; > > > > I'm not sure if it's related to the issue reported by Peter Rosin > > and > > Ralph Sennhauser, but I am still getting a kernel panic using UBIFS > > with OverlayFS on Linux v4.9.0-rc2 with this patch applied: > > Does reverting c83ed4c9dbb35 help? > And are you 100% sure you applied the fix? I double double checked. The fix was applied on the git tree, but the compiler cache (I am using Buildroot with this option enabled) fooled me by using an old copy. After disabling the compiler cache I got a fixed build of the kernel. The panic is gone! Thanks! > > Does the following WARN_ON() trigger? > diff --git a/fs/ubifs/dir.c b/fs/ubifs/dir.c > index ca16c5d7bab1..12ffc91f7ef8 100644 > --- a/fs/ubifs/dir.c > +++ b/fs/ubifs/dir.c > @@ -554,6 +554,9 @@ static int ubifs_readdir(struct file *file, > struct dir_context *ctx) > >   /* 2 is a special value indicating that there are no more > direntries */ >   ctx->pos = 2; > + > + WARN_ON(err); > + >   return err; >  } Best regards, Jörg Krause From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout02.posteo.de ([185.67.36.142]) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1c0FZ5-0002eW-SD for linux-mtd@lists.infradead.org; Fri, 28 Oct 2016 22:23:57 +0000 Message-ID: <1477693395.31471.1.camel@embedded.rocks> Subject: Re: [PATCH] ubifs: Fix regression in ubifs_readdir() From: =?ISO-8859-1?Q?J=F6rg?= Krause To: Richard Weinberger , linux-mtd@lists.infradead.org Cc: dedekind1@gmail.com, linux-kernel@vger.kernel.org, stable@vger.kernel.org, adrian.hunter@intel.com, ralph.sennhauser@gmail.com, peda@axentia.se Date: Sat, 29 Oct 2016 00:23:15 +0200 In-Reply-To: References: <1477648432-9543-1-git-send-email-richard@nod.at> <1477671544.8927.1.camel@embedded.rocks> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Richard, On Fri, 2016-10-28 at 19:07 +0200, Richard Weinberger wrote: > Jörg, > > On 28.10.2016 18:19, Jörg Krause wrote: > > Hi, > > > > On Fri, 2016-10-28 at 11:53 +0200, Richard Weinberger wrote: > > > Commit c83ed4c9dbb35 ("ubifs: Abort readdir upon error") broke > > > overlayfs support because the fix exposed an internal error > > > code to VFS. > > > > > > Reported-by: Peter Rosin > > > Tested-by: Peter Rosin > > > Reported-by: Ralph Sennhauser > > > Fixes: c83ed4c9dbb35 ("ubifs: Abort readdir upon error") > > > Cc: stable@vger.kernel.org > > > Signed-off-by: Richard Weinberger > > > --- > > >  fs/ubifs/dir.c | 8 ++++++++ > > >  1 file changed, 8 insertions(+) > > > > > > diff --git a/fs/ubifs/dir.c b/fs/ubifs/dir.c > > > index bd4a5e8ce441..ca16c5d7bab1 100644 > > > --- a/fs/ubifs/dir.c > > > +++ b/fs/ubifs/dir.c > > > @@ -543,6 +543,14 @@ static int ubifs_readdir(struct file *file, > > > struct dir_context *ctx) > > >   > > >   if (err != -ENOENT) > > >   ubifs_err(c, "cannot find next direntry, error > > > %d", > > > err); > > > + else > > > + /* > > > +  * -ENOENT is a non-fatal error in this context, > > > the > > > TNC uses > > > +  * it to indicate that the cursor moved past the > > > current directory > > > +  * and readdir() has to stop. > > > +  */ > > > + err = 0; > > > + > > >   > > >   /* 2 is a special value indicating that there are no > > > more > > > direntries */ > > >   ctx->pos = 2; > > > > I'm not sure if it's related to the issue reported by Peter Rosin > > and > > Ralph Sennhauser, but I am still getting a kernel panic using UBIFS > > with OverlayFS on Linux v4.9.0-rc2 with this patch applied: > > Does reverting c83ed4c9dbb35 help? > And are you 100% sure you applied the fix? I double double checked. The fix was applied on the git tree, but the compiler cache (I am using Buildroot with this option enabled) fooled me by using an old copy. After disabling the compiler cache I got a fixed build of the kernel. The panic is gone! Thanks! > > Does the following WARN_ON() trigger? > diff --git a/fs/ubifs/dir.c b/fs/ubifs/dir.c > index ca16c5d7bab1..12ffc91f7ef8 100644 > --- a/fs/ubifs/dir.c > +++ b/fs/ubifs/dir.c > @@ -554,6 +554,9 @@ static int ubifs_readdir(struct file *file, > struct dir_context *ctx) > >   /* 2 is a special value indicating that there are no more > direntries */ >   ctx->pos = 2; > + > + WARN_ON(err); > + >   return err; >  } Best regards, Jörg Krause