Re: [PATCH v6 3/3] tests/qtest/libqos: Add generic pci host bridge in arm-virt machine

[Alex Bennée]

Eric Auger <eric.auger@redhat.com> writes:

> Up to now the virt-machine node contains a virtio-mmio node.
> However no driver produces any PCI interface node. Hence, PCI
> tests cannot be run with aarch64 binary.
>
> Add a GPEX driver node that produces a pci interface node. This latter
> then can be consumed by all the pci tests. One of the first motivation
> was to be able to run the virtio-iommu-pci tests.
>
> We still face an issue with pci hotplug tests as hotplug cannot happen
> on the pcie root bus and require a generic root port. This will be
> addressed later on.
>
> We force cpu=max along with aarch64/virt machine as some PCI tests
> require high MMIO regions to be available.

Where would I be able to force disable-legacy=off for the PCI device
from? Building on this for GPIO I run into the following:

  subprocess_run_one_test: 
/aarch64/virt/generic-pcihost/pci-bus-generic/pci-bus/vhost-user-gpio-pci/vhost-user-gpio/vhost-user-gpio-tests/read-guest-mem/memfile/subprocess
  vhost_user_test_setup: -M virt, -cpu max -device 
vhost-user-gpio-pci,id=gpio0,addr=04.0,chardev=chr-vhost-user-test
  vu_gpio_get_protocol_features: 0x202
  qemu-system-aarch64: -device 
vhost-user-gpio-pci,id=gpio0,addr=04.0,chardev=chr-vhost-user-test: Device 
doesn't support modern mode, and legacy mode is disabled
  Set disable-legacy to off
  Broken pipe

and I think this needs to be applied to the root bus device?

Anyway otherwise it looks OK to me:

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée