Re: [PATCH 4.9 00/59] 4.9.226-rc2 review

[Sasha Levin]

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