]> git.proxmox.com Git - qemu-server.git/commit
cfg2cmd: switch off ACPI hotplug on bridges for q35 VMs
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Thu, 21 Oct 2021 07:51:22 +0000 (09:51 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Thu, 4 Nov 2021 14:30:30 +0000 (15:30 +0100)
commitcc1810363579b8d6bf7dd8d681b831f33209bbb0
tree019a6edb27a127e76184224fa3b5aab3378c3fa4
parent02cfca4b71ce8dd624bbd9408a50e1384cc80907
cfg2cmd: switch off ACPI hotplug on bridges for q35 VMs

See commit 17858a1695 (hw/acpi/ich9: Set ACPI PCI hot-plug as default
on Q35)[0] in upstream QEMU repository for details about why the change
was made.

As that change affects systemds predictable interface naming[1],
e.g., by going from a previously `ens18` name to `enp6s18`, it may
have rather bad effects for users that did not setup some .link files
to enforce a specific naming by an more stable information like the
NIC's MAC-Address

The alternative would be making the preferred mode of hotplug an
option like `hotplug-mode=<acpi|pcie>`, but it does not seems like
one would like to change that much in the first place...

Note the changes to the tests and especially the tests with q35
machines that did not change.

[0]: https://gitlab.com/qemu-project/qemu/-/commit/17858a1695
[1]: https://www.freedesktop.org/software/systemd/man/systemd.net-naming-scheme.html#Naming

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>
Tested-by: Dominik Csapak <d.csapak@proxmox.com>
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
PVE/QemuServer.pm
test/cfg2cmd/q35-linux-hostpci-multifunction.conf.cmd
test/cfg2cmd/q35-linux-hostpci.conf.cmd
test/cfg2cmd/q35-simple.conf.cmd