Hi Thomas, On 11/2/21 2:34 PM, Thomas Huth wrote: > On 01/11/2021 19.48, Eric Auger wrote: >> Add the framework to test the virtio-iommu-pci device >> and tests exercising the attach/detach, map/unmap API. >> >> Signed-off-by: Eric Auger <eric.au...@redhat.com> >> Tested-by: Jean-Philippe Brucker <jean-phili...@linaro.org> >> Reviewed-by: Jean-Philippe Brucker <jean-phili...@linaro.org> >> >> --- >> >> v4 -> v5: >> - remove printf and move a comment >> - Added Jean-Philippe's T-b and R-b >> --- >> tests/qtest/libqos/meson.build | 1 + >> tests/qtest/libqos/virtio-iommu.c | 126 ++++++++++++ >> tests/qtest/libqos/virtio-iommu.h | 40 ++++ >> tests/qtest/meson.build | 1 + >> tests/qtest/virtio-iommu-test.c | 326 ++++++++++++++++++++++++++++++ >> 5 files changed, 494 insertions(+) >> create mode 100644 tests/qtest/libqos/virtio-iommu.c >> create mode 100644 tests/qtest/libqos/virtio-iommu.h >> create mode 100644 tests/qtest/virtio-iommu-test.c > > There's another failure on Travis-CI: > > https://app.travis-ci.com/github/huth/qemu/jobs/546268575#L7240 > > ERROR:../tests/qtest/virtio-iommu-test.c:38:pci_config: > assertion failed (domain_range_end == 32): (536870912 == 32) > > 536870912 is 0x20000000, so this is likely an endianess issue? Hum It seems so. Thank you for reporting the issue. I think the get_config() cb implementation in the virtio-iommu device is wrong and needs to be fixed. I will send a fix along with the test.
Thanks! Eric > > Thomas >
