RE: [PATCH] kvm/fpu: Enable fully eager restore kvm FPU

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

 



> -----Original Message-----
> From: Avi Kivity [mailto:avi@xxxxxxxxxx]
> Sent: Monday, September 03, 2012 5:23 PM
> To: Hao, Xudong
> Cc: Roedel, Joerg; kvm@xxxxxxxxxxxxxxx; Zhang, Xiantao
> Subject: Re: [PATCH] kvm/fpu: Enable fully eager restore kvm FPU
> 
> On 08/23/2012 11:51 AM, Hao, Xudong wrote:
> >> -----Original Message-----
> >> From: kvm-owner@xxxxxxxxxxxxxxx [mailto:kvm-owner@xxxxxxxxxxxxxxx] On
> >> Behalf Of Avi Kivity
> >> Sent: Monday, August 20, 2012 6:43 PM
> >> To: Roedel, Joerg
> >> Cc: Hao, Xudong; kvm@xxxxxxxxxxxxxxx; Zhang, Xiantao
> >> Subject: Re: [PATCH] kvm/fpu: Enable fully eager restore kvm FPU
> >>
> >> On 08/20/2012 01:14 PM, Roedel, Joerg wrote:
> >> > On Mon, Aug 20, 2012 at 01:08:14PM +0300, Avi Kivity wrote:
> >> >> On 08/20/2012 12:24 PM, Roedel, Joerg wrote:
> >> >
> >> >> So it was broken all along?  Yikes.
> >> >
> >> > There is no LWP support in the kernel and thus KVM can't expose it to
> >> > guests. So for now nothing should be broken, no?
> >>
> >> Oh, we mask out xcr0 bits not supported by the host.
> >>
> >> So it's broken in another way: it isn't exposed.  Pity, it's such a nice
> >> feature.
> >>
> >
> > Hi, Avi/Joerg
> >
> > What's the decision for it? I don't understand LWP, so how about this patch?
> 
> It's fine (Joerg can send the LWP change), but there was a truncation
> issue that needs fixing, no?
> 

Yes, I think you means to expand KVM_XSTATE_LAZY to 64-bits, I'll send another version patch.
 
Thanks,
-Xudong
--
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