[PATCH-for-10.1 v3 0/9] hw/arm: GIC ITS=off ACPI tables fixes

Philippe Mathieu-Daudé posted 9 patches 7 months, 2 weeks ago
Failed in applying to current master (apply log)
include/hw/arm/virt.h                     |   1 -
include/hw/intc/arm_gicv3_its_common.h    |   2 +-
hw/arm/virt-acpi-build.c                  |  48 +++++++++++++---------
hw/arm/virt.c                             |  23 ++++-------
tests/qtest/bios-tables-test.c            |  22 ++++++++++
tests/data/acpi/aarch64/virt/APIC.its_off | Bin 0 -> 164 bytes
tests/data/acpi/aarch64/virt/FACP.its_off | Bin 0 -> 276 bytes
tests/data/acpi/aarch64/virt/IORT.its_off | Bin 0 -> 212 bytes
8 files changed, 59 insertions(+), 37 deletions(-)
create mode 100644 tests/data/acpi/aarch64/virt/APIC.its_off
create mode 100644 tests/data/acpi/aarch64/virt/FACP.its_off
create mode 100644 tests/data/acpi/aarch64/virt/IORT.its_off
[PATCH-for-10.1 v3 0/9] hw/arm: GIC ITS=off ACPI tables fixes
Posted by Philippe Mathieu-Daudé 7 months, 2 weeks ago
Since v2:
- Fixed no_tcg_its inverted logic (rth)

Fix ACPI tables for '-M its=off' CLI option.

Regards,

Phil.

Philippe Mathieu-Daudé (9):
  hw/arm/virt: Remove pointless VirtMachineState::tcg_its field
  hw/intc/gicv3_its: Do not check its_class_name() for NULL
  hw/arm/virt: Simplify create_its()
  hw/arm/virt-acpi: Factor its_enabled() helper out
  qtest/bios-tables-test: Add test for -M virt,its=off
  qtest/bios-tables-test: Whitelist aarch64/virt 'its_off' variant blobs
  hw/arm/virt-acpi: Always build IORT table (even with GIC ITS disabled)
  hw/arm/virt-acpi: Do not advertise disabled GIC ITS
  qtest/bios-tables-test: Update aarch64/virt 'its_off' variant blobs

 include/hw/arm/virt.h                     |   1 -
 include/hw/intc/arm_gicv3_its_common.h    |   2 +-
 hw/arm/virt-acpi-build.c                  |  48 +++++++++++++---------
 hw/arm/virt.c                             |  23 ++++-------
 tests/qtest/bios-tables-test.c            |  22 ++++++++++
 tests/data/acpi/aarch64/virt/APIC.its_off | Bin 0 -> 164 bytes
 tests/data/acpi/aarch64/virt/FACP.its_off | Bin 0 -> 276 bytes
 tests/data/acpi/aarch64/virt/IORT.its_off | Bin 0 -> 212 bytes
 8 files changed, 59 insertions(+), 37 deletions(-)
 create mode 100644 tests/data/acpi/aarch64/virt/APIC.its_off
 create mode 100644 tests/data/acpi/aarch64/virt/FACP.its_off
 create mode 100644 tests/data/acpi/aarch64/virt/IORT.its_off

-- 
2.47.1


Re: [PATCH-for-10.1 v3 0/9] hw/arm: GIC ITS=off ACPI tables fixes
Posted by Philippe Mathieu-Daudé 7 months ago
Hi Gustavo,

Could you take over this series? You seem better suited
with ACPI and testing with Aarch64 virt machine :)

Thanks,

Phil.

On 3/4/25 22:40, Philippe Mathieu-Daudé wrote:
> 
> Fix ACPI tables for '-M its=off' CLI option.
> 
> Regards,
> 
> Phil.
> 
> Philippe Mathieu-Daudé (9):
>    hw/arm/virt: Remove pointless VirtMachineState::tcg_its field
>    hw/intc/gicv3_its: Do not check its_class_name() for NULL
>    hw/arm/virt: Simplify create_its()
>    hw/arm/virt-acpi: Factor its_enabled() helper out
>    qtest/bios-tables-test: Add test for -M virt,its=off
>    qtest/bios-tables-test: Whitelist aarch64/virt 'its_off' variant blobs
>    hw/arm/virt-acpi: Always build IORT table (even with GIC ITS disabled)
>    hw/arm/virt-acpi: Do not advertise disabled GIC ITS
>    qtest/bios-tables-test: Update aarch64/virt 'its_off' variant blobs

Re: [PATCH-for-10.1 v3 0/9] hw/arm: GIC ITS=off ACPI tables fixes
Posted by Gustavo Romero 7 months ago
Hi Phil,

On 4/15/25 05:19, Philippe Mathieu-Daudé wrote:
> Hi Gustavo,
> 
> Could you take over this series? You seem better suited
> with ACPI and testing with Aarch64 virt machine :)
> 
> Thanks,

Sure. I just need to clarify a couple of things regarding
the IORT spec. Meanwhile I'm (hopefully) waiting Igor or
Michael confirm that the patch organization I've suggested
makes sense for them.


Cheers,
Gustavo