x86emul: mark new "cp" local var maybe-unused

[PATCH] x86emul: mark new "cp" local var maybe-unused

Posted by Jan Beulich 2 months, 2 weeks ago

Just for the sake of the 32-bit build of the test harness. This wants
reverting once unconditional uses of the variable appear (AMX, AVX10).

Fixes: b20e3fbc3ec9 ("x86emul: introduce a struct cpu_policy * local in x86_emulate()")
Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/xen/arch/x86/x86_emulate/x86_emulate.c
+++ b/xen/arch/x86/x86_emulate/x86_emulate.c
@@ -1233,7 +1233,7 @@ x86_emulate(
 {
     /* Shadow copy of register state. Committed on successful emulation. */
     struct cpu_user_regs _regs = *ctxt->regs;
-    const struct cpu_policy *cp = ctxt->cpu_policy;
+    const struct cpu_policy *__maybe_unused cp = ctxt->cpu_policy;
     struct x86_emulate_state state;
     int rc;
     uint8_t b, d, *opc = NULL;

Re: [PATCH] x86emul: mark new "cp" local var maybe-unused

Posted by Andrew Cooper 2 months, 2 weeks ago

On 06/09/2024 11:38 am, Jan Beulich wrote:
> Just for the sake of the 32-bit build of the test harness. This wants
> reverting once unconditional uses of the variable appear (AMX, AVX10).
>
> Fixes: b20e3fbc3ec9 ("x86emul: introduce a struct cpu_policy * local in x86_emulate()")
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>