linux-kernel - Re: (ltc-kernel 10702) [PATCH perf/core v3 0/8] perf-probe: Add filtering features

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: <5540071C.50007@hitachi.com>
Date:	Wed, 29 Apr 2015 07:18:04 +0900
From:	Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
To:	Arnaldo Carvalho de Melo <acme@...nel.org>
CC:	Peter Zijlstra <peterz@...radead.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	David Ahern <dsahern@...il.com>, namhyung@...nel.org,
	Jiri Olsa <jolsa@...hat.com>, Ingo Molnar <mingo@...nel.org>
Subject: Re: (ltc-kernel 10702) [PATCH perf/core v3 0/8] perf-probe: Add filtering
 features

Ping?

On 2015/04/24 18:47, Masami Hiramatsu wrote:
> Hi,
> 
> Here is a series of patches which improves perf-probe to add
> filtering features for --list,--del and --funcs. This also
> includes some bugfixes and cleanups.
> 
> I've fixed some patches according to Namhyung's review(Thanks!),
> and splitted the strfilter related patches.
> 
> Note that first 2 bugfixes are removed from this series, those
> are already picked by Arnaldo.
> 
> E.g. --list/--funcs accepts filter rules.
>   ----
>   # ./perf probe -l vfs\*
>     probe:vfs_read       (on vfs_read@...c/linux-3/fs/read_write.c)
>   # ./perf probe -l \*libc:\*
>     probe_libc:malloc    (on __libc_malloc@...loc/malloc.c in /usr/lib64/libc-2.17.so)
>   ----
>   ----
>   # ./perf probe -F *kmalloc
>   __kmalloc
>   devm_kmalloc
>   mempool_kmalloc
>   sg_kmalloc
>   sock_kmalloc
>   ----
> 
> Also, --del accepts filter rules too which is more flexible than
> simple wildcard.
>   ----
>   # ./perf probe -d 'vfs*|malloc'
>   Removed event: probe:vfs_read
>   Removed event: probe_libc:malloc
>   ----
> 
> Changes in v3:
>   - [1/8] Split strfilter__or() and add strfilter__and() from v2 [3/8].
>            Also rename strfilter__add() to strfilter__append().
>   - [2/8] Split strfilter__string from v2 [5/8].
> 
> Changes in v2:
>   - [2/8] Fix to check errors except -ENOENT on deleting events.
>   - [5/8] Ditto.
>   - [5/8] Fix to check memory allocation error in strfilter__string.
>   - [5/8] Fix to check errors of strfilter__string for verifying
>           given filter.
> 
> Thank you,
> 
> 
> ---
> 
> Masami Hiramatsu (8):
>       perf: Improve strfilter to append additional rules
>       perf: Add strfilter__string to recover rules string
>       perf probe: Accept multiple filter options
>       perf probe: Accept filter argument for --list
>       perf probe: Allow to use filter on --del command
>       perf probe: Accept filter argument for --funcs
>       perf probe: Remove redundant cleanup of params.filter
>       perf probe: Cleanup and consolidate command parsers
> 
> 
>  tools/perf/Documentation/perf-probe.txt |    9 +-
>  tools/perf/builtin-probe.c              |  160 +++++++++++++++----------------
>  tools/perf/util/probe-event.c           |  126 +++++++++++-------------
>  tools/perf/util/probe-event.h           |    4 -
>  tools/perf/util/strfilter.c             |  107 +++++++++++++++++++++
>  tools/perf/util/strfilter.h             |   35 +++++++
>  6 files changed, 281 insertions(+), 160 deletions(-)
> 
> 


-- 
Masami HIRAMATSU
Linux Technology Research Center, System Productivity Research Dept.
Center for Technology Innovation - Systems Engineering
Hitachi, Ltd., Research & Development Group
E-mail: masami.hiramatsu.pt@...achi.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/