linux-kernel - Re: [PATCH v2 0/5] perf-probe: Improve probing on versioned symbols

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]

Message-Id: <bbe6b71a-86ac-bc1c-7895-d3bcbe35b782@linux.vnet.ibm.com>
Date:   Fri, 8 Dec 2017 12:56:06 +0100
From:   Thomas-Mich Richter <tmricht@...ux.vnet.ibm.com>
To:     Masami Hiramatsu <mhiramat@...nel.org>,
        Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     bhargavb <bhargavaramudu@...il.com>, linux-kernel@...r.kernel.org,
        Paul Clarke <pc@...ibm.com>,
        Ravi Bangoria <ravi.bangoria@...ux.vnet.ibm.com>,
        linux-rt-users@...r.kernel.org, linux-perf-users@...r.kernel.org
Subject: Re: [PATCH v2 0/5] perf-probe: Improve probing on versioned symbols

On 12/07/2017 08:19 AM, Masami Hiramatsu wrote:
> Hi,
> 
> Here is the 2nd version of the series for probing on
> versioned symbols in libraries. This includes 5 patches
> to fix the issues discussed on perf-users ML 
> (https://www.spinics.net/lists/linux-perf-users/msg04637.html)
> 
> The first version is here; https://lkml.org/lkml/2017/12/5/1124
> 
> Here is the updates;
> 
> [3/5] Add a description in Documentation/perf-probe.txt
> [5/5] Add a description and examples in Documentation/perf-probe.txt
> 
> Thank you,
> ---
> 
> Masami Hiramatsu (5):
>       perf-probe: Add warning message if there is unexpected event name
>       perf-probe: Cut off the version suffix from event name
>       perf-probe: Add __return suffix for return events
>       perf-probe: Find versioned symbols from map
>       perf-probe: Support escaped character in parser
> 
> 
>  tools/perf/Documentation/perf-probe.txt     |   18 ++++++
>  tools/perf/arch/powerpc/util/sym-handling.c |    8 +++
>  tools/perf/util/probe-event.c               |   81 +++++++++++++++++++--------
>  tools/perf/util/string.c                    |   46 +++++++++++++++
>  tools/perf/util/string2.h                   |    2 +
>  tools/perf/util/symbol.c                    |    5 ++
>  tools/perf/util/symbol.h                    |    1 
>  7 files changed, 137 insertions(+), 24 deletions(-)
> 
> --
> Masami Hiramatsu (Linaro Ltd.) <mhiramat@...nel.org>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-perf-users" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

If you fix the findings for patch 4 and 5 you have my 

Reviewed-by: Thomas Richter <tmricht@...ux.vnet.ibm.com>
-- 
Thomas Richter, Dept 3303, IBM LTC Boeblingen Germany
--
Vorsitzende des Aufsichtsrats: Martina Koederitz 
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen / Registergericht: Amtsgericht Stuttgart, HRB 243294