Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 21 Feb 2019 17:02:17 -0300
From:   Arnaldo Carvalho de Melo <acme@kernel.org>
To:     Wei Li <liwei391@huawei.com>
Cc:     Jiri Olsa <jolsa@redhat.com>, peterz@infradead.org,
        mingo@redhat.com, alexander.shishkin@linux.intel.com,
        namhyung@kernel.org, yao.jin@linux.intel.com,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] perf: fix getting source line failure in annotate
Message-ID: <20190221200217.GG9755@kernel.org>
References: <20190221095716.39529-1-liwei391@huawei.com>
 <20190221110559.GF10990@krava>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20190221110559.GF10990@krava>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Em Thu, Feb 21, 2019 at 12:05:59PM +0100, Jiri Olsa escreveu:
> On Thu, Feb 21, 2019 at 05:57:16PM +0800, Wei Li wrote:
> 
> SNIP
> 
> > liwei@euler:~/main_code/hulk_work/hulk/tools/perf$ sudo ./perf record ./common_while_1
> > [ perf record: Woken up 2 times to write data ]
> > [ perf record: Captured and wrote 0.488 MB perf.data (12500 samples) ]
> > liwei@euler:~/main_code/hulk_work/hulk/tools/perf$ sudo ./perf annotate -l -s hotspot_1 --stdio
> > 
> > Sorted summary for file /home/liwei/main_code/hulk_work/hulk/tools/perf/common_while_1
> > ----------------------------------------------
> > 
> >    33.34 common_while_1.c:5
> >    33.34 common_while_1.c:6
> >    33.32 common_while_1.c:7
> >  Percent |      Source code & Disassembly of common_while_1 for cycles:ppp (12482 samples, percent: local period)
> > -----------------------------------------------------------------------------------------------------------------
> >          :
> >          :
> >          :
> >          :         Disassembly of section .text:
> >          :
> >          :         00000000000005fa <hotspot_1>:
> >          :         hotspot_1():
> >          :         void hotspot_1(void)
> >          :         {
> >     0.00 :   5fa:   push   %rbp
> >     0.00 :   5fb:   mov    %rsp,%rbp
> >          :                 volatile int i;
> >          :
> >          :                 for (i = 0; i < 0x10000000; i++);
> >     0.00 :   5fe:   movl   $0x0,-0x4(%rbp)
> >     0.00 :   605:   jmp    610 <hotspot_1+0x16>
> >     0.00 :   607:   mov    -0x4(%rbp),%eax
> >  common_while_1.c:5    4.70 :   60a:   add    $0x1,%eax
> >     4.89 :   60d:   mov    %eax,-0x4(%rbp)
> >  common_while_1.c:5   19.03 :   610:   mov    -0x4(%rbp),%eax
> >  common_while_1.c:5    4.72 :   613:   cmp    $0xfffffff,%eax
> >     0.00 :   618:   jle    607 <hotspot_1+0xd>
> >          :                 for (i = 0; i < 0x10000000; i++);
> >     0.00 :   61a:   movl   $0x0,-0x4(%rbp)
> >     0.00 :   621:   jmp    62c <hotspot_1+0x32>
> >     0.00 :   623:   mov    -0x4(%rbp),%eax
> >  common_while_1.c:6    4.54 :   626:   add    $0x1,%eax
> >     4.73 :   629:   mov    %eax,-0x4(%rbp)
> >  common_while_1.c:6   19.54 :   62c:   mov    -0x4(%rbp),%eax
> >  common_while_1.c:6    4.54 :   62f:   cmp    $0xfffffff,%eax
> > ...
> > 
> > Fixes: 425859ff0de33 ("perf annotate: No need to calculate notes->start twice")
> 
> Acked-by: Jiri Olsa <jolsa@kernel.org>

Tested and applied.

Thanks for the Fixes: tag!

- Arnaldo