linux-kernel - Re: [BUGFIX PATCH V2 0/6] perf/probe: Additional fixes for range only functions

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] [day] [month] [year] [list]

Message-Id: <20191028235347.8167bb3b56eeb86be3d93606@kernel.org>
Date:   Mon, 28 Oct 2019 23:53:47 +0900
From:   Masami Hiramatsu <mhiramat@...nel.org>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     Jiri Olsa <jolsa@...hat.com>, Namhyung Kim <namhyung@...nel.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [BUGFIX PATCH V2 0/6] perf/probe: Additional fixes for range
 only functions

On Mon, 28 Oct 2019 11:16:55 -0300
Arnaldo Carvalho de Melo <acme@...nel.org> wrote:

> Em Sat, Oct 26, 2019 at 06:00:19PM +0900, Masami Hiramatsu escreveu:
> > Hi Arnaldo,
> > 
> > I've updated examples in patch description in this v2.
> > No changes in the code itself. I ran it on Ubuntu 19.04
> > (linux-5.0.0-32-generic).
> > 
> > Please replace previous one with this.
> 
> Can you please take a look at my perf/core branch? I have these already
> there:
> 
> 5e72f4349eff perf probe: Fix to show ranges of variables in functions without entry_pc
> 50fc0fda5f2c perf probe: Fix to show inlined function callsite without entry_pc
> d7bf48229b85 perf probe: Fix to list probe event with correct line number
> 39cee497850a perf probe: Fix to probe an inline function which has no entry pc
> 6150bad27ebd perf probe: Fix to probe a function which has no entry pc
> fdaea9eea92d perf probe: Fix wrong address verification
> 
> And I added committer notes doing the tests.

Yes, I confirmed your comments on the patches. It looks good to me.

Thank you!

> 
> - Arnaldo
>  
> > Thank you,
> > 
> > ---
> > 
> > Masami Hiramatsu (6):
> >       perf/probe: Fix wrong address verification
> >       perf/probe: Fix to probe a function which has no entry pc
> >       perf/probe: Fix to probe an inline function which has no entry pc
> >       perf/probe: Fix to list probe event with correct line number
> >       perf/probe: Fix to show inlined function callsite without entry_pc
> >       perf/probe: Fix to show ranges of variables in functions without entry_pc
> > 
> > 
> >  tools/perf/util/dwarf-aux.c    |    6 +++---
> >  tools/perf/util/probe-finder.c |   40 ++++++++++++++--------------------------
> >  2 files changed, 17 insertions(+), 29 deletions(-)
> > 
> > --
> > Masami Hiramatsu (Linaro) <mhiramat@...nel.org>
> 
> -- 
> 
> - Arnaldo


-- 
Masami Hiramatsu <mhiramat@...nel.org>