- Devel - libvirt List Archives

[libvirt] [PATCH] Makefile.nonreentrant: Rebuild against Fedora 26
by Andrea Bolognani 04 Sep '17

04 Sep '17

According to the comments in the file and the git history, the list of forbidden symbols was originally built against Fedora 9 in 2009 (!) and pretty much never refreshed afterwards. Signed-off-by: Andrea Bolognani <abologna(a)redhat.com> --- I didn't put too much thought into this, but the results looks sane enough as we are only *adding* to the list. Makefile.nonreentrant | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/Makefile.nonreentrant b/Makefile.nonreentrant index 5cc64c0d5..a83298322 100644 --- a/Makefile.nonreentrant +++ b/Makefile.nonreentrant @@ -15,9 +15,9 @@ ## <http://www.gnu.org/licenses/>. # -# Generated by running the following on Fedora 9: +# Generated by running the following on Fedora 26: # -# nm -D --defined-only /lib/libc.so.6 \ +# nm -D --defined-only /lib64/libc.so.6 \ # | grep '_r$' \ # | awk '{print $3}' \ # | grep -v __ \ @@ -43,6 +43,7 @@ NON_REENTRANT += ether_ntoa NON_REENTRANT += fcvt NON_REENTRANT += fgetgrent NON_REENTRANT += fgetpwent +NON_REENTRANT += fgetsgent NON_REENTRANT += fgetspent NON_REENTRANT += getaliasbyname NON_REENTRANT += getaliasent @@ -72,6 +73,8 @@ NON_REENTRANT += getrpcent NON_REENTRANT += getservbyname NON_REENTRANT += getservbyport NON_REENTRANT += getservent +NON_REENTRANT += getsgent +NON_REENTRANT += getsgnam NON_REENTRANT += getspent NON_REENTRANT += getspnam NON_REENTRANT += getutent @@ -95,6 +98,7 @@ NON_REENTRANT += random NON_REENTRANT += rand NON_REENTRANT += seed48 NON_REENTRANT += setstate +NON_REENTRANT += sgetsgent NON_REENTRANT += sgetspent NON_REENTRANT += srand48 NON_REENTRANT += srandom -- 2.13.5

2 1

[libvirt] [PATCH] docs: Fix typo deamon -> daemon
by Andrea Bolognani 04 Sep '17

04 Sep '17

Suggested-by: Martin Kletzander <mkletzan(a)redhat.com> Signed-off-by: Andrea Bolognani <abologna(a)redhat.com> --- I would push this as trivial, but I feel like someone might not be okay with touching the release notes for libvirt versions that are already out. I think in this case it's perfectly justified, but if you disagree this is your chance to point it out :) docs/internals/rpc.html.in | 2 +- docs/news.xml | 2 +- src/libxl/libxl.conf | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/docs/internals/rpc.html.in b/docs/internals/rpc.html.in index 45c07cb61..fca9cb784 100644 --- a/docs/internals/rpc.html.in +++ b/docs/internals/rpc.html.in @@ -534,7 +534,7 @@ C <-- |32| 8 | 1 | 3 | 1 | 1 | 0 | .o.oOo | <-- S (reply) <dt><code>virNetDaemonPtr</code> (virnetdaemon.h)</dt> <dd>The virNetDaemon APIs are used to manage a daemon process. A - deamon is a process that might expose one or more servers. It + daemon is a process that might expose one or more servers. It handles most process-related details, network-related should be part of the underlying server. </dd> diff --git a/docs/news.xml b/docs/news.xml index 452711717..483f9d6d1 100644 --- a/docs/news.xml +++ b/docs/news.xml @@ -195,7 +195,7 @@ Fix --verbose option for all daemons </summary> <description> - Since v3.0.0, the option had been ignored by all libvirt deamons + Since v3.0.0, the option had been ignored by all libvirt daemons (<code>libvirtd</code>, <code>virtlogd</code> and <code>virtlockd</code>); it's now working as intended once again. </description> diff --git a/src/libxl/libxl.conf b/src/libxl/libxl.conf index 5c9bdaac2..264af7cf9 100644 --- a/src/libxl/libxl.conf +++ b/src/libxl/libxl.conf @@ -28,7 +28,7 @@ # is sent to the daemon after keepalive_interval seconds of inactivity # to check if the daemon is still responding; keepalive_count is a # maximum number of keepalive messages that are allowed to be sent to -# the deamon without getting any response before the connection is +# the daemon without getting any response before the connection is # considered broken. In other words, the connection is automatically # closed after approximately keepalive_interval * (keepalive_count + 1) # seconds since the last message was received from the daemon. If -- 2.13.5

2 1

[libvirt] [PATCH v2 0/2] dac: relabel spice rendernode
by Cole Robinson 04 Sep '17

04 Sep '17

