[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200103163022.GA1205191@kroah.com>
Date: Fri, 3 Jan 2020 17:30:22 +0100
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Guenter Roeck <linux@...ck-us.net>
Cc: Arnd Bergmann <arnd@...db.de>,
Naresh Kamboju <naresh.kamboju@...aro.org>,
open list <linux-kernel@...r.kernel.org>,
Linus Torvalds <torvalds@...ux-foundation.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Shuah Khan <shuah@...nel.org>, patches@...nelci.org,
Ben Hutchings <ben.hutchings@...ethink.co.uk>,
lkft-triage@...ts.linaro.org,
linux- stable <stable@...r.kernel.org>,
Chengguang Xu <cgxu519@...ernel.net>,
Mike Kravetz <mike.kravetz@...cle.com>,
David Howells <dhowells@...hat.com>,
Al Viro <viro@...iv.linux.org.uk>,
Sasha Levin <sashal@...nel.org>, LTP List <ltp@...ts.linux.it>,
Jan Stancek <jstancek@...hat.com>,
John Stultz <john.stultz@...aro.org>
Subject: Re: [PATCH 5.4 000/191] 5.4.8-stable review
On Fri, Jan 03, 2020 at 07:48:49AM -0800, Guenter Roeck wrote:
> On 1/3/20 7:29 AM, Arnd Bergmann wrote:
> > On Fri, Jan 3, 2020 at 4:25 PM Arnd Bergmann <arnd@...db.de> wrote:
> > >
> > > On Fri, Jan 3, 2020 at 4:03 PM Naresh Kamboju <naresh.kamboju@...aro.org> wrote:
> > > >
> > > > On Fri, 3 Jan 2020 at 03:42, Greg Kroah-Hartman
> > > > <gregkh@...uxfoundation.org> wrote:
> > >
> > > -ENOENT is what you get when hugetlbfs is not mounted, so this hints to
> > >
> > > 8fc312b32b2 mm/hugetlbfs: fix error handling when setting up mounts
> > >
> > > https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/commit/?h=linux-5.4.y&id=3f549fb42a39bea3b29c0fc12afee53c4a01bec9
> >
> > I see that Mike Kravetz suggested not putting this patch into stable in
> >
> > https://lore.kernel.org/lkml/befca227-cb8a-8f47-617d-e3bf9972bfec@oracle.com/
> >
> > but it was picked through the autosel mechanism later.
> >
>
> I think autosel is way too aggressive. This is an excellent example.
Why? It fixes a bug, the text says so, and the code shows it. This is
a great example of a patch that autosel _should_ be picking up. Now the
fact that it happens to break existing functionality is not an
autosel-detectable thing. Especially as that same functionality is now
broken in Linus's tree :)
Autosel assumes that patches are correct, it can't know that they are
buggy. That should have been weeded out by the developers and testing
before they hit Linus's tree.
thanks,
greg k-h
Powered by blists - more mailing lists