On Thu, Sep 25, 2025 at 20:07:00 +0200, Andrea Bolognani via Devel wrote:
> The microvm machine type doesn't support PCI, so adding PCI
> controllers to it doesn't make sense, and nor does adding a
> USB controller or a memballon since both are PCI devices.
> 
> Signed-off-by: Andrea Bolognani <[email protected]>
> ---
>  src/qemu/qemu_postparse.c                     |  3 ++-
>  .../microvm-pci.x86_64-latest.args            |  2 --
>  .../microvm-pci.x86_64-latest.xml             |  6 -----
>  ...ller-automatic-microvm.x86_64-latest.args} | 20 ++++++++--------
>  ...roller-automatic-microvm.x86_64-latest.err |  1 -
>  ...roller-automatic-microvm.x86_64-latest.xml | 23 +++++++++++++++++++
>  tests/qemuxmlconftest.c                       |  2 +-
>  7 files changed, 35 insertions(+), 22 deletions(-)
>  copy tests/qemuxmlconfdata/{microvm-pci.x86_64-latest.args => 
> usb-controller-automatic-microvm.x86_64-latest.args} (56%)
>  delete mode 100644 
> tests/qemuxmlconfdata/usb-controller-automatic-microvm.x86_64-latest.err
>  create mode 100644 
> tests/qemuxmlconfdata/usb-controller-automatic-microvm.x86_64-latest.xml

This patch is quite similar to the one I've proposed a year and half
ago.

Reviewed-by: Peter Krempa <[email protected]>

Reply via email to