Re: [PATCH] tools headers: Skip arm64 cputype.h check

[Mark Rutland]

On Mon, Mar 16, 2026 at 10:57:55AM +0000, Leo Yan wrote:
> Some definitions in the arm64 kernel's cputype.h are kernel specific and
> cause perf build failures when the header is synced into tools.
> 
> Stop checking arm64's cputype.h.  In the future, the header in tools
> will be updated manually as needed.
> 
> Signed-off-by: Leo Yan <leo.yan@xxxxxxx>

I think it might be worth stating explicitly that we only need to update
this header when teaching the perf tool about new CPUs, but with or
without that wording:

Acked-by: Mark Rutland <mark.rutland@xxxxxxx>

Mark.

> ---
>  tools/perf/check-headers.sh | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/tools/perf/check-headers.sh b/tools/perf/check-headers.sh
> index da3aca87457fdc56c96d4184f2dee88dab026d98..31826621eebdb003d0fd681ae89822225dda966f 100755
> --- a/tools/perf/check-headers.sh
> +++ b/tools/perf/check-headers.sh
> @@ -187,7 +187,6 @@ done
>  check arch/x86/lib/memcpy_64.S        '-I "^EXPORT_SYMBOL" -I "^#include <asm/export.h>" -I"^SYM_FUNC_START\(_LOCAL\)*(memcpy_\(erms\|orig\))" -I"^#include <linux/cfi_types.h>"'
>  check arch/x86/lib/memset_64.S        '-I "^EXPORT_SYMBOL" -I "^#include <asm/export.h>" -I"^SYM_FUNC_START\(_LOCAL\)*(memset_\(erms\|orig\))"'
>  check arch/x86/include/asm/amd/ibs.h  '-I "^#include .*/msr-index.h"'
> -check arch/arm64/include/asm/cputype.h '-I "^#include [<\"]\(asm/\)*sysreg.h"'
>  check include/linux/unaligned.h '-I "^#include <linux/unaligned/packed_struct.h>" -I "^#include <asm/byteorder.h>" -I "^#pragma GCC diagnostic"'
>  check include/uapi/asm-generic/mman.h '-I "^#include <\(uapi/\)*asm-generic/mman-common\(-tools\)*.h>"'
>  check include/uapi/linux/mman.h       '-I "^#include <\(uapi/\)*asm/mman.h>"'
> 
> ---
> base-commit: bb551508e78c886e6d3bcca6c744d3bc3fd8ad59
> change-id: 20260316-tools_remove_sync_cputype-h-2b0de59d5de5
> 
> Best regards,
> -- 
> Leo Yan <leo.yan@xxxxxxx>
>