From: Sukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: jolsa@redhat.com, linux-kernel@vger.kernel.org,
namhyung@kernel.org, acme@kernel.org, mingo@kernel.org,
peterz@infradead.organton, Paul Mackerras <paulus@samba.org>,
Michael Ellerman <michaele@au1.ibm.com>
Subject: Re: perf: Add support for full Intel event lists v8
Date: Mon, 24 Nov 2014 14:37:36 -0800 [thread overview]
Message-ID: <20141124223736.GA13184@us.ibm.com> (raw)
In-Reply-To: <1406762570-16694-1-git-send-email-andi@firstfloor.org>
Andi Kleen [andi@firstfloor.org] wrote:
| Since it seems the download mechanism is controversal,
| I removed the downloader for now until a suitable distribution
| method with kernel.org can be established. This is just
| the rest of the perf JSON event code that should hopefully
| not be controversal.
Ingo, Jiri, Andi,
Can we get this patchset merged please ? It is a step forward
in eliminating raw event codes right ?
We added hundreds of events to sysfs for Power7. We don't
want to do the same for Power8 because this JSON file solution
is clearly better.
But we currently have a regression in functionality from Power7
to Power8 :-(
Even if 'perf download' is left out this patchset is still
useful IMHO. Users (specially those who currently struggle
with raw codes, are highly qualified and) could wget the
JSON files as and when needed.
Being able to install the JSON files and download updates is
a big plus, but is something thse users can live without for
now.
Sukadev
next prev parent reply other threads:[~2014-11-24 22:37 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-30 23:22 perf: Add support for full Intel event lists v8 Andi Kleen
2014-07-30 23:22 ` [PATCH 01/10] perf, tools: Add jsmn `jasmine' JSON parser Andi Kleen
2014-07-30 23:22 ` [PATCH 02/10] perf, tools: Add support for text descriptions of events and alias add Andi Kleen
2014-07-30 23:22 ` [PATCH 03/10] perf, tools, list: Update perf list to output descriptions Andi Kleen
2014-07-30 23:22 ` [PATCH 04/10] perf, tools: Allow events with dot Andi Kleen
2014-07-30 23:22 ` [PATCH 05/10] perf, tools: Add support for reading JSON event files Andi Kleen
2014-07-30 23:22 ` [PATCH 06/10] perf, tools: Automatically look for event file name for cpu Andi Kleen
2014-07-30 23:22 ` [PATCH 07/10] perf, tools: Query terminal width and use in perf list Andi Kleen
2014-07-30 23:22 ` [PATCH 08/10] perf, tools: Add a new pmu interface to iterate over all events Andi Kleen
2014-07-30 23:22 ` [PATCH 09/10] perf, tools, test: Add test case for alias and JSON parsing Andi Kleen
2014-07-30 23:22 ` [PATCH 10/10] perf, tools: Add a --no-desc flag to perf list Andi Kleen
2014-11-24 22:37 ` Sukadev Bhattiprolu [this message]
2014-11-25 9:33 ` perf: Add support for full Intel event lists v8 Jiri Olsa
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=20141124223736.GA13184@us.ibm.com \
--to=sukadev@linux.vnet.ibm.com \
--cc=acme@kernel.org \
--cc=andi@firstfloor.org \
--cc=jolsa@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=michaele@au1.ibm.com \
--cc=mingo@kernel.org \
--cc=namhyung@kernel.org \
--cc=paulus@samba.org \
--cc=peterz@infradead.organton \
/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;
as well as URLs for NNTP newsgroup(s).