From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?Radim=20Kr=C4=8Dm=C3=A1=C5=99?= Subject: [PATCH kvm-unit-tests v2 06/12] x86/*: report skipped tests Date: Thu, 17 Dec 2015 18:53:37 +0100 Message-ID: <1450374823-7648-7-git-send-email-rkrcmar@redhat.com> References: <1450374823-7648-1-git-send-email-rkrcmar@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Paolo Bonzini , Andrew Jones To: kvm@vger.kernel.org Return-path: Received: from mx1.redhat.com ([209.132.183.28]:45426 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753614AbbLQRyN (ORCPT ); Thu, 17 Dec 2015 12:54:13 -0500 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (Postfix) with ESMTPS id 1184FA04B3 for ; Thu, 17 Dec 2015 17:54:13 +0000 (UTC) In-Reply-To: <1450374823-7648-1-git-send-email-rkrcmar@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: No care to consistency or exhaustivity was given. (svm-disabled test should be redone and it's weird that x86/hyperv_syni= c is about the only one that does report_skip when unsupported.) Signed-off-by: Radim Kr=C4=8Dm=C3=A1=C5=99 --- v2: remove double newline in enable_tsc_deadline_timer [Drew] =20 x86/apic.c | 7 +++---- x86/emulator.c | 2 +- x86/hyperv_synic.c | 2 +- x86/pku.c | 2 +- x86/pmu.c | 2 +- x86/smap.c | 2 +- x86/svm.c | 2 +- x86/tsc.c | 2 +- 8 files changed, 10 insertions(+), 11 deletions(-) diff --git a/x86/apic.c b/x86/apic.c index d4eec529e535..a97fe5e5a6c5 100644 --- a/x86/apic.c +++ b/x86/apic.c @@ -27,7 +27,7 @@ static void tsc_deadline_timer_isr(isr_regs_t *regs) ++tdt_count; } =20 -static void start_tsc_deadline_timer(void) +static void __test_tsc_deadline_timer(void) { handle_irq(TSC_DEADLINE_TIMER_VECTOR, tsc_deadline_timer_isr); irq_enable(); @@ -45,7 +45,6 @@ static int enable_tsc_deadline_timer(void) if (cpuid(1).c & (1 << 24)) { lvtt =3D TSC_DEADLINE_TIMER_MODE | TSC_DEADLINE_TIMER_VECTOR; apic_write(APIC_LVTT, lvtt); - start_tsc_deadline_timer(); return 1; } else { return 0; @@ -55,9 +54,9 @@ static int enable_tsc_deadline_timer(void) static void test_tsc_deadline_timer(void) { if(enable_tsc_deadline_timer()) { - printf("tsc deadline timer enabled\n"); + __test_tsc_deadline_timer(); } else { - printf("tsc deadline timer not detected\n"); + report_skip("tsc deadline timer not detected"); } } =20 diff --git a/x86/emulator.c b/x86/emulator.c index e5c1c6b9a2f3..b64a5fe0f3dc 100644 --- a/x86/emulator.c +++ b/x86/emulator.c @@ -1062,7 +1062,7 @@ static void illegal_movbe_handler(struct ex_regs = *regs) static void test_illegal_movbe(void) { if (!(cpuid(1).c & (1 << 22))) { - printf("SKIP: illegal movbe\n"); + report_skip("illegal movbe"); return; } =20 diff --git a/x86/hyperv_synic.c b/x86/hyperv_synic.c index 18d1295bfb37..602b79392bfd 100644 --- a/x86/hyperv_synic.c +++ b/x86/hyperv_synic.c @@ -228,7 +228,7 @@ int main(int ac, char **av) =20 report("Hyper-V SynIC test", ok); } else { - report("Hyper-V SynIC is not supported", true); + report_skip("Hyper-V SynIC is not supported"); } =20 return report_summary(); diff --git a/x86/pku.c b/x86/pku.c index 0e00b9984d70..58971d21ed05 100644 --- a/x86/pku.c +++ b/x86/pku.c @@ -91,7 +91,7 @@ int main(int ac, char **av) =20 if (!(cpuid_indexed(7, 0).c & (1 << X86_FEATURE_PKU))) { printf("PKU not enabled, exiting\n"); - exit(1); + return report_summary(); } =20 setup_vm(); diff --git a/x86/pmu.c b/x86/pmu.c index 03f80190bb25..c68980044dee 100644 --- a/x86/pmu.c +++ b/x86/pmu.c @@ -387,7 +387,7 @@ int main(int ac, char **av) =20 if (!eax.split.version_id) { printf("No pmu is detected!\n"); - return 1; + return report_summary(); } printf("PMU version: %d\n", eax.split.version_id); printf("GP counters: %d\n", eax.split.num_counters); diff --git a/x86/smap.c b/x86/smap.c index d8a7ae82dc00..0aa44054bd48 100644 --- a/x86/smap.c +++ b/x86/smap.c @@ -93,7 +93,7 @@ int main(int ac, char **av) =20 if (!(cpuid_indexed(7, 0).b & (1 << X86_FEATURE_SMAP))) { printf("SMAP not enabled, exiting\n"); - exit(1); + return report_summary(); } =20 setup_vm(); diff --git a/x86/svm.c b/x86/svm.c index 1046ddf73732..ff1a0f34b4bf 100644 --- a/x86/svm.c +++ b/x86/svm.c @@ -1064,7 +1064,7 @@ int main(int ac, char **av) =20 if (!(cpuid(0x80000001).c & 4)) { printf("SVM not availble\n"); - return 0; + return report_summary(); } =20 setup_svm(); diff --git a/x86/tsc.c b/x86/tsc.c index c71dc2a7abe0..ee247459fb42 100644 --- a/x86/tsc.c +++ b/x86/tsc.c @@ -43,5 +43,5 @@ int main() test_rdtscp(0x100); } else printf("rdtscp not supported\n"); - return 0; + return report_summary(); } --=20 2.6.4