- Devel - libvirt List Archives

[libvirt] [PATCH v2 1/3] libxl: add support for multiple IP addresses
by Marek Marczykowski-Górecki 11 Dec '17

11 Dec '17

vif-* scripts support it for a long time, and expect addresses to be separated by spaces. Add appropriate support to libxl driver. --- changed in v2: - moved libxlMakeIPList function to xenconfig/xen_common.c and renamed to xenMakeIPList --- src/libxl/libxl_conf.c | 5 +++-- src/libxl/libxl_domain.c | 12 ------------ src/xenconfig/xen_common.c | 24 ++++++++++++++++++++++++ src/xenconfig/xen_common.h | 1 + 4 files changed, 28 insertions(+), 14 deletions(-) diff --git a/src/libxl/libxl_conf.c b/src/libxl/libxl_conf.c index 2a9be69..970cff2 100644 --- a/src/libxl/libxl_conf.c +++ b/src/libxl/libxl_conf.c @@ -49,6 +49,7 @@ #include "virstoragefile.h" #include "secret_util.h" #include "cpu/cpu.h" +#include "xen_common.h" #define VIR_FROM_THIS VIR_FROM_LIBXL @@ -1144,7 +1145,7 @@ libxlMakeNic(virDomainDefPtr def, if (VIR_STRDUP(x_nic->script, l_nic->script) < 0) goto cleanup; if (l_nic->guestIP.nips > 0) { - x_nic->ip = virSocketAddrFormat(&l_nic->guestIP.ips[0]->address); + x_nic->ip = xenMakeIPList(&l_nic->guestIP); if (!x_nic->ip) goto cleanup; } @@ -1160,7 +1161,7 @@ libxlMakeNic(virDomainDefPtr def, } if (l_nic->guestIP.nips > 0) { - x_nic->ip = virSocketAddrFormat(&l_nic->guestIP.ips[0]->address); + x_nic->ip = xenMakeIPList(&l_nic->guestIP); if (!x_nic->ip) goto cleanup; } diff --git a/src/libxl/libxl_domain.c b/src/libxl/libxl_domain.c index d054b07..395c8a9 100644 --- a/src/libxl/libxl_domain.c +++ b/src/libxl/libxl_domain.c @@ -294,18 +294,6 @@ libxlDomainDeviceDefPostParse(virDomainDeviceDefPtr dev, def->os.type != VIR_DOMAIN_OSTYPE_HVM) dev->data.chr->targetType = VIR_DOMAIN_CHR_CONSOLE_TARGET_TYPE_XEN; - if (dev->type == VIR_DOMAIN_DEVICE_NET && - (dev->data.net->type == VIR_DOMAIN_NET_TYPE_BRIDGE || - dev->data.net->type == VIR_DOMAIN_NET_TYPE_ETHERNET || - dev->data.net->type == VIR_DOMAIN_NET_TYPE_NETWORK)) { - if (dev->data.net->guestIP.nips > 1) { - virReportError(VIR_ERR_CONFIG_UNSUPPORTED, - _("multiple IP addresses not supported on device type %s"), - virDomainNetTypeToString(dev->data.net->type)); - return -1; - } - } - if (dev->type == VIR_DOMAIN_DEVICE_HOSTDEV || (dev->type == VIR_DOMAIN_DEVICE_NET && dev->data.net->type == VIR_DOMAIN_NET_TYPE_HOSTDEV)) { diff --git a/src/xenconfig/xen_common.c b/src/xenconfig/xen_common.c index ded0aca..7f838b6 100644 --- a/src/xenconfig/xen_common.c +++ b/src/xenconfig/xen_common.c @@ -1166,6 +1166,30 @@ xenFormatSerial(virConfValuePtr list, virDomainChrDefPtr serial) return -1; } +char * +xenMakeIPList(virNetDevIPInfoPtr guestIP) +{ + size_t i; + char **address_array; + char *ret = NULL; + + if (VIR_ALLOC_N(address_array, guestIP->nips + 1) < 0) + return NULL; + + for (i = 0; i < guestIP->nips; i++) { + address_array[i] = virSocketAddrFormat(&guestIP->ips[i]->address); + if (!address_array[i]) + goto cleanup; + } + address_array[guestIP->nips] = NULL; + + ret = virStringListJoin((const char**)address_array, " "); + + cleanup: + while (i > 0) + VIR_FREE(address_array[--i]); + return ret; +} static int xenFormatNet(virConnectPtr conn, diff --git a/src/xenconfig/xen_common.h b/src/xenconfig/xen_common.h index 9055692..3b7a5db 100644 --- a/src/xenconfig/xen_common.h +++ b/src/xenconfig/xen_common.h @@ -67,6 +67,7 @@ int xenFormatConfigCommon(virConfPtr conf, virConnectPtr conn, const char *nativeFormat); +char *xenMakeIPList(virNetDevIPInfoPtr guestIP); int xenDomainDefAddImplicitInputDevice(virDomainDefPtr def); base-commit: 9f0ccc717ba9026c30ce38951a354dd66fa12e3b -- git-series 0.9.1

