Re: [GIT PULL 0/3] perf/annotate improvements

[Ingo Molnar]

* Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxx> wrote:

> Hi Ingo,
> 
> 	Please consider pulling,
> 
> 	Just a bit on top of yesterday's pile.
> 
> - Arnaldo
> 
> The following changes since commit 0822cc80d9aee026b1ebe43c02dc01e0a0227864:
> 
>   perf annotate browser: Don't display 0.00 percentages (2012-04-27 17:13:53 -0300)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux perf/annotate
> 
> for you to fetch changes up to 64aa17ca5a4e428fcb6d0806823a99a18c548506:
> 
>   perf annotate browser: Don't change the asm line color when toggling source (2012-05-03 13:23:00 -0300)
> 
> ----------------------------------------------------------------
> Perf annotate browser improvements.
> 
> . Get back the line separating the overheads from the disassembly, requested by
>   Peter Zijlstra, Linus agreed now that it is a solid line and more column real
>   state was harvested. Also it has the jump->arrow lines separated from it by
>   the address/jump target column.
> 
> . Don't change asm line color when toggling source code view. Requested by
>   Peter Zijlstra.
> 
> Current snapshot:
> 
> avtab_search_node
>        â      push   %rbp
>        â      mov    %rsp,%rbp
>        â    â callq  mcount
>        â      movzwl 0x6(%rsi),%edx
>        â      and    $0x7fff,%dx
>        â      test   %rdi,%rdi
>        â    â jne    20
>   0.42 â17:âââxor    %eax,%eax
>        â19:â  leaveq
>   0.42 â   ââ retq
>        â   â  nopl   0x0(%rax,%rax,1)
>        â20:â  mov    (%rdi),%rax
>   0.08 â   â  test   %rax,%rax
>        â   âââje     17
>        â      movzwl (%rsi),%ecx
>        â      movzwl 0x2(%rsi),%r9d
>        â      movzwl 0x4(%rsi),%r8d
>        â      movzwl %cx,%esi
>        â      movzwl %r9w,%r10d
>        â      shl    $0x9,%esi
>        â      lea    (%rsi,%r10,4),%esi
>        â      lea    (%r8,%rsi,1),%esi
>        â      and    0x10(%rdi),%si
>        â      movzwl %si,%esi
>        â      mov    (%rax,%rsi,8),%rax
>   1.01 â      test   %rax,%rax
>        â    â je     19
>        â      nopw   0x0(%rax,%rax,1)
>   3.19 â60:   cmp    %cx,(%rax)
>        â    â jne    7e
>   0.08 â      cmp    %r9w,0x2(%rax)
>        â    â jne    7e
>        â      cmp    %r8w,0x4(%rax)
>        â    â jne    79
>        â      test   %dx,0x6(%rax)
>        â    â jne    19
>        â79:   cmp    %r8w,0x4(%rax)
>  83.45 â7e: â ja     17
>   3.36 â      mov    0x10(%rax),%rax
>   7.98 â      test   %rax,%rax
>        â    â jne    60
>        â      leaveq
>        â    â retq
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
> 
> ----------------------------------------------------------------
> Arnaldo Carvalho de Melo (3):
>       perf ui browser: Introduce routine to draw vertical line
>       perf annotate browser: More clearly separate columns
>       perf annotate browser: Don't change the asm line color when toggling source
> 
>  tools/perf/ui/browser.c           |   23 ++++++++++++-----
>  tools/perf/ui/browser.h           |    4 ++-
>  tools/perf/ui/browsers/annotate.c |   51 ++++++++++++++++++++-----------------
>  3 files changed, 47 insertions(+), 31 deletions(-)

Hm, this regresses for some symbols, the annotation window only 
shows this:

__strlen_sse2_pminub                                                                               
       â
       â

It used to show the full thing:

__strlen_sse2_pminub                                                                               
    0.00 :          392a35dbde:   jne    392a35dc30 
<__strlen_sse2_pminub+0x2a0>                  â
    0.00 :          392a35dbe0:   pcmpeqb 0x40(%rax),%xmm3                                        
â
    0.00 :          392a35dbe5:   pmovmskb %xmm3,%edx                                             
â
    0.00 :          392a35dbe9:   sub    %rdi,%rax                                                
â
    0.00 :          392a35dbec:   bsf    %rdx,%rdx                                                
â
    0.00 :          392a35dbf0:   add    %rdx,%rax                                                
â
    0.00 :          392a35dbf3:   add    $0x40,%rax                                               
â
    0.00 :          392a35dbf7:   retq                                           

Other symbols are fine.

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/