- Devel - libvirt List Archives

[libvirt] [PATCH] libvirt.c: fix typo in comment
by Claudio Bley 17 Jan '13

17 Jan '13

Signed-off-by: Claudio Bley <cbley(a)av-test.de> --- Pushing under the trivial rule. src/libvirt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libvirt.c b/src/libvirt.c index 7db677f..d5d561c 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -17181,7 +17181,7 @@ error: * encrypted, or running over a channel which is not exposed * to eavesdropping (eg a UNIX domain socket, or pipe) * - * Returns 1 if secure, 0 if secure, -1 on error + * Returns 1 if secure, 0 if not secure, -1 on error */ int virConnectIsSecure(virConnectPtr conn) { -- 1.7.9.5

1 0

[libvirt] [PATCH] Move comment after enum members
by Claudio Bley 17 Jan '13

17 Jan '13

The api builder always associates comments to the last member it read, not to the current member even if there was a comment for the previous member and a comma was already seen. This has the effect that the comment for the previous member gets overwritten and the current member has no comment at all. Signed-off-by: Claudio Bley <cbley(a)av-test.de> --- include/libvirt/libvirt.h.in | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/include/libvirt/libvirt.h.in b/include/libvirt/libvirt.h.in index 09c89c5..9110fcf 100644 --- a/include/libvirt/libvirt.h.in +++ b/include/libvirt/libvirt.h.in @@ -127,12 +127,12 @@ typedef enum { power management */ #ifdef VIR_ENUM_SENTINELS - /* - * NB: this enum value will increase over time as new events are - * added to the libvirt API. It reflects the last state supported - * by this version of the libvirt API. - */ VIR_DOMAIN_LAST + /* + * NB: this enum value will increase over time as new events are + * added to the libvirt API. It reflects the last state supported + * by this version of the libvirt API. + */ #endif } virDomainState; @@ -2742,12 +2742,12 @@ typedef enum { VIR_STORAGE_VOL_WIPE_ALG_RANDOM = 8, /* 1-pass random */ #ifdef VIR_ENUM_SENTINELS + VIR_STORAGE_VOL_WIPE_ALG_LAST /* * NB: this enum value will increase over time as new algorithms are * added to the libvirt API. It reflects the last algorithm supported * by this version of the libvirt API. */ - VIR_STORAGE_VOL_WIPE_ALG_LAST #endif } virStorageVolWipeAlgorithm; @@ -2974,12 +2974,12 @@ typedef enum { VIR_KEYCODE_SET_RFB = 9, #ifdef VIR_ENUM_SENTINELS + VIR_KEYCODE_SET_LAST /* * NB: this enum value will increase over time as new events are * added to the libvirt API. It reflects the last keycode set supported * by this version of the libvirt API. */ - VIR_KEYCODE_SET_LAST #endif } virKeycodeSet; @@ -3533,12 +3533,12 @@ typedef enum { VIR_SECRET_USAGE_TYPE_CEPH = 2, #ifdef VIR_ENUM_SENTINELS + VIR_SECRET_USAGE_TYPE_LAST /* * NB: this enum value will increase over time as new events are * added to the libvirt API. It reflects the last secret owner ID * supported by this version of the libvirt API. */ - VIR_SECRET_USAGE_TYPE_LAST #endif } virSecretUsageType; @@ -4443,12 +4443,12 @@ typedef enum { VIR_DOMAIN_EVENT_ID_PMSUSPEND_DISK = 14, /* virConnectDomainEventPMSuspendDiskCallback */ #ifdef VIR_ENUM_SENTINELS + VIR_DOMAIN_EVENT_ID_LAST /* * NB: this enum value will increase over time as new events are * added to the libvirt API. It reflects the last event ID supported * by this version of the libvirt API. */ - VIR_DOMAIN_EVENT_ID_LAST #endif } virDomainEventID; -- 1.7.9.5

2 6

[libvirt] iptables --physdev-out warnings
by Reinier Schoof 17 Jan '13

17 Jan '13

Hi, we've experienced some issues with starting lots of KVM based VM's with libvirt. Since I couldn't find any clues on the libvirt mailing list, I'm posting the way I fixed the issues. When starting a VM, /var/log/messages was spammed with the following message: xt_physdev: using --physdev-out in the OUTPUT, FORWARD and POSTROUTING chains for non-bridged traffic is not supported anymore. With each extra VM I start, the messages get amplified exponentially. This results in longer starting times every new VM, relative the the previously started VM. When I ran a test with starting 100 equal VM's, the first VM started in about 2 seconds, the 100th VM took 48 seconds to start. I'm running a vanilla 3.7.1 kernel, but I have the same issue on VM hosts with kernel 3.2.28 or 3.2.0, running libvirt 0.9.12 and 0.9.8 respectively. Looking into the warning, it seemed that iptables need an extra argument, --physdev-is-bridged, in commands like: iptables -A libvirt-out -m physdev --physdev-is-bridged --physdev-out vnet99 -g FP-vnet99 I patched the libvirt source (version 1.0.0) to test whether this works or not: --- src/nwfilter/nwfilter_ebiptables_driver.c.orig 2013-01-16 10:51:43.000000000 +0100 +++ src/nwfilter/nwfilter_ebiptables_driver.c 2013-01-16 10:52:07.000000000 +0100 @@ -166,7 +166,7 @@ snprintf(buf, sizeof(buf), "%c%c-%s", prefix[0], prefix[1], ifname) #define PHYSDEV_IN "--physdev-in" -#define PHYSDEV_OUT "--physdev-out" +#define PHYSDEV_OUT "--physdev-is-bridged --physdev-out" static const char *m_state_out_str = "-m state --state NEW,ESTABLISHED"; static const char *m_state_in_str = "-m state --state ESTABLISHED"; The warnings in /var/log/messages are gone and running the test again proved the 100th VM started in 3.8 seconds. It suprises me I'm the first to mention this problem on the libvirt mailing list and I wondering if I'm doing something wrong. Until then, this fix helps me a lot! Reinier Schoof -- TransIP BV | https://www.transip.nl/

3 2

[libvirt] [test-API][PATCH] Add storage cases of pool list, pool lookup and volume lookup
by Wayne Sun 17 Jan '13

17 Jan '13

Add 3 simple storage realted API cases * pool_list_active is only for testing with flags: libvirt.VIR_CONNECT_LIST_STORAGE_POOLS_ACTIVE * pool_lookup is for testing lookup pool object by name, UUID, UUIDString and volume object * vol_lookup is for testing lookup volume object by name, key and path * add cases into storage_dir.conf Signed-off-by: Wayne Sun <gsun(a)redhat.com> --- cases/storage_dir.conf | 14 +++++++ repos/storage/pool_list_active.py | 50 +++++++++++++++++++++++++ repos/storage/pool_lookup.py | 74 +++++++++++++++++++++++++++++++++++++ repos/storage/vol_lookup.py | 62 +++++++++++++++++++++++++++++++ 4 files changed, 200 insertions(+), 0 deletions(-) create mode 100644 repos/storage/pool_list_active.py create mode 100644 repos/storage/pool_lookup.py create mode 100644 repos/storage/vol_lookup.py diff --git a/cases/storage_dir.conf b/cases/storage_dir.conf index 38b349d..2bdb7d6 100644 --- a/cases/storage_dir.conf +++ b/cases/storage_dir.conf @@ -2,14 +2,22 @@ storage:define_dir_pool poolname $defaultpoolname +storage:pool_lookup + poolname + $defaultpoolname + storage:build_dir_pool poolname $defaultpoolname +storage:pool_list_active + storage:activate_pool poolname $defaultpoolname +storage:pool_list_active + storage:create_dir_volume poolname $defaultpoolname @@ -20,6 +28,12 @@ storage:create_dir_volume capacity $defaultvolumesize +storage:vol_lookup + poolname + $defaultpoolname + volname + $defaultvolumename + storage:vol_clone poolname $defaultpoolname diff --git a/repos/storage/pool_list_active.py b/repos/storage/pool_list_active.py new file mode 100644 index 0000000..3e390c8 --- /dev/null +++ b/repos/storage/pool_list_active.py @@ -0,0 +1,50 @@ +#!/usr/bin/env python +# list active storage pool testing with flag: +# libvirt.VIR_CONNECT_LIST_STORAGE_POOLS_ACTIVE + +import libvirt +from libvirt import libvirtError + +from src import sharedmod + +required_params = () +optional_params = { + } + +def pool_list_active(params): + """list active storage pool testing""" + + global logger + logger = params['logger'] + namelist = [] + + conn = sharedmod.libvirtobj['conn'] + try: + pool_num = conn.numOfStoragePools() + logger.info("number of active storage pools is %s" % pool_num) + + flag = libvirt.VIR_CONNECT_LIST_STORAGE_POOLS_ACTIVE + poolobj_list = conn.listAllStoragePools(flag) + if not len(poolobj_list) == pool_num: + logger.error("active pool object number mismatched") + return 1 + + for i in poolobj_list: + pool_name = i.name() + namelist.append(pool_name) + + logger.info("active pool name list is %s" % namelist) + + active_namelist = conn.listStoragePools() + if namelist == active_namelist: + logger.info("active pool name list matched") + else: + logger.error("active pool name list mismatched") + return 1 + + except libvirtError, e: + logger.error("libvirt call failed: " + str(e)) + return 1 + + logger.info("list active storage pool succeed") + return 0 diff --git a/repos/storage/pool_lookup.py b/repos/storage/pool_lookup.py new file mode 100644 index 0000000..89d7252 --- /dev/null +++ b/repos/storage/pool_lookup.py @@ -0,0 +1,74 @@ +#!/usr/bin/env python +# storage pool lookup testing + +import libvirt +from libvirt import libvirtError + +from src import sharedmod + +required_params = ('poolname',) +optional_params = { + 'volname': None + } + +def pool_lookup(params): + """storage pool lookup testing""" + + global logger + logger = params['logger'] + poolname = params['poolname'] + volname = params.get('volname') + retval = 0 + + logger.info("the poolname is %s" % poolname) + if volname: + logger.info("the given volume name is %s" % volname) + + conn = sharedmod.libvirtobj['conn'] + try: + logger.info("lookup the pool object by name: %s" % poolname) + poolobj = conn.storagePoolLookupByName(poolname) + if poolobj.name() == poolname: + logger.info("pool object lookup by name succeed") + else: + logger.error("pool object lookup by name failed") + retval += 1 + + uuid = poolobj.UUID() + logger.info("lookup the pool object by UUID: %s" % uuid) + poolobj_uuid = conn.storagePoolLookupByUUID(uuid) + if poolobj_uuid.name() == poolname: + logger.info("pool object lookup by UUID succeed") + else: + logger.error("pool object lookup by UUID failed") + retval += 1 + + uuidstr = poolobj.UUIDString() + logger.info("lookup the pool object by UUID string: %s" % uuidstr) + poolobj_uuidstr = conn.storagePoolLookupByUUIDString(uuidstr) + if poolobj_uuidstr.name() == poolname: + logger.info("pool object lookup by UUID string succeed") + else: + logger.error("pool object lookup by UUID string failed") + retval += 1 + + if volname: + logger.info("lookup volume object by volume name: %s" % volname) + volobj = poolobj.storageVolLookupByName(volname) + logger.info("lookup the pool object by volume object") + poolobj_vol = volobj.storagePoolLookupByVolume() + + if poolobj_vol.name() == poolname: + logger.info("pool object lookup by UUID string succeed") + else: + logger.error("pool object lookup by UUID string failed") + retval += 1 + + except libvirtError, e: + logger.error("libvirt call failed: " + str(e)) + return 1 + + if retval: + return 1 + + return 0 diff --git a/repos/storage/vol_lookup.py b/repos/storage/vol_lookup.py new file mode 100644 index 0000000..30238e1 --- /dev/null +++ b/repos/storage/vol_lookup.py @@ -0,0 +1,62 @@ +#!/usr/bin/env python +# storage pool volume lookup testing + +import libvirt +from libvirt import libvirtError + +from src import sharedmod + +required_params = ('poolname', 'volname',) +optional_params = { + } + +def vol_lookup(params): + """storage pool volume lookup testing""" + + global logger + logger = params['logger'] + poolname = params['poolname'] + volname = params['volname'] + retval = 0 + + logger.info("the poolname is %s" % poolname) + logger.info("the given volume name is %s" % volname) + + conn = sharedmod.libvirtobj['conn'] + try: + poolobj = conn.storagePoolLookupByName(poolname) + + logger.info("lookup the volume object by name: %s" % volname) + volobj = poolobj.storageVolLookupByName(volname) + if volobj.name() == volname: + logger.info("volume object lookup by name succeed") + else: + logger.error("volume object lookup by name failed") + retval += 1 + + volkey = volobj.key() + logger.info("lookup the volume object by key: %s" % volkey) + volobj_key = conn.storageVolLookupByKey(volkey) + if volobj_key.name() == volname: + logger.info("volume object lookup by key succeed") + else: + logger.error("volume object lookup by key failed") + retval += 1 + + volpath = volobj.path() + logger.info("lookup the volume object by path: %s" % volpath) + volobj_path = conn.storageVolLookupByPath(volpath) + if volobj_path.name() == volname: + logger.info("volume object lookup by path succeed") + else: + logger.error("volume object lookup by path failed") + retval += 1 + + except libvirtError, e: + logger.error("libvirt call failed: " + str(e)) + return 1 + + if retval: + return 1 + + return 0 -- 1.7.1

1 0

[libvirt] [PATCH v2 0/5]use VIR_ENUM_DECL for domain NIC model and add usb-net
by Guannan Ren 17 Jan '13

17 Jan '13

v1 to v2 *removed i8* NIC models support for QEMU/KVM *updated docs This patchset uses VIR_ENUM_DECL and VIR_ENUM_IMPL macros for NIC models and changes the related codes to use them. All of NIC models supported by various hypervisors comprise the enum, so the problem is that we need to do further checking in hypervisor-specific implementation. Fortunately, VMX and Vbox did this checking very well already. In 4/5, a similar job is performed for QEMU/KVM. And, adds usb-net support. The vendorId and productID of is 0525:a4a2. The corresponding item in usb-ids is as follows: 0525 Netchip Technology, Inc. a4a2 Linux-USB Ethernet/RNDIS Gadget In my opinion, using usb-net directly is good enough, so I keep it. Any idea is welcome. Libvirt XML sample: <devices> <interface type='user'> <mac address='52:54:00:32:6a:91'/> <model type='usb-net'/> <alias name='net1'/> <address type='usb' bus='0' port='1'/> </interface> </devices> qemu commandline: qemu ${other_vm_args} -netdev user,id=hostnet1 \ -device usb-net,netdev=hostnet1,id=net1,\ mac=52:54:00:32:6a:91,bus=usb.0,port=1 Guannan Ren(2) [PATCH v2 1/5] conf: fix mismatch on two domain VIR_ENUM_IMPL macro [PATCH v2 2/5] add domain NIC model enum macro [PATCH v2 3/5] net: use virDomainNICModelType{From|To}String functions [PATCH v2 4/5] qemu: add NIC model checking for qemu hypervisor [PATCH v2 5/5] qemu: add usb-net support docs/formatdomain.html.in | 29 +++++++++++------------------ src/conf/domain_conf.c | 88 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------- src/conf/domain_conf.h | 32 +++++++++++++++++++++++++++++++- src/libvirt_private.syms | 2 ++ src/libxl/libxl_conf.c | 5 +++-- src/parallels/parallels_driver.c | 2 +- src/qemu/qemu_command.c | 41 +++++++++++++++++++++++++++-------------- src/qemu/qemu_driver.c | 2 +- src/qemu/qemu_hotplug.c | 10 ++++++---- src/qemu/qemu_process.c | 14 +++++++------- src/vbox/vbox_tmpl.c | 57 +++++++++++++++++++++++++++------------------------------ src/vmx/vmx.c | 31 ++++++++++++++++--------------- src/xenxs/xen_sxpr.c | 23 ++++++++++++----------- src/xenxs/xen_xm.c | 21 +++++++++++---------- 14 files changed, 215 insertions(+), 142 deletions(-)

1 5

[libvirt] Entering freeze for 1.0.2 in a week
by Daniel Veillard 17 Jan '13

17 Jan '13

Just for people to remember, the plan is to enter the freeze for 1.0.2 in a week, on the 24. So if you need to push APIs or large changes, please do so before that deadline :-) Then the release should show up at the end of the month, thanks ! Daniel -- Daniel Veillard | Open Source and Standards, Red Hat veillard(a)redhat.com | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | virtualization library http://libvirt.org/

1 0

[libvirt] [PATCH 0/5] use VIR_ENUM_DECL for domain NIC model and add usb-net
by Guannan Ren 17 Jan '13

17 Jan '13

This patchset uses VIR_ENUM_DECL and VIR_ENUM_IMPL macros for NIC models and changes the related codes to use them. All of NIC models supported by various hypervisors comprise the enum, so the problem is that we need to do further checking in hypervisor-specific implementation. Fortunately, VMX and Vbox did this checking very well already. In 4/5, a similar job is performed for QEMU/KVM. And, adds usb-net support. The vendorId and productID of is 0525:a4a2. The corresponding item in usb-ids is as follows: 0525 Netchip Technology, Inc. a4a2 Linux-USB Ethernet/RNDIS Gadget In my opinion, using usb-net directly is good enough, so I keep it. Any idea is welcome. Libvirt XML sample: <devices> <interface type='user'> <mac address='52:54:00:32:6a:91'/> <model type='usb-net'/> <alias name='net1'/> <address type='usb' bus='0' port='1'/> </interface> </devices> qemu commandline: qemu ${other_vm_args} -netdev user,id=hostnet1 \ -device usb-net,netdev=hostnet1,id=net1,\ mac=52:54:00:32:6a:91,bus=usb.0,port=1 Guannan Ren(5) [PATCH 1/5] conf: fix mismatch on two domain VIR_ENUM_IMPL macro [PATCH 2/5] conf: add domain NIC model enum macro [PATCH 3/5] net: use virDomainNICModelType{From|To}String functions [PATCH 4/5] qemu: add NIC model checking for qemu hypervisor [PATCH 5/5] qemu: add usb-net support docs/formatdomain.html.in | 8 +++++++- src/conf/domain_conf.c | 101 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------- src/conf/domain_conf.h | 45 ++++++++++++++++++++++++++++++++++++++++++++- src/libvirt_private.syms | 2 ++ src/libxl/libxl_conf.c | 5 +++-- src/parallels/parallels_driver.c | 2 +- src/qemu/qemu_command.c | 41 +++++++++++++++++++++++++++-------------- src/qemu/qemu_driver.c | 2 +- src/qemu/qemu_hotplug.c | 10 ++++++---- src/qemu/qemu_process.c | 14 +++++++------- src/vbox/vbox_tmpl.c | 57 +++++++++++++++++++++++++++------------------------------ src/vmx/vmx.c | 31 ++++++++++++++++--------------- src/xenxs/xen_sxpr.c | 23 ++++++++++++----------- src/xenxs/xen_xm.c | 21 +++++++++++---------- 14 files changed, 237 insertions(+), 125 deletions(-)

3 12

Re: [libvirt] Redefinition of struct in6_addr in <netinet/in.h> and <linux/in6.h>
by David Miller 17 Jan '13

17 Jan '13

From: Mike Frysinger <vapier(a)gentoo.org> Date: Wed, 16 Jan 2013 12:28:39 -0500 > if you're not calling the kernel directly, why are you including the kernel > headers ? what is the problem people are actually trying to address here (and > no, "i want to include both headers" is not the answer) ? When GLIBC doesn't provide it's own definition of some networking macros or interfaces that the kernel provides, people include the kernel header. This has been done for decades, wake up.

3 3

Re: [libvirt] Redefinition of struct in6_addr in <netinet/in.h> and <linux/in6.h>
by Cong Wang 17 Jan '13

17 Jan '13

On Wed, 2013-01-16 at 14:22 -0500, Mike Frysinger wrote: > > but this is still too vague. what headers/definitions do people want to see > simultaneously included ? changes would be needed on both sides (kernel & C > library). > Hi, Mike, Please take a look at my first email in this thread. The user application includes <linux/if_bridge.h> and <netinet/in.h>. <linux/if_bridge.h> uses struct_in6 but doesn't include <linux/in6.h> (this is my bad, sorry), an obvious fix is just including <linux/in6.h>. But this immediately breaks applications which include <linux/if_bridge.h> and <netinet/in.h>, just as what Thomas reported. And if_bridge.h is kernel-specific, there is no corresponding glibc one, so you can't blame applications which include both of them. Thanks.

1 0

[libvirt] [PATCH 0/5] libxl: Trivial fixes and style improvements
by Jim Fehlig 17 Jan '13

17 Jan '13

This patchset contains some trivial bug fixes and style improvements for the libxl driver that were noticed while investigating races between the driver and libxl. Jim Fehlig (5): libxl: Use consistent style for function definitions libxl: Use consistent parameter naming scheme libxl: Don't free domain death event libxl: Check for libxl_ctx_alloc failure libxl: Fix cleanup on domain start error src/libxl/libxl_driver.c | 106 ++++++++++++++++++++++++++--------------------- 1 file changed, 58 insertions(+), 48 deletions(-) -- 1.8.0.1

2 13