3 10

[libvirt] [PATCH] qemu: Avoid comparing size_t with -1
by Jiri Denemark 11 Dec '17

11 Dec '17

ncpus would be -1 on error and the cleanup for loop would not be skipped in this case. Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com> --- src/qemu/qemu_capabilities.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index 8c65de956e..9673ef857b 100644 --- a/src/qemu/qemu_capabilities.c +++ b/src/qemu/qemu_capabilities.c @@ -2965,7 +2965,7 @@ virQEMUCapsFetchCPUDefinitions(qemuMonitorPtr mon) size_t i; if ((ncpus = qemuMonitorGetCPUDefinitions(mon, &cpus)) < 0) - goto error; + return NULL; if (!(models = virDomainCapsCPUModelsNew(ncpus))) goto error; -- 2.15.1

2 1

[libvirt] [PATCH 0/2] qemu: Fix upgrade with running domains with host-model CPUs
by Jiri Denemark 11 Dec '17

11 Dec '17

When upgrading old libvirt on a host with EPYC host CPU (or any other CPU which was added to cpu_map.xml in newer libvirt releases), we would replace host-model CPU definition for running domains to EPYC even though a different CPU model was used to start the domain. And sync old libvirt doesn't know about EPYC, migrating the domain to another host running the same old libvirt would be impossible. https://bugzilla.redhat.com/show_bug.cgi?id=1521202 Jiri Denemark (2): qemu: Separate fetching CPU definitions from filling qemuCaps qemu: Make sure host-model uses CPU model supported by QEMU src/qemu/qemu_capabilities.c | 59 +++++++++++++++++++++++++++----------------- src/qemu/qemu_capabilities.h | 1 + src/qemu/qemu_process.c | 30 ++++++++++++++++++++++ 3 files changed, 68 insertions(+), 22 deletions(-) -- 2.15.1

3 5

[libvirt] [PATCH v3] qemuDomainUndefineFlags: Fix error message
by Michal Privoznik 11 Dec '17

11 Dec '17

https://bugzilla.redhat.com/show_bug.cgi?id=1522706 If domain is active, but the undefine API was called without the VIR_DOMAIN_UNDEFINE_KEEP_NVRAM flag set, the following incorrect error message is produced: error: Requested operation is not valid: cannot delete inactive domain with nvram Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- Diff to v2: - Drop state from the message completely src/qemu/qemu_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index aa30b119a..d18cea4d0 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -7538,7 +7538,7 @@ qemuDomainUndefineFlags(virDomainPtr dom, } } else if (!(flags & VIR_DOMAIN_UNDEFINE_KEEP_NVRAM)) { virReportError(VIR_ERR_OPERATION_INVALID, "%s", - _("cannot delete inactive domain with nvram")); + _("cannot delete domain with nvram")); goto endjob; } } -- 2.13.6

2 1

[libvirt] [PATCH] news: mention multiple IP addresses support for Xen
by Jim Fehlig 11 Dec '17

11 Dec '17

Signed-off-by: Jim Fehlig <jfehlig(a)suse.com> --- docs/news.xml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/docs/news.xml b/docs/news.xml index 0ec9857e2..7509416ec 100644 --- a/docs/news.xml +++ b/docs/news.xml @@ -37,6 +37,12 @@ <section title="New features"> </section> <section title="Improvements"> + <change> + <summary> + Xen: Add support for multiple IP addresses on + <code><interface></code> devices + </summary> + </change> </section> <section title="Bug fixes"> </section> -- 2.15.1

2 1

[libvirt] [PATCH v2] qemuDomainUndefineFlags: Fix error message
by Michal Privoznik 11 Dec '17

11 Dec '17

