Re: [PATCH 2/6] target/i386/monitor: Replace legacy cpu_physical_memory_read() calls

[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_*().
>
> Replace:
>
>  - cpu_physical_memory_read(len=4) -> address_space_ldl()
>  - cpu_physical_memory_read(len=8) -> address_space_ldq()
>
> inlining the little endianness conversion via the '_le' suffix.
> 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>