Changes in parse-events.c

5e530a82 - perf tests: Enable tests disabled due to tracepoint parsing

Mon, 18 Nov 2024 22:53:45 +0000

perf tests: Enable tests disabled due to tracepoint parsingTracepoint parsing required libtraceevent but no longer does. Removethe Build logic and #ifdefs that caused the tests not to be run. Testcode that directly uses libtraceevent is still guarded.Reviewed-by: Namhyung Kim Signed-off-by: Ian Rogers Tested-by: Arnaldo Carvalho de Melo Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Andi Kleen Cc: Athira Rajeev Cc: Ben Gainey Cc: Colin Ian King Cc: Dominique Martinet Cc: Ilkka Koskinen Cc: Ingo Molnar Cc: James Clark Cc: Jiri Olsa Cc: Kan Liang Cc: Mark Rutland Cc: Oliver Upton Cc: Paran Lee Cc: Peter Zijlstra Cc: Steinar H. Gunderson Cc: Steven Rostedt (VMware) Cc: Thomas Falcon Cc: Weilin Wang Cc: Yang Jihong Cc: Yang Li Cc: Ze Gao Cc: Zixian Cai Cc: zhaimingbing Link: https://lore.kernel.org/r/20241118225345.889810-8-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

88bc63d0 - perf tools: Do not set exclude_guest for precise_ip

Wed, 16 Oct 2024 06:23:54 +0000

perf tools: Do not set exclude_guest for precise_ipIt seems perf sets the exclude_guest bit because of Intel PEBSimplementation which uses a virtual address. IIUC now kernel disablesPEBS when it goes to the guest mode regardless of this bit so we don'tneed to set it explicitly. At least for the other archs/vendors.I found the commit 1342798cc13e set the exclude_guest for precise_ipin the tool and the commit 20b279ddb38c added kernel side enforcementwhich was reverted by commit a706d965dcfd later.Actually it doesn't set the exclude_guest for the default event(cycles:P) already. $ grep -m1 vendor /proc/cpuinfo vendor_id : GenuineIntel $ perf record -e cycles:P true [ perf record: Woken up 1 times to write data ] [ perf record: Captured and wrote 0.002 MB perf.data (9 samples) ] $ perf evlist -v | tr ',' '\n' | grep -e exclude -e precise precise_ip: 3But having lower 'p' modifier set the bit for some reason. $ perf record -e cycles:pp true [ perf record: Woken up 1 times to write data ] [ perf record: Captured and wrote 0.002 MB perf.data (9 samples) ] $ perf evlist -v | tr ',' '\n' | grep -e exclude -e precise precise_ip: 2 exclude_guest: 1Actually AMD IBS suffers from this because it doesn't support excludesand having this bit effectively disables new features in the currentimplementation (due to the missing feature check). $ grep -m1 vendor /proc/cpuinfo vendor_id : AuthenticAMD $ perf record -W -e cycles:p -vv true 2>&1 | grep switching switching off PERF_FORMAT_LOST support switching off weight struct support switching off bpf_event switching off ksymbol switching off cloexec flag switching off mmap2 switching off exclude_guest, exclude_hostBy not setting exclude_guest, we can fix this inconsistency and thetroubles.Reviewed-by: Ian Rogers Reviewed-by: James Clark Reviewed-by: Ravi Bangoria Acked-by: Kan Liang Cc: James Clark Cc: Atish Patra Cc: Mingwei Zhang Cc: Kajol Jain Cc: Thomas Richter Cc: Palmer Dabbelt Link: https://lore.kernel.org/r/20241016062359.264929-5-namhyung@kernel.orgSigned-off-by: Namhyung Kim List of files: /linux-6.15/tools/perf/tests/parse-events.c

35c8d213 - perf tools: Don't set attr.exclude_guest by default

Wed, 16 Oct 2024 06:23:52 +0000

