January 2023 - Devel - libvirt List Archives

[libvirt PATCH 0/7] Extract the integration job commands to a shell scripts
by Erik Skultety 26 Jan '23

26 Jan '23

Using shell scripts rather than inlining shell commands to YAML feels more natural, more readable, and will keep all different variations of execution consistent. Essentially the only disadvantage is that we won't see each command listed one-by-one in gitlab's log output (unless we set -x that is), but given that shell would complain if something was wrong with the script, it's fairly easy to identify the problem. Here's a successful pipeline after the change: https://gitlab.com/eskultety/libvirt/-/pipelines/753090691 Erik Skultety (7): ci: integration: Extract several hidden job definitions to a script ci: integration: Drop the 'install-deps' hidden job and reference ci: integration-template: Drop Fedora 35 check ci: integration.sh: Replace most 'sudo' usage with running as root ci: integration.sh: Add/Rewrite/Reformat commentaries ci: integration.sh: Replace 'test' with '[' operator ci: integration.sh: Define the SCRATCH_DIR variable for local execution ci/integration-template.yml | 42 ++---------------------------- ci/integration.sh | 51 +++++++++++++++++++++++++++++++++++++ 2 files changed, 53 insertions(+), 40 deletions(-) create mode 100644 ci/integration.sh -- 2.39.0

3 23

[PATCH] qemu_domain: Don't unref NULL hash table in qemuDomainRefreshStatsSchema()
by Michal Privoznik 26 Jan '23

26 Jan '23

The g_hash_table_unref() function does not accept NULL. Passing NULL results in a glib warning being triggered. Check whether the hash table is not NULL and unref it only then. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- src/qemu/qemu_domain.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index b6ad118f1f..881c88d54c 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -11923,7 +11923,8 @@ qemuDomainRefreshStatsSchema(virDomainObj *dom) if (!schema) return -1; - g_hash_table_unref(priv->statsSchema); + if (priv->statsSchema) + g_hash_table_unref(priv->statsSchema); priv->statsSchema = schema; return 0; -- 2.39.1

2 1

[PATCH 0/7] qemu: Fix implicit q35 watchdog
by Martin Kletzander 26 Jan '23

26 Jan '23

