From: Greg KH <greg@kroah.com>
To: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>,
robert.richter@amd.com, Anton Blanchard <anton@au1.ibm.com>,
linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org,
Ingo Molnar <mingo@redhat.com>, Paul Mackerras <paulus@samba.org>,
Arnaldo Carvalho de Melo <acme@ghostprotocols.net>,
Jiri Olsa <jolsa@redhat.com>
Subject: Re: [PATCH 3/5] perf/POWER7: Make generic event translations available in sysfs
Date: Wed, 19 Dec 2012 21:36:00 -0800 [thread overview]
Message-ID: <20121220053600.GC28007@kroah.com> (raw)
In-Reply-To: <20121219072848.GB30790@us.ibm.com>
On Tue, Dec 18, 2012 at 11:28:49PM -0800, Sukadev Bhattiprolu wrote:
>
> [PATCH 3/5] perf/POWER7: Make generic event translations available in sysfs
>
> Make the generic perf events in POWER7 available via sysfs.
>
> $ ls /sys/bus/event_source/devices/cpu/events
> branch-instructions
> branch-misses
> cache-misses
> cache-references
> cpu-cycles
> instructions
> stalled-cycles-backend
> stalled-cycles-frontend
>
> $ cat /sys/bus/event_source/devices/cpu/events/cache-misses
> event=0x400f0
>
> This patch is based on commits that implement this functionality on x86.
> Eg:
> commit a47473939db20e3961b200eb00acf5fcf084d755
> Author: Jiri Olsa <jolsa@redhat.com>
> Date: Wed Oct 10 14:53:11 2012 +0200
>
> perf/x86: Make hardware event translations available in sysfs
>
> Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
> ---
> arch/powerpc/include/asm/perf_event_server.h | 25 +++++++++++++++++++
> arch/powerpc/perf/core-book3s.c | 12 +++++++++
> arch/powerpc/perf/power7-pmu.c | 34 ++++++++++++++++++++++++++
> 3 files changed, 71 insertions(+), 0 deletions(-)
As you are adding new sysfs files, you must also add new
Documentation/ABI entries at the same time. Please also do this for the
other patches in this series that add new sysfs files.
thanks,
greg k-h
WARNING: multiple messages have this Message-ID (diff)
From: Greg KH <greg@kroah.com>
To: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>,
Paul Mackerras <paulus@samba.org>, Ingo Molnar <mingo@redhat.com>,
Arnaldo Carvalho de Melo <acme@ghostprotocols.net>,
Anton Blanchard <anton@au1.ibm.com>, Jiri Olsa <jolsa@redhat.com>,
robert.richter@amd.com, linuxppc-dev@ozlabs.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 3/5] perf/POWER7: Make generic event translations available in sysfs
Date: Wed, 19 Dec 2012 21:36:00 -0800 [thread overview]
Message-ID: <20121220053600.GC28007@kroah.com> (raw)
In-Reply-To: <20121219072848.GB30790@us.ibm.com>
On Tue, Dec 18, 2012 at 11:28:49PM -0800, Sukadev Bhattiprolu wrote:
>
> [PATCH 3/5] perf/POWER7: Make generic event translations available in sysfs
>
> Make the generic perf events in POWER7 available via sysfs.
>
> $ ls /sys/bus/event_source/devices/cpu/events
> branch-instructions
> branch-misses
> cache-misses
> cache-references
> cpu-cycles
> instructions
> stalled-cycles-backend
> stalled-cycles-frontend
>
> $ cat /sys/bus/event_source/devices/cpu/events/cache-misses
> event=0x400f0
>
> This patch is based on commits that implement this functionality on x86.
> Eg:
> commit a47473939db20e3961b200eb00acf5fcf084d755
> Author: Jiri Olsa <jolsa@redhat.com>
> Date: Wed Oct 10 14:53:11 2012 +0200
>
> perf/x86: Make hardware event translations available in sysfs
>
> Signed-off-by: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
> ---
> arch/powerpc/include/asm/perf_event_server.h | 25 +++++++++++++++++++
> arch/powerpc/perf/core-book3s.c | 12 +++++++++
> arch/powerpc/perf/power7-pmu.c | 34 ++++++++++++++++++++++++++
> 3 files changed, 71 insertions(+), 0 deletions(-)
As you are adding new sysfs files, you must also add new
Documentation/ABI entries at the same time. Please also do this for the
other patches in this series that add new sysfs files.
thanks,
greg k-h
next prev parent reply other threads:[~2012-12-20 5:40 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-19 7:26 [PATCH 1/5] perf/Power7: Use macros to identify perf events Sukadev Bhattiprolu
2012-12-19 7:28 ` [PATCH 2/5] perf: Make EVENT_ATTR and EVENT_PTR global Sukadev Bhattiprolu
2013-01-02 14:58 ` Jiri Olsa
2013-01-02 14:58 ` Jiri Olsa
2013-01-05 1:47 ` Sukadev Bhattiprolu
2013-01-05 1:47 ` Sukadev Bhattiprolu
2012-12-19 7:28 ` [PATCH 3/5] perf/POWER7: Make generic event translations available in sysfs Sukadev Bhattiprolu
2012-12-20 5:36 ` Greg KH [this message]
2012-12-20 5:36 ` Greg KH
2012-12-20 7:03 ` Sukadev Bhattiprolu
2012-12-20 7:03 ` Sukadev Bhattiprolu
2012-12-19 7:29 ` [PATCH 4/5] perf/POWER7: Make some POWER7 events " Sukadev Bhattiprolu
2012-12-19 7:30 ` [PATCH 5/5] perf: Create a sysfs entry for Power event format Sukadev Bhattiprolu
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=20121220053600.GC28007@kroah.com \
--to=greg@kroah.com \
--cc=a.p.zijlstra@chello.nl \
--cc=acme@ghostprotocols.net \
--cc=anton@au1.ibm.com \
--cc=jolsa@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=mingo@redhat.com \
--cc=paulus@samba.org \
--cc=robert.richter@amd.com \
--cc=sukadev@linux.vnet.ibm.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.