Re: [PATCH for-9.1] hw/loongarch: Fix length for lowram in ACPI SRAT

[Philippe Mathieu-Daudé]

On 20/8/24 20:42, Jiaxun Yang wrote:
> The size of lowram should be "gap" instead of the whole node.
>
> This is failing kernel's sanity check:
>
> [    0.000000] ACPI: SRAT: Node 0 PXM 0 [mem 0x00000000-0xffffffff]
> [    0.000000] ACPI: SRAT: Node 0 PXM 0 [mem 0x80000000-0x16fffffff]
> [    0.000000] ACPI: SRAT: Node 1 PXM 1 [mem 0x170000000-0x26fffffff]
> [    0.000000] Warning: node 0 [mem 0x00000000-0xffffffff] overlaps with itself 
> [mem 0x80000000-0x16fffffff]
>
> Fixes: fc100011f38d ("hw/loongarch: Refine acpi srat table for numa memory")
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ---
>   hw/loongarch/acpi-build.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

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

This is just a warning, so not worth delaying the release IMHO.