More info in commit messages (duh). Martin Kletzander (7): Avoid unnecessary nesting Support multiple watchdog devices Add iTCO watchdog support qemu: Enable iTCO watchdog by disabling its noreboot pin strap qemu: Add implicit watchdog for q35 machine types Document change to multiple watchdogs news: Add information about iTCO watchdog changes NEWS.rst | 6 ++ docs/formatdomain.rst | 10 ++- src/conf/domain_conf.c | 83 +++++++++++++------ src/conf/domain_conf.h | 9 +- src/conf/schemas/domaincommon.rng | 5 +- src/libvirt_private.syms | 1 + src/qemu/qemu_alias.c | 26 ++++-- src/qemu/qemu_alias.h | 5 +- src/qemu/qemu_command.c | 34 ++++++-- src/qemu/qemu_domain.c | 44 ++++++++++ src/qemu/qemu_domain_address.c | 9 +- src/qemu/qemu_driver.c | 14 ++-- src/qemu/qemu_hotplug.c | 71 +++++++--------- src/qemu/qemu_process.c | 3 +- src/qemu/qemu_validate.c | 50 +++++++++++ .../qemustatusxml2xmldata/vcpus-multi-in.xml | 1 + tests/qemuxml2argvdata/autoindex.args | 2 + .../boot-floppy-q35.x86_64-latest.args | 2 + tests/qemuxml2argvdata/boot-floppy-q35.xml | 1 + tests/qemuxml2argvdata/cpu-host-model.args | 2 + .../cpu-host-model.x86_64-4.2.0.args | 2 + .../cpu-host-model.x86_64-5.0.0.args | 2 + .../cpu-host-model.x86_64-5.1.0.args | 2 + .../cpu-host-model.x86_64-5.2.0.args | 2 + .../cpu-host-model.x86_64-6.0.0.args | 2 + .../cpu-host-model.x86_64-6.1.0.args | 2 + .../cpu-host-model.x86_64-latest.args | 2 + .../disk-floppy-q35.x86_64-latest.args | 2 + ...are-auto-bios-stateless.x86_64-latest.args | 2 + .../firmware-auto-bios.x86_64-latest.args | 2 + ...-auto-efi-enrolled-keys.x86_64-latest.args | 2 + ...-auto-efi-loader-secure.x86_64-latest.args | 2 + ...to-efi-no-enrolled-keys.x86_64-latest.args | 2 + ...are-auto-efi-no-secboot.x86_64-latest.args | 2 + ...firmware-auto-efi-nvram.x86_64-latest.args | 2 + ...rmware-auto-efi-secboot.x86_64-latest.args | 2 + ...ware-auto-efi-stateless.x86_64-latest.args | 2 + .../firmware-auto-efi.x86_64-latest.args | 2 + .../firmware-manual-efi-acpi-q35.args | 2 + .../firmware-manual-efi-secure.args | 2 + .../firmware-manual-noefi-acpi-q35.args | 2 + .../firmware-manual-noefi-noacpi-q35.args | 2 + ...dev-scsi-vhost-scsi-pcie.x86_64-4.2.0.args | 2 + ...ev-scsi-vhost-scsi-pcie.x86_64-latest.args | 2 + .../hvf-x86_64-q35-headless.args | 2 + .../intel-iommu-aw-bits.x86_64-latest.args | 2 + .../qemuxml2argvdata/intel-iommu-aw-bits.xml | 1 + ...ntel-iommu-caching-mode.x86_64-latest.args | 2 + .../intel-iommu-caching-mode.xml | 1 + ...ntel-iommu-device-iotlb.x86_64-latest.args | 2 + .../intel-iommu-device-iotlb.xml | 1 + .../intel-iommu-eim.x86_64-latest.args | 2 + tests/qemuxml2argvdata/intel-iommu-eim.xml | 1 + .../intel-iommu.x86_64-latest.args | 2 + tests/qemuxml2argvdata/intel-iommu.xml | 1 + ...othreads-ids-pool-sizes.x86_64-latest.args | 2 + .../iothreads-ids-pool-sizes.xml | 1 + .../machine-smm-off.x86_64-latest.args | 2 + .../machine-smm-on.x86_64-latest.args | 2 + tests/qemuxml2argvdata/pcie-expander-bus.args | 2 + .../pcie-root-port-model-generic.args | 2 + .../pcie-root-port-model-ioh3420.args | 2 + ...cie-root-port-nohotplug.x86_64-latest.args | 2 + tests/qemuxml2argvdata/pcie-root-port.args | 2 + tests/qemuxml2argvdata/pcie-root.args | 2 + .../pcie-switch-downstream-port.args | 2 + .../pcie-switch-upstream-port.args | 2 + tests/qemuxml2argvdata/pcihole64-q35.args | 2 + ...q35-default-devices-only.x86_64-4.2.0.args | 2 + ...35-default-devices-only.x86_64-latest.args | 2 + .../q35-multifunction.x86_64-4.2.0.args | 2 + .../q35-multifunction.x86_64-latest.args | 2 + .../q35-pci-force-address.args | 2 + .../q35-pcie-autoadd.x86_64-4.2.0.args | 2 + .../q35-pcie-autoadd.x86_64-latest.args | 2 + .../q35-pcie.x86_64-4.2.0.args | 2 + .../q35-pcie.x86_64-latest.args | 2 + .../q35-pm-disable-fallback.args | 2 + tests/qemuxml2argvdata/q35-pm-disable.args | 2 + tests/qemuxml2argvdata/q35-usb2-multi.args | 2 + tests/qemuxml2argvdata/q35-usb2-reorder.args | 2 + tests/qemuxml2argvdata/q35-usb2.args | 2 + .../q35-virt-manager-basic.x86_64-4.2.0.args | 2 + .../q35-virt-manager-basic.x86_64-latest.args | 2 + tests/qemuxml2argvdata/q35.args | 2 + .../sgx-epc.x86_64-7.0.0.args | 2 + tests/qemuxml2argvdata/sgx-epc.xml | 1 + .../tseg-explicit-size.x86_64-latest.args | 2 + .../usb-controller-default-q35.args | 2 + .../usb-controller-explicit-q35.args | 2 + .../user-aliases2.x86_64-latest.args | 2 + .../vcpu-placement-static.args | 2 + .../vcpu-placement-static.xml | 1 + ...vhost-user-fs-hugepages.x86_64-latest.args | 2 + .../vhost-user-fs-hugepages.xml | 1 + .../virtio-iommu-x86_64.x86_64-latest.args | 2 + ...virtio-non-transitional.x86_64-latest.args | 2 + .../virtio-transitional.x86_64-latest.args | 2 + .../watchdog-q35-multiple.x86_64-latest.args | 40 +++++++++ .../watchdog-q35-multiple.xml | 25 ++++++ ...default-cpu-kvm-q35-4.2.x86_64-latest.args | 2 + ...efault-cpu-tcg-features.x86_64-latest.args | 2 + .../x86_64-default-cpu-tcg-features.xml | 1 + ...default-cpu-tcg-q35-4.2.x86_64-latest.args | 2 + .../x86_64-q35-graphics.x86_64-latest.args | 2 + .../x86_64-q35-headless.x86_64-latest.args | 2 + tests/qemuxml2argvtest.c | 1 + tests/qemuxml2xmloutdata/autoindex.xml | 1 + ...ware-auto-bios-stateless.x86_64-latest.xml | 1 + .../firmware-auto-bios.x86_64-latest.xml | 1 + ...e-auto-efi-enrolled-keys.x86_64-latest.xml | 1 + ...e-auto-efi-loader-secure.x86_64-latest.xml | 1 + ...uto-efi-no-enrolled-keys.x86_64-latest.xml | 1 + ...ware-auto-efi-no-secboot.x86_64-latest.xml | 1 + .../firmware-auto-efi-nvram.x86_64-latest.xml | 1 + ...irmware-auto-efi-secboot.x86_64-latest.xml | 1 + .../firmware-auto-efi.x86_64-latest.xml | 1 + ...dev-scsi-vhost-scsi-pcie.x86_64-latest.xml | 1 + .../hvf-x86_64-q35-headless.xml | 1 + .../machine-smm-off.x86_64-latest.xml | 1 + .../machine-smm-on.x86_64-latest.xml | 1 + .../net-isolated-port.x86_64-latest.xml | 1 + .../qemuxml2xmloutdata/pcie-expander-bus.xml | 1 + .../pcie-root-port-model-generic.xml | 1 + .../pcie-root-port-model-ioh3420.xml | 1 + ...pcie-root-port-nohotplug.x86_64-latest.xml | 1 + tests/qemuxml2xmloutdata/pcie-root-port.xml | 1 + tests/qemuxml2xmloutdata/pcie-root.xml | 1 + .../pcie-switch-downstream-port.xml | 1 + .../pcie-switch-upstream-port.xml | 1 + tests/qemuxml2xmloutdata/pcihole64-q35.xml | 1 + ...q35-default-devices-only.x86_64-latest.xml | 1 + .../q35-multifunction.x86_64-latest.xml | 1 + .../q35-pci-force-address.xml | 1 + .../q35-pcie-autoadd.x86_64-latest.xml | 1 + .../q35-pcie.x86_64-latest.xml | 1 + tests/qemuxml2xmloutdata/q35-usb2-multi.xml | 1 + tests/qemuxml2xmloutdata/q35-usb2-reorder.xml | 1 + tests/qemuxml2xmloutdata/q35-usb2.xml | 1 + .../q35-virt-manager-basic.x86_64-latest.xml | 1 + tests/qemuxml2xmloutdata/q35.xml | 1 + .../tseg-explicit-size.x86_64-latest.xml | 1 + .../usb-controller-default-q35.xml | 1 + .../usb-controller-explicit-q35.xml | 1 + .../virtio-iommu-x86_64.x86_64-latest.xml | 1 + .../virtio-non-transitional.x86_64-latest.xml | 1 + .../virtio-transitional.x86_64-latest.xml | 1 + .../watchdog-q35-multiple.x86_64-latest.xml | 51 ++++++++++++ ...-default-cpu-kvm-q35-4.2.x86_64-latest.xml | 1 + ...-default-cpu-tcg-q35-4.2.x86_64-latest.xml | 1 + .../x86_64-q35-graphics.x86_64-latest.xml | 1 + .../x86_64-q35-headless.x86_64-latest.xml | 1 + tests/qemuxml2xmltest.c | 1 + 153 files changed, 601 insertions(+), 97 deletions(-) create mode 100644 tests/qemuxml2argvdata/watchdog-q35-multiple.x86_64-latest.args create mode 100644 tests/qemuxml2argvdata/watchdog-q35-multiple.xml create mode 100644 tests/qemuxml2xmloutdata/watchdog-q35-multiple.x86_64-latest.xml -- 2.39.1

