I kept R-B tags from Fabiano and Phil despite a little change in how patch 1 looks and splitting it, hope that is okay.
Since RFC: - Split endian fix + invalid value assert into two paches. - Changed the invalid message assertion in what is now patch 1 to use g_assert_cmphex so the failing value can be seen. - Added patch 3 to factor duplicated code we just changed. Some patches I have for e1000e and xhci tests also want to use the same function... - Fixed stray hunk from mis-rebase in the spapr patch. Thanks, Nick Nicholas Piggin (4): tests/qtest: Enforce zero for the "un-fired" msix message value tests/qtest: Fix virtio msix message endianness tests/qtest: Add libqos function for testing msix interrupt status tests/qtest: Enable spapr dma with linear iommu map tests/qtest/libqos/pci.h | 6 +-- hw/ppc/spapr_iommu.c | 9 +++- tests/qtest/e1000e-test.c | 21 --------- tests/qtest/igb-test.c | 21 --------- tests/qtest/libqos/generic-pcihost.c | 1 - tests/qtest/libqos/pci-pc.c | 3 -- tests/qtest/libqos/pci-spapr.c | 7 +-- tests/qtest/libqos/pci.c | 62 ++++++++++++++++++++------ tests/qtest/libqos/virtio-pci-modern.c | 30 +++---------- tests/qtest/libqos/virtio-pci.c | 38 ++++------------ tests/qtest/vhost-user-blk-test.c | 6 --- tests/qtest/virtio-blk-test.c | 12 ----- 12 files changed, 77 insertions(+), 139 deletions(-) -- 2.47.1