Re: [PATCH] perf kvm stat: Fix build error

[Namhyung Kim <namhyung@kernel.org>]

Hi Ian,

On Fri, Feb 06, 2026 at 11:00:20AM +0000, Leo Yan wrote:
> Since commit ceea279f9376 ("perf kvm stat: Remove use of the arch
> directory"), a native build on Arm64 machine reports:
> 
>   util/kvm-stat-arch/kvm-stat-x86.c:7:10: fatal error: asm/svm.h: No such file or directory
>     7 | #include <asm/svm.h>
>       |          ^~~~~~~~~~~
>   compilation terminated.
> 
> The build fails to find x86's asm headers when building for Arm64.  Fix
> this by including asm headers with relative path instead.
> 
> Fixes: ceea279f9376 ("perf kvm stat: Remove use of the arch directory")
> Signed-off-by: Leo Yan <leo.yan@arm.com>

Probably not from this patch, but this change seems to introduce this
error on my i386.

  In file included from util/kvm-stat-arch/kvm-stat-x86.c:4:                      
  /linux/tools/include/../../arch/x86/include/uapi/asm/svm.h:137:32: error: conversion from 'long long unsigned int' to 'long unsigned int' changes value from '18446744073709551615' to '4294967295' [-Werror=overflow]
    137 | #define SVM_EXIT_ERR           -1ull                                    
        |                                ^                                        
  util/kvm-stat-arch/kvm-stat-x86.c:13:1: note: in expansion of macro 'define_exit_reasons_table'
     13 | define_exit_reasons_table(svm_exit_reasons, SVM_EXIT_REASONS);          
        | ^~~~~~~~~~~~~~~~~~~~~~~~~                                               
  /linux/tools/include/../../arch/x86/include/uapi/asm/svm.h:249:11: note: in expansion of macro 'SVM_EXIT_ERR'
    249 |         { SVM_EXIT_ERR,         "invalid_guest_state" }                 
        |           ^~~~~~~~~~~~                                                  
  util/kvm-stat-arch/kvm-stat-x86.c:13:45: note: in expansion of macro 'SVM_EXIT_REASONS'
     13 | define_exit_reasons_table(svm_exit_reasons, SVM_EXIT_REASONS);          
        |                                             ^~~~~~~~~~~~~~~~     

Thanks,
Namhyung

> ---
>  tools/perf/util/kvm-stat-arch/kvm-stat-x86.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/perf/util/kvm-stat-arch/kvm-stat-x86.c b/tools/perf/util/kvm-stat-arch/kvm-stat-x86.c
> index 1cf541385a4bdee4930b67a054906ccade7301fd..43275d25b6cbccc477ba45a901d69c67422847ae 100644
> --- a/tools/perf/util/kvm-stat-arch/kvm-stat-x86.c
> +++ b/tools/perf/util/kvm-stat-arch/kvm-stat-x86.c
> @@ -4,9 +4,9 @@
>  #include "../kvm-stat.h"
>  #include "../evsel.h"
>  #include "../env.h"
> -#include <asm/svm.h>
> -#include <asm/vmx.h>
> -#include <asm/kvm.h>
> +#include "../../arch/x86/include/uapi/asm/svm.h"
> +#include "../../arch/x86/include/uapi/asm/vmx.h"
> +#include "../../arch/x86/include/uapi/asm/kvm.h"
>  #include <subcmd/parse-options.h>
>  
>  define_exit_reasons_table(vmx_exit_reasons, VMX_EXIT_REASONS);
> 
> ---
> base-commit: 84cb36da81413c2dff805150b9f4db1524460269
> change-id: 20260206-perf_fix_kvm_stat_error-a795a1dd67a2
> 
> Best regards,
> -- 
> Leo Yan <leo.yan@arm.com>
>