3 13

[PATCH v2 0/5] Support crypto device
by zhenwei pi 25 Jan '23

25 Jan '23

v1 -> v2: - Rebase code against the latest commit: 46aee2a9255adf842ab44a9292acb46189a726f7 - Merge previous 1/7 and 2/7 into a single patch: conf: introduce crypto device - Add missing qemuxml2xmltest. - Merge previous 3/7 and 4/7 into a single patch: capabilities: introduce crypto - Add missing formatdomaincaps.rst. - Several fixes from Michal Prívozník' crypto_review branch. - Add caps and test for qemu-8.0. Thanks to Michal for lots of suggestions! v1: This series supports crypto device, also add support for QEMU. The basic xml schema looks like: <crypto model='virtio' type='qemu'> <backend model='builtin' queues='1'/> <address type='pci' domain='0x0000' bus='0x00' slot='0x0a' function='0x0'/> </crypto> <crypto model='virtio' type='qemu'> <backend model='lkcf'/> <address type='pci' domain='0x0000' bus='0x00' slot='0x0b' function='0x0'/> </crypto> Each patch has been tested by 'ninja -C build test', and launch a QEMU with crypto devices, it works fine. zhenwei pi (5): conf: introduce crypto device capabilities: introduce crypto qemu: alias: support crypto device qemu: command: support crypto device NEWS: Document 'crypto' device NEWS.rst | 3 + docs/formatdomain.rst | 21 +++ docs/formatdomaincaps.rst | 33 ++++ src/ch/ch_domain.c | 1 + src/conf/domain_capabilities.c | 15 ++ src/conf/domain_capabilities.h | 12 ++ src/conf/domain_conf.c | 158 ++++++++++++++++++ src/conf/domain_conf.h | 39 +++++ src/conf/domain_postparse.c | 1 + src/conf/domain_validate.c | 18 ++ src/conf/schemas/domaincaps.rng | 10 ++ src/conf/schemas/domaincommon.rng | 58 +++++++ src/conf/virconftypes.h | 2 + src/libvirt_private.syms | 1 + src/qemu/qemu_alias.c | 23 +++ src/qemu/qemu_capabilities.c | 30 ++++ src/qemu/qemu_capabilities.h | 6 + src/qemu/qemu_command.c | 109 ++++++++++++ src/qemu/qemu_domain.c | 3 + src/qemu/qemu_domain_address.c | 26 +++ src/qemu/qemu_driver.c | 5 + src/qemu/qemu_hotplug.c | 3 + src/qemu/qemu_validate.c | 22 +++ .../domaincapsdata/qemu_4.2.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_4.2.0-tcg.x86_64.xml | 11 ++ .../qemu_4.2.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.s390x.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.0.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.0.0-tcg.x86_64.xml | 11 ++ .../qemu_5.0.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.0.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.0.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_5.0.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.1.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.1.0-tcg.x86_64.xml | 11 ++ tests/domaincapsdata/qemu_5.1.0.sparc.xml | 9 + tests/domaincapsdata/qemu_5.1.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.2.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.2.0-tcg.x86_64.xml | 11 ++ .../qemu_5.2.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.s390x.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.0.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.0.0-tcg.x86_64.xml | 11 ++ .../qemu_6.0.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.0.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.0.0.s390x.xml | 11 ++ tests/domaincapsdata/qemu_6.0.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.1.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.1.0-tcg.x86_64.xml | 11 ++ tests/domaincapsdata/qemu_6.1.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.2.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.2.0-tcg.x86_64.xml | 11 ++ .../qemu_6.2.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.2.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.2.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_6.2.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.0.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.0.0-tcg.x86_64.xml | 11 ++ .../qemu_7.0.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_7.0.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_7.0.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_7.0.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.1.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.1.0-tcg.x86_64.xml | 11 ++ tests/domaincapsdata/qemu_7.1.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_7.1.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.2.0-q35.x86_64.xml | 12 ++ .../domaincapsdata/qemu_7.2.0-tcg.x86_64.xml | 12 ++ tests/domaincapsdata/qemu_7.2.0.x86_64.xml | 12 ++ .../domaincapsdata/qemu_8.0.0-q35.x86_64.xml | 12 ++ .../domaincapsdata/qemu_8.0.0-tcg.x86_64.xml | 12 ++ tests/domaincapsdata/qemu_8.0.0.x86_64.xml | 12 ++ .../caps_4.2.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_4.2.0.ppc64.xml | 2 + .../qemucapabilitiesdata/caps_4.2.0.s390x.xml | 2 + .../caps_4.2.0.x86_64.xml | 2 + .../caps_5.0.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_5.0.0.ppc64.xml | 2 + .../caps_5.0.0.riscv64.xml | 2 + .../caps_5.0.0.x86_64.xml | 2 + .../qemucapabilitiesdata/caps_5.1.0.sparc.xml | 1 + .../caps_5.1.0.x86_64.xml | 2 + .../caps_5.2.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_5.2.0.ppc64.xml | 2 + .../caps_5.2.0.riscv64.xml | 2 + .../qemucapabilitiesdata/caps_5.2.0.s390x.xml | 2 + .../caps_5.2.0.x86_64.xml | 2 + .../caps_6.0.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_6.0.0.s390x.xml | 2 + .../caps_6.0.0.x86_64.xml | 2 + .../caps_6.1.0.x86_64.xml | 2 + .../caps_6.2.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_6.2.0.ppc64.xml | 2 + .../caps_6.2.0.x86_64.xml | 2 + .../caps_7.0.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_7.0.0.ppc64.xml | 2 + .../caps_7.0.0.x86_64.xml | 2 + .../qemucapabilitiesdata/caps_7.1.0.ppc64.xml | 2 + .../caps_7.1.0.x86_64.xml | 2 + .../caps_7.2.0.x86_64.xml | 3 + .../caps_8.0.0.x86_64.xml | 3 + tests/qemuxml2argvdata/crypto-builtin.xml | 51 ++++++ .../crypto-builtin.x86_64-latest.xml | 1 + tests/qemuxml2xmltest.c | 2 + 110 files changed, 1321 insertions(+) create mode 100644 tests/qemuxml2argvdata/crypto-builtin.xml create mode 120000 tests/qemuxml2xmloutdata/crypto-builtin.x86_64-latest.xml -- 2.34.1

