mbox series

[v5,0/5] test and QEMU fixes to ensure proper PCIE device usage

Message ID 20230626160156.3938-1-anisinha@redhat.com (mailing list archive)
Headers show
Series test and QEMU fixes to ensure proper PCIE device usage | expand

Message

Ani Sinha June 26, 2023, 4:01 p.m. UTC
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:
=========
v5: no code changes - correct a mistake in the commit log message.

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 (4):
  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

Peter Maydell (1):
  Revert "cputlb: Restrict SavedIOTLB to system emulation"

 include/hw/core/cpu.h             |   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, 12 insertions(+), 16 deletions(-)

Comments

Ani Sinha June 26, 2023, 4:06 p.m. UTC | #1
Scratch this … I will resend …


> On 26-Jun-2023, at 9:31 PM, Ani Sinha <anisinha@redhat.com> 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:
> =========
> v5: no code changes - correct a mistake in the commit log message.
> 
> 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 (4):
>  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
> 
> Peter Maydell (1):
>  Revert "cputlb: Restrict SavedIOTLB to system emulation"
> 
> include/hw/core/cpu.h             |   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, 12 insertions(+), 16 deletions(-)
> 
> -- 
> 2.39.1
>