This fixes the last issue preventing qemu:///system spice GL from working out of the box: chown'ing the rendernode path so we have permissions to open it. We skip this if mount namespaces are disabled, so the chown'ing won't interfere with other rendernode users on the host. https://bugzilla.redhat.com/show_bug.cgi?id=1460804 v2: Add the MOUNT_NAMESPACE handling Drop DAC restore of rendernode Cole Robinson (2): security: add MANAGER_MOUNT_NAMESPACE flag security: dac: relabel spice rendernode src/qemu/qemu_driver.c | 2 ++ src/security/security_dac.c | 68 +++++++++++++++++++++++++++++++++++++++++ src/security/security_dac.h | 3 ++ src/security/security_manager.c | 4 ++- src/security/security_manager.h | 1 + 5 files changed, 77 insertions(+), 1 deletion(-) -- 2.13.5

2 3

[libvirt] [PATCH] Add libxslt as build requires for mingw RPMs
by Daniel P. Berrange 04 Sep '17

04 Sep '17

The libxslt package is needed since: commit 94d2d6429d686c5af95115d09c01f3c6bd5ea7c6 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Wed Jul 26 17:40:44 2017 +0100 docs: make xmllint & xsltproc compulsory The native RPM had it already, but mingw build was missing it. Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> --- Pushed as a build breaker & trivial fix mingw-libvirt.spec.in | 1 + 1 file changed, 1 insertion(+) diff --git a/mingw-libvirt.spec.in b/mingw-libvirt.spec.in index 6b0ce0bf5..dc18d055b 100644 --- a/mingw-libvirt.spec.in +++ b/mingw-libvirt.spec.in @@ -60,6 +60,7 @@ BuildRequires: mingw64-dlfcn BuildRequires: pkgconfig # Need native version for msgfmt BuildRequires: gettext +BuildRequires: libxslt BuildRequires: python %if 0%{?fedora} >= 27 BuildRequires: perl-interpreter -- 2.13.5

1 0

[libvirt] Availability of libvirt 3.7.0
by Daniel Veillard 04 Sep '17

04 Sep '17

I pushed the final version earlier today, it is tagged in git and I made available sigend tarball and rpms at the usual place: ftp://libvirt.org/libvirt/ I also rolled out the 3.7.0 version of libvirt-python bindings, those are available from: ftp://libvirt.org/libvirt/python/ This is a balanced release with new features available from virsh command line and API, as well a general improvements and a fair amount of bug fixes: * New features - qemu: Add managedsave-edit commands Using managedsave-dumpxml, managedsave-define and managedsave-edit commands, now we can dump and edit the XML configuration of domain which has managedsave image. - qemu: Add migrate-getmaxdowntime command Currently, the maximum tolerable downtime for a domain being migrated is write-only from libvirt, via migrate-setmaxdowntime. This implements a complementary migrate-getmaxdowntime command - bhyve: Support autoport for VNC ports It's no longer necessary to explicitly specify VNC port for the bhyve guests. With the autoport feature it will be allocated automatically. Please refer to the bhyve driver documentation for examples. - qemu: Added support for setting heads of virtio GPU - qemu: Added support to configure reconnect timeout for chardev devices When you have a TCP or UNIX chardev device and it's connected somewhere you can configure reconnect timeout if the connection is closed. * Improvements - qemu: Report a clear error when dropping a VM during startup "Failed to load config for domain 'DOMNAME'" is now reported if a VM config can't be parsed for some reason, and thus provides a clear indication for users (and devs). - apparmor: Update for QEMU 2.10 compatibility Starting with QEMU 2.10, disk images and NVRAM files get automatically locked to prevent them from being corrupted; however, file locking needs to be explicitly allowed through virt-aa-helper or AppArmor will reject the requests and the guest will not be able to run. - virsh: List Unix sockets in 'domdisplay' output VNC and SPICE graphics can use Unix sockets instead of TCP/IP sockets as connection endpoints, but such a configuration was not handled correctly by virsh domdisplay, causing the respective endpoints to be missing from the output. - qemu: Don't check whether offline migration is safe Since offline migration only copies the guest definition to the destination host, data corruption is not a concern and the operation can always be performed safely. - virt-host-validate: Fix IOMMU detection on ppc64 * Bug fixes - qemu: Better support for international domain names (with wide characters) There were some issues with multi-byte domains getting lost on daemon restart due to truncation, so the code now handles multi-byte names a bit better. - qemu: Support long domain names with namespaces Domains with extremely long names would fail to start due to temporary namespace paths being created with the whole name. The path is now generated with shortened name instead. - qemu: Tolerate missing emulator binary during libvirtd restart For some time libvirt required qemu capabilities being present when parsing VM configs during startup. As a side effect VM configs would fail to parse and thus vanish, if the emulator binary would be uninstalled or broken. Libvirt now tolerates when capabilities are missing during startup. - qemu: Prevent pSeries guests from disappearing in some situations pSeries guest would disappear if any of the host devices they were configured to use was not available during libvirtd startup, which could easily happen for SR-IOV Virtual Functions. This scenario is now handled correctly. - qemu: Honor <on_reboot/> setting The setting was accepted by the parser, but not actually implemented. - Fix --verbose option for all daemons Since v3.0.0, the option had been ignored by all libvirt deamons (libvirtd, virtlogd and virtlockd); it's now working as intended once again. Thanks everybody for your help with this release, be it with code, patches, review, documentation, but reports, etc ... Enjoy this release ! Daniel -- Daniel Veillard | Red Hat Developers Tools http://developer.redhat.com/ veillard(a)redhat.com | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | virtualization library http://libvirt.org/

