On 10/12/2018 11:32, Jan Beulich wrote:
> The avx512_vlen_check() invocation needs to be conditional.
>
> Signed-off-by: Jan Beulich <[email protected]>

I'm not sure if I've asked before, but do LIG instructions really #UD
for L=3 ?  I don't see any documentation to this effect.

>
> --- a/xen/arch/x86/x86_emulate/x86_emulate.c
> +++ b/xen/arch/x86/x86_emulate/x86_emulate.c
> @@ -6179,7 +6179,8 @@ x86_emulate(
>                                 evex.w != evex.pfx),
>                                EXC_UD);
>          host_and_vcpu_must_have(avx512f);
> -        avx512_vlen_check(true);
> +        if ( !evex.br )

On the subject of ineligibility of the code, what about #define sae br ?

That way, this would read "if ( !evex.sae ) check_vlen()"

~Andrew

> +            avx512_vlen_check(true);
>          get_fpu(X86EMUL_FPU_zmm);
>  
>          opc = init_evex(stub);
>
>


_______________________________________________
Xen-devel mailing list
[email protected]
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to