Re: [PATCH kvm-unit-tests 00/10] report is nice, use it more

[Andrew Jones <drjones@redhat.com>]

On Fri, Dec 12, 2014 at 05:06:07PM +0100, Andrew Jones wrote:
> This series applies report() to more x86 unit tests. It also
> cleans up some uses of it in vmx_tests. Finally, it also
> introduces report prefix support, and applies that a few places.
> There should be no functional changes with this series, except of
> course the output formatting (which makes the churn debatable).
> Anyway, I believe this series helps us to start converging on a
> "standard" output format for all unit tests.
> 
> I'm afraid I didn't test all the changes, as not all unit tests
> could run on my test machine.
>   svm 		- didn't run
>   xsave		- didn't run 'have xsave' tests
>   asyncpf	- is commented out of unittests.cfg,
> 		  and I didn't uncomment it out
>   pmu		- didn't run
> 
> (I probably should add a bunch of kvm-unit-tests contributors to
>  the CC in order to alert them that I'm touching their unit tests,
>  but I don't think this series warrants all the git blaming, so
>  I'm not adding anybody at all :-)

Forgot to mention that these patches are available here

https://github.com/rhdrjones/kvm-unit-tests/tree/more-report

>  
> 
> Andrew Jones (10):
>   x86: hypercall: add missing report_summary
>   x86: pmu: report takes a fmt
>   x86: vmx: better use of report
>   x86: tsc_adjust: use report
>   x86: svm: use report
>   x86: asyncpf: use report
>   x86: xsave: use report
>   report: add prefix support
>   x86: use report prefixes
>   arm: use report prefixes
> 
>  arm/selftest.c   |  31 ++++-----
>  lib/libcflat.h   |   2 +
>  lib/report.c     |  22 ++++++
>  lib/string.c     |  17 +++++
>  lib/string.h     |   1 +
>  x86/apic.c       |   8 ++-
>  x86/asyncpf.c    |  15 ++--
>  x86/hypercall.c  |   2 +-
>  x86/pmu.c        |  28 +++++---
>  x86/svm.c        |  25 +++----
>  x86/tsc_adjust.c |  44 ++++--------
>  x86/vmx_tests.c  | 204 +++++++++++++------------------------------------------
>  x86/xsave.c      | 150 +++++++++++++++-------------------------
>  13 files changed, 210 insertions(+), 339 deletions(-)
> 
> -- 
> 1.9.3
>