stable.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Guillaume Tucker <guillaume.tucker@collabora.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-kernel@vger.kernel.org
Cc: torvalds@linux-foundation.org, akpm@linux-foundation.org,
	linux@roeck-us.net, shuah@kernel.org, patches@kernelci.org,
	lkft-triage@lists.linaro.org, pavel@denx.de,
	jonathanh@nvidia.com, f.fainelli@gmail.com,
	stable@vger.kernel.org, Suram Suram <suram@nxp.com>
Subject: Re: [PATCH 5.10 000/661] 5.10.20-rc2 review
Date: Tue, 2 Mar 2021 11:38:36 +0000	[thread overview]
Message-ID: <32a6c609-642c-71cf-0a84-d5e8ccd104b1@collabora.com> (raw)
In-Reply-To: <20210301193642.707301430@linuxfoundation.org>

On 01/03/2021 19:37, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 5.10.20 release.
> There are 661 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 Wed, 03 Mar 2021 19:34:53 +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.20-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.10.y
> and the diffstat can be found below.
> 
> thanks,
> 
> greg k-h


I've been through the KernelCI results for v5.10.20-rc2 and made
this manual reply, hoping to eventually get it all automated.



First there was one build regression with the arm
realview_defconfig:

kernel/rcu/tree.c:683:2: error: implicit declaration of function ‘IRQ_WORK_INIT’; did you mean ‘IRQMASK_I_BIT’? [-Werror=implicit-function-declaration]
  IRQ_WORK_INIT(late_wakeup_func);
  ^~~~~~~~~~~~~
  IRQMASK_I_BIT
kernel/rcu/tree.c:683:2: error: invalid initializer


Full log:

  https://storage.kernelci.org/stable-rc/linux-5.10.y/v5.10.19-662-g92929e15cdc0/arm/realview_defconfig/gcc-8/build.log


There were also a few new build warnings.  Here's a comparison of
the number of builds that completed with no warnings, with at
least one warning, and with an error between current stable and
stable-rc:

              pass  warn  error
v5.10.19      188      6      0  
v5.10.20-rc2  180     15      1

Full details for these 2 revisions respectively:

  https://kernelci.org/build/stable/branch/linux-5.10.y/kernel/v5.10.19/
  https://kernelci.org/build/stable-rc/branch/linux-5.10.y/kernel/v5.10.19-662-g92929e15cdc0/



Then on the runtime testing side, there was one boot regression
detected on imx8mp-evk as detailed here:

  https://kernelci.org/test/case/id/603d69ec2924db6b9baddcb2/

I've re-run a couple of tests with both v5.10.19 and v5.10.20-rc2
and also got a failure with v5.10.19, so it looks like it's not
really a new regression but more of an intermittent problem.
Bisections are not enabled in NXP's lab so we don't have results
about which commit caused it.  We should chase this up, I've
already asked if they're OK to enable bisection.  Then we may
bisect with an older revision that is really booting to find the
root cause...



Presumably it's not OK to have this build error in the v5.10.20
release, assuming the boot regression is not new and can be
ignored, but that's your call.  So it seems a bit early for
KernelCI to stamp it with Tested-by, even though it was tested
but it's more a matter of clarifying the semantics and whether
Tested-by implicitly means "works for me".  What do you think?

Best wishes,
Guillaume

  parent reply	other threads:[~2021-03-03  0:20 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-01 19:37 [PATCH 5.10 000/661] 5.10.20-rc2 review Greg Kroah-Hartman
2021-03-01 21:19 ` Pavel Machek
2021-03-04 10:39   ` Greg Kroah-Hartman
2021-03-01 21:43 ` Shuah Khan
2021-03-04 10:39   ` Greg Kroah-Hartman
2021-03-01 22:42 ` Florian Fainelli
2021-03-02  7:15 ` Naresh Kamboju
2021-03-02  9:50   ` Naresh Kamboju
2021-03-02  9:56     ` Greg Kroah-Hartman
2021-03-02 10:12       ` Naresh Kamboju
2021-03-02 10:15       ` Naresh Kamboju
2021-03-02  9:21 ` Jon Hunter
2021-03-04 10:39   ` Greg Kroah-Hartman
2021-03-02 11:38 ` Guillaume Tucker [this message]
2021-03-02 12:40   ` Greg Kroah-Hartman
2021-03-03 10:34     ` Guillaume Tucker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=32a6c609-642c-71cf-0a84-d5e8ccd104b1@collabora.com \
    --to=guillaume.tucker@collabora.com \
    --cc=akpm@linux-foundation.org \
    --cc=f.fainelli@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=jonathanh@nvidia.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=lkft-triage@lists.linaro.org \
    --cc=patches@kernelci.org \
    --cc=pavel@denx.de \
    --cc=shuah@kernel.org \
    --cc=stable@vger.kernel.org \
    --cc=suram@nxp.com \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).