Re: [PATCH -next] x86/virt/tdx: Remove duplicated include in tdx.c

From: Huang, Kai
Date: Mon Dec 11 2023 - 20:05:56 EST


On Tue, 2023-12-12 at 08:38 +0800, Yang Li wrote:
> The header files linux/acpi.h and asm/cpufeature.h are both
> included twice in C, so one inclusion of each can be removed.
>
> Reported-by: Abaci Robot <abaci@xxxxxxxxxxxxxxxxx>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=7713
> Signed-off-by: Yang Li <yang.lee@xxxxxxxxxxxxxxxxx>

Reviewed-by: Kai Huang <kai.huang@xxxxxxxxx>

> ---
> arch/x86/virt/vmx/tdx/tdx.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/x86/virt/vmx/tdx/tdx.c b/arch/x86/virt/vmx/tdx/tdx.c
> index 13df68ef40b5..49a1c6890b55 100644
> --- a/arch/x86/virt/vmx/tdx/tdx.c
> +++ b/arch/x86/virt/vmx/tdx/tdx.c
> @@ -27,14 +27,12 @@
> #include <linux/log2.h>
> #include <linux/acpi.h>
> #include <linux/suspend.h>
> -#include <linux/acpi.h>
> #include <asm/page.h>
> #include <asm/special_insns.h>
> #include <asm/msr-index.h>
> #include <asm/msr.h>
> #include <asm/cpufeature.h>
> #include <asm/tdx.h>
> -#include <asm/cpufeature.h>
> #include <asm/intel-family.h>
> #include <asm/processor.h>
> #include <asm/mce.h>