Attachment 876074 Details for Bug 1052090 – [RHEL7 PATCH 1/2] KVM: x86 emulator: emulate MOVAPS

[patch] [RHEL7 PATCH 1/2] KVM: x86 emulator: emulate MOVAPS

0001-KVM-x86-emulator-emulate-MOVAPS.patch (text/plain), 2.33 KB, created by Igor Mammedov on 2014-03-18 19:19:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Igor Mammedov

Created: 2014-03-18 19:19:56 UTC

Size: 2.33 KB

patch

obsolete

>From 9f29571660599834a87ddf293e593c0126b4bdef Mon Sep 17 00:00:00 2001
>From: Igor Mammedov <imammedo@redhat.com>
>Date: Sat, 15 Mar 2014 21:01:59 +0100
>Subject: [RHEL7 PATCH 1/2] KVM: x86 emulator: emulate MOVAPS
>
>Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1052090
>upstream: queued into kvm/next 27ce825823a145eb72bd5a5832c6dbb3168b720e
>Brew: https://brewweb.devel.redhat.com/taskinfo?taskID=7216057
>
>HCK memory driver test fails when testing 32-bit Windows 8.1
>with baloon driver.
>
>tracing KVM shows error:
>reason EXIT_ERR rip 0x81c18326 info 0 0
>
>x/10i 0x81c18326-20
>0x0000000081c18312:  add    %al,(%eax)
>0x0000000081c18314:  add    %cl,-0x7127711d(%esi)
>0x0000000081c1831a:  rolb   $0x0,0x80ec(%ecx)
>0x0000000081c18321:  and    $0xfffffff0,%esp
>0x0000000081c18324:  mov    %esp,%esi
>0x0000000081c18326:  movaps %xmm0,(%esi)
>0x0000000081c18329:  movaps %xmm1,0x10(%esi)
>0x0000000081c1832d:  movaps %xmm2,0x20(%esi)
>0x0000000081c18331:  movaps %xmm3,0x30(%esi)
>0x0000000081c18335:  movaps %xmm4,0x40(%esi)
>
>which points to MOVAPS instruction currently no emulated by KVM.
>Fix it by adding appropriate entries to opcode table in KVM's emulator.
>
>Signed-off-by: Igor Mammedov <imammedo@redhat.com>
>Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>---
> arch/x86/kvm/emulate.c |    8 +++++++-
> 1 files changed, 7 insertions(+), 1 deletions(-)
>
>diff --git a/arch/x86/kvm/emulate.c b/arch/x86/kvm/emulate.c
>index 44bdaa6..85263da 100644
>--- a/arch/x86/kvm/emulate.c
>+++ b/arch/x86/kvm/emulate.c
>@@ -3623,6 +3623,10 @@ static const struct gprefix pfx_vmovntpx = {
> 	I(0, em_mov), N, N, N,
> };
> 
>+static const struct gprefix pfx_0f_28_0f_29 = {
>+	I(Aligned, em_mov), N, N, N,
>+};
>+
> static const struct escape escape_d9 = { {
> 	N, N, N, N, N, N, N, I(DstMem, em_fnstcw),
> }, {
>@@ -3824,7 +3828,9 @@ static const struct opcode twobyte_table[256] = {
> 	IIP(ModRM | SrcMem | Priv | Op3264, em_cr_write, cr_write, check_cr_write),
> 	IIP(ModRM | SrcMem | Priv | Op3264, em_dr_write, dr_write, check_dr_write),
> 	N, N, N, N,
>-	N, N, N, GP(ModRM | DstMem | SrcReg | Sse | Mov | Aligned, &pfx_vmovntpx),
>+	GP(ModRM | DstReg | SrcMem | Mov | Sse, &pfx_0f_28_0f_29),
>+	GP(ModRM | DstMem | SrcReg | Mov | Sse, &pfx_0f_28_0f_29),
>+	N, GP(ModRM | DstMem | SrcReg | Sse | Mov | Aligned, &pfx_vmovntpx),
> 	N, N, N, N,
> 	/* 0x30 - 0x3F */
> 	II(ImplicitOps | Priv, em_wrmsr, wrmsr),
>-- 
>1.7.1
>

Actions: View | Diff

Attachments on bug 1052090: 849245 | 876074 | 876075