June 2012 - Devel - libvirt List Archives

[libvirt] [PATCH] fix sync issue in virnetclientstream
by Radu Caragea 04 Jun '12

04 Jun '12

Hello, I have come across a synchronization issue in virnetclientstream.c The stream lock is unlocked twice instead of being locked and then unlocked. Probably a typo. Best regards, Radu Caragea

2 1

[libvirt] can't get right capabilities
by Li Zhang 04 Jun '12

04 Jun '12

Hi, I am trying to use x86 machine to create VMs. But one error occurs. [root@mcd kvm-test]# virsh create test.xml error: Failed to create domain from test.xml error: unknown OS type hvm My xml file is as the following: <domain type='kvm'> <name>test</name> <memory>524288</memory> <memorybacking> <hugepages/> </memorybacking> <vcpu>2</vcpu> <os> <type arch='x86_64' machine='pc'>hvm</type> </os> <features> <acpi/> </features> <clock offset='utc'/> <devices> <emulator>/usr/bin/qemu-system-x86_64</emulator> <disk type='file' device='disk' > <driver name='qemu' type='raw'/> <source file='/home/zhlbj/kvm-test/disk1'/> <target dev='sda' bus='scsi'/> </disk> <interface type='bridge'> <source bridge='virbr0'/> <mac address='52:54:00:19:bc:0c'/> </interface> <serial type='pty'> <target port='0'/> </serial> <console type='pty'> <target type='serial' port='0'/> </console> <memballoon model="none"/> </devices> </domain> I check kvm module, it is available. [root@mcd kvm-test]# lsmod |grep kvm kvm_intel 119308 4 kvm 344876 1 kvm_intel But I tried the capabilities, it is not right. There is no guest capabilities. <capabilities> <host> <uuid>d4ff0166-0f8b-e111-ab17-78a277034000</uuid> <cpu> <arch>x86_64</arch> <model>Nehalem</model> <vendor>Intel</vendor> <topology sockets='1' cores='4' threads='1'/> <feature name='rdtscp'/> <feature name='avx'/> <feature name='osxsave'/> <feature name='xsave'/> <feature name='tsc-deadline'/> <feature name='x2apic'/> <feature name='pdcm'/> <feature name='xtpr'/> <feature name='tm2'/> <feature name='est'/> <feature name='smx'/> <feature name='vmx'/> <feature name='ds_cpl'/> <feature name='monitor'/> <feature name='dtes64'/> <feature name='pclmuldq'/> <feature name='pbe'/> <feature name='tm'/> <feature name='ht'/> <feature name='ss'/> <feature name='acpi'/> <feature name='ds'/> <feature name='vme'/> </cpu> <power_management> <suspend_mem/> <suspend_disk/> <suspend_hybrid/> </power_management> <migration_features> <live/> <uri_transports> <uri_transport>tcp</uri_transport> </uri_transports> </migration_features> <secmodel> <model>selinux</model> <doi>0</doi> </secmodel> </host> </capabilities> This problem occurs some times. When I tried to find this problem, it didn't occur. Now it occurs again. It's a little strange. Is there anything I miss? -- Best Regards Li IBM LTC, China System&Technology Lab, Beijing

4 6

[libvirt] [PATCH v3 0/2] Make virsh *edit more bearable
by Michal Privoznik 02 Jun '12

02 Jun '12

If there's an error in XML, all changes made by user are lost. Users (read /me) tends to get angry with this. Allow them to get back and re-edit. diff to v2: -some bugfixes (like flawn in snapshot edit) -new EDIT_FREE to avoid leaks (Both suggested by Eric) diff to v1: -Eric's review suggestions included Michal Privoznik (2): virsh: Switch from generated cmd*Edit commands to nongenerated virsh: Allow users to reedit rejected XML cfg.mk | 4 +- po/POTFILES.in | 1 + tools/Makefile.am | 40 +---- tools/console.c | 40 +++-- tools/console.h | 2 + tools/virsh-edit.c | 148 +++++++++++++++ tools/virsh.c | 509 +++++++++++++++++++++++----------------------------- 7 files changed, 407 insertions(+), 337 deletions(-) create mode 100644 tools/virsh-edit.c -- 1.7.8.5

