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

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

[Anders Roxell]

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

metadata:
--------
* kernel: 5.15.165-rc2
* git: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
* git commit: aff234a5be72ef6e0a85246b992f6ec19cd59812
* git describe: v5.15.164-484-gaff234a5be72
* test details:
https://qa-reports.linaro.org/lkft/linux-stable-rc-linux-5.15.y/build/v5.15.164-484-gaff234a5be72
* arch: s390
* toolchain: gcc-12, clang
* config: https://storage.tuxsuite.com/public/linaro/lkft/builds/2kjlB8pPtrlqwxjAoqVX8sFQDpN/config
* download_url:
https://storage.tuxsuite.com/public/linaro/lkft/builds/2kjlB8pPtrlqwxjAoqVX8sFQDpN/

--
Linaro LKFT
https://lkft.linaro.org

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

[Greg Kroah-Hartman]

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