From: Arnaldo Carvalho de Melo <acme@kernel.org> To: German Gomez <german.gomez@arm.com> Cc: James Clark <james.clark@arm.com>, linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, broonie@kernel.org, leo.yan@linaro.org, mathieu.poirier@linaro.org, john.garry@huawei.com, Will Deacon <will@kernel.org>, Mike Leach <mike.leach@linaro.org>, Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com>, Mark Rutland <mark.rutland@arm.com>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, Jiri Olsa <jolsa@kernel.org>, Namhyung Kim <namhyung@kernel.org>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 0/2] perf: arm64: Tools support for Dwarf unwinding through SVE functions Date: Fri, 20 May 2022 09:32:36 -0300 [thread overview] Message-ID: <YoeKZFXO6zqFaqme@kernel.org> (raw) In-Reply-To: <995099d3-a170-69dc-d12e-c9b2e4753f35@arm.com> Em Fri, May 20, 2022 at 12:46:24PM +0100, German Gomez escreveu: > > On 17/05/2022 15:58, Arnaldo Carvalho de Melo wrote: > > Em Tue, May 17, 2022 at 11:20:01AM +0100, James Clark escreveu: > >> Changes since v1: > >> > >> * Split patchset into kernel side and Perf tool changes > > Thanks, now I'll wait for the kernel side to be merged. > > > > - Arnaldo > > > > Hi, > > I think we also need to fix the below import (it's trying to import the > perf_regs.h from the kernel, not the tools/ dir). Sure, that is why it is important to do a: make -C tools/perf build-test before posting patches, as it will do, among other tests, a detached tarball test using: ⬢[acme@toolbox perf-urgent]$ make help | grep perf perf-tar-src-pkg - Build perf-5.18.0-rc7.tar source tarball perf-targz-src-pkg - Build perf-5.18.0-rc7.tar.gz source tarball perf-tarbz2-src-pkg - Build perf-5.18.0-rc7.tar.bz2 source tarball perf-tarxz-src-pkg - Build perf-5.18.0-rc7.tar.xz source tarball perf-tarzst-src-pkg - Build perf-5.18.0-rc7.tar.zst source tarball ⬢[acme@toolbox perf-urgent]$ That will pick just what is in tools/perf/MANIFEST and that doesn´t include the kernel headers, just what is in tools/ then building with this patchkit would fail. To recap: ⬢[acme@toolbox perf-urgent]$ make perf-tarxz-src-pkg TAR PERF_VERSION = 5.18.rc7.gc645054ef517 ⬢[acme@toolbox perf-urgent]$ ls -la perf-5.18.0-rc7.tar.xz -rw-r--r--. 1 acme acme 2364832 May 20 09:32 perf-5.18.0-rc7.tar.xz ⬢[acme@toolbox perf-urgent]$ tar tvf perf-5.18.0-rc7.tar.xz | tail -rw-rw-r-- root/root 7364 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/vdso.c -rw-rw-r-- root/root 639 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/vdso.h -rw-rw-r-- root/root 1793 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/zlib.c -rw-rw-r-- root/root 2672 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/zstd.c drwxrwxr-x root/root 0 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/ -rw-rw-r-- root/root 1091 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/Makefile.arch -rw-rw-r-- root/root 5905 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/Makefile.include -rw-rw-r-- root/root 5288 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/utilities.mak -rw-r--r-- acme/acme 41 2022-05-20 09:32 perf-5.18.0-rc7/HEAD -rw-r--r-- acme/acme 46 2022-05-20 09:32 perf-5.18.0-rc7/PERF-VERSION-FILE ⬢[acme@toolbox perf-urgent]$ - Arnaldo > diff --git a/tools/perf/util/libunwind/arm64.c b/tools/perf/util/libunwind/arm64.c > index 15f60fd09424..014d82159656 100644 > --- a/tools/perf/util/libunwind/arm64.c > +++ b/tools/perf/util/libunwind/arm64.c > @@ -24,7 +24,7 @@ > #include "unwind.h" > #include "libunwind-aarch64.h" > #define perf_event_arm_regs perf_event_arm64_regs > -#include <../../../../arch/arm64/include/uapi/asm/perf_regs.h> > +#include <../../../arch/arm64/include/uapi/asm/perf_regs.h> > #undef perf_event_arm_regs > #include "../../arch/arm64/util/unwind-libunwind.c" > -- - Arnaldo
WARNING: multiple messages have this Message-ID (diff)
From: Arnaldo Carvalho de Melo <acme@kernel.org> To: German Gomez <german.gomez@arm.com> Cc: James Clark <james.clark@arm.com>, linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, broonie@kernel.org, leo.yan@linaro.org, mathieu.poirier@linaro.org, john.garry@huawei.com, Will Deacon <will@kernel.org>, Mike Leach <mike.leach@linaro.org>, Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com>, Mark Rutland <mark.rutland@arm.com>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, Jiri Olsa <jolsa@kernel.org>, Namhyung Kim <namhyung@kernel.org>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 0/2] perf: arm64: Tools support for Dwarf unwinding through SVE functions Date: Fri, 20 May 2022 09:32:36 -0300 [thread overview] Message-ID: <YoeKZFXO6zqFaqme@kernel.org> (raw) In-Reply-To: <995099d3-a170-69dc-d12e-c9b2e4753f35@arm.com> Em Fri, May 20, 2022 at 12:46:24PM +0100, German Gomez escreveu: > > On 17/05/2022 15:58, Arnaldo Carvalho de Melo wrote: > > Em Tue, May 17, 2022 at 11:20:01AM +0100, James Clark escreveu: > >> Changes since v1: > >> > >> * Split patchset into kernel side and Perf tool changes > > Thanks, now I'll wait for the kernel side to be merged. > > > > - Arnaldo > > > > Hi, > > I think we also need to fix the below import (it's trying to import the > perf_regs.h from the kernel, not the tools/ dir). Sure, that is why it is important to do a: make -C tools/perf build-test before posting patches, as it will do, among other tests, a detached tarball test using: ⬢[acme@toolbox perf-urgent]$ make help | grep perf perf-tar-src-pkg - Build perf-5.18.0-rc7.tar source tarball perf-targz-src-pkg - Build perf-5.18.0-rc7.tar.gz source tarball perf-tarbz2-src-pkg - Build perf-5.18.0-rc7.tar.bz2 source tarball perf-tarxz-src-pkg - Build perf-5.18.0-rc7.tar.xz source tarball perf-tarzst-src-pkg - Build perf-5.18.0-rc7.tar.zst source tarball ⬢[acme@toolbox perf-urgent]$ That will pick just what is in tools/perf/MANIFEST and that doesn´t include the kernel headers, just what is in tools/ then building with this patchkit would fail. To recap: ⬢[acme@toolbox perf-urgent]$ make perf-tarxz-src-pkg TAR PERF_VERSION = 5.18.rc7.gc645054ef517 ⬢[acme@toolbox perf-urgent]$ ls -la perf-5.18.0-rc7.tar.xz -rw-r--r--. 1 acme acme 2364832 May 20 09:32 perf-5.18.0-rc7.tar.xz ⬢[acme@toolbox perf-urgent]$ tar tvf perf-5.18.0-rc7.tar.xz | tail -rw-rw-r-- root/root 7364 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/vdso.c -rw-rw-r-- root/root 639 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/vdso.h -rw-rw-r-- root/root 1793 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/zlib.c -rw-rw-r-- root/root 2672 2022-05-20 09:32 perf-5.18.0-rc7/tools/perf/util/zstd.c drwxrwxr-x root/root 0 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/ -rw-rw-r-- root/root 1091 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/Makefile.arch -rw-rw-r-- root/root 5905 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/Makefile.include -rw-rw-r-- root/root 5288 2022-05-20 09:32 perf-5.18.0-rc7/tools/scripts/utilities.mak -rw-r--r-- acme/acme 41 2022-05-20 09:32 perf-5.18.0-rc7/HEAD -rw-r--r-- acme/acme 46 2022-05-20 09:32 perf-5.18.0-rc7/PERF-VERSION-FILE ⬢[acme@toolbox perf-urgent]$ - Arnaldo > diff --git a/tools/perf/util/libunwind/arm64.c b/tools/perf/util/libunwind/arm64.c > index 15f60fd09424..014d82159656 100644 > --- a/tools/perf/util/libunwind/arm64.c > +++ b/tools/perf/util/libunwind/arm64.c > @@ -24,7 +24,7 @@ > #include "unwind.h" > #include "libunwind-aarch64.h" > #define perf_event_arm_regs perf_event_arm64_regs > -#include <../../../../arch/arm64/include/uapi/asm/perf_regs.h> > +#include <../../../arch/arm64/include/uapi/asm/perf_regs.h> > #undef perf_event_arm_regs > #include "../../arch/arm64/util/unwind-libunwind.c" > -- - Arnaldo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-05-20 12:32 UTC|newest] Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-05-17 10:20 [PATCH v2 0/2] perf: arm64: Tools support for Dwarf unwinding through SVE functions James Clark 2022-05-17 10:20 ` James Clark 2022-05-17 10:20 ` [PATCH v2 1/4] perf tools: arm64: Copy perf_regs.h from the kernel James Clark 2022-05-17 10:20 ` James Clark 2022-05-17 10:50 ` Leo Yan 2022-05-17 10:50 ` Leo Yan 2022-05-17 10:20 ` [PATCH v2 2/4] perf tools: Use dynamic register set for Dwarf unwind James Clark 2022-05-17 10:20 ` James Clark 2022-05-17 11:03 ` Leo Yan 2022-05-17 11:03 ` Leo Yan 2022-05-18 13:25 ` James Clark 2022-05-18 13:25 ` James Clark 2022-05-18 14:00 ` Leo Yan 2022-05-18 14:00 ` Leo Yan 2022-05-17 10:20 ` [PATCH v2 3/4] perf tools: arm64: Decouple Libunwind register names from Perf James Clark 2022-05-17 10:20 ` James Clark 2022-05-17 12:54 ` Leo Yan 2022-05-17 12:54 ` Leo Yan 2022-05-17 10:20 ` [PATCH v2 4/4] perf tools: arm64: Add support for VG register James Clark 2022-05-17 10:20 ` James Clark 2022-05-17 13:19 ` Leo Yan 2022-05-17 13:19 ` Leo Yan 2022-05-18 9:44 ` James Clark 2022-05-18 9:44 ` James Clark 2022-05-18 9:57 ` Leo Yan 2022-05-18 9:57 ` Leo Yan 2022-05-17 14:58 ` [PATCH v2 0/2] perf: arm64: Tools support for Dwarf unwinding through SVE functions Arnaldo Carvalho de Melo 2022-05-17 14:58 ` Arnaldo Carvalho de Melo 2022-05-20 11:46 ` German Gomez 2022-05-20 11:46 ` German Gomez 2022-05-20 12:32 ` Arnaldo Carvalho de Melo [this message] 2022-05-20 12:32 ` Arnaldo Carvalho de Melo 2022-05-20 14:52 ` James Clark 2022-05-20 14:52 ` James Clark
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=YoeKZFXO6zqFaqme@kernel.org \ --to=acme@kernel.org \ --cc=alexander.shishkin@linux.intel.com \ --cc=broonie@kernel.org \ --cc=german.gomez@arm.com \ --cc=james.clark@arm.com \ --cc=john.garry@huawei.com \ --cc=jolsa@kernel.org \ --cc=leo.yan@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-perf-users@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mathieu.poirier@linaro.org \ --cc=mike.leach@linaro.org \ --cc=mingo@redhat.com \ --cc=namhyung@kernel.org \ --cc=peterz@infradead.org \ --cc=will@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.