Re: [PATCH] x86/kvm: Include linux/types.h in asm/kvm_para.h

[Sean Christopherson <seanjc@google.com>]

On Tue, May 05, 2026, WEI-HONG, YE wrote:
> arch/x86/include/asm/kvm_para.h uses u32 in function declarations, but
> does not include linux/types.h itself. One user, mtrr.c, carried an
> explicit workaround for this dependency.
> 
> Include linux/types.h from asm/kvm_para.h and remove the workaround from
> mtrr.c.
> 
> Build-tested with x86_64 defconfig.
> 
> Signed-off-by: WEI-HONG, YE <1234567weewee457@gmail.com>
> ---
>  arch/x86/include/asm/kvm_para.h | 2 ++
>  arch/x86/kernel/cpu/mtrr/mtrr.c | 2 --
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/x86/include/asm/kvm_para.h b/arch/x86/include/asm/kvm_para.h
> index 4a47c16e2df8..26e86db37b05 100644
> --- a/arch/x86/include/asm/kvm_para.h
> +++ b/arch/x86/include/asm/kvm_para.h
> @@ -2,6 +2,8 @@
>  #ifndef _ASM_X86_KVM_PARA_H
>  #define _ASM_X86_KVM_PARA_H
>  
> +#include <linux/types.h>
> +
>  #include <asm/processor.h>
>  #include <asm/alternative.h>
>  #include <linux/interrupt.h>
> diff --git a/arch/x86/kernel/cpu/mtrr/mtrr.c b/arch/x86/kernel/cpu/mtrr/mtrr.c
> index 4b3d492afe17..cdba5bf4a51d 100644
> --- a/arch/x86/kernel/cpu/mtrr/mtrr.c
> +++ b/arch/x86/kernel/cpu/mtrr/mtrr.c
> @@ -18,8 +18,6 @@
>      System Programming Guide; Section 9.11. (1997 edition - PPro).
>  */
>  
> -#include <linux/types.h> /* FIXME: kvm_para.h needs this */
> -
>  #include <linux/stop_machine.h>
>  #include <linux/kvm_para.h>

Just drop the kvm_para.h include, it has been needed since commit bf3647c44bc7
("x86: tone down mtrr_trim_uncached_memory() warning") from 2009.  I.e.

diff --git a/arch/x86/kernel/cpu/mtrr/mtrr.c b/arch/x86/kernel/cpu/mtrr/mtrr.c
index 4b3d492afe17..db5bf4adfd28 100644
--- a/arch/x86/kernel/cpu/mtrr/mtrr.c
+++ b/arch/x86/kernel/cpu/mtrr/mtrr.c
@@ -18,10 +18,7 @@
     System Programming Guide; Section 9.11. (1997 edition - PPro).
 */
 
-#include <linux/types.h> /* FIXME: kvm_para.h needs this */
-
 #include <linux/stop_machine.h>
-#include <linux/kvm_para.h>
 #include <linux/uaccess.h>
 #include <linux/export.h>
 #include <linux/mutex.h>