New test in bios-tables-test.c for the ACPI PCI hotplug on ARM series

New test in bios-tables-test.c for the ACPI PCI hotplug on ARM series

[Gustavo Romero]

Hi Eric,

While we go through the ACPI PCI Hotplug series [0] and review it, I'm
starting to take a look at tests/qtest/bios-tables-test.c.

You mentioned testing the combinations of "acpi-root-pci-hotplug"
and "acpi-pci-hotplug-with-bridge-support" flags [1], but taking a
look now I see that "acpi-root-pci-hotplug" is only used in a PCI
context (not in PCIE, so not meaningful in GPEX), hence
"acpi-pci-hotplug-with-bridge-support" would be the interesting
flag for testing the ACPI PCI HP series for ARM. Since this maps to
'use_acpi_hotplug_bridge' variable in i386 (e.g., in ICH9 controller)
and this variable is being controlled by machine flag "acpi_pcihp"
in the series, I understand it's only a matter of testing the ACPI for
acpi_pcihp=off and acpi_pcihp=on. What do you think?


Cheers,
Gustavo

[0] https://lists.nongnu.org/archive/html/qemu-devel/2025-05/msg03487.html
[1] https://lists.nongnu.org/archive/html/qemu-trivial/2025-05/msg00020.html

Re: New test in bios-tables-test.c for the ACPI PCI hotplug on ARM series

[Gustavo Romero]

Hi Eric,

On 5/22/25 22:13, Gustavo Romero wrote:
> Hi Eric,
> 
> While we go through the ACPI PCI Hotplug series [0] and review it, I'm
> starting to take a look at tests/qtest/bios-tables-test.c.
> 
> You mentioned testing the combinations of "acpi-root-pci-hotplug"
> and "acpi-pci-hotplug-with-bridge-support" flags [1], but taking a
> look now I see that "acpi-root-pci-hotplug" is only used in a PCI
> context (not in PCIE, so not meaningful in GPEX), hence
> "acpi-pci-hotplug-with-bridge-support" would be the interesting
> flag for testing the ACPI PCI HP series for ARM. Since this maps to
> 'use_acpi_hotplug_bridge' variable in i386 (e.g., in ICH9 controller)
> and this variable is being controlled by machine flag "acpi_pcihp"
> in the series, I understand it's only a matter of testing the ACPI for
> acpi_pcihp=off and acpi_pcihp=on. What do you think?

I've posted v1:

https://mail.gnu.org/archive/html/qemu-devel/2025-05/msg05824.html


Cheers,
Gustavo