Re: [PATCH 4/6] target/riscv/monitor: Replace legacy cpu_physical_memory_read() call

[Manos Pitsidianakis]

On Thu, Oct 2, 2025 at 6:00 PM Philippe Mathieu-Daudé <phi...@linaro.org> wrote:
>
> Commit b7ecba0f6f6 ("docs/devel/loads-stores.rst: Document our
> various load and store APIs") mentioned cpu_physical_memory_*()
> methods are legacy, the replacement being address_space_*().
>
> Propagate the address space to walk_pte(), then replace the
> cpu_physical_memory_read() by address_space_read(). Since the
> monitor command are run with a vCPU context, use its default
> address space. As with the previous implementation, ignore
> whether the memory read succeeded or failed.
>
> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
> ---


Reviewed-by: Manos Pitsidianakis <manos.pitsidiana...@linaro.org>