Re: [PATCH 5.15 000/483] 5.15.165-rc2 review

[Greg Kroah-Hartman <gregkh@linuxfoundation.org>]

On Sat, Aug 17, 2024 at 07:55:42AM +0200, Anders Roxell wrote:
> On Fri, 16 Aug 2024 at 12:22, Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> >
> > This is the start of the stable review cycle for the 5.15.165 release.
> > There are 483 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 Sun, 18 Aug 2024 10:14:00 +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.15.165-rc2.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.15.y
> > and the diffstat can be found below.
> >
> > thanks,
> >
> > greg k-h
> 
> The following S390 build failed on stable-rc 5.15.y with gcc-12 and clang due
> to following warnings and errors [1].
> 
> s390:
>   build:
>     * gcc-8-defconfig-fe40093d
>     * gcc-12-defconfig
>     * clang-18-defconfig
> 
> 
> Reported-by: Linux Kernel Functional Testing <lkft@linaro.org>
> 
> The bisect points to 85cf9455e504 ("KVM: s390: pv: avoid stalls when
> making pages secure")
> as the problematic commit [ Upstream commit
> f0a1a0615a6ff6d38af2c65a522698fb4bb85df6 ].
> 
> Build log:
> --------
> /builds/linux/arch/s390/kernel/uv.c: In function 'make_page_secure':
> /builds/linux/arch/s390/kernel/uv.c:219:19: error: 'UVC_CC_OK'
> undeclared (first use in this function)
>   219 |         if (cc == UVC_CC_OK)
>       |                   ^~~~~~~~~
> /builds/linux/arch/s390/kernel/uv.c:219:19: note: each undeclared
> identifier is reported only once for each function it appears in
> /builds/linux/arch/s390/kernel/uv.c:221:24: error: 'UVC_CC_BUSY'
> undeclared (first use in this function); did you mean 'SIGP_CC_BUSY'?
>   221 |         else if (cc == UVC_CC_BUSY || cc == UVC_CC_PARTIAL)
>       |                        ^~~~~~~~~~~
>       |                        SIGP_CC_BUSY
> /builds/linux/arch/s390/kernel/uv.c:221:45: error: 'UVC_CC_PARTIAL'
> undeclared (first use in this function)
>   221 |         else if (cc == UVC_CC_BUSY || cc == UVC_CC_PARTIAL)
>       |                                             ^~~~~~~~~~~~~~
> /builds/linux/arch/s390/kernel/uv.c: In function 'should_export_before_import':
> /builds/linux/arch/s390/kernel/uv.c:249:40: error: 'mm_context_t' has
> no member named 'protected_count'
>   249 |         return atomic_read(&mm->context.protected_count) > 1;
>       |                                        ^
> /builds/linux/arch/s390/kernel/uv.c:250:1: error: control reaches end
> of non-void function [-Werror=return-type]
>   250 | }
>       | ^
> cc1: some warnings being treated as errors
> 
> 
> Build log link:
> --------
> [1] https://qa-reports.linaro.org/lkft/linux-stable-rc-linux-5.15.y/build/v5.15.164-484-gaff234a5be72/testrun/24890486/suite/build/test/gcc-12-defconfig/log

I'll go fix this up and push out a rc3, thanks.

greg k-h