On 06.02.2026 17:15, Alejandro Vallejo wrote:
> ... but keep the current behaviour of using the policy vendor when
> compiled for userspace, where cross-vendor configurations are expected.
> 
> Not a functional change.
> 
> Signed-off-by: Alejandro Vallejo <[email protected]>

Hmm, ...

> --- a/xen/arch/x86/x86_emulate/private.h
> +++ b/xen/arch/x86/x86_emulate/private.h
> @@ -15,6 +15,7 @@
>  # include <xen/kernel.h>
>  
>  # include <asm/cpu-user-regs.h>
> +# include <asm/cpufeature.h>
>  # include <asm/endbr.h>
>  # include <asm/msr-index.h>
>  # include <asm/stubs.h>
> @@ -30,8 +31,11 @@ void BUG(void);
>  #  define X86EMUL_NO_SIMD
>  # endif
>  
> +# define x86emul_cpu(cp) ({ (void)(cp); cpu_vendor(); })
> +
>  #else /* !__XEN__ */
>  # include "x86-emulate.h"
> +# define x86emul_cpu(cp) ((cp)->x86_vendor)
>  #endif

... this looked familiar, yet different at the same time. It was only after
some eyebrow raising that I noticed that patch 07/12 appears twice in the
series, with different titles.

Jan

Reply via email to