- Devel - libvirt List Archives

[libvirt] [libvirt-test-API][PATCH 0/2] Add test cases for setTime and getTime
by Jincheng Miao 11 Mar '15

11 Mar '15

create some test cases for libvirt-python API setTime and getTime Jincheng Miao (2): Add guest setTime and getTime testing Add test cases for linux_domain cases/linux_domain.conf | 26 +++++++++ repos/domain/guest_time.py | 98 ++++++++++++++++++++++++++++++++ repos/domain/set_guest_time.py | 122 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 246 insertions(+), 0 deletions(-) create mode 100644 repos/domain/guest_time.py create mode 100644 repos/domain/set_guest_time.py

2 3

[libvirt] [PATCH] virnetdev: fix build with old kernel
by Pavel Hrdina 11 Mar '15

11 Mar '15

Commit c9027d8f added a detection of NIC HW features, but some of them are not available in old kernel. Very old kernels lack enum ethtool_flags and even if this enum is present, not all values are available for all kernels. To be sure that we have everything in kernel that we need, we must check for existence of newest value which is ETH_FLAG_TXVLAN. Only then we can be sure that the feature detection can be successfully built. Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com> --- configure.ac | 6 ++++++ src/util/virnetdev.c | 6 ++++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index e071813..3cffb84 100644 --- a/configure.ac +++ b/configure.ac @@ -383,6 +383,12 @@ AC_CHECK_TYPE([struct ifreq], #include <net/if.h> ]]) +AC_CHECK_DECL([ETH_FLAG_TXVLAN], + [AC_DEFINE([HAVE_ETH_FLAG_TXVLAN],[1], + [Defined if ETH_FLAG_TXVLAN exists in linux/ethtool.h])], + [], [[#include <linux/ethtool.h> + ]]) + dnl Our only use of libtasn1.h is in the testsuite, and can be skipped dnl if the header is not present. Assume -ltasn1 is present if the dnl header could be found. diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c index 971db43..7519579 100644 --- a/src/util/virnetdev.c +++ b/src/util/virnetdev.c @@ -47,7 +47,8 @@ # undef HAVE_STRUCT_IFREQ #endif -#if defined(SIOCETHTOOL) && defined(HAVE_STRUCT_IFREQ) +#if defined(SIOCETHTOOL) && defined(HAVE_STRUCT_IFREQ) && \ + defined(HAVE_ETH_FLAG_TXVLAN) # include <linux/ethtool.h> #endif @@ -2733,7 +2734,8 @@ int virNetDevGetRxFilter(const char *ifname, return ret; } -#if defined(SIOCETHTOOL) && defined(HAVE_STRUCT_IFREQ) +#if defined(SIOCETHTOOL) && defined(HAVE_STRUCT_IFREQ) && \ + defined(HAVE_ETH_FLAG_TXVLAN) /** * virNetDevFeatureAvailable -- 2.0.5

3 2

[libvirt] [PATCH] RNG: Allow multiple parameters to be passed to an interface filter
by Michal Privoznik 11 Mar '15

11 Mar '15

Our code supports that for ages. When using a <filterref/> to an <interface/> several parameters can be passed to the filter. Later, when building firewall rules, parameters are substituted for their values. However, our RNG schema allowed only one parameter to be passed. Reported-by: Brian Rak <brak(a)gameservers.com> Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- docs/schemas/domaincommon.rng | 4 +- .../domain-qemu-interface-filterref.xml | 170 +++++++++++++++++++++ 2 files changed, 172 insertions(+), 2 deletions(-) create mode 100644 tests/domainschemadata/domain-qemu-interface-filterref.xml diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng index 56ea6a4..b1d883f 100644 --- a/docs/schemas/domaincommon.rng +++ b/docs/schemas/domaincommon.rng @@ -4530,7 +4530,7 @@ <attribute name="filter"> <data type="NCName"/> </attribute> - <optional> + <zeroOrMore> <element name="parameter"> <attribute name="name"> <ref name="filter-param-name"/> @@ -4539,7 +4539,7 @@ <ref name="filter-param-value"/> </attribute> </element> - </optional> + </zeroOrMore> </define> <define name="deviceBoot"> diff --git a/tests/domainschemadata/domain-qemu-interface-filterref.xml b/tests/domainschemadata/domain-qemu-interface-filterref.xml new file mode 100644 index 0000000..0aa0c99 --- /dev/null +++ b/tests/domainschemadata/domain-qemu-interface-filterref.xml @@ -0,0 +1,170 @@ +<domain type='kvm'> + <name>gentoo</name> + <uuid>a75aca4b-a02f-2bcb-4a91-c93cd848c34b</uuid> + <memory unit='KiB'>4194304</memory> + <currentMemory unit='KiB'>4194304</currentMemory> + <memoryBacking> + <hugepages> + <page size='1048576' unit='KiB' nodeset='0-3'/> + </hugepages> + </memoryBacking> + <vcpu placement='static'>4</vcpu> + <os> + <type arch='x86_64' machine='pc-i440fx-1.4'>hvm</type> + <boot dev='hd'/> + <boot dev='cdrom'/> + </os> + <features> + <acpi/> + <apic/> + <pae/> + </features> + <cpu mode='custom' match='exact'> + <model fallback='allow'>Haswell</model> + <vendor>Intel</vendor> + <feature policy='require' name='tm2'/> + <feature policy='require' name='est'/> + <feature policy='require' name='vmx'/> + <feature policy='require' name='osxsave'/> + <feature policy='require' name='smx'/> + <feature policy='require' name='ss'/> + <feature policy='require' name='ds'/> + <feature policy='require' name='vme'/> + <feature policy='require' name='dtes64'/> + <feature policy='require' name='abm'/> + <feature policy='require' name='ht'/> + <feature policy='require' name='acpi'/> + <feature policy='require' name='pbe'/> + <feature policy='require' name='tm'/> + <feature policy='require' name='pdcm'/> + <feature policy='require' name='pdpe1gb'/> + <feature policy='require' name='ds_cpl'/> + <feature policy='require' name='rdrand'/> + <feature policy='require' name='f16c'/> + <feature policy='require' name='xtpr'/> + <feature policy='require' name='monitor'/> + <numa> + <cell id='0' cpus='0' memory='1048576' unit='KiB'/> + <cell id='1' cpus='1' memory='1048576' unit='KiB'/> + <cell id='2' cpus='2' memory='1048576' unit='KiB'/> + <cell id='3' cpus='3' memory='1048576' unit='KiB'/> + </numa> + </cpu> + <clock offset='utc'/> + <on_poweroff>destroy</on_poweroff> + <on_reboot>restart</on_reboot> + <on_crash>restart</on_crash> + <pm> + <suspend-to-mem enabled='yes'/> + <suspend-to-disk enabled='yes'/> + </pm> + <devices> + <emulator>/usr/bin/qemu-system-x86_64</emulator> + <disk type='file' device='floppy'> + <driver name='qemu' type='raw' cache='none'/> + <source file='/var/lib/libvirt/images/fd.img'/> + <target dev='fda' bus='fdc'/> + <address type='drive' controller='0' bus='0' target='0' unit='0'/> + </disk> + <disk type='file' device='disk'> + <driver name='qemu' type='qcow2'/> + <source file='/var/lib/libvirt/images/gentoo.qcow2'/> + <target dev='vda' bus='virtio'/> + <address type='pci' domain='0x0000' bus='0x00' slot='0x05' function='0x0'/> + </disk> + <disk type='file' device='disk'> + <driver name='qemu' type='qcow2'/> + <source file='/var/lib/libvirt/images/OtherDemo.img'/> + <target dev='vdb' bus='virtio'/> + <encryption format='qcow'> + <secret type='passphrase' uuid='e78d4b51-a2af-485f-b0f5-afca709a80f4'/> + </encryption> + <address type='pci' domain='0x0000' bus='0x00' slot='0x07' function='0x0'/> + </disk> + <disk type='file' device='cdrom'> + <driver name='qemu' type='raw' cache='none'/> + <source file='/home/zippy/tmp/install-amd64-minimal-20140619.iso'/> + <target dev='hdc' bus='ide'/> + <readonly/> + <shareable/> + <address type='drive' controller='0' bus='1' target='0' unit='0'/> + </disk> + <controller type='usb' index='0'> + <address type='pci' domain='0x0000' bus='0x00' slot='0x01' function='0x2'/> + </controller> + <controller type='pci' index='0' model='pci-root'/> + <controller type='ide' index='0'> + <address type='pci' domain='0x0000' bus='0x00' slot='0x01' function='0x1'/> + </controller> + <controller type='virtio-serial' index='0'> + <address type='pci' domain='0x0000' bus='0x00' slot='0x08' function='0x0'/> + </controller> + <controller type='fdc' index='0'/> + <interface type='network'> + <mac address='52:54:00:d6:c0:0b'/> + <source network='default'/> + <bandwidth> + <inbound average='100'/> + <outbound average='100'/> + </bandwidth> + <model type='virtio'/> + <address type='pci' domain='0x0000' bus='0x00' slot='0x03' function='0x0'/> + </interface> + <interface type='bridge'> + <mac address='52:54:00:35:82:59'/> + <source bridge='br0'/> + <model type='rtl8139'/> + <filterref filter='myfilter'> + <parameter name='CTRL_IP_LEARNING' value='none'/> + <parameter name='DHCPSERVER' value='104.156.226.10'/> + <parameter name='IP' value='104.207.129.11'/> + <parameter name='IP6_ADDR' value='2001:19f0:300:2102::'/> + <parameter name='IP6_MASK' value='64'/> + </filterref> + <address type='pci' domain='0x0000' bus='0x00' slot='0x0b' function='0x0'/> + </interface> + <interface type='server'> + <mac address='52:54:00:22:c9:42'/> + <source address='127.0.0.1' port='1234'/> + <bandwidth> + <inbound average='1234'/> + <outbound average='5678'/> + </bandwidth> + <model type='rtl8139'/> + <address type='pci' domain='0x0000' bus='0x00' slot='0x09' function='0x0'/> + </interface> + <interface type='client'> + <mac address='52:54:00:8c:b1:f8'/> + <source address='127.0.0.1' port='1234'/> + <model type='rtl8139'/> + <address type='pci' domain='0x0000' bus='0x00' slot='0x0a' function='0x0'/> + </interface> + <serial type='pty'> + <target port='0'/> + </serial> + <serial type='pty'> + <target port='1'/> + </serial> + <console type='pty'> + <target type='serial' port='0'/> + </console> + <channel type='unix'> + <source mode='bind' path='/var/lib/libvirt/qemu/channel/target/gentoo.org.qemu.guest_agent.0'/> + <target type='virtio' name='org.qemu.guest_agent.0'/> + <address type='virtio-serial' controller='0' bus='0' port='1'/> + </channel> + <input type='mouse' bus='ps2'/> + <input type='keyboard' bus='ps2'/> + <graphics type='vnc' port='-1' autoport='yes'/> + <sound model='ich6'> + <address type='pci' domain='0x0000' bus='0x00' slot='0x04' function='0x0'/> + </sound> + <video> + <model type='cirrus' vram='16384' heads='1'/> + <address type='pci' domain='0x0000' bus='0x00' slot='0x02' function='0x0'/> + </video> + <memballoon model='virtio'> + <address type='pci' domain='0x0000' bus='0x00' slot='0x06' function='0x0'/> + </memballoon> + </devices> +</domain> -- 2.0.5

2 1

[libvirt] [PATCH 0/4] parallels: fix different domain define parameters
by Maxim Nestratov 11 Mar '15

11 Mar '15

Actually our checking for unsupported parameters is too strict about cpumask, vcpupin and NUMA parameters and in general wrong. First of all, we do support cpumask and consequently we don't have to prevent domain definition in case cpumask is specified. Also we should allow vcpupin parameters if they don't contradict with common cpumask. NUMA checking was also wrong because def->numa is always allocated. Here we check that its content is not specified. Maxim Nestratov (4): parallels: cpumask support parallels: don't forget to unlock domain in parallelsDomainHasManagedSaveImage parallels: prevent domain define only if NUMA is really specified parallels: prevent domain define only if vcpupin is specified

2 8

[libvirt] [libvirt-test-API][PATCH 0/2] Add securitylabel test case
by jiahu 11 Mar '15

11 Mar '15

The test case will check securityLabel() and securityLabelList() APIs. jiahu (2): Add securitylabel test case to linux_domain.conf Add securitylabel test case cases/linux_domain.conf | 4 + repos/domain/securitylabel.py | 170 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 174 insertions(+) create mode 100644 repos/domain/securitylabel.py -- 1.8.3.1

2 3

[libvirt] [PATCH v3 0/5] Add support for graphics to get a IPv6 address
by John Ferlan 11 Mar '15

11 Mar '15

Update to Luyao's series - figured I'd repost just to make it easier rather than continued replies... v2 here: http://www.redhat.com/archives/libvir-list/2015-February/msg01228.html Changes since v2: * Split patch 1 into two parts - one to create the IPAddress shim that just calls the existing (and now static) IPv4 code. * Adjust remainder of patches according to my review comments (hopefully and thus why I'm reposting) John Ferlan (1): util: Replace virNetDevGetIPv4Address with virNetDevGetIPAddress Luyao Huang (4): util: Update virNetDevGetIPAddress to add IPv6 boolean conf: Add 'family' attribute to <graphics> 'listen' element network: Allow networkGetNetworkAddress to return IPv6 address qemu: Remove unnecessary virReportError on networkGetNetworkAddress return docs/formatdomain.html.in | 14 ++- docs/schemas/domaincommon.rng | 8 ++ src/conf/domain_conf.c | 21 ++++ src/conf/domain_conf.h | 10 ++ src/libvirt_private.syms | 2 +- src/network/bridge_driver.c | 17 +-- src/network/bridge_driver.h | 6 +- src/qemu/qemu_command.c | 22 ++-- src/util/virnetdev.c | 121 +++++++++++++++++++-- src/util/virnetdev.h | 7 +- .../qemuxml2argv-graphics-listen-network-ipv4.xml | 35 ++++++ .../qemuxml2argv-graphics-listen-network-ipv6.xml | 35 ++++++ tests/qemuxml2xmltest.c | 2 + 13 files changed, 263 insertions(+), 37 deletions(-) create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-graphics-listen-network-ipv4.xml create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-graphics-listen-network-ipv6.xml -- 2.1.0

2 10

[libvirt] [libvirt-test-API][PATCH V2 0/2] Add freepages test cases
by Jincheng Miao 11 Mar '15

11 Mar '15

Add freepages test cases. V2: Assign ps.strip().upper() to ps variable in free_page.py Jincheng Miao (2): Add freepage test Add freepage test case to test_connection.conf cases/test_connection.conf | 6 +++ repos/virconn/free_pages.py | 97 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 103 insertions(+), 0 deletions(-) create mode 100644 repos/virconn/free_pages.py

3 4

[libvirt] [PATCH] domain_conf: fix crash in virDomainObjListFindByUUIDInternal
by Michael Chapman 11 Mar '15

11 Mar '15

If a domain object is being removed and looked up concurrently we must ensure we unlock the object before unreferencing it, since the latter might free the object. Signed-off-by: Michael Chapman <mike(a)very.puzzling.org> --- src/conf/domain_conf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index cc8616b..ce0e173 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -1068,9 +1068,9 @@ virDomainObjListFindByUUIDInternal(virDomainObjListPtr doms, if (obj) { virObjectLock(obj); if (obj->removing) { + virObjectUnlock(obj); if (ref) virObjectUnref(obj); - virObjectUnlock(obj); obj = NULL; } } -- 2.1.0

2 1

[libvirt] [PATCH 0/3] virBitmapGetBit cleanups
by Ján Tomko 11 Mar '15

11 Mar '15

Return bool instead of ignoring the return value almost everywhere. Ján Tomko (3): Use virBitmapNextClearBit in networkNextClassID Return bool in virBitmapGetBit Reverse the logic in virbitmaptest src/conf/domain_conf.c | 4 +--- src/conf/node_device_conf.c | 4 +--- src/conf/snapshot_conf.c | 6 ++---- src/conf/storage_conf.c | 4 +--- src/libxl/libxl_domain.c | 4 +--- src/libxl/libxl_driver.c | 5 +---- src/network/bridge_driver.c | 8 +++----- src/nodeinfo.c | 6 +----- src/qemu/qemu_capabilities.c | 8 +++----- src/qemu/qemu_command.c | 12 +----------- src/qemu/qemu_driver.c | 15 +++------------ src/qemu/qemu_process.c | 13 ++++--------- src/storage/storage_backend.c | 4 +--- src/test/test_driver.c | 5 +---- src/util/virbitmap.c | 13 +++++-------- src/util/virbitmap.h | 6 +++--- src/util/vircgroup.c | 4 +--- src/util/virdnsmasq.c | 7 ++----- src/util/virportallocator.c | 14 ++------------ src/util/virprocess.c | 9 ++------- src/xen/xen_driver.c | 4 +--- src/xen/xend_internal.c | 5 +---- tests/virbitmaptest.c | 18 ++++-------------- tools/virsh-domain.c | 6 ++---- 24 files changed, 47 insertions(+), 137 deletions(-) -- 2.0.5

3 8

[libvirt] [PATCH] {domain, network}_conf: disable autostart when deleting config
by Michael Chapman 11 Mar '15

11 Mar '15

Undefining a running, autostarted domain removes the autostart link, but dom->autostart is not cleared. If the domain is subsequently redefined, libvirt thinks it is already autostarted and will not create the link even if requested: # virsh dominfo example | grep Autostart Autostart: enable # ls /etc/libvirt/qemu/autostart/example.xml /etc/libvirt/qemu/autostart/example.xml # virsh undefine example Domain example has been undefined # virsh define example.xml Domain example defined from example.xml # virsh dominfo example | grep Autostart Autostart: enable # virsh autostart example Domain example marked as autostarted # ls /etc/libvirt/qemu/autostart/example.xml ls: cannot access /etc/libvirt/qemu/autostart/example.xml: No such file or directory This commit ensures dom->autostart is cleared whenever the config and autostart link (if present) are removed. The bridge network driver cleared this flag itself in networkUndefine. This commit moves this into virNetworkDeleteConfig for symmetry with virDomainDeleteConfig, and to ensure it is not missed in future network drivers. Signed-off-by: Michael Chapman <mike(a)very.puzzling.org> --- src/conf/domain_conf.c | 1 + src/conf/network_conf.c | 1 + src/network/bridge_driver.c | 1 - 3 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index cc8616b..a8f4ce2 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -20998,6 +20998,7 @@ virDomainDeleteConfig(const char *configDir, /* Not fatal if this doesn't work */ unlink(autostartLink); + dom->autostart = 0; if (unlink(configFile) < 0 && errno != ENOENT) { diff --git a/src/conf/network_conf.c b/src/conf/network_conf.c index 9c1d578..779a08a 100644 --- a/src/conf/network_conf.c +++ b/src/conf/network_conf.c @@ -3077,6 +3077,7 @@ int virNetworkDeleteConfig(const char *configDir, /* Not fatal if this doesn't work */ unlink(autostartLink); + net->autostart = 0; if (unlink(configFile) < 0) { virReportSystemError(errno, diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c index 5752acb..5158078 100644 --- a/src/network/bridge_driver.c +++ b/src/network/bridge_driver.c @@ -3050,7 +3050,6 @@ networkUndefine(virNetworkPtr net) driver->networkAutostartDir, network) < 0) goto cleanup; - network->autostart = 0; event = virNetworkEventLifecycleNew(network->def->name, network->def->uuid, -- 2.1.0

2 1