perf tools: Don't set attr.exclude_guest by defaultThe exclude_guest in the event attribute is to limit profiling in thehost environment. But I'm not sure why we want to set it by defaultcause we don't care about it in most cases and I feel like it justmakes new PMU implementation complicated.Of course it's useful for perf kvm command so I added theexclude_GH_default variable to preserve the old behavior for perf kvmand other commands like perf record and stat won't set the exclude bit.This is helpful for AMD IBS case since having exclude_guest bit willclear new feature bit due to the missing feature check logic. $ sysctl kernel.perf_event_paranoid kernel.perf_event_paranoid = 0 $ perf record -W -e ibs_op// -vv true 2>&1 | grep switching switching off PERF_FORMAT_LOST support switching off weight struct support switching off bpf_event switching off ksymbol switching off cloexec flag switching off mmap2 switching off exclude_guest, exclude_hostIntestingly, I found it sets the exclude_bit if "u" modifier is used.I don't know why but it's neither intuitive nor consistent. Let'sremove the bit there too.Reviewed-by: Ian Rogers Reviewed-by: James Clark Reviewed-by: Ravi Bangoria Acked-by: Kan Liang Cc: James Clark Cc: Atish Patra Cc: Mingwei Zhang Cc: Kajol Jain Cc: Thomas Richter Cc: Palmer Dabbelt Link: https://lore.kernel.org/r/20241016062359.264929-3-namhyung@kernel.orgSigned-off-by: Namhyung Kim List of files: /linux-6.15/tools/perf/tests/parse-events.c

d7d156fc - perf evsel: Remove pmu_name

Thu, 26 Sep 2024 14:48:36 +0000

perf evsel: Remove pmu_name"evsel->pmu_name" is only ever assigned a strdup of "pmu->name", astrdup of "evsel->pmu_name" or NULL. As such, prefer to use"pmu->name" directly and even to directly compare PMUs than PMUnames. For safety, add some additional NULL tests.Acked-by: Namhyung Kim Signed-off-by: Ian Rogers [ Fix arm-spe.c usage of pmu_name and empty PMU name ]Acked-by: Kan Liang Signed-off-by: James Clark Cc: Yang Jihong Cc: Dominique Martinet Cc: Howard Chu Cc: Ze Gao Cc: Yicong Yang Cc: Weilin Wang Cc: Will Deacon Cc: Mike Leach Cc: Jing Zhang Cc: Yang Li Cc: Leo Yan Cc: ak@linux.intel.comCc: Athira Rajeev Cc: linux-arm-kernel@lists.infradead.orgCc: Sun Haiyong Cc: John Garry Link: https://lore.kernel.org/r/20240926144851.245903-6-james.clark@linaro.orgSigned-off-by: Namhyung Kim List of files: /linux-6.15/tools/perf/tests/parse-events.c

92532073 - perf pmus: Fake PMU clean up

Sat, 07 Sep 2024 05:08:17 +0000

perf pmus: Fake PMU clean upRather than passing a fake PMU around, just pass that the fake PMUshould be used - true when doing testing. Move the fake PMU intopmus.[ch] and try to abstract the PMU's properties in pmu.c, ie sothere is less "if fake_pmu" in non-PMU code. Give the fake PMU a madeup type number.Signed-off-by: Ian Rogers Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Andi Kleen Cc: Athira Rajeev Cc: Benjamin Gray Cc: Changbin Du Cc: Clément Le Goffic Cc: Colin Ian King Cc: Dominique Martinet Cc: Dr. David Alan Gilbert Cc: Howard Chu Cc: Ingo Molnar Cc: James Clark Cc: Jing Zhang Cc: Jiri Olsa Cc: John Garry Cc: Junhao He Cc: Kan Liang Cc: Leo Yan Cc: Mark Rutland Cc: Mike Leach Cc: Namhyung Kim Cc: Oliver Upton Cc: Peter Zijlstra Cc: Ravi Bangoria Cc: Sandipan Das Cc: Sun Haiyong Cc: Tiezhu Yang Cc: Veronika Molnarova Cc: Weilin Wang Cc: Will Deacon Cc: Xu Yang Cc: Yang Jihong Cc: Yicong Yang Cc: Ze Gao Cc: linux-arm-kernel@lists.infradead.orgLink: https://lore.kernel.org/r/20240907050830.6752-3-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

e37b315c - perf parse-events: Remove duplicated include in parse-events.c

Tue, 10 Sep 2024 00:55:22 +0000

perf parse-events: Remove duplicated include in parse-events.cThe header files parse-events.h is included twice in parse-events.c,so one inclusion of each can be removed.Reported-by: Abaci Robot Signed-off-by: Yang Li Reviewed-by: James Clark Cc: Ingo Molnar Cc: Namhyung Kim Cc: Peter Zijlstra Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=10822Link: https://lore.kernel.org/r/20240910005522.35994-1-yang.lee@linux.alibaba.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

