* [PATCH v2] x86emul: fix vector-length check for AVX512F scalar fused-multiply-add insns
@ 2018-12-11 8:48 Jan Beulich
2018-12-18 13:48 ` Andrew Cooper
0 siblings, 1 reply; 2+ messages in thread
From: Jan Beulich @ 2018-12-11 8:48 UTC (permalink / raw)
To: xen-devel; +Cc: Andrew Cooper, Wei Liu, Roger Pau Monne
The check needs to happen whenever EVEX.b (SDM nomenclature) is clear,
not just in the memory operand case.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
---
v2: Clarify naming (to address apparent disconnect between description
and code change).
--- a/xen/arch/x86/x86_emulate/x86_emulate.c
+++ b/xen/arch/x86/x86_emulate/x86_emulate.c
@@ -8526,11 +8526,9 @@ x86_emulate(
case X86EMUL_OPC_EVEX_66(0x0f38, 0xbd): /* vfnmadd231s{s,d} xmm/mem,xmm,xmm{k} */
case X86EMUL_OPC_EVEX_66(0x0f38, 0xbf): /* vfnmsub231s{s,d} xmm/mem,xmm,xmm{k} */
host_and_vcpu_must_have(avx512f);
- if ( ea.type == OP_MEM )
- {
- generate_exception_if(evex.br, EXC_UD);
+ generate_exception_if(ea.type != OP_REG && evex.br, EXC_UD);
+ if ( !evex.br )
avx512_vlen_check(true);
- }
goto simd_zmm;
case X86EMUL_OPC(0x0f38, 0xc8): /* sha1nexte xmm/m128,xmm */
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v2] x86emul: fix vector-length check for AVX512F scalar fused-multiply-add insns
2018-12-11 8:48 [PATCH v2] x86emul: fix vector-length check for AVX512F scalar fused-multiply-add insns Jan Beulich
@ 2018-12-18 13:48 ` Andrew Cooper
0 siblings, 0 replies; 2+ messages in thread
From: Andrew Cooper @ 2018-12-18 13:48 UTC (permalink / raw)
To: Jan Beulich, xen-devel; +Cc: Wei Liu, Roger Pau Monne
On 11/12/2018 08:48, Jan Beulich wrote:
> The check needs to happen whenever EVEX.b (SDM nomenclature) is clear,
> not just in the memory operand case.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> ---
> v2: Clarify naming (to address apparent disconnect between description
> and code change).
Thanks - this is much easier to follow.
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-12-18 13:48 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-11 8:48 [PATCH v2] x86emul: fix vector-length check for AVX512F scalar fused-multiply-add insns Jan Beulich
2018-12-18 13:48 ` Andrew Cooper
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).