2 15

[libvirt PATCH] rpc: don't try to spawn non-existant daemon
by Daniel P. Berrangé 25 Jan '23

25 Jan '23

If libvirt is built in client only mode, the libvirtd/virtqemud/etc daemons won't exist. If the client is told to connect to a local hypervisor, it'll see the socket doesn't exist, try to spawn the daemon and then re-try connecting to the socket for a few seconds. Ultimately this will fail because the daemon doesn't exist and the user gets an error message error: Failed to connect socket to '/run/user/1000/libvirt/virtqemud-sock': No such file or directory technically this is accurate, but it doesn't help identify the root cause. With this change it will now report error: binary 'virtqemud' does not exist in $PATH: No such file or directory and will skip all the socket connect retries Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com> --- src/rpc/virnetsocket.c | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/src/rpc/virnetsocket.c b/src/rpc/virnetsocket.c index 8280bda007..bb2e0c5d3d 100644 --- a/src/rpc/virnetsocket.c +++ b/src/rpc/virnetsocket.c @@ -123,9 +123,19 @@ VIR_ONCE_GLOBAL_INIT(virNetSocket); #ifndef WIN32 static int virNetSocketForkDaemon(const char *binary) { - g_autoptr(virCommand) cmd = virCommandNewArgList(binary, - "--timeout=120", - NULL); + g_autofree char *binarypath = virFindFileInPath(binary); + g_autoptr(virCommand) cmd = NULL; + + if (!binarypath) { + virReportSystemError(ENOENT, + _("binary '%s' does not exist in $PATH"), + binary); + return -1; + } + + cmd = virCommandNewArgList(binarypath, + "--timeout=120", + NULL); virCommandAddEnvPassCommon(cmd); virCommandAddEnvPass(cmd, "XDG_CACHE_HOME"); -- 2.39.0