https://bugzilla.redhat.com/show_bug.cgi?id=1522706 If domain is active, but the undefine API was called without the VIR_DOMAIN_UNDEFINE_KEEP_NVRAM flag set, the following incorrect error message is produced: error: Requested operation is not valid: cannot delete inactive domain with nvram Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- Diff to v1: - switched to different code pattern to make it easier for translators src/qemu/qemu_driver.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index aa30b119a..cc5520c54 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -7537,8 +7537,13 @@ qemuDomainUndefineFlags(virDomainPtr dom, goto endjob; } } else if (!(flags & VIR_DOMAIN_UNDEFINE_KEEP_NVRAM)) { - virReportError(VIR_ERR_OPERATION_INVALID, "%s", - _("cannot delete inactive domain with nvram")); + if (virDomainObjIsActive(vm)) { + virReportError(VIR_ERR_OPERATION_INVALID, "%s", + _("cannot delete active domain with nvram")); + } else { + virReportError(VIR_ERR_OPERATION_INVALID, "%s", + _("cannot delete inactive domain with nvram")); + } goto endjob; } } -- 2.13.6

3 2

[libvirt] [libvirt-php][PATCH 0/5] Couple of fixes
by Michal Privoznik 11 Dec '17

11 Dec '17

Mostly memleaks. Michal Privoznik (5): maint: Drop INSTALL install-sh and missing src: Don't leak @uuid in libvirt_nwfilter_get_uuid_string src: Fix a typo in php_libvirt_nwfilter_dtor src: Free @filters array in libvirt_list_all_nwfilters src: Don't leak connection_get_emulator retval INSTALL | 370 ---------------------------------- install-sh | 501 ----------------------------------------------- missing | 215 -------------------- src/libvirt-connection.c | 2 + src/libvirt-nwfilter.c | 7 +- src/libvirt-php.c | 11 +- 6 files changed, 14 insertions(+), 1092 deletions(-) delete mode 100644 INSTALL delete mode 100755 install-sh delete mode 100755 missing -- 2.13.6

2 12

[libvirt] [PATCH] qemuDomainUndefineFlags: Fix error message
by Michal Privoznik 11 Dec '17

11 Dec '17

https://bugzilla.redhat.com/show_bug.cgi?id=1522706 If domain is active, but the undefine API was called without the VIR_DOMAIN_UNDEFINE_KEEP_NVRAM flag set, the following incorrect error message is produced: error: Requested operation is not valid: cannot delete inactive domain with nvram Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- src/qemu/qemu_driver.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index aa30b119a..f2e8f3edc 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -7537,8 +7537,10 @@ qemuDomainUndefineFlags(virDomainPtr dom, goto endjob; } } else if (!(flags & VIR_DOMAIN_UNDEFINE_KEEP_NVRAM)) { - virReportError(VIR_ERR_OPERATION_INVALID, "%s", - _("cannot delete inactive domain with nvram")); + virReportError(VIR_ERR_OPERATION_INVALID, + _("cannot delete %s domain with nvram"), + virDomainObjIsActive(vm) ? + _("active") : _("inactive")); goto endjob; } } -- 2.13.6

2 1

[libvirt] [PATCH v3 00/12] Move qemu command line controller checks to qemuDomainDeviceDefValidateController* checks
by John Ferlan 09 Dec '17

09 Dec '17

v2: https://www.redhat.com/archives/libvir-list/2017-December/msg00173.html Changes since v2: * Push a few patches from v1 that were ACK'd * Make the Skip helper private to qemu_command.c and just copied the various skip checks from the new helper into the specific controller validate helper to make it painfully obvious what was being skipped rather than having common code handle that and possibly obsfuscate those details from the casual reader. John Ferlan (11): qemu: Introduce qemuBuildSkipController qemu: Introduce qemuDomainDeviceDefValidateController qemu: Move CCW S390 Address check to controller def validate qemu: Introduce qemuDomainDeviceDefValidateControllerSCSI qemu: Introduce qemuDomainDeviceDefValidateControllerPCI qemu: Use virDomainPCIControllerOpts in qemuBuildControllerDevStr qemu: Move PCI command modelName check to controller def validate qemu: Move PCI command modelName TypeToString to controller def validate qemu: Move PCI more command checks to controller def validate qemu: Complete PCI command checks to controller def validate qemu: Introduce qemuDomainDeviceDefValidateControllerSATA Lin Ma (1): qemu: Introduce qemuDomainDeviceDefValidateControllerIDE src/qemu/qemu_command.c | 497 ++++++++---------------------------------------- src/qemu/qemu_domain.c | 469 ++++++++++++++++++++++++++++++++++++++++++++- tests/qemumemlocktest.c | 14 ++ tests/qemuxml2xmltest.c | 5 +- 4 files changed, 564 insertions(+), 421 deletions(-) -- 2.13.6

3 27

[libvirt] [PATCH] migration.html: Clarify configuration file handling docs
by Jiri Denemark 08 Dec '17

08 Dec '17

