From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D7488F4643B for ; Mon, 16 Mar 2026 09:43:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Transfer-Encoding:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=yFwryLR44CPbV4KpoikTCaFI/MrFEyqhT5hAKdB8Tnc=; b=i5Xa+duYkgLBwKL9ReAzfbxGsV o5V3OLUe1Dexr4HhJq4HFTbhBrL4xIBZAMxwEtaGvZT9TOod2IJ8m3tFEP9KLXJvULfnbxuYkYx19 knJlzVFtCBJ83Wv1kwBUSJVeMsI6RM5mvYHQe04RYQp82YBPqf2/ij7ilkg4ySNZ3qdBHFAXU3d1v B23YnicTZRtL8eKQenveAfH0LHQ2RX+dlAXdMfN0qQLnDak2vm2UjWfqwaLCAkuqn5vVKM+EqNf03 pXcM68UfLSkM9+w5zxbC9do9ANTqEMknS0a5Pf/W7galxx2vdZBoMqrXCEziQfIllHb9C5RPbuyJe TIr2MZBQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1w24UL-00000003gVS-2uBd; Mon, 16 Mar 2026 09:43:53 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1w24UI-00000003gUy-004p for linux-arm-kernel@lists.infradead.org; Mon, 16 Mar 2026 09:43:51 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 97CC51477; Mon, 16 Mar 2026 02:43:40 -0700 (PDT) Received: from localhost (e132581.arm.com [10.1.196.87]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 622E63F778; Mon, 16 Mar 2026 02:43:46 -0700 (PDT) Date: Mon, 16 Mar 2026 09:43:44 +0000 From: Leo Yan To: Arnaldo Carvalho de Melo , Mark Rutland Cc: linux-arm-kernel@lists.infradead.org, Oliver Upton , Shameer Kolothum , Adrian Hunter , Ian Rogers , James Clark , Jiri Olsa , Namhyung Kim , Linux Kernel Mailing List , linux-perf-users@vger.kernel.org Subject: Re: REQUEST: Syncing tools/arch/arm64/include/asm/cputype.h with the kernel sources Message-ID: <20260316094344.GA8048@e132581.arm.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260316_024350_163400_36B2D8AF X-CRM114-Status: GOOD ( 29.44 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Arnaldo, [ + Mark ] On Sun, Mar 15, 2026 at 10:43:03AM -0300, Arnaldo Carvalho de Melo wrote: > Hi, > > Can someone please address this perf build warning: > > make: Entering directory '/home/acme/git/perf-tools/tools/perf' > BUILD: Doing 'make -j32' parallel build > Warning: Kernel ABI header differences: > diff -u tools/arch/arm64/include/asm/cputype.h arch/arm64/include/asm/cputype.h > > I tried updating that header and got the problems below. > > I just merged perf-tools with upstream, will push to tmp.perf-tools at: > > https://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools.git tmp.perf-tools > > Thanks in advance, Sorry that this issue has come up again. A quick summary: The plan was to extract the CPU ID definitions from cputype.h so that kernel and userspace can share the same definitions. This would avoid keep syncing cputype.h between the kernel and userspace [1]. James worked on this a bit, and later Mark wanted to try a different implementation. However, we haven't posted formal patches yet. To avoid extra burden on perf maintainers, I suggest removing the cputype.h check in check-headers.sh. In the short term, Arm developers will take responsibility for keeping it up to date. In the long term, once the CPU ID refactoring is completed, we can do a proper cleanup of cputype.h. James, Mark, is this reasonable? [1] https://lore.kernel.org/linux-perf-users/aFJ8bQh_30JMzF_-@J2N7QTR9R3/ > - Arnaldo > > ⬢ [acme@toolbx perf-tools]$ git diff > diff --git a/tools/arch/arm64/include/asm/cputype.h b/tools/arch/arm64/include/asm/cputype.h > index 9b73c1aa3ad747ec..08860d482e60009f 100644 > --- a/tools/arch/arm64/include/asm/cputype.h > +++ b/tools/arch/arm64/include/asm/cputype.h > @@ -255,6 +255,16 @@ > > #define read_cpuid(reg) read_sysreg_s(SYS_ ## reg) > > +/* > + * The CPU ID never changes at run time, so we might as well tell the > + * compiler that it's constant. Use this function to read the CPU ID > + * rather than directly reading processor_id or read_cpuid() directly. > + */ > +static inline u32 __attribute_const__ read_cpuid_id(void) > +{ > + return read_cpuid(MIDR_EL1); > +} > + > /* > * Represent a range of MIDR values for a given CPU model and a > * range of variant/revision values. > @@ -290,31 +300,6 @@ static inline bool midr_is_cpu_model_range(u32 midr, u32 model, u32 rv_min, > return _model == model && rv >= rv_min && rv <= rv_max; > } > > -static inline bool is_midr_in_range(u32 midr, struct midr_range const *range) > -{ > - return midr_is_cpu_model_range(midr, range->model, > - range->rv_min, range->rv_max); > -} > - > -static inline bool > -is_midr_in_range_list(u32 midr, struct midr_range const *ranges) > -{ > - while (ranges->model) > - if (is_midr_in_range(midr, ranges++)) > - return true; > - return false; > ⬢ [acme@toolbx perf-tools]$ m > make: Entering directory '/home/acme/git/perf-tools/tools/perf' > BUILD: Doing 'make -j32' parallel build > Warning: Kernel ABI header differences: > diff -u tools/arch/x86/include/asm/msr-index.h arch/x86/include/asm/msr-index.h > diff -u tools/arch/x86/include/uapi/asm/svm.h arch/x86/include/uapi/asm/svm.h > > Auto-detecting system features: > ... libdw: [ on ] > ... glibc: [ on ] > ... libelf: [ on ] > ... libnuma: [ on ] > ... numa_num_possible_cpus: [ on ] > ... libpython: [ on ] > ... libcapstone: [ on ] > ... llvm-perf: [ on ] > ... zlib: [ on ] > ... lzma: [ on ] > ... bpf: [ on ] > ... libaio: [ on ] > ... libzstd: [ on ] > ... libopenssl: [ on ] > ... rust: [ on ] > > INSTALL libsubcmd_headers > INSTALL libapi_headers > INSTALL libperf_headers > INSTALL libsymbol_headers > INSTALL libbpf_headers > CC /tmp/build/perf-tools/util/arm-spe.o > util/arm-spe.c: In function ‘arm_spe__synth_ds’: > util/arm-spe.c:986:43: error: passing argument 1 of ‘is_midr_in_range_list’ makes pointer from integer without a cast [-Wint-conversion] > 986 | if (is_midr_in_range_list(midr, data_source_handles[i].midr_ranges)) { > | ^~~~ > | | > | u64 {aka long unsigned int} > In file included from util/arm-spe.c:37: > util/../../arch/arm64/include/asm/cputype.h:310:53: note: expected ‘const struct midr_range *’ but argument is of type ‘u64’ {aka ‘long unsigned int’} > 310 | bool is_midr_in_range_list(struct midr_range const *ranges); > | ~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~ > util/arm-spe.c:986:21: error: too many arguments to function ‘is_midr_in_range_list’; expected 1, have 2 > 986 | if (is_midr_in_range_list(midr, data_source_handles[i].midr_ranges)) { > | ^~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > util/../../arch/arm64/include/asm/cputype.h:310:6: note: declared here > 310 | bool is_midr_in_range_list(struct midr_range const *ranges); > | ^~~~~~~~~~~~~~~~~~~~~ > make[4]: *** [/home/acme/git/perf-tools/tools/build/Makefile.build:95: /tmp/build/perf-tools/util/arm-spe.o] Error 1 > make[3]: *** [/home/acme/git/perf-tools/tools/build/Makefile.build:158: util] Error 2 > make[2]: *** [Makefile.perf:797: /tmp/build/perf-tools/perf-util-in.o] Error 2 > make[2]: *** Waiting for unfinished jobs.... > make[1]: *** [Makefile.perf:289: sub-make] Error 2 > make: *** [Makefile:119: install-bin] Error 2 > make: Leaving directory '/home/acme/git/perf-tools/tools/perf' > ⬢ [acme@toolbx perf-tools]$