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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 9 Nov 2009 08:55:12 +0100 From: Ingo Molnar <mingo@...e.hu> To: Hitoshi Mitake <mitake@....info.waseda.ac.jp> Cc: linux-kernel@...r.kernel.org, rusty@...tcorp.com.au, tglx@...utronix.de, a.p.zijlstra@...llo.nl, efault@....de, acme@...hat.com, fweisbec@...il.com Subject: Re: [PATCH v2 3/7] Adding general performance benchmarking subcommand to perf. * Hitoshi Mitake <mitake@....info.waseda.ac.jp> wrote: > From: Ingo Molnar <mingo@...e.hu> > Subject: Re: [PATCH v2 3/7] Adding general performance benchmarking subcommand to perf. > Date: Sun, 8 Nov 2009 12:30:13 +0100 > > > > > > > > > Shouldnt we output the unit of measurement, i.e. '4.575 usecs'? Also, we > > > > should perhaps print something like: > > > > > > > > % perf bench sched pipe > > > > > > > > (executing 1000000 pipe operations between two tasks) > > > > > > > > 4.575 usecs per op > > > > 218579 ops/sec > > > > > > > > ? > > > > > > I have to admit that single float value output is too simple. > > > So I'll fix the default output. > > > > > > But, I believe that simple form makes sense for > > > processing by scripts or graph tools like gnuplot. > > > I'll add the option (may be --simple) to switch > > > friendliness of outputs. > > > > Btw., could you make it Git-ish, i.e.: > > > > --format=short > > > > or: > > > > --format=simple > > > > Eventually more format options might be added. > > It's good idea. > I have to admit that reserving -s for simple output is not good idea. > I'll do this. I think --format=simple will be used by scripts mostly, so it doesnt matter that it's longer to type. We try to save the shorter options for humans and be conservative with them. Another angle is coherency between different subcommands - and '-s' is already used as -s/--sort in other perf subcommands, which does not match up with the '-s/--simple' usage. We try to match what the Git project does here - a good deal of infrastructure code in perf came from Git and i find Git very easy to use and it's managed well. It's not a hard rule: not all option name incoherencies are fixable or avoidable, and there's no big problem if something slips in - i just wanted to mention so that you can keep an eye on it when developing new features for perf bench. 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/
Powered by blists - more mailing lists