70b27c75 - perf parse-events: Add default_breakpoint_len helper

Wed, 04 Sep 2024 05:06:04 +0000

perf parse-events: Add default_breakpoint_len helperThe default breakpoint length is "sizeof(long)" however this isincorrect on platforms like Aarch64 where sizeof(long) is 8 but thebreakpoint length is 4. Add a helper function that can be used todetermine the correct breakpoint length, in this change it justreturns the existing default sizeof(long) value.Use the helper in the bp_account test so that, when modifying theevent from a watchpoint to a breakpoint, the breakpoint length isappropriate for the architecture and not just sizeof(long).Signed-off-by: Ian Rogers Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Athira Rajeev Cc: Chaitanya S Prakash Cc: Colin Ian King Cc: Dominique Martinet Cc: Ingo Molnar Cc: James Clark Cc: Jiri Olsa Cc: John Garry Cc: Junhao He Cc: Kan Liang Cc: Mark Rutland Cc: Masami Hiramatsu Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Yang Jihong Link: https://lore.kernel.org/r/20240904050606.752788-5-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

5ceb5799 - perf parse: Allow tracepoint names to start with digits

Thu, 09 May 2024 22:24:32 +0000

perf parse: Allow tracepoint names to start with digitsTracepoints can start with digits, although we don't have many of these: $ rg -g '*.h' '\bTRACE_EVENT\([0-9]' net/mac802154/trace.h 53:TRACE_EVENT(802154_drv_return_int, ... net/ieee802154/trace.h 66:TRACE_EVENT(802154_rdev_add_virtual_intf, ... include/trace/events/9p.h 124:TRACE_EVENT(9p_client_req, ...Just allow names to start with digits too so e.g. "perf trace -e '9p:*'"worksReviewed-by: Ian Rogers Signed-off-by: Dominique Martinet Tested-by: Ian Rogers Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Ingo Molnar Cc: Jiri Olsa Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Link: https://lore.kernel.org/r/20240510-perf_digit-v4-3-db1553f3233b@codewreck.orgSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

a2a6604e - perf parse-events: Add new 'fake_tp' parameter for tests

Thu, 09 May 2024 22:24:31 +0000

perf parse-events: Add new 'fake_tp' parameter for testsThe next commit will allow tracepoints starting with digits, but mostsystems do not have any available by default so tests should skip theactual "check if it exists in /sys/kernel/debug/tracing" step.In order to do that, add a new boolean flag specifying if we shouldactually "format" the probe or not.Originally-by: Jiri Olsa Reviewed-by: Ian Rogers Signed-off-by: Dominique Martinet Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Ingo Molnar Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Link: https://lore.kernel.org/r/20240510-perf_digit-v4-2-db1553f3233b@codewreck.orgSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

78fae207 - perf tests parse-events: Use "branches" rather than "cache-references"

Tue, 16 Apr 2024 06:15:21 +0000

perf tests parse-events: Use "branches" rather than "cache-references"Switch from "cache-references" to "branches" in test as Intel has asysfs event for "cache-references" and changing the priority for sysfsover legacy causes the test to fail.Signed-off-by: Ian Rogers Reviewed-by: Kan Liang Tested-by: Atish Patra Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Beeman Strong Cc: Ingo Molnar Cc: James Clark Cc: Jiri Olsa Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Link: https://lore.kernel.org/r/20240416061533.921723-6-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

09d2056e - perf evsel: Use evsel__name_is() helper

Mon, 01 Apr 2024 06:27:24 +0000

perf evsel: Use evsel__name_is() helperCode cleanup, replace strcmp(evsel__name(evsel, {NAME})) withevsel__name_is() helper.No functional change.Committer notes:Fix this build error: trace.syscalls.events.bpf_output = evlist__last(trace.evlist); - assert(evsel__name_is(trace.syscalls.events.bpf_output), "__augmented_syscalls__"); + assert(evsel__name_is(trace.syscalls.events.bpf_output, "__augmented_syscalls__"));Reviewed-by: Ian Rogers Signed-off-by: Yang Jihong Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Ingo Molnar Cc: James Clark Cc: Jiri Olsa Cc: Kan Liang Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Link: https://lore.kernel.org/r/20240401062724.1006010-3-yangjihong@bytedance.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

fd7b8e8f - perf parse-events: Print all errors

Wed, 31 Jan 2024 13:49:40 +0000

perf parse-events: Print all errorsPrior to this patch the first and the last error encountered duringparsing are printed. To see other errors verbose needsenabling. Unfortunately this can drop useful errors, in particular onterms. This patch changes the errors so that instead of the first andlast all errors are recorded and printed, the underlying datastructure is changed to a list.Before:```$ perf stat -e 'slots/edge=2/' trueevent syntax error: 'slots/edge=2/' \___ Bad event or PMUUnable to find PMU or event on a PMU of 'slots'Initial error:event syntax error: 'slots/edge=2/' \___ Cannot find PMU `slots'. Missing kernel support?Run 'perf list' for a list of valid events Usage: perf stat [] [] -e, --event event selector. use 'perf list' to list available events```After:```$ perf stat -e 'slots/edge=2/' trueevent syntax error: 'slots/edge=2/' \___ Bad event or PMUUnable to find PMU or event on a PMU of 'slots'event syntax error: 'slots/edge=2/' \___ value too big for format (edge), maximum is 1event syntax error: 'slots/edge=2/' \___ Cannot find PMU `slots'. Missing kernel support?Run 'perf list' for a list of valid events Usage: perf stat [] [] -e, --event event selector. use 'perf list' to list available events```Signed-off-by: Ian Rogers Reviewed-by: James Clark Cc: Mark Rutland Cc: tchen168@asu.eduCc: Kan Liang Cc: Michael Petlan Signed-off-by: Namhyung Kim Link: https://lore.kernel.org/r/20240131134940.593788-3-irogers@google.com List of files: /linux-6.15/tools/perf/tests/parse-events.c

a24d9d9d - perf parse-events: Make legacy events lower priority than sysfs/JSON

Thu, 23 Nov 2023 04:29:22 +0000

perf parse-events: Make legacy events lower priority than sysfs/JSONThe perf tool has previously made legacy events the priority so withor without a PMU the legacy event would be opened: $ perf stat -e cpu-cycles,cpu/cpu-cycles/ true Using CPUID GenuineIntel-6-8D-1 intel_pt default config: tsc,mtc,mtc_period=3,psb_period=3,pt,branch Attempting to add event pmu 'cpu' with 'cpu-cycles,' that may result in non-fatal errors After aliases, add event pmu 'cpu' with 'cpu-cycles,' that may result in non-fatal errors Control descriptor is not initialized ------------------------------------------------------------ perf_event_attr: type 0 (PERF_TYPE_HARDWARE) size 136 config 0 (PERF_COUNT_HW_CPU_CYCLES) sample_type IDENTIFIER read_format TOTAL_TIME_ENABLED|TOTAL_TIME_RUNNING disabled 1 inherit 1 enable_on_exec 1 exclude_guest 1 ------------------------------------------------------------ sys_perf_event_open: pid 833967 cpu -1 group_fd -1 flags 0x8 = 3 ------------------------------------------------------------ perf_event_attr: type 0 (PERF_TYPE_HARDWARE) size 136 config 0 (PERF_COUNT_HW_CPU_CYCLES) sample_type IDENTIFIER read_format TOTAL_TIME_ENABLED|TOTAL_TIME_RUNNING disabled 1 inherit 1 enable_on_exec 1 exclude_guest 1 ------------------------------------------------------------ ...Fixes to make hybrid/BIG.little PMUs behave correctly, ie as core PMUscapable of opening legacy events on each, removing hard coded "cpu_core"and "cpu_atom" Intel PMU names, etc. caused a behavioral difference onApple/ARM due to latent issues in the PMU driver reported in:https://lore.kernel.org/lkml/08f1f185-e259-4014-9ca4-6411d5c1bc65@marcan.st/As part of that report Mark Rutland requestedthat legacy events not be higher in priority when a PMU is specifiedreversing what has until this change been perf's default behavior. Withthis change the above becomes: $ perf stat -e cpu-cycles,cpu/cpu-cycles/ true Using CPUID GenuineIntel-6-8D-1 Attempt to add: cpu/cpu-cycles=0/ ..after resolving event: cpu/event=0x3c/ Control descriptor is not initialized ------------------------------------------------------------ perf_event_attr: type 0 (PERF_TYPE_HARDWARE) size 136 config 0 (PERF_COUNT_HW_CPU_CYCLES) sample_type IDENTIFIER read_format TOTAL_TIME_ENABLED|TOTAL_TIME_RUNNING disabled 1 inherit 1 enable_on_exec 1 exclude_guest 1 ------------------------------------------------------------ sys_perf_event_open: pid 827628 cpu -1 group_fd -1 flags 0x8 = 3 ------------------------------------------------------------ perf_event_attr: type 4 (PERF_TYPE_RAW) size 136 config 0x3c sample_type IDENTIFIER read_format TOTAL_TIME_ENABLED|TOTAL_TIME_RUNNING disabled 1 inherit 1 enable_on_exec 1 exclude_guest 1 ------------------------------------------------------------ ...So the second event has become a raw event as/sys/devices/cpu/events/cpu-cycles exists.A fix was necessary to config_term_pmu in parse-events.c as check_aliasexpansion needs to happen after config_term_pmu, and config_term_pmu mayneed calling a second time because of this.config_term_pmu is updated to not use the legacy event when the PMU hassuch a named event (either from JSON or sysfs).The bulk of this change is updating all of the parse-events testexpectations so that if a sysfs/JSON event exists for a PMU the testdoesn't fail - a further sign, if it were needed, that the legacy eventpriority was a known and tested behavior of the perf tool.Reported-by: Hector Martin Signed-off-by: Ian Rogers Tested-by: Hector Martin Tested-by: Marc Zyngier Acked-by: Mark Rutland Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Ingo Molnar Cc: James Clark Cc: Jiri Olsa Cc: Kan Liang Cc: Namhyung Kim Cc: Peter Zijlstra Link: https://lore.kernel.org/r/20231123042922.834425-1-irogers@google.com[ Initialize the 'alias_rewrote_terms' variable to false to address a clang warning ]Signed-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

7ff7b7af - perf tools: Fix spelling mistake "parametrized" -> "parameterized"

Tue, 03 Oct 2023 07:49:11 +0000

perf tools: Fix spelling mistake "parametrized" -> "parameterized"There are spelling mistakes in comments and a pr_debug message. Fix them.Reviewed-by: Athira Jajeev Reviewed-by: Ian Rogers Signed-off-by: Colin Ian King Cc: Alexander Shishkin Cc: Ingo Molnar Cc: Jiri Olsa Cc: Mark Rutland Cc: Peter Zijlstra Cc: kernel-janitors@vger.kernel.orgLink: https://lore.kernel.org/r/20231003074911.220216-1-colin.i.king@gmail.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

ee33a0ef - perf test: Fix parse-events tests to skip parametrized events

Wed, 27 Sep 2023 18:17:03 +0000

perf test: Fix parse-events tests to skip parametrized eventsTestcase "Parsing of all PMU events from sysfs" parse events forall PMUs, and not just cpu. In case of powerpc, the PowerVMenvironment supports events from hv_24x7 and hv_gpci PMU whichis of example format like below:- hv_24x7/CPM_ADJUNCT_INST,domain=?,core=?/- hv_gpci/event,partition_id=?/The value for "?" needs to be filled in depending on systemconfiguration. It is better to skip these parametrized eventsin this test as it is done in:'commit b50d691e50e6 ("perf test: Fix "all PMU test" to skipparametrized events")' which handled a simialr instance with"all PMU test".Fix parse-events test to skip parametrized events sinceit needs proper setup of the parameters.Signed-off-by: Athira Rajeev Tested-by: Ian Rogers Tested-by: Sachin Sant Reviewed-by: Kajol Jain Cc: maddy@linux.ibm.comCc: disgoel@linux.vnet.ibm.comCc: linuxppc-dev@lists.ozlabs.orgLink: https://lore.kernel.org/r/20230927181703.80936-1-atrajeev@linux.vnet.ibm.comSigned-off-by: Namhyung Kim List of files: /linux-6.15/tools/perf/tests/parse-events.c

0d3f0e6f - perf parse-events: Introduce 'struct parse_events_terms'

Fri, 01 Sep 2023 23:39:49 +0000

perf parse-events: Introduce 'struct parse_events_terms'parse_events_terms() existed in function names but was passed a'struct list_head'.As many parse_events functions take an evsel_config list as well as aparse_event_term list, and the naming head_terms and head_config isinconsistent, there's a potential to switch the lists and get errors.Introduce a 'struct parse_events_terms', that just wraps a list_head, toavoid this. Add the regular init/exit functions and transition the codeto use them.Reviewed-by: James Clark Signed-off-by: Ian Rogers Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Ingo Molnar Cc: Jiri Olsa Cc: Kan Liang Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Rob Herring Link: https://lore.kernel.org/r/20230901233949.2930562-6-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

9ea150a8 - perf parse-events: Fixes relating to no_value terms

Fri, 01 Sep 2023 23:39:44 +0000

perf parse-events: Fixes relating to no_value termsA term may have no value in which case it is assumed to have a valueof 1. It doesn't just apply to alias/event terms so change theparse_events_term__to_strbuf assert.Commit 99e7138eb7897aa0 ("perf tools: Fail on using multiple bits longterms without value") made it so that no_value terms could only be for asingle bit. Prior to commit 64199ae4b8a3 ("perf parse-events: Fixpropagation of term's no_value when cloning") this missed a test casewhere config1 had no_value.Fixes: 64199ae4b8a36038 ("perf parse-events: Fix propagation of term's no_value when cloning")Signed-off-by: Ian Rogers Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Ian Rogers Cc: Ingo Molnar Cc: James Clark Cc: Jiri Olsa Cc: Kan Liang Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Rob Herring Link: https://lore.kernel.org/r/20230901233949.2930562-1-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

edb217ff - perf pmu: Parse sysfs events directly from a file

Thu, 24 Aug 2023 04:13:21 +0000

perf pmu: Parse sysfs events directly from a fileRather than read a sysfs events file into a 256 byte char buffer, passthe FILE* directly to the lex/yacc parser.This avoids there being a maximum events file size.While changing the API, constify some arguments to remove unnecessarycasts.Allocating the read buffer decreases the performance of pmu-scan byaround 3%.Signed-off-by: Ian Rogers Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Gaosheng Cui Cc: Ingo Molnar Cc: James Clark Cc: Jing Zhang Cc: Jiri Olsa Cc: John Garry Cc: Kajol Jain Cc: Kan Liang Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Ravi Bangoria Cc: Rob Herring Link: https://lore.kernel.org/r/20230824041330.266337-10-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

07d2b820 - perf test parse-events: Test complex name has required event format

Thu, 06 Jul 2023 18:37:05 +0000

perf test parse-events: Test complex name has required event formattest__checkevent_complex_name will use an "event" format which if notpresent, such as with a placeholder PMU, will cause test failures. Skipthe test in this case to avoid failures in restricted environments.Add perf_pmu__has_format utility as a general PMU utility.Fixes: 628eaa4e877af823 ("perf pmus: Add placeholder core PMU")Signed-off-by: Ian Rogers Tested-by: Thomas Richter Cc: Adrian Hunter Cc: Alexander Shishkin Cc: Ian Rogers Cc: Ingo Molnar Cc: James Clark Cc: Jiri Olsa Cc: Kan Liang Cc: Mark Rutland Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Ravi Bangoria Cc: Rob Herring Link: https://lore.kernel.org/r/20230706183705.601412-2-irogers@google.comSigned-off-by: Arnaldo Carvalho de Melo List of files: /linux-6.15/tools/perf/tests/parse-events.c

bcd981db - perf test: Fix event parsing test when PERF_PMU_CAP_EXTENDED_HW_TYPE isn't supported.

Wed, 05 Jul 2023 08:26:52 +0000

perf test: Fix event parsing test when PERF_PMU_CAP_EXTENDED_HW_TYPE isn't supported.Arm has multiple PMU types for heterogeneous systems, but doesn'tcurrently support PERF_PMU_CAP_EXTENDED_HW_TYPE. Make the testssupport both scenarios so that they pass on Arm, and will still passonce PERF_PMU_CAP_EXTENDED_HW_TYPE support is added.Fixes: 27c9fcfc1e14 ("perf test: Update parse-events expectations to test for multiple events")Acked-by: Ian Rogers Signed-off-by: James Clark Cc: Mark Rutland Cc: Peter Zijlstra Cc: Adrian Hunter Cc: Arnaldo Carvalho de Melo Cc: Jiri Olsa Cc: Alexander Shishkin Cc: Ingo Molnar Link: https://lore.kernel.org/r/20230705082653.23566-3-james.clark@arm.comSigned-off-by: Namhyung Kim List of files: /linux-6.15/tools/perf/tests/parse-events.c