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

WEI-HONG, YE posted 1 patch 1 month, 1 week ago
arch/x86/include/asm/kvm_para.h | 2 ++
arch/x86/kernel/cpu/mtrr/mtrr.c | 2 --
2 files changed, 2 insertions(+), 2 deletions(-)
[PATCH] x86/kvm: Include linux/types.h in asm/kvm_para.h
Posted by WEI-HONG, YE 1 month, 1 week ago
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>
 #include <linux/uaccess.h>
-- 
2.48.1
Re: [PATCH] x86/kvm: Include linux/types.h in asm/kvm_para.h
Posted by Thomas Huth 1 month, 1 week ago
On 05/05/2026 08.58, 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(-)

Reviewed-by: Thomas Huth <thuth@redhat.com>