summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorAmit Shah <amit.shah@qumranet.com>2007-11-19 17:57:35 +0200
committerAvi Kivity <avi@qumranet.com>2007-11-27 15:38:18 +0200
commit404fb881b82cf0cf6981832f8d31a7484e4dee81 (patch)
tree5da55ec34573b2f691f66f3a0b35a1698393a06f /drivers
parent8d379a7c069179a98616c9cac6bb2a06a500de49 (diff)
downloadop-kernel-dev-404fb881b82cf0cf6981832f8d31a7484e4dee81.zip
op-kernel-dev-404fb881b82cf0cf6981832f8d31a7484e4dee81.tar.gz
KVM: SVM: Fix FPU leak while emulating clts
The clts code didn't use set_cr0 properly, so our lazy FPU processing wasn't being done by the clts instruction at all. (this isn't called on Intel as the hardware does the decode for us) Signed-off-by: Amit Shah <amit.shah@qumranet.com> Signed-off-by: Avi Kivity <avi@qumranet.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/kvm/kvm_main.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
index 07ae280..47c10b8 100644
--- a/drivers/kvm/kvm_main.c
+++ b/drivers/kvm/kvm_main.c
@@ -1188,8 +1188,7 @@ int emulate_invlpg(struct kvm_vcpu *vcpu, gva_t address)
int emulate_clts(struct kvm_vcpu *vcpu)
{
- vcpu->cr0 &= ~X86_CR0_TS;
- kvm_x86_ops->set_cr0(vcpu, vcpu->cr0);
+ kvm_x86_ops->set_cr0(vcpu, vcpu->cr0 & ~X86_CR0_TS);
return X86EMUL_CONTINUE;
}
OpenPOWER on IntegriCloud