Re: [PATCH] KVM: Move KVM context switch into own function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 20, 2010 at 06:12:11PM +0200, Avi Kivity wrote:
>  On 10/20/2010 05:56 PM, Andi Kleen wrote:
> >From: Andi Kleen<ak@xxxxxxxxxxxxxxx>
> >
> >gcc 4.5 with some special options is able to duplicate the VMX
> >context switch asm in vmx_vcpu_run(). This results in a compile error
> >because the inline asm sequence uses an on local label. The non local
> >label is needed because other code wants to set up the return address.
> >
> 
> Curious, any idea why gcc does that?  I guess it's the unlikely()
> near the top.

Don't know why, i wasn't motivated to go through its debugging dumps
for this. But strictly the problem could always happen,
not even a "volatile" stops that.

-Andi
--
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


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux