[PATCH V2 0/3] perf tests: Fix Track with sched_switch test for hybrid case

Adrian Hunter posted 3 patches 3 years, 8 months ago
tools/perf/arch/arm/util/cs-etm.c       |  2 +-
tools/perf/arch/arm64/util/arm-spe.c    |  2 +-
tools/perf/arch/x86/tests/intel-cqm.c   |  2 +-
tools/perf/arch/x86/util/intel-bts.c    |  2 +-
tools/perf/arch/x86/util/intel-pt.c     |  2 +-
tools/perf/arch/x86/util/iostat.c       |  2 +-
tools/perf/arch/x86/util/topdown.c      |  2 +-
tools/perf/tests/code-reading.c         |  2 +-
tools/perf/tests/event-times.c          |  2 +-
tools/perf/tests/evsel-roundtrip-name.c |  4 ++--
tools/perf/tests/hists_cumulate.c       |  2 +-
tools/perf/tests/hists_filter.c         |  4 ++--
tools/perf/tests/hists_link.c           |  4 ++--
tools/perf/tests/hists_output.c         |  2 +-
tools/perf/tests/keep-tracking.c        |  4 ++--
tools/perf/tests/perf-time-to-tsc.c     |  2 +-
tools/perf/tests/switch-tracking.c      | 24 ++++++++++++++++--------
tools/perf/util/bpf-loader.c            |  2 +-
tools/perf/util/parse-events.c          | 22 ++++++++++++++++++----
tools/perf/util/parse-events.h          |  4 ++++
tools/perf/util/perf_api_probe.c        |  2 +-
tools/perf/util/record.c                |  2 +-
22 files changed, 61 insertions(+), 35 deletions(-)
[PATCH V2 0/3] perf tests: Fix Track with sched_switch test for hybrid case
Posted by Adrian Hunter 3 years, 8 months ago
Hi

Here are a couple of fixes (V2) related to the Track with sched_switch test,
and a subsequent tidy-up.


Changes in V2:

      perf parse-events: Fix segfault when event parser gets an error
	Amend commit message to refer to subsequent tidy-up patch

      perf tools: Do not pass NULL to parse_events()
	New patch


Adrian Hunter (3):
      perf parse-events: Fix segfault when event parser gets an error
      perf tests: Fix Track with sched_switch test for hybrid case
      perf tools: Do not pass NULL to parse_events()

 tools/perf/arch/arm/util/cs-etm.c       |  2 +-
 tools/perf/arch/arm64/util/arm-spe.c    |  2 +-
 tools/perf/arch/x86/tests/intel-cqm.c   |  2 +-
 tools/perf/arch/x86/util/intel-bts.c    |  2 +-
 tools/perf/arch/x86/util/intel-pt.c     |  2 +-
 tools/perf/arch/x86/util/iostat.c       |  2 +-
 tools/perf/arch/x86/util/topdown.c      |  2 +-
 tools/perf/tests/code-reading.c         |  2 +-
 tools/perf/tests/event-times.c          |  2 +-
 tools/perf/tests/evsel-roundtrip-name.c |  4 ++--
 tools/perf/tests/hists_cumulate.c       |  2 +-
 tools/perf/tests/hists_filter.c         |  4 ++--
 tools/perf/tests/hists_link.c           |  4 ++--
 tools/perf/tests/hists_output.c         |  2 +-
 tools/perf/tests/keep-tracking.c        |  4 ++--
 tools/perf/tests/perf-time-to-tsc.c     |  2 +-
 tools/perf/tests/switch-tracking.c      | 24 ++++++++++++++++--------
 tools/perf/util/bpf-loader.c            |  2 +-
 tools/perf/util/parse-events.c          | 22 ++++++++++++++++++----
 tools/perf/util/parse-events.h          |  4 ++++
 tools/perf/util/perf_api_probe.c        |  2 +-
 tools/perf/util/record.c                |  2 +-
 22 files changed, 61 insertions(+), 35 deletions(-)


Regards
Adrian
Re: [PATCH V2 0/3] perf tests: Fix Track with sched_switch test for hybrid case
Posted by Arnaldo Carvalho de Melo 3 years, 8 months ago
Em Tue, Aug 09, 2022 at 11:06:59AM +0300, Adrian Hunter escreveu:
> Hi
> 
> Here are a couple of fixes (V2) related to the Track with sched_switch test,
> and a subsequent tidy-up.
> 
> 
> Changes in V2:
> 
>       perf parse-events: Fix segfault when event parser gets an error
> 	Amend commit message to refer to subsequent tidy-up patch
> 
>       perf tools: Do not pass NULL to parse_events()
> 	New patch

Thanks, applied to my local branch.

- Arnaldo

 
> 
> Adrian Hunter (3):
>       perf parse-events: Fix segfault when event parser gets an error
>       perf tests: Fix Track with sched_switch test for hybrid case
>       perf tools: Do not pass NULL to parse_events()
> 
>  tools/perf/arch/arm/util/cs-etm.c       |  2 +-
>  tools/perf/arch/arm64/util/arm-spe.c    |  2 +-
>  tools/perf/arch/x86/tests/intel-cqm.c   |  2 +-
>  tools/perf/arch/x86/util/intel-bts.c    |  2 +-
>  tools/perf/arch/x86/util/intel-pt.c     |  2 +-
>  tools/perf/arch/x86/util/iostat.c       |  2 +-
>  tools/perf/arch/x86/util/topdown.c      |  2 +-
>  tools/perf/tests/code-reading.c         |  2 +-
>  tools/perf/tests/event-times.c          |  2 +-
>  tools/perf/tests/evsel-roundtrip-name.c |  4 ++--
>  tools/perf/tests/hists_cumulate.c       |  2 +-
>  tools/perf/tests/hists_filter.c         |  4 ++--
>  tools/perf/tests/hists_link.c           |  4 ++--
>  tools/perf/tests/hists_output.c         |  2 +-
>  tools/perf/tests/keep-tracking.c        |  4 ++--
>  tools/perf/tests/perf-time-to-tsc.c     |  2 +-
>  tools/perf/tests/switch-tracking.c      | 24 ++++++++++++++++--------
>  tools/perf/util/bpf-loader.c            |  2 +-
>  tools/perf/util/parse-events.c          | 22 ++++++++++++++++++----
>  tools/perf/util/parse-events.h          |  4 ++++
>  tools/perf/util/perf_api_probe.c        |  2 +-
>  tools/perf/util/record.c                |  2 +-
>  22 files changed, 61 insertions(+), 35 deletions(-)
> 
> 
> Regards
> Adrian

-- 

- Arnaldo