Re: [PATCH kvm-unit-tests] build: enable -Werror

[Paolo Bonzini <pbonzini@redhat.com>]

On 04/03/2016 09:47, Thomas Huth wrote:
>> >  	asm volatile("mov %%dr7,%0" : "=r" (dr7));
>> >  	debugctl = rdmsr(MSR_IA32_DEBUGCTLMSR);
>> >  	/* Commented out: KVM does not support DEBUGCTL so far */
>> > +	assert(debugctl == debugctl);
>> >  	report("Guest=host debug controls", dr7 == 0x402 /* && debugctl == 0x1 */);
> Now these assert()s are really ugly. Wouldn't it be better do comment
> out the "debugctl = rdmsr(MSR_IA32_DEBUGCTLMSR);" line instead?

This also looks better than the asserts:

	(void)debugctl;

Thomas, if you're okay with it I can do the change locally.

Paolo