linux-kernel - Re: [RFC][PATCH 7/7] perf/annotate: Add branch stack / basic block information

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 8 Sep 2016 18:41:35 +0200
From:   Peter Zijlstra <peterz@...radead.org>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     Ingo Molnar <mingo@...nel.org>, linux-kernel@...r.kernel.org,
        andi@...stfloor.org, eranian@...gle.com, jolsa@...nel.org,
        torvalds@...ux-foundation.org, davidcc@...gle.com,
        alexander.shishkin@...ux.intel.com, namhyung@...nel.org,
        kan.liang@...el.com, khandual@...ux.vnet.ibm.com
Subject: Re: [RFC][PATCH 7/7] perf/annotate: Add branch stack / basic block
 information

On Thu, Sep 08, 2016 at 01:18:57PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Fri, Jul 08, 2016 at 06:36:32PM +0200, Peter Zijlstra escreveu:
> > On Fri, Jul 08, 2016 at 06:27:33PM +0200, Peter Zijlstra wrote:
> > 
> > > I've been thinking of filtering all targets and branches that are
> > > smaller than 0.1% in order to avoid this, but so far I've just been
> > > ignoring these things.
> > 
> > Like so... seems to 'work'.
> 
> So I merged this one with 7/7 and this is the result, screenshot to
> capture the colors:
> 
>   http://vger.kernel.org/~acme/perf/annotate_basic_blocks.png
> 
> Please let me know if I should go ahead and push with the combined
> patch, that is now at:
> 
> https://git.kernel.org/cgit/linux/kernel/git/acme/linux.git/commit/?h=perf/annotate_basic_blocks&id=baf41a43fa439ac534d21e41882a7858d5cee1e5
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/annotate_basic_blocks
> 
> Is that ok?
> 
> The problem with it is that it is done only for --stdio, I'll check how
> to properly make it UI agnostic...

Yep, much thanks!