RE: [tip:x86/hyperv] x86/Hyper-V/hv_apic: Include asm/apic.h

From: KY Srinivasan
Date: Sun May 20 2018 - 13:00:18 EST




> -----Original Message-----
> From: tip tree robot <tipbot@xxxxxxxxx>
> Sent: Saturday, May 19, 2018 7:46 AM
> To: linux-tip-commits@xxxxxxxxxxxxxxx
> Cc: hpa@xxxxxxxxx; Michael Kelley (EOSG)
> <Michael.H.Kelley@xxxxxxxxxxxxx>; KY Srinivasan <kys@xxxxxxxxxxxxx>;
> tglx@xxxxxxxxxxxxx; mingo@xxxxxxxxxx; lkp@xxxxxxxxx
> Subject: [tip:x86/hyperv] x86/Hyper-V/hv_apic: Include asm/apic.h
>
> Commit-ID: 61eeb1f6d1f2648a218855d7c8d44f16df242ef3
> Gitweb:
> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.ker
> nel.org%2Ftip%2F61eeb1f6d1f2648a218855d7c8d44f16df242ef3&data=02%7
> C01%7Ckys%40microsoft.com%7C9469b1fea91344d65b9f08d5bd973c7c%7C7
> 2f988bf86f141af91ab2d7cd011db47%7C1%7C1%7C636623379628726325&sdat
> a=mbprYiPRtCsbYHGm%2BMsmUUt4MGGMFloyTN5Y40JrnKg%3D&reserved
> =0
> Author: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> AuthorDate: Sat, 19 May 2018 16:38:59 +0200
> Committer: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> CommitDate: Sat, 19 May 2018 16:41:59 +0200
>
> x86/Hyper-V/hv_apic: Include asm/apic.h
>
> Not all configurations magically include asm/apic.h, but the Hyper-V code
> requires it. Include it explicitely.

Thanks Thomas. I was planning to send this patch today after I saw the kbuild robot mail.

Regards,

K. Y
>
> Fixes: 6b48cb5f8347 ("X86/Hyper-V: Enlighten APIC access")
> Reported-by: kbuild test robot <lkp@xxxxxxxxx>
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
> Cc: Michael Kelley <mikelley@xxxxxxxxxxxxx>
> ---
> arch/x86/hyperv/hv_apic.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/x86/hyperv/hv_apic.c b/arch/x86/hyperv/hv_apic.c
> index 192b6ad6a361..d3ff6e255924 100644
> --- a/arch/x86/hyperv/hv_apic.c
> +++ b/arch/x86/hyperv/hv_apic.c
> @@ -29,6 +29,7 @@
> #include <linux/cpuhotplug.h>
> #include <asm/hypervisor.h>
> #include <asm/mshyperv.h>
> +#include <asm/apic.h>
>
> #ifdef CONFIG_X86_64
> #if IS_ENABLED(CONFIG_HYPERV)