login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2010
»
March
»
11
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent symbolic event names to "perf stat"
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From: Ingo Molnar
Subject:
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent symbolic event names to "perf stat"
Date: Thursday, March 11, 2010 - 12:14 pm
* Corey Ashford <cjashfor@linux.vnet.ibm.com> wrote:
quoted text
> On 3/11/2010 4:46 AM, Ingo Molnar wrote: > [snip] > >If you want extensible events you can already do it by providing an ftrace > >tracepoint event via TRACE_EVENT. They are easy to add and ad-hoc, and are > >supported throughout by perf. > > Is TRACE_EVENT an appropriate way to add hardware-specific counter > events? I will look into this. Thanks for the pointer. > > > > >That could be librarized further by providing an /eventfs or /proc/events > >interface to enumerate them. > > We can enumerate events this way, but there are other aspects to > events than just their names (see below). > > > > >Or if you want to extend the perf events namespace ABI you can send patches > >for that as well. (It's not a big issue if a particular event is currently > >only supported on Power for example - as long as you make a good effort naming > >and structuring it in a reasonably generic way.) > > I'm not sure how that would work. The issue I am trying to solve > here is that Power arch chips have a large number of very > hardware-specific events that are not generalizable. Many of these > events not only have names, but other user-configurable bits as well > that select or narrow the scope of which exact events are recorded. > This issue is dealt with nicely in libpfm4, as it has mechanisms for > parsing event names and attributes (aka modifiers or unit masks), > and then produces a usable config field for the perf_events_attr > struct. > > Should I take it from the above that you are completely against the > idea of using an external library for hardware-specific event and > attribute naming?
Could you give a few relevant examples of events in question, and the kind of configurability/attributes they have on Power? Thanks, Ingo --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
[RFC] [PATCH 1/1] perf: add support for arch-dependent sym ...
, Corey Ashford
, (Wed Mar 3, 7:30 pm)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Corey Ashford
, (Thu Mar 4, 11:39 am)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Corey Ashford
, (Fri Mar 5, 10:42 am)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Ingo Molnar
, (Thu Mar 11, 5:46 am)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Corey Ashford
, (Thu Mar 11, 11:47 am)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Ingo Molnar
, (Thu Mar 11, 12:14 pm)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Corey Ashford
, (Thu Mar 11, 1:46 pm)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Paul Mackerras
, (Thu Mar 11, 7:41 pm)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Corey Ashford
, (Thu Mar 11, 11:53 pm)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Corey Ashford
, (Mon Mar 15, 4:38 pm)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Ingo Molnar
, (Tue Mar 16, 2:34 am)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Ingo Molnar
, (Tue Mar 16, 2:40 am)
Re: [RFC] [PATCH 1/1] perf: add support for arch-dependent ...
, Corey Ashford
, (Tue Mar 16, 11:24 am)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Mel Gorman
Re: [PATCH 1/4] vmstat: remove zone->lock from walk_zones_in_node
Guenter Roeck
Re: [lm-sensors] Location for thermal drivers
David Woodhouse
Re: RFC: Moving firmware blobs out of the kernel.
Siddha, Suresh B
Re: [PATCH 2.6.21 review I] [11/25] x86: default to physical mode on hotplug CPU k...
Peter Zijlstra
Re: [patch 4/6] mm: merge populate and nopage into fault (fixes nonlinear)
git-commits-head
:
Linux Kernel Mailing List
[MIPS] Fix potential latency problem due to non-atomic cpu_wait.
Linux Kernel Mailing List
USB: rename USB_SPEED_VARIABLE to USB_SPEED_WIRELESS
Linux Kernel Mailing List
lib/vsprintf.c: fix bug omitting minus sign of numbers (module_param)
Linux Kernel Mailing List
[Bluetooth] Initiate authentication during connection establishment
Linux Kernel Mailing List
[POWERPC] 4xx: Add ppc40x_defconfig
linux-netdev
:
MERCEDES
Your mail id has won 950,000.00 in the MERCEDES Benz Online Promo.for claims send:
David Miller
Re: [PATCH] xen/netfront: do not mark packets of length < MSS as GSO
David Miller
Re: skb_segment() questions
Shan Wei
[RFC PATCH net-next 2/5]IPv6:netfilter: Send an ICMPv6 "Fragment Reassembly Timeou...
Stanislaw Gruszka
[PATCH 1/4] bnx2x: use smp_mb() to keep ordering of read write operations
git
:
Nicolas Sebrecht
git-svn died of signal 11 (was "3 failures on test t9100 (svn)")
Junio C Hamano
Re: [PATCH 2/2] Add url.<base>.pushInsteadOf: URL rewriting for push only
Martin Langhoff
Re: [PATCH] GIT commit statistics.
Alexandre Julliard
[PATCH] gitweb: Put back shortlog instead of graphiclog in the project list.
Josh Triplett
[PATCH 2/2] Add url.<base>.pushInsteadOf: URL rewriting for push only
openbsd-misc
:
Taisto Qvist XX
Re: AMD GEODE LX-800 just works with kernel from install42.iso and kernelpanics wi...
Nico Meijer
Re: gOS Develop Kit with VIA pc-1 Processor Platform VIA C7-D
Andreas Bihlmaier
Re: jetway board sensors (Fintek F71805F)
admin
Drive a 2009 car from R799p/m
Antti Harri
Re: how to create a sha256 hash
Colocation donated by:
Syndicate