On 12/12/2014 20:54, Eugene Korenevsky wrote: > Remove unused variable to get rid of compiler warning. > And remove commented out code (it can always be restored > from git logs).
This is also specifying that the behavior is incorrect. It should be changed to an XFAIL instead. Paolo > Signed-off-by: Eugene Korenevsky <[email protected]> > --- > x86/vmx_tests.c | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c > index 184fafc..0daaacf 100644 > --- a/x86/vmx_tests.c > +++ b/x86/vmx_tests.c > @@ -1439,12 +1439,10 @@ static int dbgctls_init(struct vmcs *vmcs) > > static void dbgctls_main(void) > { > - u64 dr7, debugctl; > + u64 dr7; > > asm volatile("mov %%dr7,%0" : "=r" (dr7)); > - debugctl = rdmsr(MSR_IA32_DEBUGCTLMSR); > - /* Commented out: KVM does not support DEBUGCTL so far */ > - report("Load debug controls", dr7 == 0x404 /* && debugctl == 0x2 */); > + report("Load debug controls", dr7 == 0x404); > > dr7 = 0x408; > asm volatile("mov %0,%%dr7" : : "r" (dr7)); > @@ -1463,9 +1461,7 @@ static void dbgctls_main(void) > vmcall(); > > asm volatile("mov %%dr7,%0" : "=r" (dr7)); > - debugctl = rdmsr(MSR_IA32_DEBUGCTLMSR); > - /* Commented out: KVM does not support DEBUGCTL so far */ > - report("Guest=host debug controls", dr7 == 0x402 /* && debugctl == 0x1 > */); > + report("Guest=host debug controls", dr7 == 0x402); > > dr7 = 0x408; > asm volatile("mov %0,%%dr7" : : "r" (dr7)); > -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
