Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 4 Nov 2021 17:52:42 +0100
From:   Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To:     Guenter Roeck <linux@roeck-us.net>
Cc:     Daniel =?iso-8859-1?Q?D=EDaz?= <daniel.diaz@linaro.org>,
        linux-kernel@vger.kernel.org, shuah@kernel.org,
        f.fainelli@gmail.com, patches@kernelci.org,
        lkft-triage@lists.linaro.org, jonathanh@nvidia.com,
        stable@vger.kernel.org, pavel@denx.de, akpm@linux-foundation.org,
        torvalds@linux-foundation.org
Subject: Re: [PATCH 5.10 00/16] 5.10.78-rc1 review
Message-ID: <YYQP2jGVxhMUlcMH@kroah.com>
References: <20211104141159.561284732@linuxfoundation.org>
 <3971a9b4-ebb6-a789-2143-31cf257d0d38@linaro.org>
 <YYQIUhHkv3kUY+UC@kroah.com>
 <acabc414-164b-cd65-6a1a-cf912d8621d7@roeck-us.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <acabc414-164b-cd65-6a1a-cf912d8621d7@roeck-us.net>
Precedence: bulk

On Thu, Nov 04, 2021 at 09:46:49AM -0700, Guenter Roeck wrote:
> On 11/4/21 9:20 AM, Greg Kroah-Hartman wrote:
> > On Thu, Nov 04, 2021 at 09:53:57AM -0600, Daniel D?az wrote:
> > > Hello!
> > > 
> > > On 11/4/21 8:12 AM, Greg Kroah-Hartman wrote:
> > > > This is the start of the stable review cycle for the 5.10.78 release.
> > > > There are 16 patches in this series, all will be posted as a response
> > > > to this one.  If anyone has any issues with these being applied, please
> > > > let me know.
> > > > 
> > > > Responses should be made by Sat, 06 Nov 2021 14:11:51 +0000.
> > > > Anything received after that time might be too late.
> > > > 
> > > > The whole patch series can be found in one patch at:
> > > > 	https://www.kernel.org/pub/linux/kernel/v5.x/stable-review/patch-5.10.78-rc1.gz
> > > > or in the git tree and branch at:
> > > > 	git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.10.y
> > > > and the diffstat can be found below.
> > > > 
> > > > thanks,
> > > > 
> > > > greg k-h
> > > 
> > > Regressions detected.
> > > 
> > > Build failures on all architectures and all toolchains (GCC 8, 9, 10, 11; Clang 10, 11, 12, 13, nightly):
> > > - arc
> > > - arm (32-bits)
> > > - arm (64-bits)
> > > - i386
> > > - mips
> > > - parisc
> > > - ppc
> > > - riscv
> > > - s390
> > > - sh
> > > - sparc
> > > - x86
> > > 
> > > Failures look like this:
> > > 
> > >    In file included from /builds/linux/include/linux/kernel.h:11,
> > >                     from /builds/linux/include/linux/list.h:9,
> > >                     from /builds/linux/include/linux/smp.h:12,
> > >                     from /builds/linux/include/linux/kernel_stat.h:5,
> > >                     from /builds/linux/mm/memory.c:42:
> > >    /builds/linux/mm/memory.c: In function 'finish_fault':
> > >    /builds/linux/mm/memory.c:3929:15: error: implicit declaration of function 'PageHasHWPoisoned'; did you mean 'PageHWPoison'? [-Werror=implicit-function-declaration]
> > >     3929 |  if (unlikely(PageHasHWPoisoned(page)))
> > >          |               ^~~~~~~~~~~~~~~~~
> > >    /builds/linux/include/linux/compiler.h:78:42: note: in definition of macro 'unlikely'
> > >       78 | # define unlikely(x) __builtin_expect(!!(x), 0)
> > >          |                                          ^
> > >    cc1: some warnings being treated as errors
> > > 
> > > and this:
> > > 
> > >    /builds/linux/mm/memory.c:3929:15: error: implicit declaration of function 'PageHasHWPoisoned' [-Werror,-Wimplicit-function-declaration]
> > >            if (unlikely(PageHasHWPoisoned(page)))
> > >                         ^
> > > 
> > >    /builds/linux/mm/page_alloc.c:1237:4: error: implicit declaration of function 'ClearPageHasHWPoisoned' [-Werror,-Wimplicit-function-declaration]
> > >                            ClearPageHasHWPoisoned(page);
> > >                            ^
> > >    /builds/linux/mm/page_alloc.c:1237:4: note: did you mean 'ClearPageHWPoison'?
> > > 
> > 
> > What configuration?  This builds for me on x86 here on allmodconfig.
> > 
> 
> defconfig, and anything with CONFIG_MEMORY_FAILURE=n or CONFIG_TRANSPARENT_HUGEPAGE=n.
> Fix needs upstream commit e66435936756d (presumably, I did not check).

Odd, no, I don't think that commit will help.

I'll go drop the offending commit now and push out a -rc2.

thanks,

greg k-h