Migration never removes any configuration files on the destination host. Thus when the domain is already defined on the destination, it will stay persistent even after migration without --persist. https://bugzilla.redhat.com/show_bug.cgi?id=1514930 Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com> --- docs/migration.html.in | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/docs/migration.html.in b/docs/migration.html.in index de7d4def75..7c345b65b7 100644 --- a/docs/migration.html.in +++ b/docs/migration.html.in @@ -206,14 +206,14 @@ <h2><a id="config">Configuration file handling</a></h2> <p> - There are two types of virtual machine known to libvirt. A <em>transient</em> + There are two types of virtual machines known to libvirt. A <em>transient</em> guest only exists while it is running, and has no configuration file stored on disk. A <em>persistent</em> guest maintains a configuration file on disk even when it is not running. </p> <p> - By default, a migration operation will not attempt to change any configuration + By default, a migration operation will not attempt to modify any configuration files that may be stored on either the source or destination host. It is the administrator, or management application's, responsibility to manage distribution of configuration files (if desired). It is important to note that the <code>/etc/libvirt</code> @@ -230,14 +230,14 @@ host and removed on the original. </li> <li>Centralized configuration files outside libvirt, in a database. A data center - management application may not storage configuration files at all. Instead it + management application may not store configuration files at all. Instead it may generate libvirt XML on the fly when a guest is booted. It will typically use transient guests, and thus not have to consider configuration files during migration. </li> <li>Distributed configuration inside libvirt. The configuration file for each guest is copied to every host where the guest is able to run. Upon migration - the existing config merely needs to be updated with any changes + the existing config merely needs to be updated with any changes. </li> <li>Ad-hoc configuration management inside libvirt. Each guest is tied to a specific host and rarely migrated. When migration is required, the config @@ -246,7 +246,7 @@ </ul> <p> - As mentioned above, libvirt will not touch configuration files during + As mentioned above, libvirt will not modify configuration files during migration by default. The <code>virsh</code> command has two flags to influence this behaviour. The <code>--undefine-source</code> flag will cause the configuration file to be removed on the source host @@ -265,12 +265,12 @@ <th colspan="3">After migration</th> </tr> <tr class="subhead"> - <th>Guest type</th> + <th>Source type</th> <th>Source config</th> <th>Dest config</th> <th>--undefine-source</th> <th>--persist</th> - <th>Guest type</th> + <th>Dest type</th> <th>Source config</th> <th>Dest config</th> </tr> @@ -325,9 +325,9 @@ <td class="y">Y</td> <td class="n">N</td> <td class="n">N</td> - <td>Transient</td> - <td class="n">N</td> + <td>Persistent</td> <td class="n">N</td> + <td class="y">Y<br/>(unchanged dest config)</td> </tr> <tr> <td>Transient</td> @@ -335,9 +335,9 @@ <td class="y">Y</td> <td class="y">Y</td> <td class="n">N</td> - <td>Transient</td> - <td class="n">N</td> + <td>Persistent</td> <td class="n">N</td> + <td class="y">Y<br/>(unchanged dest config)</td> </tr> <tr> <td>Transient</td> @@ -347,7 +347,7 @@ <td class="y">Y</td> <td>Persistent</td> <td class="n">N</td> - <td class="y">Y</td> + <td class="y">Y<br/>(replaced with source)</td> </tr> <tr> <td>Transient</td> @@ -357,7 +357,7 @@ <td class="y">Y</td> <td>Persistent</td> <td class="n">N</td> - <td class="y">Y</td> + <td class="y">Y<br/>(replaced with source)</td> </tr>  @@ -411,7 +411,7 @@ <td class="n">N</td> <td>Persistent</td> <td class="y">Y</td> - <td class="y">Y</td> + <td class="y">Y<br/>(unchanged dest config)</td> </tr> <tr> <td>Persistent</td> @@ -421,7 +421,7 @@ <td class="n">N</td> <td>Persistent</td> <td class="n">N</td> - <td class="y">Y</td> + <td class="y">Y<br/>(unchanged dest config)</td> </tr> <tr> <td>Persistent</td> @@ -431,7 +431,7 @@ <td class="y">Y</td> <td>Persistent</td> <td class="y">Y</td> - <td class="y">Y</td> + <td class="y">Y<br/>(replaced with source)</td> </tr> <tr> <td>Persistent</td> @@ -441,7 +441,7 @@ <td class="y">Y</td> <td>Persistent</td> <td class="n">N</td> - <td class="y">Y</td> + <td class="y">Y<br/>(replaced with source)</td> </tr> </tbody> </table> -- 2.15.1

2 1