> On 16 Jul 2025, at 19.28, Philippe Mathieu-Daudé <phi...@linaro.org> wrote: > > Do not abort in hvf_arm_get_default_ipa_bit_size() > and hvf_arm_get_max_ipa_bit_size() when the IPA can > not be fetched. Return 0 (and document it). > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> > --- > target/arm/hvf_arm.h | 11 +++++++++++ > target/arm/hvf/hvf.c | 8 ++------ > 2 files changed, 13 insertions(+), 6 deletions(-)
I looked up the different uses of hvf_arm_get_max_ipa_bit_size. There's an unchecked use in clamp_id_aa64mmfr0_parange_to_ipa_size, but I guess it'll never reach that far, with your changes to virt_hvf_get_physical_address_range. clamp_id_aa64mmfr0_parange_to_ipa_size is already surrounded by other hv-calls and assert_hvf_ok, so either leave it, or assert ipa_size!=0?