2 4

[PATCH 0/3] update RISC-V QEMU caps for QEMU 8.0.0
by Daniel Henrique Barboza 24 Jan '23

24 Jan '23

Hi, This series updates RISC-V QEMU caps that weren't being updated since QEMU 5.0.0. To do that I had to add a RISC-V stub driver in src/cpu to avoid test failures after the caps update. This is a very bare-bone driver that is basically a no-op. I also took the opportunity to enable the RISC-V 'virt' machine type in domaincapstest.c to increase the test coverage for the architecture. Daniel Henrique Barboza (3): src/cpu: add a basic RiscV64 cpu driver tests: update RISC-V QEMU caps for QEMU 8.0.0 tests: add QEMU RISC-V "virt" machine in domaincapstest src/cpu/cpu.c | 2 + src/cpu/cpu_riscv64.c | 59 + src/cpu/cpu_riscv64.h | 25 + src/cpu/meson.build | 1 + .../qemu_5.0.0-tcg-virt.riscv64.xml | 142 + .../qemu_5.0.0-virt.riscv64.xml | 145 + .../qemu_5.2.0-tcg-virt.riscv64.xml | 142 + .../qemu_5.2.0-virt.riscv64.xml | 145 + .../qemu_8.0.0-tcg-virt.riscv64.xml | 148 + .../qemu_8.0.0-virt.riscv64.xml | 151 + tests/domaincapstest.c | 14 +- .../caps_8.0.0.riscv64.replies | 28554 ++++++++++++++++ .../caps_8.0.0.riscv64.xml | 157 + ...ult-video-type-riscv64.riscv64-latest.args | 11 +- .../riscv64-virt-graphics.riscv64-latest.args | 45 +- .../riscv64-virt-headless.riscv64-latest.args | 37 +- tests/testutilshostcpus.h | 10 + 17 files changed, 29741 insertions(+), 47 deletions(-) create mode 100644 src/cpu/cpu_riscv64.c create mode 100644 src/cpu/cpu_riscv64.h create mode 100644 tests/domaincapsdata/qemu_5.0.0-tcg-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_5.0.0-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_5.2.0-tcg-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_5.2.0-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_8.0.0-tcg-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_8.0.0-virt.riscv64.xml create mode 100644 tests/qemucapabilitiesdata/caps_8.0.0.riscv64.replies create mode 100644 tests/qemucapabilitiesdata/caps_8.0.0.riscv64.xml -- 2.39.0

