linux-kernel - Re: [PATCH 6.6 00/77] 6.6.85-rc1 review

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0001e7e020145983055d154488f5c460@manjaro.org>
Date: Wed, 26 Mar 2025 04:56:18 +0100
From: Dragan Simic <dsimic@...jaro.org>
To: Harshit Mogalapalli <harshit.m.mogalapalli@...cle.com>
Cc: Naresh Kamboju <naresh.kamboju@...aro.org>, Greg Kroah-Hartman
 <gregkh@...uxfoundation.org>, stable@...r.kernel.org,
 patches@...ts.linux.dev, linux-kernel@...r.kernel.org,
 torvalds@...ux-foundation.org, akpm@...ux-foundation.org,
 linux@...ck-us.net, shuah@...nel.org, patches@...nelci.org,
 lkft-triage@...ts.linaro.org, pavel@...x.de, jonathanh@...dia.com,
 f.fainelli@...il.com, sudipm.mukherjee@...il.com, srw@...dewatkins.net,
 rwarsow@....de, conor@...nel.org, hargar@...rosoft.com, broonie@...nel.org,
 Heiko Stuebner <heiko@...ech.de>, Vegard Nossum <vegard.nossum@...cle.com>,
 Darren Kenny <darren.kenny@...cle.com>, Diederik de Haas
 <didi.debian@...ow.org>
Subject: Re: [PATCH 6.6 00/77] 6.6.85-rc1 review

Hello Harshit and Greg,

On 2025-03-26 03:33, Harshit Mogalapalli wrote:
> On 25/03/25 21:37, Dragan Simic wrote:
>> On 2025-03-25 16:07, Naresh Kamboju wrote:
> ...
>>> Build regression: arm64 dtb rockchip non-existent node or label 
>>> "vcca_0v9"
>>> Reported-by: Linux Kernel Functional Testing <lkft@...aro.org>
>>> 
>>> ## Build log
>>> arch/arm64/boot/dts/rockchip/rk3399.dtsi:221.23-266.4: ERROR
>>> (phandle_references):
>>> /pcie@...00000: Reference to non-existent node or label "vcca_0v9"
>>> 
>>>   also defined at arch/arm64/boot/dts/rockchip/rk3399- 
>>> rockpro64.dtsi:659.8-669.3
> ...
>> 
>> This is caused by another patch from the original series failing
>> to apply due to some bulk regulator renaming.  I'll send backported
>> version of that patch soon, which should make everything fine.
> 
> On ARM configs, we do see the same issue that Naresh reported.
> 
> arch/arm64/boot/dts/rockchip/rk3399.dtsi:221.23-266.4: ERROR
> (phandle_references): /pcie@...00000: Reference to non-existent node
> or label "vcca_0v9"
>  also defined at 
> arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi:659.8-669.3
> ERROR: Input tree has errors, aborting (use -f to force output)
> make[3]: *** [scripts/Makefile.lib:423:
> arch/arm64/boot/dts/rockchip/rk3399-rockpro64-v2.dtb] Error 2
> make[3]: *** Waiting for unfinished jobs....
> arch/arm64/boot/dts/rockchip/rk3399.dtsi:221.23-266.4: ERROR
> (phandle_references): /pcie@...00000: Reference to non-existent node
> or label "vcca_0v9"
>  also defined at 
> arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtsi:659.8-669.3
> ERROR: Input tree has errors, aborting (use -f to force output)
> make[3]: *** [scripts/Makefile.lib:423:
> arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dtb] Error 2
> make[2]: *** [scripts/Makefile.build:480: arch/arm64/boot/dts/rockchip] 
> Error 2
> make[2]: *** Waiting for unfinished jobs....
> 
> Caused by commit: 1e4bd0ec5a47 ("arm64: dts: rockchip: Add missing
> PCIe supplies to RockPro64 board dtsi") -- PATCH 42/77 of this series.
> 
> We see same problem with 6.12.21-rc1 as well.
> 
> Notes:
> -----
> I think Dragan was referring to upstream commit: bd1c959f37f3 ("arm64:
> dts: rockchip: Add avdd HDMI supplies to RockPro64 board dtsi") which
> will fix this problem but fails to apply due to regulator renaming in
> commit: 5c96e6330197 ("arm64: dts: rockchip: adapt regulator nodenames
> to preferred form") which is not in stable kernels(6.6.y and 6.12.y)

Exactly, the direct dependency is the commit bd1c959f37f3 ("arm64:
dts: rockchip: Add avdd HDMI supplies to RockPro64 board dtsi"),
for which I haven't sent the backported version yet.

As Diederik pointed it out already in a separate message [*] from
a couple of days ago, it might be the best to include the commit
5c96e6330197 ("arm64: dts: rockchip: adapt regulator nodenames to
preferred form") into stable kernels as well.

Obviously, including the commit 5c96e6330197 pretty much goes
against the rules of stable kernels, but it would save a lot of
time and effort in the future.

[*] https://lore.kernel.org/stable/D8ONE4WEF7A2.1OE1YY8J34MM3@cknow.org/