linux-kernel - Re: [PATCH 4.9 00/59] 4.9.226-rc2 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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Tue, 2 Jun 2020 12:33:46 -0400
From:   Sasha Levin <sashal@...nel.org>
To:     Guenter Roeck <linux@...ck-us.net>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
        akpm@...ux-foundation.org, shuah@...nel.org, patches@...nelci.org,
        ben.hutchings@...ethink.co.uk, lkft-triage@...ts.linaro.org,
        stable@...r.kernel.org
Subject: Re: [PATCH 4.9 00/59] 4.9.226-rc2 review

On Tue, Jun 02, 2020 at 08:37:26AM -0700, Guenter Roeck wrote:
>On 6/2/20 3:23 AM, Greg Kroah-Hartman wrote:
>> This is the start of the stable review cycle for the 4.9.226 release.
>> There are 59 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 Thu, 04 Jun 2020 10:16:52 +0000.
>> Anything received after that time might be too late.
>>
>
>Many arm builds still fail as attached. Is it really only me seeing this problem ?
>
>FWIW, if we need/want to use unified assembler in v4.9.y, shouldn't all unified
>assembler patches be applied ?

We don't - I took 71f8af111010 as a dependency rather than on its own
merit.

>$ git log --oneline v4.9..c001899a5d6 arch/arm | grep unified
>c001899a5d6c ARM: 8843/1: use unified assembler in headers
>a216376add73 ARM: 8841/1: use unified assembler in macros
>eb7ff9023e4f ARM: 8829/1: spinlock: use unified assembler language syntax
>32fdb046ac43 ARM: 8828/1: uaccess: use unified assembler language syntax
>1293c2b5d790 ARM: dts: berlin2q: add "cache-unified" to l2 node
>75fea300d73a ARM: 8723/2: always assume the "unified" syntax for assembly code
>
>I am quite concerned especially about missing commit 75fea300d73a,
>which removes the ARM_ASM_UNIFIED configuration option. That means it is
>still present in v4.9.y, but the failing builds don't enable it. Given that,
>the build failures don't seem to be surprising.

I'm just going to drop this series from 4.9 for now, until we can figure
out how to do it right.

-- 
Thanks,
Sasha