1 0

[libvirt] [PATCH v2] vmx: Expose VMware Managed Object Reference (moref) in
by Richard W.M. Jones 04 Sep '17

04 Sep '17

v1 -> v2: - Implemented all the review recommendations. I didn't add a test. Is it even possible to test this without a VMware server? I did however test it myself with a real server and it works as expected for me. Rich.

2 2

[libvirt] [PATCH] event: ignore attempts to replace the event loop impl
by Daniel P. Berrange 04 Sep '17

04 Sep '17

Although not previously explicitly documented, the expectation for the libvirt event loop is that an implementation is registered early in application startup, before calling any libvirt APIs and then run forever after. Replacing a previously registered event loop is not safe & subject to races even if virConnectClose has been called on open handles, due to delayed deregistration of callbacks during conenction close. Signed-off-by: Daniel P. Berrange <berrange(a)redhat.com> --- src/util/virevent.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/src/util/virevent.c b/src/util/virevent.c index e0fd35e41..51d8714df 100644 --- a/src/util/virevent.c +++ b/src/util/virevent.c @@ -220,6 +220,13 @@ virEventRemoveTimeout(int timer) * existing event loop implementation, then the * virEventRegisterDefaultImpl() method can be used to setup * the generic libvirt implementation. + * + * Once registered, the event loop implementation cannot be + * changed, and must be run continuously. Note that callbacks + * may remain registered for a short time even after calling + * virConnectClose on all open connections, so it is not safe + * to stop running the event loop immediately after closing + * the connection. */ void virEventRegisterImpl(virEventAddHandleFunc addHandle, virEventUpdateHandleFunc updateHandle, @@ -233,6 +240,12 @@ void virEventRegisterImpl(virEventAddHandleFunc addHandle, addHandle, updateHandle, removeHandle, addTimeout, updateTimeout, removeTimeout); + if (addHandleImpl || updateHandleImpl || removeHandleImpl || + addTimeoutImpl || updateTimeoutImpl || removeHandleImpl) { + VIR_WARN("Ignoring attempt to replace registered event loop"); + return; + } + addHandleImpl = addHandle; updateHandleImpl = updateHandle; removeHandleImpl = removeHandle; -- 2.13.5

2 1

[libvirt] [PATCH] docs: Remove <code> from <summary> entries due to RNG error
by John Ferlan 04 Sep '17

04 Sep '17

commit '96e55048' caused make check failure for virschematest: 1929) Checking ../docs/news.xml against ../news.rng ... libvirt: XML Util error : XML document failed to validate against schema: Unable to validate doc against /home/jferlan/git/libvirt.work/docs/schemas/../news.rng Datatype element summary has child elements Element summary failed to validate content Datatype element summary has child elements Element summary failed to validate content ^[[31m^[[1mFAILED^[[0m That's because <code> elements don't appear to be allowed in the schema. Rather than attempt to fix the schema, figured it was simpler to just remove them and let the schema fix happen later. Signed-off-by: John Ferlan <jferlan(a)redhat.com> --- Pushed as build breaker. docs/news.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/news.xml b/docs/news.xml index 7e9395b..dd2e653 100644 --- a/docs/news.xml +++ b/docs/news.xml @@ -176,7 +176,7 @@ </change> <change> <summary> - qemu: Honor <code><on_reboot/></code> setting + qemu: Honor <on_reboot/> setting </summary> <description> The setting was accepted by the parser, but not actually implemented. @@ -184,7 +184,7 @@ </change> <change> <summary> - Fix <code>--verbose</code> option for all daemons + Fix --verbose option for all daemons </summary> <description> Since v3.0.0, the option had been ignored by all libvirt deamons -- 2.9.5

2 1

[libvirt] [PATCH] qemu: Remove unused params from qemuDomainDeviceDefValidate
by John Ferlan 04 Sep '17

04 Sep '17

Neither @cfg nor (now) @driver is used in the API, so remove them and mark @opaque as UNUSED. NB: Commit id 'fa3c558596' dropped the unused @qemuCaps which was the last consumer of @driver other than @cfg, but even @cfg was never used even in the original implementation from commit id 'd987f63a'. Signed-off-by: John Ferlan <jferlan(a)redhat.com> --- Another rogue piece of unused data seen whilst working through the VxHS patches. src/qemu/qemu_domain.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index b98ffff..cd53c6f 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -3302,10 +3302,8 @@ qemuDomainRedirdevDefValidate(const virDomainRedirdevDef *def) static int qemuDomainDeviceDefValidate(const virDomainDeviceDef *dev, const virDomainDef *def ATTRIBUTE_UNUSED, - void *opaque) + void *opaque ATTRIBUTE_UNUSED) { - virQEMUDriverPtr driver = opaque; - virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver); int ret = -1; if (dev->type == VIR_DOMAIN_DEVICE_NET) { @@ -3389,7 +3387,6 @@ qemuDomainDeviceDefValidate(const virDomainDeviceDef *dev, ret = 0; cleanup: - virObjectUnref(cfg); return ret; } -- 2.9.5

2 1

[libvirt] [PATCH v2 0/2] Add support for qcow2 cache
by Liu Qing 04 Sep '17

04 Sep '17

Qcow2 small IO random write performance will drop dramatically if the l2 cache table could not cover the whole disk. This will be a lot of l2 cache table RW operations if cache miss happens frequently. This patch exports the qcow2 driver parameter l2-cache-size/refcount-cache-size, first added in Qemu 2.2, and cache-clean-interval, first added in Qemu 2.5, in libvirt. *** BLURB HERE *** Liu Qing (2): conf, docs: Add qcow2 cache configuration support qemu: add capability checking for qcow2 cache configuration docs/formatdomain.html.in | 21 ++++++++++ docs/schemas/domaincommon.rng | 24 +++++++++++ src/conf/domain_conf.c | 30 ++++++++++++++ src/qemu/qemu_capabilities.c | 15 +++++++ src/qemu/qemu_capabilities.h | 3 ++ src/qemu/qemu_command.c | 47 ++++++++++++++++++++++ src/util/virstoragefile.c | 3 ++ src/util/virstoragefile.h | 4 ++ tests/qemucapabilitiesdata/caps_2.4.0.x86_64.xml | 2 + tests/qemucapabilitiesdata/caps_2.5.0.x86_64.xml | 3 ++ .../caps_2.6.0-gicv2.aarch64.xml | 3 ++ .../caps_2.6.0-gicv3.aarch64.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.6.0.ppc64le.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.6.0.x86_64.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.7.0.s390x.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.7.0.x86_64.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.8.0.s390x.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.8.0.x86_64.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.9.0.ppc64le.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.9.0.s390x.xml | 3 ++ tests/qemucapabilitiesdata/caps_2.9.0.x86_64.xml | 3 ++ ...argv-disk-drive-qcow2-cache-clean-interval.args | 27 +++++++++++++ ...2argv-disk-drive-qcow2-cache-clean-interval.xml | 37 +++++++++++++++++ ...emuxml2argv-disk-drive-qcow2-l2-cache-size.args | 27 +++++++++++++ ...qemuxml2argv-disk-drive-qcow2-l2-cache-size.xml | 37 +++++++++++++++++ ...2argv-disk-drive-qcow2-refcount-cache-size.args | 27 +++++++++++++ ...l2argv-disk-drive-qcow2-refcount-cache-size.xml | 37 +++++++++++++++++ tests/qemuxml2argvtest.c | 6 +++ ...mlout-disk-drive-qcow2-cache-clean-interval.xml | 41 +++++++++++++++++++ ...muxml2xmlout-disk-drive-qcow2-l2-cache-size.xml | 41 +++++++++++++++++++ ...xmlout-disk-drive-qcow2-refcount-cache-size.xml | 41 +++++++++++++++++++ tests/qemuxml2xmltest.c | 3 ++ 32 files changed, 509 insertions(+) create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-drive-qcow2-cache-clean-interval.args create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-drive-qcow2-cache-clean-interval.xml create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-drive-qcow2-l2-cache-size.args create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-drive-qcow2-l2-cache-size.xml create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-drive-qcow2-refcount-cache-size.args create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-disk-drive-qcow2-refcount-cache-size.xml create mode 100644 tests/qemuxml2xmloutdata/qemuxml2xmlout-disk-drive-qcow2-cache-clean-interval.xml create mode 100644 tests/qemuxml2xmloutdata/qemuxml2xmlout-disk-drive-qcow2-l2-cache-size.xml create mode 100644 tests/qemuxml2xmloutdata/qemuxml2xmlout-disk-drive-qcow2-refcount-cache-size.xml -- 1.8.3.1

1 0