There are so many tests, no point in reporting successful ones. Leave the
capability optional in case a failure triggers a kvm or testsuite crash before
the report is complete and we need to see which test failed.
Signed-off-by: Avi Kivity <a...@redhat.com>
---
kvm/user/test/x86/access.c | 15 +++++++++++++--
1 files changed, 13 insertions(+), 2 deletions(-)
diff --git a/kvm/user/test/x86/access.c b/kvm/user/test/x86/access.c
index 0906691..c7a7075 100644
--- a/kvm/user/test/x86/access.c
+++ b/kvm/user/test/x86/access.c
@@ -6,6 +6,8 @@
#define true 1
#define false 0
+static _Bool verbose = false;
+
typedef unsigned long pt_element_t;
#define PAGE_SIZE ((pt_element_t)4096)
@@ -145,6 +147,9 @@ typedef struct {
unsigned long linear_addr;
} __attribute__((packed)) descriptor_table_t;
+
+static void ac_test_show(ac_test_t *at);
+
void lidt(idt_entry_t *idt, int nentries)
{
descriptor_table_t dt;
@@ -469,6 +474,10 @@ static void ac_test_check(ac_test_t *at, _Bool
*success_ret, _Bool cond,
*success_ret = false;
+ if (!verbose) {
+ ac_test_show(at);
+ }
+
va_start(ap, fmt);
vsnprintf(buf, sizeof(buf), fmt, ap);
va_end(ap);
@@ -565,7 +574,7 @@ int ac_test_do_access(ac_test_t *at)
ac_test_check(at, &success, *at->pdep != at->expected_pde,
"pde %x expected %x", *at->pdep, at->expected_pde);
- if (success) {
+ if (success && verbose) {
printf("PASS\n");
}
return success;
@@ -590,7 +599,9 @@ int ac_test_exec(ac_test_t *at)
{
int r;
- ac_test_show(at);
+ if (verbose) {
+ ac_test_show(at);
+ }
ac_test_setup_pte(at);
r = ac_test_do_access(at);
return r;
--
1.7.0.4
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
