Re: [PATCH kvm-unit-tests 3/6] x86/pmu: expect failure with nmi_watchdog

Radim Krčmář
2015-12-15T13:01:04Z

2015-12-14 16:05-0600, Andrew Jones:
> On Mon, Dec 14, 2015 at 10:24:18PM +0100, Radim Krčmář wrote:
>> diff --git a/x86/pmu.c b/x86/pmu.c
>> @@ -291,7 +292,7 @@ static void check_counters_many(void)
>> if (!verify_counter(&cnt[i]))
>> break;
>>
>> - report("all counters", i == n);
>> + report_xfail("all counters", host_nmi_watchdog, i == n);
>
> How about outputting "host_nmi_watchdog=%d" as well?

It's already implied in the output. Prefix will be XPASS/XFAIL if
host_nmi_watchdog=1 and PASS/FAIL otherwise.

Should it still be explicitly printed?