From: Jiri Olsa <jolsa@redhat.com>
To: Ian Rogers <irogers@google.com>
Cc: Peter Zijlstra <peterz@infradead.org>,
Ingo Molnar <mingo@redhat.com>,
Arnaldo Carvalho de Melo <acme@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Namhyung Kim <namhyung@kernel.org>,
Alexei Starovoitov <ast@kernel.org>,
Daniel Borkmann <daniel@iogearbox.net>,
Martin KaFai Lau <kafai@fb.com>, Song Liu <songliubraving@fb.com>,
Yonghong Song <yhs@fb.com>, Andrii Nakryiko <andriin@fb.com>,
John Fastabend <john.fastabend@gmail.com>,
KP Singh <kpsingh@chromium.org>,
Adrian Hunter <adrian.hunter@intel.com>,
Andi Kleen <ak@linux.intel.com>,
Athira Rajeev <atrajeev@linux.vnet.ibm.com>,
linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
bpf@vger.kernel.org, Stephane Eranian <eranian@google.com>
Subject: Re: [PATCH v3 0/4] Fixes for setting event freq/periods
Date: Sat, 12 Sep 2020 22:49:02 +0200 [thread overview]
Message-ID: <20200912204902.GE1714160@krava> (raw)
In-Reply-To: <20200912025655.1337192-1-irogers@google.com>
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
>
prev parent reply other threads:[~2020-09-12 20:49 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-12 2:56 [PATCH v3 0/4] Fixes for setting event freq/periods Ian Rogers
2020-09-12 2:56 ` [PATCH v3 1/4] perf record: Set PERF_RECORD_PERIOD if attr->freq is set Ian Rogers
2020-09-12 2:56 ` [PATCH v3 2/4] perf record: Prevent override of attr->sample_period for libpfm4 events Ian Rogers
2020-09-12 2:56 ` [PATCH v3 3/4] perf record: Don't clear event's period if set by a term Ian Rogers
2020-09-14 6:02 ` Adrian Hunter
2020-09-14 21:46 ` Arnaldo Carvalho de Melo
2020-09-14 21:51 ` Arnaldo Carvalho de Melo
2020-09-14 21:52 ` Ian Rogers
2020-09-14 22:34 ` Arnaldo Carvalho de Melo
2020-09-12 2:56 ` [PATCH v3 4/4] perf test: Leader sampling shouldn't clear sample period Ian Rogers
2020-09-14 22:36 ` Arnaldo Carvalho de Melo
2020-09-12 20:49 ` Jiri Olsa [this message]
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=20200912204902.GE1714160@krava \
--to=jolsa@redhat.com \
--cc=acme@kernel.org \
--cc=adrian.hunter@intel.com \
--cc=ak@linux.intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=andriin@fb.com \
--cc=ast@kernel.org \
--cc=atrajeev@linux.vnet.ibm.com \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=eranian@google.com \
--cc=irogers@google.com \
--cc=john.fastabend@gmail.com \
--cc=kafai@fb.com \
--cc=kpsingh@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=mingo@redhat.com \
--cc=namhyung@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=peterz@infradead.org \
--cc=songliubraving@fb.com \
--cc=yhs@fb.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: link
Be 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.