From: ashwinc@codeaurora.org (Ashwin Chaugule)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC] Extending ARM perf-events for multiple PMUs
Date: Mon, 11 Apr 2011 13:44:14 -0400 [thread overview]
Message-ID: <4DA33DEE.9080709@codeaurora.org> (raw)
In-Reply-To: <1302521393.24286.66.camel@e102144-lin.cambridge.arm.com>
Hi Will,
On 4/11/2011 7:29 AM, Will Deacon wrote:
>> I don't see this distinction, both will have to count, and telling it
>> what to count is a function of perf_event_attr::config* and how the
>> hardware implements that is of no interest.
>
> Sure, fundamentally we're just writing bits rather than interpreting
> them. The reason I mention the difference is that filtering PMUs will
> always need their own struct pmu because of the lack of an event
> namespace. The other problem is only an issue for some userspace tools
> (like Oprofile) which require lists of events and their hex codes.
>
If you mean namespace = perf_event_attr::config, its 64 bits + another 64
bits of config_base + event_base on ARM ? Not too sure, but it would seem
like that should be enough to setup such event chaining.
>
> Would this result in userspace attributing all of the data to a
> particular CPU? We could consider allowing events where the cpu is -1
> and the task pid is -1 as well. Non system-wide PMUs could reject these
> and demand multiple events instead.
Agreed. perf stat -a on PMU's that are not CPU-aware, would report
incorrect output. Task counting on such PMU's would be pointless.
Cheers,
Ashwin
--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
next prev parent reply other threads:[~2011-04-11 17:44 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-08 17:15 [RFC] Extending ARM perf-events for multiple PMUs Will Deacon
2011-04-08 18:10 ` Linus Walleij
2011-04-11 11:12 ` Will Deacon
2011-04-09 11:40 ` Peter Zijlstra
2011-04-11 11:29 ` Will Deacon
2011-04-11 12:47 ` Peter Zijlstra
2011-04-11 17:44 ` Ashwin Chaugule [this message]
2011-04-12 17:45 ` Will Deacon
2011-04-11 18:00 ` Ashwin Chaugule
2011-04-12 7:39 ` Ming Lei
2011-04-12 10:30 ` Peter Zijlstra
2011-04-12 11:12 ` Ming Lei
2011-04-11 17:29 ` Ashwin Chaugule
2011-04-11 18:00 ` Will Deacon
2011-04-11 20:46 ` Ashwin Chaugule
2011-04-12 18:08 ` Will Deacon
2011-04-13 5:09 ` Ashwin Chaugule
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=4DA33DEE.9080709@codeaurora.org \
--to=ashwinc@codeaurora.org \
--cc=linux-arm-kernel@lists.infradead.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 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.