From: Song Liu <songliubraving@fb.com>
To: Jiri Olsa <jolsa@redhat.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
Kernel Team <Kernel-team@fb.com>,
"acme@kernel.org" <acme@kernel.org>,
"acme@redhat.com" <acme@redhat.com>,
"namhyung@kernel.org" <namhyung@kernel.org>,
"jolsa@kernel.org" <jolsa@kernel.org>
Subject: Re: [PATCH] perf-stat: introduce bperf, share hardware PMCs with BPF
Date: Fri, 12 Mar 2021 15:45:13 +0000 [thread overview]
Message-ID: <45BCAC33-1626-42D1-A170-92DC8D7BAAF8@fb.com> (raw)
In-Reply-To: <YEtawFnompBDKpK0@krava>
> On Mar 12, 2021, at 4:12 AM, Jiri Olsa <jolsa@redhat.com> wrote:
>
> On Thu, Mar 11, 2021 at 06:02:57PM -0800, Song Liu wrote:
>> perf uses performance monitoring counters (PMCs) to monitor system
>> performance. The PMCs are limited hardware resources. For example,
>> Intel CPUs have 3x fixed PMCs and 4x programmable PMCs per cpu.
>>
>> Modern data center systems use these PMCs in many different ways:
>> system level monitoring, (maybe nested) container level monitoring, per
>> process monitoring, profiling (in sample mode), etc. In some cases,
>> there are more active perf_events than available hardware PMCs. To allow
>> all perf_events to have a chance to run, it is necessary to do expensive
>> time multiplexing of events.
>>
>> On the other hand, many monitoring tools count the common metrics (cycles,
>> instructions). It is a waste to have multiple tools create multiple
>> perf_events of "cycles" and occupy multiple PMCs.
>>
>> bperf tries to reduce such wastes by allowing multiple perf_events of
>> "cycles" or "instructions" (at different scopes) to share PMUs. Instead
>> of having each perf-stat session to read its own perf_events, bperf uses
>> BPF programs to read the perf_events and aggregate readings to BPF maps.
>> Then, the perf-stat session(s) reads the values from these BPF maps.
>>
>> Please refer to the comment before the definition of bperf_ops for the
>> description of bperf architecture.
>>
>> bperf is off by default. To enable it, pass --use-bpf option to perf-stat.
>> bperf uses a BPF hashmap to share information about BPF programs and maps
>> used by bperf. This map is pinned to bpffs. The default address is
>> /sys/fs/bpf/bperf_attr_map. The user could change the address with option
>> --attr-map.
>
> nice, I recall the presentation about that and was wondering
> when this will come up ;-)
The progress is slower than I expected. But I finished some dependencies of
this in the last year:
1. BPF_PROG_TEST_RUN for raw_tp event;
2. perf-stat -b, which introduced skeleton and bpf_counter;
3. BPF task local storage, I didn't use it in this version, but it could,
help optimize bperf in the future.
>
>>
>> ---
>> Known limitations:
>> 1. Do not support per cgroup events;
>> 2. Do not support monitoring of BPF program (perf-stat -b);
>> 3. Do not support event groups.
>>
>> The following commands have been tested:
>>
>> perf stat --use-bpf -e cycles -a
>> perf stat --use-bpf -e cycles -C 1,3,4
>> perf stat --use-bpf -e cycles -p 123
>> perf stat --use-bpf -e cycles -t 100,101
>
> I assume the output is same as standard perf?
Yes, the output is identical to that without --use-bpf option.
Thanks,
Song
next prev parent reply other threads:[~2021-03-12 15:46 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-12 2:02 Song Liu
2021-03-12 8:36 ` Namhyung Kim
2021-03-12 15:38 ` Song Liu
2021-03-13 2:47 ` Namhyung Kim
2021-03-13 19:37 ` Song Liu
2021-03-12 12:12 ` Jiri Olsa
2021-03-12 15:45 ` Song Liu [this message]
2021-03-12 16:09 ` Song Liu
2021-03-13 22:06 ` Jiri Olsa
2021-03-13 23:03 ` Song Liu
2021-03-12 14:24 ` Arnaldo Carvalho de Melo
2021-03-12 16:07 ` Song Liu
2021-03-12 17:37 ` Arnaldo Carvalho de Melo
2021-03-12 18:52 ` Song Liu
2021-03-15 13:10 ` Arnaldo Carvalho de Melo
2021-03-15 15:34 ` Andi Kleen
2021-03-15 18:29 ` Song Liu
2021-03-14 22:48 ` Jiri Olsa
2021-03-15 7:51 ` Song Liu
2021-03-15 14:09 ` Jiri Olsa
2021-03-15 18:24 ` Song Liu
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=45BCAC33-1626-42D1-A170-92DC8D7BAAF8@fb.com \
--to=songliubraving@fb.com \
--cc=Kernel-team@fb.com \
--cc=acme@kernel.org \
--cc=acme@redhat.com \
--cc=jolsa@kernel.org \
--cc=jolsa@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=namhyung@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®