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

[xen staging] x86emul: set (fake) operand size for AVX512CD broadcast insns



commit 6fa6b7feaafd622db3a2f3436750cf07782f4c12
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Fri Aug 23 09:12:24 2024 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Fri Aug 23 09:12:24 2024 +0200

    x86emul: set (fake) operand size for AVX512CD broadcast insns
    
    Back at the time I failed to pay attention to op_bytes still being zero
    when reaching the respective case block: With the ext0f38_table[]
    entries having simd_packed_int, the defaulting at the bottom of
    x86emul_decode() won't set the field to non-zero for F3-prefixed insns.
    
    Fixes: 37ccca740c26 ("x86emul: support AVX512CD insns")
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
 xen/arch/x86/x86_emulate/x86_emulate.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/xen/arch/x86/x86_emulate/x86_emulate.c 
b/xen/arch/x86/x86_emulate/x86_emulate.c
index eda691868d..0a7552de96 100644
--- a/xen/arch/x86/x86_emulate/x86_emulate.c
+++ b/xen/arch/x86/x86_emulate/x86_emulate.c
@@ -5929,6 +5929,7 @@ x86_emulate(
                                evex.w == ((b >> 4) & 1)),
                               X86_EXC_UD);
         d |= TwoOp;
+        op_bytes = 1; /* fake */
         /* fall through */
     case X86EMUL_OPC_EVEX_66(0x0f38, 0xc4): /* vpconflict{d,q} 
[xyz]mm/mem,[xyz]mm{k} */
         fault_suppression = false;
--
generated by git-patchbot for /home/xen/git/xen.git#staging



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.