Re: [PATCH 36/37] target/riscv: Remove ifndef CONFIG_USER_ONLY from cpu_cfg.h

[Pierrick Bouvier]

On 3/12/25 20:45, Richard Henderson wrote:

While RISCVCPUConfig.satp_mode is unused for user-only,
this header is used from disas/riscv.h, whose users are
only built once.  The savings of 4 bytes isn't worth it.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
---
  target/riscv/cpu_cfg.h | 2 --
  1 file changed, 2 deletions(-)

diff --git a/target/riscv/cpu_cfg.h b/target/riscv/cpu_cfg.h
index 8a843482cc..cfe371b829 100644
--- a/target/riscv/cpu_cfg.h
+++ b/target/riscv/cpu_cfg.h
@@ -196,9 +196,7 @@ struct RISCVCPUConfig {

bool short_isa_string; -#ifndef CONFIG_USER_ONLY

      RISCVSATPMap satp_mode;
-#endif
  };

typedef struct RISCVCPUConfig RISCVCPUConfig;

Reviewed-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>