Re: [PATCH 1/6] RAS: ACPI: APEI: add conditional compilation to ARM error report functions

From: Mauro Carvalho Chehab
Date: Thu Jul 11 2024 - 01:27:07 EST


Em Mon, 8 Jul 2024 16:43:12 +0200
Borislav Petkov <bp@xxxxxxxxx> escreveu:

> On Mon, Jul 08, 2024 at 02:10:25PM +0200, Mauro Carvalho Chehab wrote:
> > This patch itself just add conditionals to optimize out code on
> > non-ARM architectures. The next one will add some ARM-specific bits
> > inside ARM processor CPER trace, thus causing compilation breakages
> > on non-ARM, due to arm-specific kAPI bits that will be used then.
>
> Are you sure?

That is what reviews to past attempts to merge patch 2 implied.

> I have both patches applied and then practically reverting the second one
> builds an allmodconfig just fine.

I double-checked the logic: I noticed just one kABI symbol that
it is arm-specific (CPU midr), and there is has already a wrapper
for it.

I also did a cross-compilation for both x86_64 and s390 to verify,
and indeed it is building fine without the ifdefs.

So, I'll drop patch 1.

Thanks,
Mauro