[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 0/8] AArch64/HMAT support and tests
From: |
Yicong Yang |
Subject: |
Re: [PATCH v2 0/8] AArch64/HMAT support and tests |
Date: |
Thu, 15 Sep 2022 15:26:40 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.5.1 |
On 2022/7/19 17:49, Hesham Almatary wrote:
> This patchset adds support for AArch64/HMAT including a test.
> It relies on other two patch sets from:
>
> Brice Goglin: to support -numa without initiators on q35/x86.
> https://lore.kernel.org/all/ed23accb-2c8b-90f4-a7a3-f81cc57bf678@inria.fr/
> Xiang Chen: to enable/support HMAT on AArch64.
>
> https://lore.kernel.org/all/1643102134-15506-1-git-send-email-chenxiang66@hisilicon.com/
>
> I further add a test with ACPI/HMAT tables that uses the two
> patch sets.
>
Hi Hesham,
I've used this feature locally for a period on aarch64 and it works fine. Also
tested the qemu
test added based on v7.1.0-rc4 and `make test` show:
108/130 qemu:qtest+qtest-aarch64 / qtest-aarch64/bios-tables-test OK
99.80s 7 subtests passed
Wish to see this in mainline!
Tested-by: Yicong Yang <yangyicong@hisilicon.com>
Thanks!
> Changes from v1:
> - Generate APIC and PPTT ACPI tables for AArch64/virt
> - Avoid using legacy syntax in numa/bios tests
> - Delete unchanged FACP tables
>
> Brice Goglin (4):
> hmat acpi: Don't require initiator value in -numa
> tests: acpi: add and whitelist *.hmat-noinitiator expected blobs
> tests: acpi: q35: add test for hmat nodes without initiators
> tests: acpi: q35: update expected blobs *.hmat-noinitiators expected
> HMAT:
>
> Hesham Almatary (3):
> tests: Add HMAT AArch64/virt empty table files
> tests: acpi: aarch64/virt: add a test for hmat nodes with no initiators
> tests: virt: Update expected *.acpihmatvirt tables
>
> Xiang Chen (1):
> hw/arm/virt: Enable HMAT on arm virt machine
>
> hw/arm/Kconfig | 1 +
> hw/arm/virt-acpi-build.c | 7 ++
> hw/core/machine.c | 4 +-
> tests/data/acpi/q35/APIC.acpihmat-noinitiator | Bin 0 -> 144 bytes
> tests/data/acpi/q35/DSDT.acpihmat-noinitiator | Bin 0 -> 8553 bytes
> tests/data/acpi/q35/HMAT.acpihmat-noinitiator | Bin 0 -> 288 bytes
> tests/data/acpi/q35/SRAT.acpihmat-noinitiator | Bin 0 -> 312 bytes
> tests/data/acpi/virt/APIC.acpihmatvirt | Bin 0 -> 396 bytes
> tests/data/acpi/virt/DSDT.acpihmatvirt | Bin 0 -> 5282 bytes
> tests/data/acpi/virt/HMAT.acpihmatvirt | Bin 0 -> 288 bytes
> tests/data/acpi/virt/PPTT.acpihmatvirt | Bin 0 -> 196 bytes
> tests/data/acpi/virt/SRAT.acpihmatvirt | Bin 0 -> 240 bytes
> tests/qtest/bios-tables-test.c | 109 ++++++++++++++++++
> 13 files changed, 118 insertions(+), 3 deletions(-)
> create mode 100644 tests/data/acpi/q35/APIC.acpihmat-noinitiator
> create mode 100644 tests/data/acpi/q35/DSDT.acpihmat-noinitiator
> create mode 100644 tests/data/acpi/q35/HMAT.acpihmat-noinitiator
> create mode 100644 tests/data/acpi/q35/SRAT.acpihmat-noinitiator
> create mode 100644 tests/data/acpi/virt/APIC.acpihmatvirt
> create mode 100644 tests/data/acpi/virt/DSDT.acpihmatvirt
> create mode 100644 tests/data/acpi/virt/HMAT.acpihmatvirt
> create mode 100644 tests/data/acpi/virt/PPTT.acpihmatvirt
> create mode 100644 tests/data/acpi/virt/SRAT.acpihmatvirt
>