On 03/21/2012 08:40 PM, Avi Kivity wrote:
>
> +static int flush_pending_x87_faults(struct x86_emulate_ctxt *ctxt)
> +{
> + bool fault = false;
> +
> + ctxt->ops->get_fpu(ctxt);
> + asm volatile("1: fwait \n\t"
> + "2: \n\t"
> + ".pushsection .fixup,\"ax\" \n\t"
> + "3: \n\t"
> + "movb $1, %[fault] \n\t"
> + "jmp 2b \n\t"
> + ".popsection \n\t"
> + _ASM_EXTABLE(1b, 3b)
> + : [fault]"=rm"(fault));
>
"=" tells gcc that the asm section unconditionally writes 'fault', so it
skips the initialization. Use "+" instead.
--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
