On 5/3/25 22:29, Pierrick Bouvier wrote:
This exposes two new subsections for arm: vmstate_sve and vmstate_za.
Those sections have a ".needed" callback, which already allow to skip
them when not needed.

vmstate_sve .needed is checking cpu_isar_feature(aa64_sve, cpu).
vmstate_za .needed is checking ZA flag in cpu->env.svcr.

Signed-off-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>
---
  target/arm/machine.c | 4 ----
  1 file changed, 4 deletions(-)

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

r~


diff --git a/target/arm/machine.c b/target/arm/machine.c
index f7956898fa1..868246a98c0 100644
--- a/target/arm/machine.c
+++ b/target/arm/machine.c
@@ -241,7 +241,6 @@ static const VMStateDescription vmstate_iwmmxt = {
      }
  };
-#ifdef TARGET_AARCH64
  /* The expression ARM_MAX_VQ - 2 is 0 for pure AArch32 build,
   * and ARMPredicateReg is actively empty.  This triggers errors
   * in the expansion of the VMSTATE macros.
@@ -321,7 +320,6 @@ static const VMStateDescription vmstate_za = {
          VMSTATE_END_OF_LIST()
      }
  };
-#endif /* AARCH64 */
static bool serror_needed(void *opaque)
  {
@@ -1102,10 +1100,8 @@ const VMStateDescription vmstate_arm_cpu = {
          &vmstate_pmsav7,
          &vmstate_pmsav8,
          &vmstate_m_security,
-#ifdef TARGET_AARCH64
          &vmstate_sve,
          &vmstate_za,
-#endif
          &vmstate_serror,
          &vmstate_irq_line_state,
          &vmstate_wfxt_timer,


Reply via email to