Re: [PATCH 10/20] target/arm: Restrict system register properties to system binary

[Philippe Mathieu-Daudé]

On 19/6/25 15:13, Philippe Mathieu-Daudé wrote:

Do not expose system-specific properties on user-mode binaries.

Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
---
  target/arm/cpu.c | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/target/arm/cpu.c b/target/arm/cpu.c
index eb0639de719..c99981d88f8 100644
--- a/target/arm/cpu.c
+++ b/target/arm/cpu.c
@@ -1500,6 +1500,7 @@ static void arm_cpu_initfn(Object *obj)
   * 0 means "unset, use the default value". That default might vary depending
   * on the CPU type, and is set in the realize fn.
   */
+#ifndef CONFIG_USER_ONLY
  static const Property arm_cpu_gt_cntfrq_property =
              DEFINE_PROP_UINT64("cntfrq", ARMCPU, gt_cntfrq_hz, 0);

@@ -1509,7 +1510,6 @@ static const Property arm_cpu_reset_cbar_property =

  static const Property arm_cpu_reset_hivecs_property =
              DEFINE_PROP_BOOL("reset-hivecs", ARMCPU, reset_hivecs, false);

-#ifndef CONFIG_USER_ONLY

  static const Property arm_cpu_has_el2_property =
              DEFINE_PROP_BOOL("has_el2", ARMCPU, has_el2, true);

@@ -1532,6 +1532,7 @@ static const Property arm_cpu_has_neon_property =

  static const Property arm_cpu_has_dsp_property =
              DEFINE_PROP_BOOL("dsp", ARMCPU, has_dsp, true);

+#ifndef CONFIG_USER_ONLY

  static const Property arm_cpu_has_mpu_property =
              DEFINE_PROP_BOOL("has-mpu", ARMCPU, has_mpu, true);

@@ -1544,6 +1545,7 @@ static const Property arm_cpu_pmsav7_dregion_property =

              DEFINE_PROP_UNSIGNED_NODEFAULT("pmsav7-dregion", ARMCPU,
                                             pmsav7_dregion,
                                             qdev_prop_uint32, uint32_t);
+#endif

Sorry, this patch is incomplete...