Re: [PATCH v4 0/5] test and QEMU fixes to ensure proper PCIE device usage

[Michael S. Tsirkin]

Ani, two versions a day is too much, pls don't unless first one
has been sent by mistake or something. then explain what
it was pls.

On Mon, Jun 26, 2023 at 07:23:19PM +0530, Ani Sinha wrote:
> Patches 1-4:
> Fix tests so that devices do not use non-zero slots on the pcie root
> ports. PCIE ports only have one slot, so PCIE devices can only be
> plugged into slot 0 on a PCIE port.
> 
> Patch 5:
> Enforce only one slot on PCIE port.
> 
> The test fixes must be applied before the QEMU change that checks for use
> of a single slot in PCIE port.
> 
> CC: mst@redhat.com
> CC: imammedo@redhat.com
> CC: jusual@redhat.com
> CC: thuth@redhat.com
> CC: lvivier@redhat.com
> CC: michael.labiuk@virtuozzo.com
> 
> Changelog:
> v4: reword commit log for patch 4.
> v3: tags added. reword the error description in patch 5. Reword commit log in 
> patch 4. 
> v2: add hd-geo-test fix as well as the actual QEMU code fix to the patchset.
> The patches are added in the right order.
> 
> Ani Sinha (5):
>   tests/acpi: allow changes in DSDT.noacpihp table blob
>   tests/acpi/bios-tables-test: use the correct slot on the
>     pcie-root-port
>   tests/acpi/bios-tables-test: update acpi blob q35/DSDT.noacpihp
>   tests/qtest/hd-geo-test: fix incorrect pcie-root-port usage and
>     simplify test
>   hw/pci: ensure PCIE devices are plugged into only slot 0 of PCIE port
> 
>  hw/pci/pci.c                      |   6 ++++++
>  tests/data/acpi/q35/DSDT.noacpihp | Bin 8248 -> 8241 bytes
>  tests/qtest/bios-tables-test.c    |   4 ++--
>  tests/qtest/hd-geo-test.c         |  18 ++++++++----------
>  4 files changed, 16 insertions(+), 12 deletions(-)
> 
> -- 
> 2.39.1