linux-kernel - Re: [PATCH V8 07/14] rtla/timerlat: Add timerlat hist mode

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: <aabde006-ce0e-fc68-63fb-58eb43648ef1@kernel.org>
Date:   Fri, 3 Dec 2021 18:42:04 +0100
From:   Daniel Bristot de Oliveira <bristot@...nel.org>
To:     Tao Zhou <tao.zhou@...ux.dev>
Cc:     Steven Rostedt <rostedt@...dmis.org>,
        Ingo Molnar <mingo@...hat.com>,
        Tom Zanussi <zanussi@...nel.org>,
        Masami Hiramatsu <mhiramat@...nel.org>,
        Juri Lelli <juri.lelli@...hat.com>,
        Clark Williams <williams@...hat.com>,
        John Kacur <jkacur@...hat.com>,
        Peter Zijlstra <peterz@...radead.org>,
        Thomas Gleixner <tglx@...utronix.de>,
        Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
        linux-rt-users@...r.kernel.org, linux-trace-devel@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH V8 07/14] rtla/timerlat: Add timerlat hist mode

On 12/3/21 10:17, Tao Zhou wrote:
> On Mon, Nov 29, 2021 at 12:07:45PM +0100,
> Daniel Bristot de Oliveira wrote:
> 
>> +static void timerlat_hist_usage(char *usage)
>> +{
>> +	int i;
>> +
>> +	char *msg[] = {
>> +		"",
>> +		"  usage: [rtla] timerlat hist [-h] [-q] [-p us] [-i us] [-t us] [-s us] [-T[=file]] \\",
>> +		"         [-c cpu-list] [-P priority] [--no-irq] [--no-thread] [--no-header] [--no-summary] \\",
>> +		"         [--no-index] [--with-zeros]",
>> +		"",
>> +		"	  -h/--help: print this menu",
>> +		"	  -p/--period us: timerlat period in us",
>> +		"	  -i/--irq us: stop trace if the irq latency is higher than the argument in us",
>> +		"	  -T/--thread us: stop trace if the thread latency is higher than the argument in us",
>> +		"	  -s/--stack us: save the stack trace at the IRQ if a thread latency is higher than the argument in us",
>> +		"	  -c/--cpus cpus: run the tracer only on the given cpus",
>> +		"	  -d/--duration time[m|h|d]: duration of the session in seconds",
>> +		"	  -T/--trace[=file]: save the stopped trace to [file|timerlat_trace.txt]",
>> +		"	  -n/--nano: display data in nanoseconds",
>> +		"	  -b/--bucket-size N: set the histogram bucket size (default 1)",
>> +		"	  -e/--entries N: set the number of entries of the histogram (default 256)",
>> +		"	     --no-irq: ignore IRQ latencies",
>> +		"	     --no-thread: ignore thread latencies",
>> +		"	     --no-header: do not print header",
>> +		"	     --no-summary: do not print summary",
>> +		"	     --no-index: do not print index",
>> +		"	     --with-zeros: print zero only entries",
>> +		"	  -P/--priority o:prio|r:prio|f:prio|d:runtime:period : set scheduling parameters",
>> +		"		o:prio - use SCHED_OTHER with prio",
>> +		"		r:prio - use SCHED_RR with prio",
>> +		"		f:prio - use SCHED_FIFO with prio",
>> +		"		d:runtime[us|ms|s]:period[us|ms|s] - use SCHED_DEADLINE with runtime and period",
>> +		"						       in nanoseconds",
>> +		NULL,
>> +	};
> 
> [-t us] --> [-T us]
> [-T[=file]] --> [-t[=file]]
> [-d] [-b] [-e] [-n] lack in usage info. this is not important tho.
> 

Fixed in V9

-- Daniel