Messages in this thread | | | From | Punit Agrawal <> | Subject | Re: [PATCH v5] cper, apei, mce: Pass x86 CPER through the MCA handling chain | Date | Mon, 09 Nov 2020 10:18:46 +0900 |
| |
Borislav Petkov <bp@alien8.de> writes:
> On Fri, Nov 06, 2020 at 02:36:46PM +0900, Punit Agrawal wrote: >> > diff --git a/drivers/firmware/efi/cper-x86.c b/drivers/firmware/efi/cper-x86.c >> > index 2531de49f56c..438ed9eff6d0 100644 >> > --- a/drivers/firmware/efi/cper-x86.c >> > +++ b/drivers/firmware/efi/cper-x86.c >> > @@ -2,6 +2,7 @@ >> > // Copyright (C) 2018, Advanced Micro Devices, Inc. >> > >> > #include <linux/cper.h> >> > +#include <linux/acpi.h> >> >> Did you mean to include <asm/acpi.h>? > > Why?
Because arch_apei_report_x86_error() used in the patch is defined there. The indirect include works but pulls in additional definitions not needed by the patch.
Do you prefer the more generic include?
Thanks, Punit
| |