From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Jones Subject: Re: [PATCH kvm-unit-tests 00/10] report is nice, use it more Date: Fri, 12 Dec 2014 17:19:54 +0100 Message-ID: <20141212161954.GA17674@hawk.usersys.redhat.com> References: <1418400377-17388-1-git-send-email-drjones@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii To: kvm@vger.kernel.org Return-path: Received: from mx1.redhat.com ([209.132.183.28]:44908 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932622AbaLLQT6 (ORCPT ); Fri, 12 Dec 2014 11:19:58 -0500 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id sBCGJviH005680 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Fri, 12 Dec 2014 11:19:58 -0500 Content-Disposition: inline In-Reply-To: <1418400377-17388-1-git-send-email-drjones@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: 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 >