From: "Jin, Yao" <yao.jin@linux.intel.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: acme@kernel.org, jolsa@kernel.org, mingo@redhat.com,
alexander.shishkin@linux.intel.com, Linux-kernel@vger.kernel.org,
ak@linux.intel.com, kan.liang@intel.com, yao.jin@intel.com
Subject: Re: [PATCH] perf annotate: Support to display the LBR data in tui mode
Date: Fri, 23 Feb 2018 17:08:56 +0800 [thread overview]
Message-ID: <164d5cb8-58fc-d77d-ff30-e25049f9299b@linux.intel.com> (raw)
In-Reply-To: <20180223082500.GT25181@hirez.programming.kicks-ass.net>
On 2/23/2018 4:25 PM, Peter Zijlstra wrote:
> On Fri, Feb 23, 2018 at 10:35:58PM +0800, Jin Yao wrote:
>> Unlike the perf report interactive annotate mode, the perf annotate
>> doesn't display the LBR data.
>>
>> perf record -b ...
>> perf annotate function
>>
>> It should show IPC/cycle, but it doesn't.
>
> There is far more than IPC/cycle for the LBR data, so this Changelog is
> misleading.
>
I will change the changelog to make it more clear.
> Also, I think that this patch goes the wrong way, we should reduce the
> divergence of the various modes, not make it worse.
>
I do plan to support stdio mode. While stdio mode needs more changes
than tui mode, so I plan to do it in a follow-up patch.
Posting this patch now is because I want to listen from community first
for this feature. If the tui patch could be accepted, then it's worth
putting more efforts on stdio version. That's my thoughts.
Thanks
Jin Yao
next prev parent reply other threads:[~2018-02-23 9:09 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-23 14:35 [PATCH] perf annotate: Support to display the LBR data in tui mode Jin Yao
2018-02-23 8:25 ` Peter Zijlstra
2018-02-23 9:08 ` Jin, Yao [this message]
2018-02-23 15:29 ` Arnaldo Carvalho de Melo
2018-02-23 16:59 ` Peter Zijlstra
2018-02-23 17:02 ` Andi Kleen
2018-02-27 9:38 ` [PATCH v2] perf annotate: Support to display the IPC/Cycle " Jin Yao
2018-03-08 5:22 ` Jin, Yao
2018-03-08 13:12 ` Arnaldo Carvalho de Melo
2018-03-09 8:57 ` [tip:perf/core] perf annotate: Support to display the IPC/Cycle in TUI mode tip-bot for Jin Yao
2018-02-24 1:40 ` [PATCH] perf annotate: Support to display the LBR data in tui mode Jin, Yao
2018-02-26 13:57 ` Arnaldo Carvalho de Melo
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=164d5cb8-58fc-d77d-ff30-e25049f9299b@linux.intel.com \
--to=yao.jin@linux.intel.com \
--cc=Linux-kernel@vger.kernel.org \
--cc=acme@kernel.org \
--cc=ak@linux.intel.com \
--cc=alexander.shishkin@linux.intel.com \
--cc=jolsa@kernel.org \
--cc=kan.liang@intel.com \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=yao.jin@intel.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.