2 5

[libvirt] [PATCH] qemu: Move -name arg to be the 1st so it's visible in "ps x"
by Marti Raudsepp 02 Jun '12

02 Jun '12

Hi list, Currently, monitoring QEMU virtual machines with standard Unix sysadmin tools is harder than it has to be. The QEMU command line is often miles long and mostly redundant, it's hard to tell which process is which. This patch reorders the QEMU -name argument to be the first, so it's immediately visible in "ps x", htop and "atop -c" output. Patch attached and also available from my github fork (master @ git://github.com/intgr/libvirt.git) Regards, Marti

2 3

[libvirt] [PATCH] qemu: fix netdev alias name assignment wrt type='hostdev'
by Laine Stump 02 Jun '12

02 Jun '12

This patch resolves: https://bugzilla.redhat.com/show_bug.cgi?id=827519 The problem is that an interface with type='hostdev' will have an alias of the form "hostdev%d", while the function that looks through existing netdevs to determine the name to use for a new addition will fail if there's an existing entry that does not match the form "net%d". This is another of the handful of places that need an exception due to the hybrid nature of <interface type='hostdev'> (which is not exactly an <interface> or a <hostdev>, but is both at the same time). --- src/qemu/qemu_command.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 9f99dce..1a240fb 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -548,6 +548,11 @@ qemuAssignDeviceNetAlias(virDomainDefPtr def, virDomainNetDefPtr net, int idx) idx = 0; for (i = 0 ; i < def->nnets ; i++) { int thisidx; + + if (def->nets[i]->type == VIR_DOMAIN_NET_TYPE_HOSTDEV) { + /* type='hostdev' interfaces have a hostdev%d alias */ + continue; + } if ((thisidx = qemuDomainDeviceAliasIndex(&def->nets[i]->info, "net")) < 0) { qemuReportError(VIR_ERR_INTERNAL_ERROR, "%s", _("Unable to determine device index for network device")); -- 1.7.10.2

2 1

Re: [libvirt] hello:about libvirt collectd monitor plugin
by Richard W.M. Jones 01 Jun '12

01 Jun '12

On Fri, Jun 01, 2012 at 04:36:17PM +0800, xingxing gao wrote: > hi,rjones: > i am using libvirt and qemu-kvm on rhel,in these days i need to > monitor the vm's performance on my computer,and i've found the > wonderful tool---collectd and the lovely plugin for libvirt,but i > found that the libvirt plugin seems didnot collect the memory info of > the vm,so i ask you for help,thanks!!!^^ The libvirt collectd plugin predates support for memory statistics. Therefore it would need to be extended to add this feature. Patches for this best sent to the collectd project. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://libguestfs.org

2 1

[libvirt] [PATCH][libvirt-glib] glib-events: Allow zero timeouts for timer
by Michal Privoznik 01 Jun '12

01 Jun '12

In libvirt, it's perfectly possible and widely used to have disabled timers (timeout=-1) and fire them up 'randomly' with timeout=0. However, with current mapping into glib mainloop it's not possible and causing troubles. --- libvirt-glib/libvirt-glib-event.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libvirt-glib/libvirt-glib-event.c b/libvirt-glib/libvirt-glib-event.c index 94f4de8..c3eb3c8 100644 --- a/libvirt-glib/libvirt-glib-event.c +++ b/libvirt-glib/libvirt-glib-event.c @@ -384,7 +384,7 @@ gvir_event_timeout_update(int timer, if (interval >= 0) { if (data->source) - goto cleanup; + g_source_remove(data->source); data->interval = interval; data->source = g_timeout_add(data->interval, -- 1.7.8.5

2 2