* [PATCH] KVM: Make unloading of FPU state when putting vcpu arch-independent
@ 2007-11-18 17:24 Amit Shah
[not found] ` <1195406685648-git-send-email-amit.shah-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Amit Shah @ 2007-11-18 17:24 UTC (permalink / raw)
To: avi-atKUWr5tajBWk0Htik3J/w
Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f, Amit Shah
Instead of having each architecture do it individually, we
do this in the arch-independent code (just x86 as of now).
Turns out SVM did not do this at all.
Signed-off-by: Amit Shah <amit.shah-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
---
drivers/kvm/vmx.c | 1 -
drivers/kvm/x86.c | 1 +
2 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/kvm/vmx.c b/drivers/kvm/vmx.c
index da3a339..16e3935 100644
--- a/drivers/kvm/vmx.c
+++ b/drivers/kvm/vmx.c
@@ -531,7 +531,6 @@ static void vmx_vcpu_load(struct kvm_vcpu *vcpu, int cpu)
static void vmx_vcpu_put(struct kvm_vcpu *vcpu)
{
vmx_load_host_state(to_vmx(vcpu));
- kvm_put_guest_fpu(vcpu);
}
static void vmx_fpu_activate(struct kvm_vcpu *vcpu)
diff --git a/drivers/kvm/x86.c b/drivers/kvm/x86.c
index e905d46..f48138b 100644
--- a/drivers/kvm/x86.c
+++ b/drivers/kvm/x86.c
@@ -609,6 +609,7 @@ void kvm_arch_vcpu_load(struct kvm_vcpu *vcpu, int cpu)
void kvm_arch_vcpu_put(struct kvm_vcpu *vcpu)
{
kvm_x86_ops->vcpu_put(vcpu);
+ kvm_put_guest_fpu(vcpu);
}
static void cpuid_fix_nx_cap(struct kvm_vcpu *vcpu)
--
1.5.3
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] KVM: Make unloading of FPU state when putting vcpu arch-independent
[not found] ` <1195406685648-git-send-email-amit.shah-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
@ 2007-11-18 17:29 ` Avi Kivity
0 siblings, 0 replies; 2+ messages in thread
From: Avi Kivity @ 2007-11-18 17:29 UTC (permalink / raw)
To: Amit Shah; +Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f
Amit Shah wrote:
> Instead of having each architecture do it individually, we
> do this in the arch-independent code (just x86 as of now).
> Turns out SVM did not do this at all.
>
>
Applied, thanks.
--
error compiling committee.c: too many arguments to function
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-11-18 17:29 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-11-18 17:24 [PATCH] KVM: Make unloading of FPU state when putting vcpu arch-independent Amit Shah
[not found] ` <1195406685648-git-send-email-amit.shah-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-11-18 17:29 ` Avi Kivity
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox