Re: [PATCH v3 0/4] Fixes for setting event freq/periods

Jiri Olsa
2020-09-12T20:49:46Z

On Fri, Sep 11, 2020 at 07:56:51PM -0700, Ian Rogers wrote:
> Some fixes that address issues for regular and pfm4 events with 2
> additional perf_event_attr tests. Various authors, David Sharp isn't
> currently at Google.
>
> v3. moved a loop into a helper following Adrian Hunter's suggestion.
> v2. corrects the commit message following Athira Rajeev's suggestion.
>
> David Sharp (1):
> perf record: Set PERF_RECORD_PERIOD if attr->freq is set.
>
> Ian Rogers (2):
> perf record: Don't clear event's period if set by a term
> perf test: Leader sampling shouldn't clear sample period
>
> Stephane Eranian (1):
> perf record: Prevent override of attr->sample_period for libpfm4
> events

Acked-by: Jiri Olsa <jolsa@redhat.com>

thanks,
jirka

>
> tools/perf/tests/attr/README | 1 +
> tools/perf/tests/attr/test-record-group2 | 29 ++++++++++++++++++++
> tools/perf/util/evsel.c | 10 ++++---
> tools/perf/util/record.c | 34 ++++++++++++++++++------
> 4 files changed, 63 insertions(+), 11 deletions(-)
> create mode 100644 tools/perf/tests/attr/test-record-group2
>
> --
> 2.28.0.618.gf4bc123cb7-goog
>