On Sat, Dec 07, 2013 at 01:36:47PM +0100, Daniel Vetter wrote: > On Sat, Dec 7, 2013 at 5:35 AM, Ben Widawsky > <benjamin.widawsky@xxxxxxxxx> wrote: > > Provide two arguments version, and verbose, which allow printing from an > > arbitrary igt test. It will show system information (from build time, > > not runtime), as well as the git SHA being used. > > > > This will help reduce errors when people try to reproduce problems. > > > > As an example if I want to verify someone is running the correct version > > of a test, I could ask them to do: > > bwidawsk@ironside ~/intel-gfx/intel-gpu-tools (master)$ sudo ./tests/gem_exec_nop --verbose > > gem_exec_nop-git-3c5423b (Linux ironside 3.12.0-1-ARCH #1 SMP PREEMPT > > Wed Nov 6 09:06:27 CET 2013 x86_64 GNU/Linux) > > Time to exec x 1: 35.000µs (ring=render) > > Time to exec x 2: 28.000µs (ring=render) > > Time to exec x 4: 20.000µs (ring=render) > > Time to exec x 8: 14.625µs (ring=render) > > Time to exec x 16: 11.188µs (ring=render) > > Time to exec x 32: 11.125µs (ring=render) > > Time to exec x 64: 10.328µs (ring=render) > > Time to exec x 128: 10.172µs (ring=render) > > Time to exec x 256: 10.234µs (ring=render) > > Time to exec x 512: 10.232µs (ring=render) > > Time to exec x 1024: 10.121µs (ring=render) > > Time to exec x 2048: 10.151µs (ring=render) > > Time to exec x 4096: 11.474µs (ring=render) > > Time to exec x 8192: 9.432µs (ring=render) > > Time to exec x 16384: 6.003µs (ring=render) > > Time to exec x 32768: 5.029µs (ring=render) > > Time to exec x 65536: 4.206µs (ring=render) > > Time to exec x 131072: 3.630µs (ring=render) > > Subtest render: SUCCESS > > > > --verbose is provided for completeness, but doesn't seem too useful at > > the moement. > > > > bwidawsk@ironside ~/intel-gfx/intel-gpu-tools (master)$ sudo ./tests/gem_exec_nop --version > > gem_exec_nop-git-3c5423b (Linux ironside 3.12.0-1-ARCH #1 SMP PREEMPT > > Wed Nov 6 09:06:27 CET 2013 x86_64 GNU/Linux) > > > > I've put version.h in the root directory so that any subdir can access > > it. I've added the tests usage since it's immediately useful, and done > > easily via Daniels igt infrastructure work. > > > > Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx> > > > Yeah, this is definitely very useful. But I'd just print it by default > so that we don't need to ask for this information. Of course we need > to be careful to not print it when listing subtest. Also: > - We don't have any init stuff for simple testcases without subtests, > we need to roll out a bit of infrastructure to make that happen. > - Shell script tests might also benefit from this. > - uname information needs to be gathered at runtime, since QA has a > special build server. > > If you don't mind I'll apply this and then frob it some more. > -Daniel Sounds great. I want to test it out on some of the BDW bugs QA is reporting, so ping me when you're done. -- Ben Widawsky, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