2 6

[PATCH] rpc: Fix error message in virNetServerSetClientLimits
by Martin Kletzander 24 Jan '23

24 Jan '23

Commit f007940cb25a tried to change the error message so that it is unified later in 35afa1d2d6c1, but various rewrites missed this particular error message which does not make sense. Fix it so that it is the same as the other two messages checking the same thing in this file. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2033879 Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com> --- src/rpc/virnetserver.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/rpc/virnetserver.c b/src/rpc/virnetserver.c index bf0fda04ee89..e97dfe81369f 100644 --- a/src/rpc/virnetserver.c +++ b/src/rpc/virnetserver.c @@ -1127,9 +1127,8 @@ virNetServerSetClientLimits(virNetServer *srv, if (max < max_unauth) { virReportError(VIR_ERR_INVALID_ARG, "%s", - _("The overall maximum number of clients waiting " - "for authentication must not be less than the overall " - "maximum number of clients")); + _("The overall maximum number of clients must not be less " + "than the number of clients waiting for authentication")); return -1; } -- 2.39.1

2 1

[PATCH 0/8] vbox: Report VirtualBox exceptions too
by Michal Privoznik 24 Jan '23

24 Jan '23

See 5/8 for motivation. Michal Prívozník (8): vbox: Introduce VBOX_QUERY_INTERFACE() vbox: Introduce IVirtualBoxErrorInfo interface vbox: Introduce vboxUniformedPFN::GetException() vbox: Introduce vboxUniformedPFN::ClearException() vbox: Introduce vboxReportError() vbox: Replace virReportError() with vboxReportError() vbox: Move error messages onto a single line vbox: Stop reporting RC in error messages src/vbox/vbox_common.c | 1223 +++++++++++++++++---------------- src/vbox/vbox_common.h | 6 + src/vbox/vbox_tmpl.c | 52 ++ src/vbox/vbox_uniformed_api.h | 11 + 4 files changed, 712 insertions(+), 580 deletions(-) -- 2.39.1

