Re: [PATCH v4 14/17] hw/arm, accel/hvf, whpx: unify get_physical_address_range between WHPX and HVF

[Pierrick Bouvier]

On 8/4/25 4:03 PM, Philippe Mathieu-Daudé wrote:
On 4/8/25 16:23, Mohamed Mediouni wrote:
Signed-off-by: Mohamed Mediouni <moha...@unpredictable.fr>
---
   accel/hvf/hvf-all.c        |  7 +++++--
   hw/arm/virt.c              | 41 ++++----------------------------------
   include/hw/boards.h        |  4 ++--
   include/system/hvf_int.h   |  2 ++
   target/arm/hvf-stub.c      | 20 -------------------
   target/arm/hvf/hvf.c       |  6 +++---
   target/arm/hvf_arm.h       |  3 ---
   target/arm/meson.build     |  1 -
   target/arm/whpx/whpx-all.c |  5 +++--
   target/i386/hvf/hvf.c      | 10 ++++++++++
   10 files changed, 29 insertions(+), 70 deletions(-)
   delete mode 100644 target/arm/hvf-stub.c


diff --git a/target/i386/hvf/hvf.c b/target/i386/hvf/hvf.c
index 818b50419f..fa06e3b815 100644
--- a/target/i386/hvf/hvf.c
+++ b/target/i386/hvf/hvf.c
@@ -225,6 +225,16 @@ int hvf_arch_init(void)
       return 0;
   }
   
+uint32_t hvf_arch_get_default_ipa_bit_size(void)
+{
+    return 48;
+}
+
+uint32_t hvf_arch_get_max_ipa_bit_size(void)
+{
+    return 48;

These don't make much sense on x86...


Common path in accel/hvf/hvf-all.c makes use of it, so it's not a bad 
idea to have an architecture agnostic function for it, instead of:
if (arm_function) {...}.

It happens that on x64, VA space is 48 bits.

+}