* Re: [PATCH 4.14 00/21] 4.14.335-rc1 review
2024-01-05 14:38 [PATCH 4.14 00/21] 4.14.335-rc1 review Greg Kroah-Hartman
@ 2024-01-05 16:23 ` Jon Hunter
2024-01-06 8:39 ` Greg Kroah-Hartman
2024-01-05 17:13 ` Daniel Díaz
` (2 subsequent siblings)
3 siblings, 1 reply; 8+ messages in thread
From: Jon Hunter @ 2024-01-05 16:23 UTC (permalink / raw)
To: Greg Kroah-Hartman, stable
Cc: patches, linux-kernel, torvalds, akpm, linux, shuah, patches,
lkft-triage, pavel, f.fainelli, sudipm.mukherjee, srw, rwarsow,
conor, allen.lkml, linux-tegra@vger.kernel.org
Hi Greg,
On 05/01/2024 14:38, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.14.335 release.
> There are 21 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, 07 Jan 2024 14:38:02 +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/v4.x/stable-review/patch-4.14.335-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-4.14.y
> and the diffstat can be found below.
>
> thanks,
>
> greg k-h
>
> -------------
> Pseudo-Shortlog of commits:
...
> Alexis Lothoré <alexis.lothore@bootlin.com>
> pinctrl: at91-pio4: use dedicated lock class for IRQ
I am seeing the following build error for ARM ...
drivers/pinctrl/pinctrl-at91-pio4.c: In function ‘atmel_pinctrl_probe’:
drivers/pinctrl/pinctrl-at91-pio4.c:1054:3: error: too many arguments to function ‘irq_set_lockdep_class’
irq_set_lockdep_class(irq, &atmel_lock_key, &atmel_request_key);
^~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/irq.h:517:0,
from include/linux/gpio/driver.h:7,
from drivers/pinctrl/pinctrl-at91-pio4.c:18:
include/linux/irqdesc.h:250:1: note: declared here
irq_set_lockdep_class(unsigned int irq, struct lock_class_key *class)
^~~~~~~~~~~~~~~~~~~~~
I am guessing it is the above commit.
Jon
--
nvpublic
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH 4.14 00/21] 4.14.335-rc1 review
2024-01-05 16:23 ` Jon Hunter
@ 2024-01-06 8:39 ` Greg Kroah-Hartman
0 siblings, 0 replies; 8+ messages in thread
From: Greg Kroah-Hartman @ 2024-01-06 8:39 UTC (permalink / raw)
To: Jon Hunter
Cc: stable, patches, linux-kernel, torvalds, akpm, linux, shuah,
patches, lkft-triage, pavel, f.fainelli, sudipm.mukherjee, srw,
rwarsow, conor, allen.lkml, linux-tegra@vger.kernel.org
On Fri, Jan 05, 2024 at 04:23:00PM +0000, Jon Hunter wrote:
> Hi Greg,
>
> On 05/01/2024 14:38, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.14.335 release.
> > There are 21 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, 07 Jan 2024 14:38:02 +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/v4.x/stable-review/patch-4.14.335-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-4.14.y
> > and the diffstat can be found below.
> >
> > thanks,
> >
> > greg k-h
> >
> > -------------
> > Pseudo-Shortlog of commits:
>
> ...
>
>
> > Alexis Lothoré <alexis.lothore@bootlin.com>
> > pinctrl: at91-pio4: use dedicated lock class for IRQ
>
>
> I am seeing the following build error for ARM ...
>
> drivers/pinctrl/pinctrl-at91-pio4.c: In function ‘atmel_pinctrl_probe’:
> drivers/pinctrl/pinctrl-at91-pio4.c:1054:3: error: too many arguments to function ‘irq_set_lockdep_class’
> irq_set_lockdep_class(irq, &atmel_lock_key, &atmel_request_key);
> ^~~~~~~~~~~~~~~~~~~~~
> In file included from include/linux/irq.h:517:0,
> from include/linux/gpio/driver.h:7,
> from drivers/pinctrl/pinctrl-at91-pio4.c:18:
> include/linux/irqdesc.h:250:1: note: declared here
> irq_set_lockdep_class(unsigned int irq, struct lock_class_key *class)
> ^~~~~~~~~~~~~~~~~~~~~
>
> I am guessing it is the above commit.
Thanks, now dropped.
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 4.14 00/21] 4.14.335-rc1 review
2024-01-05 14:38 [PATCH 4.14 00/21] 4.14.335-rc1 review Greg Kroah-Hartman
2024-01-05 16:23 ` Jon Hunter
@ 2024-01-05 17:13 ` Daniel Díaz
2024-01-05 20:39 ` Alexis Lothoré
2024-01-05 21:32 ` Pavel Machek
2024-01-06 5:56 ` Harshit Mogalapalli
3 siblings, 1 reply; 8+ messages in thread
From: Daniel Díaz @ 2024-01-05 17:13 UTC (permalink / raw)
To: Greg Kroah-Hartman, stable
Cc: patches, linux-kernel, torvalds, akpm, linux, shuah, patches,
lkft-triage, pavel, jonathanh, f.fainelli, sudipm.mukherjee, srw,
rwarsow, conor, allen.lkml, alexis.lothore, liujian56
Hello!
On 05/01/24 8:38 a. m., Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.14.335 release.
> There are 21 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, 07 Jan 2024 14:38:02 +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/v4.x/stable-review/patch-4.14.335-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-4.14.y
> and the diffstat can be found below.
>
> thanks,
>
> greg k-h
We are seeing build problems on Arm32 and Arm64. On Arm32, for the following configurations:
* clang-17-keystone_defconfig
* clang-17-lkftconfig
* clang-17-mini2440_defconfig
* clang-17-sama5_defconfig
* gcc-8-defconfig
* gcc-8-sama5_defconfig
* gcc-12-defconfig
* gcc-12-lkftconfig
* gcc-12-lkftconfig-debug
* gcc-12-lkftconfig-debug-kmemleak
* gcc-12-lkftconfig-kasan
* gcc-12-lkftconfig-kselftest-kernel
* gcc-12-lkftconfig-kunit
* gcc-12-lkftconfig-libgpiod
* gcc-12-lkftconfig-rcutorture
* gcc-12-sama5_defconfig
Failure looks like this:
-----8<-----
/builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c: In function 'atmel_pinctrl_probe':
/builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c:1054:3: error: too many arguments to function 'irq_set_lockdep_class'
irq_set_lockdep_class(irq, &atmel_lock_key, &atmel_request_key);
^~~~~~~~~~~~~~~~~~~~~
In file included from /builds/linux/include/linux/irq.h:517,
from /builds/linux/include/linux/gpio/driver.h:7,
from /builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c:18:
/builds/linux/include/linux/irqdesc.h:250:1: note: declared here
irq_set_lockdep_class(unsigned int irq, struct lock_class_key *class)
^~~~~~~~~~~~~~~~~~~~~
make[3]: *** [/builds/linux/scripts/Makefile.build:330: drivers/pinctrl/pinctrl-at91-pio4.o] Error 1
----->8-----
Bisection points to:
commit 6c47537ba008b3affe99360f65592ff5b797818f
Author: Alexis Lothoré <alexis.lothore@bootlin.com>
Date: Fri Dec 15 22:34:24 2023 +0100
pinctrl: at91-pio4: use dedicated lock class for IRQ
[ Upstream commit 14694179e561b5f2f7e56a0f590e2cb49a9cc7ab ]
Reproducer:
tuxmake --runtime podman --target-arch arm --toolchain clang-17 --kconfig defconfig LLVM_IAS=0
Then on Arm64 with Clang 17:
-----8<-----
/builds/linux/net/8021q/vlan_core.c:332:39: error: incompatible integer to pointer conversion passing '__be16' (aka 'unsigned short') to parameter of type 'const struct vlan_vi
d_info *' [-Wint-conversion]
332 | if (!vlan_hw_filter_capable(by_dev, vid_info->proto))
| ^~~~~~~~~~~~~~~
/builds/linux/net/8021q/vlan_core.c:169:38: note: passing argument to parameter 'vid_info' here
169 | const struct vlan_vid_info *vid_info)
| ^
/builds/linux/net/8021q/vlan_core.c:344:39: error: incompatible integer to pointer conversion passing '__be16' (aka 'unsigned short') to parameter of type 'const struct vlan_vi
d_info *' [-Wint-conversion]
344 | if (!vlan_hw_filter_capable(by_dev, vid_info->proto))
| ^~~~~~~~~~~~~~~
/builds/linux/net/8021q/vlan_core.c:169:38: note: passing argument to parameter 'vid_info' here
169 | const struct vlan_vid_info *vid_info)
| ^
/builds/linux/net/8021q/vlan_core.c:366:39: error: incompatible integer to pointer conversion passing '__be16' (aka 'unsigned short') to parameter of type 'const struct vlan_vid_info *' [-Wint-conversion]
366 | if (!vlan_hw_filter_capable(by_dev, vid_info->proto))
| ^~~~~~~~~~~~~~~
/builds/linux/net/8021q/vlan_core.c:169:38: note: passing argument to parameter 'vid_info' here
169 | const struct vlan_vid_info *vid_info)
| ^
3 errors generated.
make[3]: *** [/builds/linux/scripts/Makefile.build:329: net/8021q/vlan_core.o] Error 1
----->8-----
Bisection in this case points to:
commit f33474044777425b3578d4e8b22a003f034ae214
Author: Liu Jian <liujian56@huawei.com>
Date: Sat Dec 16 15:52:18 2023 +0800
net: check vlan filter feature in vlan_vids_add_by_dev() and vlan_vids_del_by_dev()
[ Upstream commit 01a564bab4876007ce35f312e16797dfe40e4823 ]
Reproducer:
tuxmake --runtime podman --target-arch arm64 --toolchain clang-17 --kconfig defconfig LD=ld.lld LLVM_IAS=0
Reported-by: Linux Kernel Functional Testing <lkft@linaro.org>
Greetings!
Daniel Díaz
daniel.diaz@linaro.org
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH 4.14 00/21] 4.14.335-rc1 review
2024-01-05 17:13 ` Daniel Díaz
@ 2024-01-05 20:39 ` Alexis Lothoré
2024-01-06 8:39 ` Greg Kroah-Hartman
0 siblings, 1 reply; 8+ messages in thread
From: Alexis Lothoré @ 2024-01-05 20:39 UTC (permalink / raw)
To: Daniel Díaz, Greg Kroah-Hartman, stable
Cc: patches, linux-kernel, torvalds, akpm, linux, shuah, patches,
lkft-triage, pavel, jonathanh, f.fainelli, sudipm.mukherjee, srw,
rwarsow, conor, allen.lkml, liujian56
Hello,
On 1/5/24 18:13, Daniel Díaz wrote:
> Hello!
>
> On 05/01/24 8:38 a. m., Greg Kroah-Hartman wrote:
[...]
> Failure looks like this:
>
> -----8<-----
> /builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c: In function
> 'atmel_pinctrl_probe':
> /builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c:1054:3: error: too many
> arguments to function 'irq_set_lockdep_class'
> irq_set_lockdep_class(irq, &atmel_lock_key, &atmel_request_key);
> ^~~~~~~~~~~~~~~~~~~~~
> In file included from /builds/linux/include/linux/irq.h:517,
> from /builds/linux/include/linux/gpio/driver.h:7,
> from /builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c:18:
> /builds/linux/include/linux/irqdesc.h:250:1: note: declared here
> irq_set_lockdep_class(unsigned int irq, struct lock_class_key *class)
> ^~~~~~~~~~~~~~~~~~~~~
> make[3]: *** [/builds/linux/scripts/Makefile.build:330:
> drivers/pinctrl/pinctrl-at91-pio4.o] Error 1
> ----->8-----
>
> Bisection points to:
>
> commit 6c47537ba008b3affe99360f65592ff5b797818f
> Author: Alexis Lothoré <alexis.lothore@bootlin.com>
> Date: Fri Dec 15 22:34:24 2023 +0100
>
> pinctrl: at91-pio4: use dedicated lock class for IRQ
> [ Upstream commit 14694179e561b5f2f7e56a0f590e2cb49a9cc7ab ]
This fails because linux-4.14.y is older than 39c3fd58952d: kernel/irq: Extend
lockdep class for request mutex, which updates irq_set_lockdep_class API.
Looking at irq_set_lockdep_class update patch, what appears to be a simple fix is
to simply remove the last parameter from irq_set_lockdep_class call in
pinctrl-at91-pio4.c, but I am not even sure if the fix remains legitimate in this
case (ie if the corresponding lockdep splat reappears), and I do not have access
to corresponding hardware before the deadline, so if that's not already the plan,
I suggest to drop this from 4.14 branch.
Alexis
--
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 4.14 00/21] 4.14.335-rc1 review
2024-01-05 20:39 ` Alexis Lothoré
@ 2024-01-06 8:39 ` Greg Kroah-Hartman
0 siblings, 0 replies; 8+ messages in thread
From: Greg Kroah-Hartman @ 2024-01-06 8:39 UTC (permalink / raw)
To: Alexis Lothoré
Cc: Daniel Díaz, stable, patches, linux-kernel, torvalds, akpm,
linux, shuah, patches, lkft-triage, pavel, jonathanh, f.fainelli,
sudipm.mukherjee, srw, rwarsow, conor, allen.lkml, liujian56
On Fri, Jan 05, 2024 at 09:39:58PM +0100, Alexis Lothoré wrote:
> Hello,
>
> On 1/5/24 18:13, Daniel Díaz wrote:
> > Hello!
> >
> > On 05/01/24 8:38 a. m., Greg Kroah-Hartman wrote:
>
> [...]
>
> > Failure looks like this:
> >
> > -----8<-----
> > /builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c: In function
> > 'atmel_pinctrl_probe':
> > /builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c:1054:3: error: too many
> > arguments to function 'irq_set_lockdep_class'
> > irq_set_lockdep_class(irq, &atmel_lock_key, &atmel_request_key);
> > ^~~~~~~~~~~~~~~~~~~~~
> > In file included from /builds/linux/include/linux/irq.h:517,
> > from /builds/linux/include/linux/gpio/driver.h:7,
> > from /builds/linux/drivers/pinctrl/pinctrl-at91-pio4.c:18:
> > /builds/linux/include/linux/irqdesc.h:250:1: note: declared here
> > irq_set_lockdep_class(unsigned int irq, struct lock_class_key *class)
> > ^~~~~~~~~~~~~~~~~~~~~
> > make[3]: *** [/builds/linux/scripts/Makefile.build:330:
> > drivers/pinctrl/pinctrl-at91-pio4.o] Error 1
> > ----->8-----
> >
> > Bisection points to:
> >
> > commit 6c47537ba008b3affe99360f65592ff5b797818f
> > Author: Alexis Lothoré <alexis.lothore@bootlin.com>
> > Date: Fri Dec 15 22:34:24 2023 +0100
> >
> > pinctrl: at91-pio4: use dedicated lock class for IRQ
> > [ Upstream commit 14694179e561b5f2f7e56a0f590e2cb49a9cc7ab ]
>
> This fails because linux-4.14.y is older than 39c3fd58952d: kernel/irq: Extend
> lockdep class for request mutex, which updates irq_set_lockdep_class API.
>
> Looking at irq_set_lockdep_class update patch, what appears to be a simple fix is
> to simply remove the last parameter from irq_set_lockdep_class call in
> pinctrl-at91-pio4.c, but I am not even sure if the fix remains legitimate in this
> case (ie if the corresponding lockdep splat reappears), and I do not have access
> to corresponding hardware before the deadline, so if that's not already the plan,
> I suggest to drop this from 4.14 branch.
Now dropped, thanks.
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 4.14 00/21] 4.14.335-rc1 review
2024-01-05 14:38 [PATCH 4.14 00/21] 4.14.335-rc1 review Greg Kroah-Hartman
2024-01-05 16:23 ` Jon Hunter
2024-01-05 17:13 ` Daniel Díaz
@ 2024-01-05 21:32 ` Pavel Machek
2024-01-06 5:56 ` Harshit Mogalapalli
3 siblings, 0 replies; 8+ messages in thread
From: Pavel Machek @ 2024-01-05 21:32 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: stable, patches, linux-kernel, torvalds, akpm, linux, shuah,
patches, lkft-triage, pavel, jonathanh, f.fainelli,
sudipm.mukherjee, srw, rwarsow, conor, allen.lkml
[-- Attachment #1: Type: text/plain, Size: 661 bytes --]
Hi!
> This is the start of the stable review cycle for the 4.14.335 release.
> There are 21 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.
CIP testing did not find any problems here:
https://gitlab.com/cip-project/cip-testing/linux-stable-rc-ci/-/tree/linux-4.14.y
Tested-by: Pavel Machek (CIP) <pavel@denx.de>
Best regards,
Pavel
--
DENX Software Engineering GmbH, Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH 4.14 00/21] 4.14.335-rc1 review
2024-01-05 14:38 [PATCH 4.14 00/21] 4.14.335-rc1 review Greg Kroah-Hartman
` (2 preceding siblings ...)
2024-01-05 21:32 ` Pavel Machek
@ 2024-01-06 5:56 ` Harshit Mogalapalli
3 siblings, 0 replies; 8+ messages in thread
From: Harshit Mogalapalli @ 2024-01-06 5:56 UTC (permalink / raw)
To: Greg Kroah-Hartman, stable
Cc: patches, linux-kernel, torvalds, akpm, linux, shuah, patches,
lkft-triage, pavel, jonathanh, f.fainelli, sudipm.mukherjee, srw,
rwarsow, conor, allen.lkml, Vegard Nossum, Darren Kenny
Hi Greg,
On 05/01/24 8:08 pm, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.14.335 release.
> There are 21 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, 07 Jan 2024 14:38:02 +0000.
> Anything received after that time might be too late.
>
No problems seen on x86_64 and aarch64 with our testing.
Tested-by: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Thanks,
Harshit
> The whole patch series can be found in one patch at:
> https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.14.335-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-4.14.y
> and the diffstat can be found below.
>
> thanks,
>
> greg k-h
> fix a memleak in atl1e_setup_ring_resources
>
> Eric Dumazet <edumazet@google.com>
> net: sched: ife: fix potential use-after-free
>
> Johannes Berg <johannes.berg@intel.com>
> wifi: mac80211: mesh_plink: fix matches_local logic
>
> Heiko Carstens <hca@linux.ibm.com>
> s390/vx: fix save/restore of fpu kernel context
>
> Kunwu Chan <chentao@kylinos.cn>
> ARM: OMAP2+: Fix null pointer dereference and memory leak in omap_soc_device_init
>
^ permalink raw reply [flat|nested] 8+ messages in thread