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]

Date:	Mon, 20 Jul 2015 22:37:14 +0900
From:	Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>
To:	Tom Zanussi <tom.zanussi@...ux.intel.com>, rostedt@...dmis.org
CC:	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 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 ?


Thank you,


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