* [PATCH for-9.1] hw/loongarch: Fix length for lowram in ACPI SRAT
@ 2024-08-20 18:42 Jiaxun Yang
2024-08-21 2:11 ` maobibo
2024-08-21 9:27 ` Philippe Mathieu-Daudé
0 siblings, 2 replies; 3+ messages in thread
From: Jiaxun Yang @ 2024-08-20 18:42 UTC (permalink / raw)
To: qemu-devel; +Cc: Song Gao, Bibo Mao, Jiaxun Yang
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(-)
diff --git a/hw/loongarch/acpi-build.c b/hw/loongarch/acpi-build.c
index 72bfc35ae6c2..2638f8743463 100644
--- a/hw/loongarch/acpi-build.c
+++ b/hw/loongarch/acpi-build.c
@@ -218,7 +218,7 @@ build_srat(GArray *table_data, BIOSLinker *linker, MachineState *machine)
* highram: [VIRT_HIGHMEM_BASE, +(len - gap))
*/
if (len >= gap) {
- build_srat_memory(table_data, base, len, i, MEM_AFFINITY_ENABLED);
+ build_srat_memory(table_data, base, gap, i, MEM_AFFINITY_ENABLED);
len -= gap;
base = VIRT_HIGHMEM_BASE;
gap = machine->ram_size - VIRT_LOWMEM_SIZE;
---
base-commit: 075fd020afe3150a0e6c4b049705b358b597b65a
change-id: 20240820-fix-numa-range-f1f0302e138d
Best regards,
--
Jiaxun Yang <jiaxun.yang@flygoat.com>
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH for-9.1] hw/loongarch: Fix length for lowram in ACPI SRAT
2024-08-20 18:42 [PATCH for-9.1] hw/loongarch: Fix length for lowram in ACPI SRAT Jiaxun Yang
@ 2024-08-21 2:11 ` maobibo
2024-08-21 9:27 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 3+ messages in thread
From: maobibo @ 2024-08-21 2:11 UTC (permalink / raw)
To: Jiaxun Yang, qemu-devel; +Cc: Song Gao
On 2024/8/21 上午2: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(-)
>
> diff --git a/hw/loongarch/acpi-build.c b/hw/loongarch/acpi-build.c
> index 72bfc35ae6c2..2638f8743463 100644
> --- a/hw/loongarch/acpi-build.c
> +++ b/hw/loongarch/acpi-build.c
> @@ -218,7 +218,7 @@ build_srat(GArray *table_data, BIOSLinker *linker, MachineState *machine)
> * highram: [VIRT_HIGHMEM_BASE, +(len - gap))
> */
> if (len >= gap) {
> - build_srat_memory(table_data, base, len, i, MEM_AFFINITY_ENABLED);
> + build_srat_memory(table_data, base, gap, i, MEM_AFFINITY_ENABLED);
> len -= gap;
> base = VIRT_HIGHMEM_BASE;
> gap = machine->ram_size - VIRT_LOWMEM_SIZE;
>
> ---
> base-commit: 075fd020afe3150a0e6c4b049705b358b597b65a
> change-id: 20240820-fix-numa-range-f1f0302e138d
>
> Best regards,
>
Thanks for catching this.
Reviewed-by: Bibo Mao <maobibo@loongson.cn>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH for-9.1] hw/loongarch: Fix length for lowram in ACPI SRAT
2024-08-20 18:42 [PATCH for-9.1] hw/loongarch: Fix length for lowram in ACPI SRAT Jiaxun Yang
2024-08-21 2:11 ` maobibo
@ 2024-08-21 9:27 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2024-08-21 9:27 UTC (permalink / raw)
To: Jiaxun Yang, qemu-devel; +Cc: Song Gao, Bibo Mao
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.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-08-21 9:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-20 18:42 [PATCH for-9.1] hw/loongarch: Fix length for lowram in ACPI SRAT Jiaxun Yang
2024-08-21 2:11 ` maobibo
2024-08-21 9:27 ` Philippe Mathieu-Daudé
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).