File 5ca773d1-x86emul-dont-read-mask-reg-without-AVX... of Package xen

Overview Repositories Revisions Requests Users Attributes Meta

File 5ca773d1-x86emul-dont-read-mask-reg-without-AVX512F.patch of Package xen (Revision 06956a7f6e8ce636fc3529e94c6d82b0)

Currently displaying revision 06956a7f6e8ce636fc3529e94c6d82b0 , Show latest

# Commit 6cb7e52edf823fd89fe14da94f9bf3e5cf99d1ff
# Date 2019-04-05 17:27:13 +0200
# Author Jan Beulich <jbeulich@suse.com>
# Committer Jan Beulich <jbeulich@suse.com>
x86emul: don't read mask register on AVX512F-incapable platforms

Nor when register state isn't sufficiently enabled.

Reported-by: George Dunlap <george.dunlap@citrix.com>
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>

--- a/xen/arch/x86/x86_emulate/x86_emulate.c
+++ b/xen/arch/x86/x86_emulate/x86_emulate.c
@@ -3511,7 +3511,8 @@ x86_emulate(
     }
 
     /* With a memory operand, fetch the mask register in use (if any). */
-    if ( ea.type == OP_MEM && evex.opmsk )
+    if ( ea.type == OP_MEM && evex.opmsk &&
+         _get_fpu(fpu_type = X86EMUL_FPU_opmask, ctxt, ops) == X86EMUL_OKAY )
     {
         uint8_t *stb = get_stub(stub);
 
@@ -3532,6 +3533,14 @@ x86_emulate(
         fault_suppression = true;
     }
 
+    if ( fpu_type == X86EMUL_FPU_opmask )
+    {
+        /* Squash (side) effects of the _get_fpu() above. */
+        x86_emul_reset_event(ctxt);
+        put_fpu(X86EMUL_FPU_opmask, false, state, ctxt, ops);
+        fpu_type = X86EMUL_FPU_none;
+    }
+
     /* Decode (but don't fetch) the destination operand: register or memory. */
     switch ( d & DstMask )
     {

Places

File 5ca773d1-x86emul-dont-read-mask-reg-without-AVX512F.patch of Package xen (Revision 06956a7f6e8ce636fc3529e94c6d82b0)

Places