From: Ravi Bangoria <ravi.bangoria@amd.com> To: <irogers@google.com>, <acme@kernel.org>, <sfr@canb.auug.org.au> Cc: <ravi.bangoria@amd.com>, <suzuki.poulose@arm.com>, <mike.leach@linaro.org>, <leo.yan@linaro.org>, <john.g.garry@oracle.com>, <will@kernel.org>, <james.clark@arm.com>, <peterz@infradead.org>, <mingo@redhat.com>, <mark.rutland@arm.com>, <alexander.shishkin@linux.intel.com>, <jolsa@kernel.org>, <namhyung@kernel.org>, <adrian.hunter@intel.com>, <kjain@linux.ibm.com>, <renyu.zj@linux.alibaba.com>, <kan.liang@linux.intel.com>, <zhengjun.xing@linux.intel.com>, <maddy@linux.ibm.com>, <atrajeev@linux.vnet.ibm.com>, <wangming01@loongson.cn>, <chenhuacai@kernel.org>, <sandipan.das@amd.com>, <9erthalion6@gmail.com>, <seanjc@google.com>, <robh@kernel.org>, <tmricht@linux.ibm.com>, <tegongkang@gmail.com>, <linux-kernel@vger.kernel.org>, <coresight@lists.linaro.org>, <linux-arm-kernel@lists.infradead.org>, <linux-perf-users@vger.kernel.org> Subject: [PATCH v2] perf test amd: Fix build failure with amd-ibs-via-core-pmu.c Date: Sat, 3 Jun 2023 10:16:50 +0530 [thread overview] Message-ID: <20230603044650.286-1-ravi.bangoria@amd.com> (raw) In-Reply-To: <CAP-5=fUPZ7+nAcxBp5sFBfxzkOH8kzWkLV0uziHqxYC0cDbbDw@mail.gmail.com> Since amd-ibs-via-core-pmu.c was applied via Peter's tree, some of the changes came via Arnaldo's tree did not reflect in this file, which is causing build failures. arch/x86/tests/amd-ibs-via-core-pmu.c:47:25: error: ‘pmus’ undeclared (first use in this function) 47 | if (list_empty(&pmus)) | ^~~~ arch/x86/tests/amd-ibs-via-core-pmu.c:48:17: error: implicit declaration of function ‘perf_pmu__scan’; did you mean perf_pmus__scan’? [-Werror=implicit-function-declaration] 48 | perf_pmu__scan(NULL); | ^~~~~~~~~~~~~~ | perf_pmus__scan arch/x86/tests/amd-ibs-via-core-pmu.c:50:19: error: implicit declaration of function ‘perf_pmu__find’; did you mean perf_pmus__find’? [-Werror=implicit-function-declaration] 50 | ibs_pmu = perf_pmu__find("ibs_op"); | ^~~~~~~~~~~~~~ | perf_pmus__find Fix those. Fixes: 1eaf496ed386 ("perf pmu: Separate pmu and pmus") Signed-off-by: Ravi Bangoria <ravi.bangoria@amd.com> --- v1->v2: - Remove unnecessary perf_pmus__scan(NULL) as suggested by Ian. tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c b/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c index 2902798ca5c1..78b1902f6f59 100644 --- a/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c +++ b/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c @@ -44,10 +44,7 @@ int test__amd_ibs_via_core_pmu(struct test_suite *test __maybe_unused, int ret = TEST_OK; int fd, i; - if (list_empty(&pmus)) - perf_pmu__scan(NULL); - - ibs_pmu = perf_pmu__find("ibs_op"); + ibs_pmu = perf_pmus__find("ibs_op"); if (!ibs_pmu) return TEST_SKIP; -- 2.40.1
WARNING: multiple messages have this Message-ID (diff)
From: Ravi Bangoria <ravi.bangoria@amd.com> To: <irogers@google.com>, <acme@kernel.org>, <sfr@canb.auug.org.au> Cc: <ravi.bangoria@amd.com>, <suzuki.poulose@arm.com>, <mike.leach@linaro.org>, <leo.yan@linaro.org>, <john.g.garry@oracle.com>, <will@kernel.org>, <james.clark@arm.com>, <peterz@infradead.org>, <mingo@redhat.com>, <mark.rutland@arm.com>, <alexander.shishkin@linux.intel.com>, <jolsa@kernel.org>, <namhyung@kernel.org>, <adrian.hunter@intel.com>, <kjain@linux.ibm.com>, <renyu.zj@linux.alibaba.com>, <kan.liang@linux.intel.com>, <zhengjun.xing@linux.intel.com>, <maddy@linux.ibm.com>, <atrajeev@linux.vnet.ibm.com>, <wangming01@loongson.cn>, <chenhuacai@kernel.org>, <sandipan.das@amd.com>, <9erthalion6@gmail.com>, <seanjc@google.com>, <robh@kernel.org>, <tmricht@linux.ibm.com>, <tegongkang@gmail.com>, <linux-kernel@vger.kernel.org>, <coresight@lists.linaro.org>, <linux-arm-kernel@lists.infradead.org>, <linux-perf-users@vger.kernel.org> Subject: [PATCH v2] perf test amd: Fix build failure with amd-ibs-via-core-pmu.c Date: Sat, 3 Jun 2023 10:16:50 +0530 [thread overview] Message-ID: <20230603044650.286-1-ravi.bangoria@amd.com> (raw) In-Reply-To: <CAP-5=fUPZ7+nAcxBp5sFBfxzkOH8kzWkLV0uziHqxYC0cDbbDw@mail.gmail.com> Since amd-ibs-via-core-pmu.c was applied via Peter's tree, some of the changes came via Arnaldo's tree did not reflect in this file, which is causing build failures. arch/x86/tests/amd-ibs-via-core-pmu.c:47:25: error: ‘pmus’ undeclared (first use in this function) 47 | if (list_empty(&pmus)) | ^~~~ arch/x86/tests/amd-ibs-via-core-pmu.c:48:17: error: implicit declaration of function ‘perf_pmu__scan’; did you mean perf_pmus__scan’? [-Werror=implicit-function-declaration] 48 | perf_pmu__scan(NULL); | ^~~~~~~~~~~~~~ | perf_pmus__scan arch/x86/tests/amd-ibs-via-core-pmu.c:50:19: error: implicit declaration of function ‘perf_pmu__find’; did you mean perf_pmus__find’? [-Werror=implicit-function-declaration] 50 | ibs_pmu = perf_pmu__find("ibs_op"); | ^~~~~~~~~~~~~~ | perf_pmus__find Fix those. Fixes: 1eaf496ed386 ("perf pmu: Separate pmu and pmus") Signed-off-by: Ravi Bangoria <ravi.bangoria@amd.com> --- v1->v2: - Remove unnecessary perf_pmus__scan(NULL) as suggested by Ian. tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c b/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c index 2902798ca5c1..78b1902f6f59 100644 --- a/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c +++ b/tools/perf/arch/x86/tests/amd-ibs-via-core-pmu.c @@ -44,10 +44,7 @@ int test__amd_ibs_via_core_pmu(struct test_suite *test __maybe_unused, int ret = TEST_OK; int fd, i; - if (list_empty(&pmus)) - perf_pmu__scan(NULL); - - ibs_pmu = perf_pmu__find("ibs_op"); + ibs_pmu = perf_pmus__find("ibs_op"); if (!ibs_pmu) return TEST_SKIP; -- 2.40.1 _______________________________________________ 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:[~2023-06-03 4:47 UTC|newest] Thread overview: 103+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-05-27 7:21 [PATCH v5 00/34] PMU refactoring and improvements Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 01/34] perf cpumap: Add internal nr and cpu accessors Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 02/34] perf cpumap: Add equal function Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 03/34] libperf cpumap: Add "any CPU"/dummy test function Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 04/34] perf pmu: Detect ARM and hybrid PMUs with sysfs Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 05/34] perf pmu: Add is_core to pmu Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 06/34] perf evsel: Add is_pmu_core inorder to interpret own_cpus Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 07/34] perf pmu: Add CPU map for "cpu" PMUs Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 08/34] perf evlist: Propagate user CPU maps intersecting core PMU maps Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 09/34] perf evlist: Allow has_user_cpus to be set on hybrid Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 10/34] perf target: Remove unused hybrid value Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 11/34] perf tools: Warn if no user requested CPUs match PMU's CPUs Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 12/34] perf evlist: Remove evlist__warn_hybrid_group Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 13/34] perf evlist: Remove __evlist__add_default Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 14/34] perf evlist: Reduce scope of evlist__has_hybrid Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 15/34] perf pmu: Remove perf_pmu__hybrid_mounted Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 16/34] perf pmu: Rewrite perf_pmu__has_hybrid to avoid list Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 17/34] perf x86: Iterate hybrid PMUs as core PMUs Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 18/34] perf topology: Avoid hybrid list for hybrid topology Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 19/34] perf evsel: Compute is_hybrid from PMU being core Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 20/34] perf header: Avoid hybrid PMU list in write_pmu_caps Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 21/34] perf metrics: Remove perf_pmu__is_hybrid use Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 22/34] perf stat: Avoid hybrid PMU list Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:21 ` [PATCH v5 23/34] perf mem: " Ian Rogers 2023-05-27 7:21 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 24/34] perf pmu: Remove perf_pmu__hybrid_pmus list Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 25/34] perf pmus: Prefer perf_pmu__scan over perf_pmus__for_each_pmu Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 26/34] perf x86 mem: minor refactor to is_mem_loads_aux_event Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 27/34] perf pmu: Separate pmu and pmus Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-06-02 5:29 ` [PATCH] perf test amd: Fix build failure with amd-ibs-via-core-pmu.c -- Was: " Ravi Bangoria 2023-06-02 5:29 ` Ravi Bangoria 2023-06-02 6:42 ` Ian Rogers 2023-06-02 6:42 ` Ian Rogers 2023-06-03 4:46 ` Ravi Bangoria [this message] 2023-06-03 4:46 ` [PATCH v2] perf test amd: Fix build failure with amd-ibs-via-core-pmu.c Ravi Bangoria 2023-06-05 14:27 ` Arnaldo Carvalho de Melo 2023-06-05 14:27 ` Arnaldo Carvalho de Melo 2023-06-06 3:12 ` Ravi Bangoria 2023-06-06 3:12 ` Ravi Bangoria 2023-06-06 4:24 ` Stephen Rothwell 2023-06-06 4:24 ` Stephen Rothwell 2023-06-07 0:56 ` Stephen Rothwell 2023-06-07 0:56 ` Stephen Rothwell 2023-05-27 7:22 ` [PATCH v5 28/34] perf pmus: Split pmus list into core and other Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-06-09 3:59 ` Ravi Bangoria 2023-06-09 3:59 ` Ravi Bangoria 2023-06-09 4:40 ` Ian Rogers 2023-06-09 4:40 ` Ian Rogers 2023-06-09 5:30 ` Ravi Bangoria 2023-06-09 5:30 ` Ravi Bangoria 2023-06-09 5:35 ` Ian Rogers 2023-06-09 5:35 ` Ian Rogers 2023-06-09 5:55 ` Ravi Bangoria 2023-06-09 5:55 ` Ravi Bangoria 2023-06-09 6:00 ` Ian Rogers 2023-06-09 6:00 ` Ian Rogers 2023-06-09 6:02 ` Ravi Bangoria 2023-06-09 6:02 ` Ravi Bangoria 2023-06-09 7:58 ` Mark Rutland 2023-06-09 7:58 ` Mark Rutland 2023-06-11 3:55 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 29/34] perf pmus: Allow just core PMU scanning Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-06-09 6:12 ` Ravi Bangoria 2023-06-09 6:12 ` Ravi Bangoria 2023-05-27 7:22 ` [PATCH v5 30/34] perf pmus: Avoid repeated sysfs scanning Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 31/34] perf pmus: Ensure all PMUs are read for find_by_type Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 32/34] perf pmus: Add function to return count of core PMUs Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 33/34] perf pmus: Remove perf_pmus__has_hybrid Ian Rogers 2023-05-27 7:22 ` Ian Rogers 2023-05-27 7:22 ` [PATCH v5 34/34] perf pmu: Remove is_pmu_hybrid Ian Rogers 2023-05-27 7:22 ` Ian Rogers
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=20230603044650.286-1-ravi.bangoria@amd.com \ --to=ravi.bangoria@amd.com \ --cc=9erthalion6@gmail.com \ --cc=acme@kernel.org \ --cc=adrian.hunter@intel.com \ --cc=alexander.shishkin@linux.intel.com \ --cc=atrajeev@linux.vnet.ibm.com \ --cc=chenhuacai@kernel.org \ --cc=coresight@lists.linaro.org \ --cc=irogers@google.com \ --cc=james.clark@arm.com \ --cc=john.g.garry@oracle.com \ --cc=jolsa@kernel.org \ --cc=kan.liang@linux.intel.com \ --cc=kjain@linux.ibm.com \ --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=maddy@linux.ibm.com \ --cc=mark.rutland@arm.com \ --cc=mike.leach@linaro.org \ --cc=mingo@redhat.com \ --cc=namhyung@kernel.org \ --cc=peterz@infradead.org \ --cc=renyu.zj@linux.alibaba.com \ --cc=robh@kernel.org \ --cc=sandipan.das@amd.com \ --cc=seanjc@google.com \ --cc=sfr@canb.auug.org.au \ --cc=suzuki.poulose@arm.com \ --cc=tegongkang@gmail.com \ --cc=tmricht@linux.ibm.com \ --cc=wangming01@loongson.cn \ --cc=will@kernel.org \ --cc=zhengjun.xing@linux.intel.com \ /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.