2 12

[PATCH 0/3] qemu: Fix alias generation for 'dimm' devices
by Peter Krempa 24 Jan '23

24 Jan '23

Peter Krempa (3): qemu: hotplug: Remove legacy quirk for 'dimm' address generation qemu: alias: Remove 'oldAlias' argument of qemuAssignDeviceMemoryAlias qemu: Remove 'memAliasOrderMismatch' field from VM private data src/qemu/qemu_alias.c | 13 ++++--------- src/qemu/qemu_alias.h | 3 +-- src/qemu/qemu_domain.h | 3 --- src/qemu/qemu_hotplug.c | 4 +--- src/qemu/qemu_process.c | 24 ------------------------ 5 files changed, 6 insertions(+), 41 deletions(-) -- 2.38.1

2 4

[PATCH 00/15] vbox: Add support for version 7.0 SDK and drop old
by Michal Privoznik 24 Jan '23

24 Jan '23

For full experience you'll need to apply these on top of: https://listman.redhat.com/archives/libvir-list/2023-January/237248.html Michal Prívozník (15): vbox: Drop misleading G_GNUC_UNUSED annotation vbox: Drop @mediaChangeOnly from vboxDomainAttachDeviceImpl() vbox: Drop @iid from UIMachine::LaunchVMProcess() vbox: Drop @iid from UISession::Open() vbox: Drop @iid from UISession::OpenExisting() vbox: Drop @data and @name from UIHost::CreateHostOnlyNetworkInterface() vbox: Drop UIUSBCommon::Enable() vbox: Drop UIUSBCommon::GetEnabled() vbox: Drop support for virtualbox-5.2.0 vbox: Drop support for virtualbox-6.0.0 vbox: Drop @networkName from UIDHCPServer::Start() vbox: Rename #include guard macro in header files vbox: add version 7.0 CAPI header vbox: Add support for version 7.0 SDK NEWS: Document recent vbox changes NEWS.rst | 11 + docs/drvvbox.rst | 2 +- src/vbox/meson.build | 3 +- src/vbox/vbox_CAPI_v5_2.h | 26870 ---------------- src/vbox/vbox_CAPI_v6_1.h | 6 +- .../{vbox_CAPI_v6_0.h => vbox_CAPI_v7_0.h} | 9280 +++++- src/vbox/vbox_V6_0.c | 13 - src/vbox/{vbox_V5_2.c => vbox_V7_0.c} | 6 +- src/vbox/vbox_XPCOMCGlue.h | 2 +- src/vbox/vbox_common.c | 65 +- src/vbox/vbox_common.h | 8 +- src/vbox/vbox_network.c | 5 +- src/vbox/vbox_storage.c | 10 +- src/vbox/vbox_tmpl.c | 204 +- src/vbox/vbox_uniformed_api.h | 15 +- 15 files changed, 8713 insertions(+), 27787 deletions(-) delete mode 100644 src/vbox/vbox_CAPI_v5_2.h rename src/vbox/{vbox_CAPI_v6_0.h => vbox_CAPI_v7_0.h} (75%) delete mode 100644 src/vbox/vbox_V6_0.c rename src/vbox/{vbox_V5_2.c => vbox_V7_0.c} (68%) -- 2.39.1

3 19