linux-kernel - Re: [PATCH 10/13] perf kvm: add live 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: <20130509085137.GA8813@gmail.com>
Date:	Thu, 9 May 2013 10:51:37 +0200
From:	Ingo Molnar <mingo@...nel.org>
To:	David Ahern <dsahern@...il.com>
Cc:	linux-kernel@...r.kernel.org, acme@...stprotocols.net,
	fweisbec@...il.com, peterz@...radead.org, jolsa@...hat.com,
	namhyung@...nel.org, xiaoguangrong@...ux.vnet.ibm.com
Subject: Re: [PATCH 10/13] perf kvm: add live mode


* David Ahern <dsahern@...il.com> wrote:

> perf kvm stat currently requires back to back record and report
> commands to see stats. e.g,.
> 
>   perf kvm stat record -p $pid -- sleep 1
>   perf kvm stat report
> 
> This is inconvenvient for on box monitoring of a VM. This patch
> introduces a 'live' mode that in effect combines the record plus
> report into one command. e.g., to monitor a single VM:
> 
>   perf kvm stat live -p $pid
> 
> or all VMs:
> 
>   perf kvm stat live
> 
> Same stats options for the record+report path work with the live mode.
> Display rate defaults to 1 second and can be changed using the -d option.

Nice!

1)

Regarding the basic syntax - is the 'perf stat kvm' command already used 
up? If not then it might make sense to alias 'perf kvm stat live' to 'perf 
kvm stat', making it really easy to use:

  perf kvm stat
  perf kvm stat -p <pid>

etc. I'd expect most 'perf kvm stat' users to utilize this syntax - so it 
makes sense to make 'live' the default and main command.

2)

Another suggestion: it would be nice to allow a "--repeat <n>" option as 
well, for a limited number of iterations - similar to what the 'top' tool 
allows:

  top -b -d <delay> -n <iterations>

--repeat 0 means infinite repeats, --repeat 1 means a single step.

3)

It might make sense to create 'perf kvm stat top' which is a small 
variation to 'perf kvm stat live': it outputs a perf stat output per 
second, but also outputs a clear-screen sequence before the output.

To keep it simple you could do this with plain console text output, not 
libslang, the C code to generate a clear screen sequence can be generated 
via this shell oneliner:

  echo 'prinf("'$(clear)'");' > clearscreen.c

This way people could see constantly updating top-alike KVM statistics.

Thanks,

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