Re: [PATCH v2 2/6] hw/i386: Have x86_bios_rom_init() take X86MachineState rather than MachineState

[Philippe Mathieu-Daudé]

On 30/4/24 17:06, Bernhard Beschow wrote:
> The function creates and leaks two MemoryRegion objects regarding the BIOS which
> will be moved into X86MachineState in the next steps to avoid the leakage.
>
> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
> ---
>   include/hw/i386/x86.h | 2 +-
>   hw/i386/microvm.c     | 2 +-
>   hw/i386/pc_sysfw.c    | 4 ++--
>   hw/i386/x86.c         | 4 ++--
>   4 files changed, 6 insertions(+), 6 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>