linux-kernel - Re: [PATCH v9 08/22] tracing: Add 'hist' event trigger command

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: <1437488226.22524.4.camel@tzanussi-mobl.amr.corp.intel.com>
Date:	Tue, 21 Jul 2015 09:17:06 -0500
From:	Tom Zanussi <tom.zanussi@...ux.intel.com>
To:	Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
Cc:	rostedt@...dmis.org, daniel.wagner@...-carit.de,
	namhyung@...nel.org, josh@...htriplett.org, andi@...stfloor.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v9 08/22] tracing: Add 'hist' event trigger command

Hi Masami,

On Mon, 2015-07-20 at 22:37 +0900, Masami Hiramatsu wrote:
> Hi Tom,
> 
> On 2015/07/17 2:22, Tom Zanussi wrote:
> 
> > @@ -3782,6 +3785,32 @@ static const char readme_msg[] =
> >  	"\t   To remove a trigger with a count:\n"
> >  	"\t     echo '!<trigger>:0 > <system>/<event>/trigger\n"
> >  	"\t   Filters can be ignored when removing a trigger.\n"
> > +#ifdef CONFIG_HIST_TRIGGERS
> > +	"      hist trigger\t- If set, event hits are aggregated into a hash table\n"
> > +	"\t    Format: hist:keys=<field1>\n"
> > +	"\t            [:size=#entries]\n"
> > +	"\t            [if <filter>]\n\n"
> > +	"\t    When a matching event is hit, an entry is added to a hash\n"
> > +	"\t    table using the key named.  Keys correspond to fields in the\n"
> > +	"\t    event's format description.  On an event hit, the value of a\n"
> > +	"\t    sum called 'hitcount' is incremented, which is simply a count\n"
> > +	"\t    of event hits.  Keys can be any field.\n\n"
> > +	"\t    Reading the 'hist' file for the event will dump the hash\n"
> > +	"\t    table in its entirety to stdout.  Each printed hash table\n"
> > +	"\t    entry is a simple list of the keys and values comprising the\n"
> > +	"\t    entry; keys are printed first and are delineated by curly\n"
> > +	"\t    braces, and are followed by the set of value fields for the\n"
> > +	"\t    entry.  Numeric fields are displayed as base-10 integers.\n"
> > +	"\t    By default, the size of the hash table is 2048 entries.  The\n"
> > +	"\t    'size' param can be used to specify more or fewer than that.\n"
> > +	"\t    The units are in terms of hashtable entries - if a run uses\n"
> > +	"\t    more entries than specified, the results will show the number\n"
> > +	"\t    of 'drops', the number of hits that were ignored.  The size\n"
> > +	"\t    should be a power of 2 between 128 and 131072 (any non-\n"
> > +	"\t    power-of-2 number specified will be rounded up).\n\n"
> > +	"\t    The entries are sorted by 'hitcount' and the sort order is\n"
> > +	"\t    'ascending'.\n\n"
> 
> 
> Hmm, this seems too much about implementation of histogram. Could you shorten this
> to be a half ?
> 

OK, I'll try to cut it down to just usage essentials.

Tom

> 
> Thank you,
> 
> 


--
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/