[libvirt] [PATCH 1/3] Implementation deficiency in virInitctlSetRunLevel v2

2013-12-19 Thread Reco
Hello, list. Refuse following symlinks in virInitctlSetRunLevel. A reasonable fallback for the next two patches, which apply fork-setns technique recommended on this list. --- src/util/virinitctl.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/virinitctl.c

[libvirt] [PATCH 2/3] Implementation deficiency in virInitctlSetRunLevel v2

2013-12-19 Thread Reco
Implement fork setns for lxcDomainShutdownFlags --- src/lxc/lxc_driver.c | 45 + 1 file changed, 45 insertions(+) diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c index c499182..9d200b2 100644 --- a/src/lxc/lxc_driver.c +++

[libvirt] [PATCH 3/3] Implementation deficiency in virInitctlSetRunLevel v2

2013-12-19 Thread Reco
Implement fork setns for lxcDomainShutdownFlags --- src/lxc/lxc_driver.c | 45 + 1 file changed, 45 insertions(+) diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c index 9d200b2..1227736 100644 --- a/src/lxc/lxc_driver.c +++

[libvirt] [PATCH 1/3] Implementation deficiency in virInitctlSetRunLevel v2

2013-12-19 Thread Reco
Hello, list. Refuse following symlinks in virInitctlSetRunLevel. A reasonable fallback for the next two patches, which apply fork-setns technique recommended on this list. --- src/util/virinitctl.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/virinitctl.c

Re: [libvirt] [PATCH 0/5] Fill qemucapabilitiesdata with some data

2013-12-19 Thread Eric Blake
On 10/03/2013 07:25 AM, Michal Privoznik wrote: The actual patches are accessible at: git://gitorious.org/libvirt/michal-staging.git branch test_qemu_capabilities_data I'm not sending the actual patches as it's big junk of JSON qemu replies. The patches has from 35KiB to 59KiB. I don't

Re: [libvirt] Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]

2013-12-19 Thread Dario Faggioli
On gio, 2013-12-19 at 11:46 +, Ian Jackson wrote: Dario Faggioli writes (Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]): [Moving this to libvir, libvir-users in Bcc. Also, added xen-devel] 4.2.1 is lacking libxl: fix stale timeout

Re: [libvirt] lvm storage pool destroy problem

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 01:54:19AM +0500, Umar Draz wrote: Hi I am trying to stop the storage pool through php-libvirt api which is a LVM volume group. when I tried to stop it I got error error: internal error Child process (/sbin/vgchange -aln mylvmgrp) unexpected exit status 5: Can't

[libvirt] [PATCH] virnettlscontexttest fails with GNUTLS 3.0.28

2013-12-19 Thread Cédric Bosdonnat
Changed the constraints on gnutls to 3.1+ --- tests/virnettlscontexttest.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/virnettlscontexttest.c b/tests/virnettlscontexttest.c index fc512fc..d9a4e9d 100644 --- a/tests/virnettlscontexttest.c +++

Re: [libvirt] [PATCH] storage: fix bogus target in gluster volume xml

2013-12-19 Thread Eric Blake
On 12/19/2013 02:18 AM, Peter Krempa wrote: On 12/19/13 05:38, Eric Blake wrote: Commit 6cd60b6 was flat out broken - it tried to print into the wrong variable. My testing was obviously too cursory (did the name get a slash added?); valgrind would have caught the error. Thankfully it didn't

Re: [libvirt] [PATCH] lxcDomainShutdownFlags: Cleanup @flags usage

2013-12-19 Thread Daniel P. Berrange
On Wed, Dec 18, 2013 at 07:19:31PM +0100, Michal Privoznik wrote: Currently, the @flags usage is a bit unclear at first sight to say the least. There's no need for such unclear code especially when we can borrow the working code from qemuDomainShutdownFlags(). In addition, this fixes one bug

Re: [libvirt] [PATCH V2 RESEND 7/8] qemu: allow to setup throttle blkio cgroup through virsh

2013-12-19 Thread Daniel P. Berrange
On Fri, Dec 13, 2013 at 11:08:10AM +0800, Gao feng wrote: With this patch, user can setup throttle blkio cgroup through virsh for qemu domain. Signed-off-by: Guan Qiang hzguanqi...@corp.netease.com Signed-off-by: Gao feng gaof...@cn.fujitsu.com --- src/qemu/qemu_driver.c | 442

Re: [libvirt] [PATCH 2/2] maint: improve debug of libvirt-{qemu, lxc} apis

2013-12-19 Thread Eric Blake
On 12/19/2013 08:17 AM, Daniel P. Berrange wrote: On Thu, Dec 19, 2013 at 08:13:36AM -0700, Eric Blake wrote: diff --git a/src/libvirt_internal.h b/src/libvirt_internal.h index 115d8d1..b8c842d 100644 --- a/src/libvirt_internal.h +++ b/src/libvirt_internal.h @@ -27,6 +27,113 @@ # include

Re: [libvirt] [PATCH V2 RESEND 8/8] lxc: allow to setup throttle blkio cgroup through virsh

2013-12-19 Thread Daniel P. Berrange
On Fri, Dec 13, 2013 at 11:09:01AM +0800, Gao feng wrote: With this patch,user can set throttle blkio cgroup for lxc domain through virsh tool. Signed-off-by: Guan Qiang hzguanqi...@corp.netease.com Signed-off-by: Gao feng gaof...@cn.fujitsu.com --- src/lxc/lxc_driver.c | 671

Re: [libvirt] [PATCH 2/2] maint: improve debug of libvirt-{qemu, lxc} apis

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 09:35:23AM -0700, Eric Blake wrote: On 12/19/2013 08:17 AM, Daniel P. Berrange wrote: On Thu, Dec 19, 2013 at 08:13:36AM -0700, Eric Blake wrote: diff --git a/src/libvirt_internal.h b/src/libvirt_internal.h index 115d8d1..b8c842d 100644 --- a/src/libvirt_internal.h

Re: [libvirt] [PATCH 1/3] Implementation deficiency in virInitctlSetRunLevel v2

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 08:07:39PM +0400, Reco wrote: Hello, list. Refuse following symlinks in virInitctlSetRunLevel. A reasonable fallback for the next two patches, which apply fork-setns technique recommended on this list. --- src/util/virinitctl.c |2 +- 1 file changed, 1

[libvirt] [PATCH] Fix explicit usage of default video PCI slots

2013-12-19 Thread Ján Tomko
Don't set the PCI address of primary video to the default if it's already occupied by another device. Without this, a primary video card that doesn't have an address gets the default one on the 1st pass of PCI address allocation, even if it's occupied. This address gets picked up by the second

Re: [libvirt] [PATCH 2/3] Implementation deficiency in virInitctlSetRunLevel v2

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 08:07:50PM +0400, Reco wrote: Implement fork setns for lxcDomainShutdownFlags --- src/lxc/lxc_driver.c | 45 + 1 file changed, 45 insertions(+) diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c index

Re: [libvirt] [PATCH 1/2] docs: improve event-related documentation

2013-12-19 Thread Eric Blake
On 12/19/2013 08:18 AM, Daniel P. Berrange wrote: On Thu, Dec 19, 2013 at 08:13:35AM -0700, Eric Blake wrote: While looking at event code, I noticed that the documentation was trying to refer me to functions that don't exist. Also fix some typos and poor formatting. * src/libvirt.c

Re: [libvirt] lvm storage pool destroy problem

2013-12-19 Thread Umar Draz
Hi Daniel, my Virtual machines are running, so how I can stop the volumes? Br. Umar On Thu, Dec 19, 2013 at 9:21 PM, Daniel P. Berrange berra...@redhat.comwrote: On Thu, Dec 19, 2013 at 01:54:19AM +0500, Umar Draz wrote: Hi I am trying to stop the storage pool through php-libvirt api

Re: [libvirt] lvm storage pool destroy problem

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 09:51:31PM +0500, Umar Draz wrote: Hi Daniel, my Virtual machines are running, so how I can stop the volumes? If you have VMs using volumes in a pool then you cannot stop that pool until the VMs are stopped. Daniel -- |: http://berrange.com -o-

Re: [libvirt] [PATCH] network: only prevent forwarding of DNS requests for unqualified names

2013-12-19 Thread Daniel P. Berrange
On Mon, Dec 16, 2013 at 04:41:31PM +0200, Laine Stump wrote: On 12/10/2013 04:45 PM, Daniel P. Berrange wrote: So considering your example XML config above we're debating the behaviour of the following 5 possible DNS requests - myguest - myguest.example.com - notmyguest -

Re: [libvirt] [Xen-devel] Setting devid for emulated NICs (Xen 4.3.1 / libvirt 1.2.0) using libxl driver

2013-12-19 Thread Stefan Bader
On 19.12.2013 11:19, Ian Campbell wrote: On Wed, 2013-12-18 at 17:44 -0700, Jim Fehlig wrote: Stefan Bader wrote: On 18.12.2013 14:28, Ian Campbell wrote: On Wed, 2013-12-18 at 14:12 +0100, Stefan Bader wrote: On 18.12.2013 13:27, Ian Campbell wrote: On Tue, 2013-12-17 at

Re: [libvirt] [PATCH] virkeycode: Allow ANSI_A

2013-12-19 Thread Jiri Denemark
On Thu, Dec 19, 2013 at 16:46:50 +0100, Michal Privoznik wrote: https://bugzilla.redhat.com/show_bug.cgi?id=1044806 Currently, sending the ANSI_A keycode from os_x codepage doesn't work as it has a special value of 0x0. Our internal code handles that no different to other not defined

Re: [libvirt] [PATCH] Fix explicit usage of default video PCI slots

2013-12-19 Thread Jiri Denemark
On Thu, Dec 19, 2013 at 17:43:33 +0100, Jano Tomko wrote: Don't set the PCI address of primary video to the default if it's already occupied by another device. Without this, a primary video card that doesn't have an address gets the default one on the 1st pass of PCI address allocation,

Re: [libvirt] [PATCH] Fix explicit usage of default video PCI slots

2013-12-19 Thread Ján Tomko
On 12/19/2013 06:18 PM, Jiri Denemark wrote: On Thu, Dec 19, 2013 at 17:43:33 +0100, Jano Tomko wrote: Don't set the PCI address of primary video to the default if it's already occupied by another device. Without this, a primary video card that doesn't have an address gets the default one on

Re: [libvirt] [Xen-devel] Setting devid for emulated NICs (Xen 4.3.1 / libvirt 1.2.0) using libxl driver

2013-12-19 Thread Ian Campbell
On Thu, 2013-12-19 at 18:06 +0100, Stefan Bader wrote: How about we: * move the init to setdefault to catch the single NIC added via hotplug case Init of devid? Yes, sorry for not being clear. Hm, would that work as I am not sure there is a simple way of differentiating

Re: [libvirt] [RFC PATCH] qemu: new API for tracking arbitrary monitor events

2013-12-19 Thread Eric Blake
On 12/19/2013 08:23 AM, Daniel P. Berrange wrote: On Thu, Dec 19, 2013 at 08:15:09AM -0700, Eric Blake wrote: diff --git a/src/libvirt-qemu.c b/src/libvirt-qemu.c index db52c65..849932d 100644 +if (dom +!(VIR_IS_CONNECTED_DOMAIN(dom) dom-conn == conn)) { +

Re: [libvirt] [Xen-devel] Setting devid for emulated NICs (Xen 4.3.1 / libvirt 1.2.0) using libxl driver

2013-12-19 Thread Jim Fehlig
Stefan Bader wrote: Oh, just while talking about setdefault. Jim, this is one of the odd things when moving from xm to xl stack from libvirt: libvirt defaults to the netfront NIC when no model is specified and sets the type. The libxl setdefault function sets the model to rtl8139 but

Re: [libvirt] [RFC PATCH] qemu: new API for tracking arbitrary monitor events

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 11:32:32AM -0700, Eric Blake wrote: On 12/19/2013 08:23 AM, Daniel P. Berrange wrote: On Thu, Dec 19, 2013 at 08:15:09AM -0700, Eric Blake wrote: diff --git a/src/libvirt-qemu.c b/src/libvirt-qemu.c index db52c65..849932d 100644 +if (dom +

Re: [libvirt] Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]

2013-12-19 Thread Jim Fehlig
Dario Faggioli wrote: On gio, 2013-12-19 at 11:46 +, Ian Jackson wrote: Dario Faggioli writes (Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]): [Moving this to libvir, libvir-users in Bcc. Also, added xen-devel] 4.2.1

Re: [libvirt] [PATCH] Fix explicit usage of default video PCI slots

2013-12-19 Thread Jiri Denemark
On Thu, Dec 19, 2013 at 18:41:25 +0100, Jano Tomko wrote: On 12/19/2013 06:18 PM, Jiri Denemark wrote: On Thu, Dec 19, 2013 at 17:43:33 +0100, Jano Tomko wrote: Don't set the PCI address of primary video to the default if it's already occupied by another device. Without this, a primary

Re: [libvirt] [RFC PATCH] qemu: new API for tracking arbitrary monitor events

2013-12-19 Thread Eric Blake
On 12/19/2013 09:01 AM, Daniel P. Berrange wrote: +typedef void (*virConnectDomainQemuMonitorEventCallback)(virConnectPtr conn, + virDomainPtr dom, + const char *event, +

Re: [libvirt] [PATCH 1/3] Implementation deficiency in virInitctlSetRunLevel v2

2013-12-19 Thread Eric Blake
On 12/19/2013 09:07 AM, Reco wrote: Hello, list. Refuse following symlinks in virInitctlSetRunLevel. A reasonable fallback for the next two patches, which apply fork-setns technique recommended on this list. Your patches came through as independent top-level threads, rather than in-reply

Re: [libvirt] [PATCH] virnettlscontexttest fails with GNUTLS 3.0.28

2013-12-19 Thread Eric Blake
On 12/19/2013 09:23 AM, Cédric Bosdonnat wrote: Changed the constraints on gnutls to 3.1+ --- tests/virnettlscontexttest.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/virnettlscontexttest.c b/tests/virnettlscontexttest.c index fc512fc..d9a4e9d 100644 ---

Re: [libvirt] [PATCH 2/2] maint: improve debug of libvirt-{qemu, lxc} apis

2013-12-19 Thread Eric Blake
On 12/19/2013 09:37 AM, Daniel P. Berrange wrote: +#define virLibDomainSnapshotError(code, ...) \ +virReportErrorHelper(VIR_FROM_DOMAIN_SNAPSHOT, code, __FILE__, \ + __FUNCTION__, __LINE__, __VA_ARGS__) I'd venture to sugggest that these

Re: [libvirt] [PATCH] Block info query: Add flag to allow failure if not active

2013-12-19 Thread Federico Simoncelli
- Original Message - From: Daniel P. Berrange berra...@redhat.com To: John Ferlan jfer...@redhat.com Cc: libvir-list@redhat.com Sent: Wednesday, December 18, 2013 3:57:22 PM Subject: Re: [libvirt] [PATCH] Block info query: Add flag to allow failure if not active On Wed, Dec 18,

[libvirt] [PATCHv2] maint: improve debug of libvirt-{qemu, lxc} apis

2013-12-19 Thread Eric Blake
I noticed that the virDomainQemuMonitorCommand debug output wasn't telling me the name of the domain it was working on. While it was easy enough to determine which pointer matches the domain based on other log messages, it is nicer to be consistent. * src/util/viruuid.h (VIR_UUID_DEBUG): Moved

Re: [libvirt] [PATCH 1/3] libxl: implement virDomainGetVcpuPinInfo

2013-12-19 Thread Jim Fehlig
Dario Faggioli wrote: So that it is possible to query vcpu related information of a persistent but not running domain, like it is for the QEMU driver. In fact, before this patch, we have: # virsh list --all IdName State

[libvirt] [PATCH] libxl: fix segfault when domain create fail

2013-12-19 Thread Bamvor Jian Zhang
there is a segfault in libxl logging in libxl_ctx_free when domain create fail. because the log output handler vmessage is freed by xtl_logger_destroy before libxl_ctx_free in virDomainObjListRemove. move xtl_logger_destroy after libxl_ctx_free could fix this bug. Signed-off-by: Bamvor Jian Zhang

[libvirt] [PATCH] Add helper program to create custom leases

2013-12-19 Thread Nehal J Wani
Introduce helper program to catch events from dnsmasq and maintain a custom lease file per network. It supports dhcpv4 and dhcpv6. The file is saved as interface-name.status. The format of each lease is: expiry-time (epoch time) mac iaid ip-address hostname clientid Example of custom leases

Re: [libvirt] [PATCH] Fix segmentation fault when accessing default qemu machine type

2013-12-19 Thread Yudai Yamagishi
Yudai, would you be comfortable with updating our test suite to test this corner case? Okay, I've taken look at the test suite and I can update it. However, my time is pretty limited right now so it may take a little while to get the update done. If you need this ASAP, maybe someone else should

Re: [libvirt] [PATCH 1/1] virsh nodecpustats erroneously returns stats for offline cpus on Linux

2013-12-19 Thread Bing Bu Cao
On 12/19/2013 12:40 PM, Pradipta Kr. Banerjee wrote: From: Pradipta Kr. Banerjee bpra...@in.ibm.com virsh nodecpustats erroneously returns stats for offline cpus on Linux To retrieve node cpu statistics on Linux system, the linuxNodeGetCPUstats function performs a minimal match of the input

Re: [libvirt] [PATCH] storage: fix bogus target in gluster volume xml

2013-12-19 Thread Peter Krempa
On 12/19/13 05:38, Eric Blake wrote: Commit 6cd60b6 was flat out broken - it tried to print into the wrong variable. My testing was obviously too cursory (did the name get a slash added?); valgrind would have caught the error. Thankfully it didn't hit any release. Reported by Peter Krempa.

Re: [libvirt] [Xen-devel] Setting devid for emulated NICs (Xen 4.3.1 / libvirt 1.2.0) using libxl driver

2013-12-19 Thread Ian Campbell
On Wed, 2013-12-18 at 17:44 -0700, Jim Fehlig wrote: Stefan Bader wrote: On 18.12.2013 14:28, Ian Campbell wrote: On Wed, 2013-12-18 at 14:12 +0100, Stefan Bader wrote: On 18.12.2013 13:27, Ian Campbell wrote: On Tue, 2013-12-17 at 18:32 +0100, Stefan Bader wrote:

Re: [libvirt] [PATCH 1/1] virsh nodecpustats erroneously returns stats for offline cpus on Linux

2013-12-19 Thread Pradipta Kumar Banerjee
On 12/19/2013 02:02 PM, Bing Bu Cao wrote: On 12/19/2013 12:40 PM, Pradipta Kr. Banerjee wrote: From: Pradipta Kr. Banerjee bpra...@in.ibm.com virsh nodecpustats erroneously returns stats for offline cpus on Linux To retrieve node cpu statistics on Linux system, the linuxNodeGetCPUstats

Re: [libvirt] [PATCH RFC 09/27] storage: Add gluster pool filter and fix virsh pool listing

2013-12-19 Thread Peter Krempa
On 12/19/13 05:54, Eric Blake wrote: On 12/16/2013 09:32 AM, Peter Krempa wrote: Recent adition of the gluster pool type omitted fixing the virsh and s/adition/addition/ virConnectListAllStoragePool filters. A typecast of the converting function in virsh showed that also the sheepdog pool

[libvirt] Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]

2013-12-19 Thread Dario Faggioli
[Moving this to libvir, libvir-users in Bcc. Also, added xen-devel] Jim, cooldharma06 reports having issues when destroying VMs with libvirt 1.2.0 and Xen 4.2.1 (is that the case, cooldharma06)? Look at the stack trace at the end of this message, or here:

Re: [libvirt] [PATCH RFC 10/27] storage: Avoid forward declaration of virStorageVolDelete

2013-12-19 Thread Peter Krempa
On 12/19/13 05:56, Eric Blake wrote: On 12/16/2013 09:32 AM, Peter Krempa wrote: Move the code around so that the forward declaration isn't needed. --- src/storage/storage_driver.c | 181 ++- 1 file changed, 91 insertions(+), 90 deletions(-) ACK;

Re: [libvirt] Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]

2013-12-19 Thread Ian Jackson
Dario Faggioli writes (Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]): [Moving this to libvir, libvir-users in Bcc. Also, added xen-devel] 4.2.1 is lacking libxl: fix stale timeout event callback race libxl: fix stale fd event callback

Re: [libvirt] Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]

2013-12-19 Thread Dario Faggioli
On gio, 2013-12-19 at 11:46 +, Ian Jackson wrote: Dario Faggioli writes (Segfault in libxl_osevent_occurred_timeout [Was: Re: [libvirt-users] About debugging of libvirt.]): [Moving this to libvir, libvir-users in Bcc. Also, added xen-devel] 4.2.1 is lacking libxl: fix stale timeout

[libvirt] [PATCH 2/3] qemu: avoid duplicate security label restore on hostdev attach failure

2013-12-19 Thread Laine Stump
This eliminates the misleading error message that was being logged when a vfio hostdev hotplug failed: error: unable to set user and group to '107:107' on '/dev/vfio/22': No such file or directory as documented in: https://bugzilla.redhat.com/show_bug.cgi?id=1035490 Commit

[libvirt] [PATCH 3/3] qemu: re-add hostdev interfaces to hostdev array on libvirtd restart

2013-12-19 Thread Laine Stump
This resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1045002 If a domain has an interface type='hostdev' or an interface type='network' where the network itself is a pool of hostdev devices, then libvirt will internally keep that device on both the interface list *and* the hostdev list

[libvirt] [PATCH 0/3] PCI device assignment bugfixes

2013-12-19 Thread Laine Stump
These were all found as a result of researching Bug 1035490. Patch 1 fixes the originally reported problem, Patch 2 fixes the cause of incorrect error reporting in a later comment of that bug, and Patch 3 fixes a behavior that I encountered while searching for the cause. Laine Stump (3): qemu:

[libvirt] [PATCH 1/3] qemu: properly set MaxMemLock when hotplugging with VFIO

2013-12-19 Thread Laine Stump
This resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1035490 virProcessSetMaxMemLock() (which is a wrapper over prlimit(3)) expects the memory size in bytes, but libvirt's domain definition (which was being used by qemuDomainAttachHostPciDevice()) stores all memory tuning parameters in

[libvirt] [PATCH 1/2] docs: improve event-related documentation

2013-12-19 Thread Eric Blake
While looking at event code, I noticed that the documentation was trying to refer me to functions that don't exist. Also fix some typos and poor formatting. * src/libvirt.c (virConnectDomainEventDeregister) (virConnectDomainEventRegisterAny) (virConnectDomainEventDeregisterAny)

[libvirt] [PATCH 2/2] maint: improve debug of libvirt-{qemu, lxc} apis

2013-12-19 Thread Eric Blake
I noticed that the virDomainQemuMonitorCommand debug output wasn't telling me the name of the domain it was working on. While it was easy enough to determine which pointer matches the domain based on other log messages, it is nicer to be consistent. Along the same lines, having

[libvirt] [PATCH 0/2] event cleanup patches

2013-12-19 Thread Eric Blake
Noticed these while working on another patch. Although it is mostly cosmetic, it is not small enough to push under the trivial rule, so I'll wait for a review. Eric Blake (2): docs: improve event-related documentation maint: improve debug of libvirt-{qemu,lxc} apis

[libvirt] [RFC PATCH] qemu: new API for tracking arbitrary monitor events

2013-12-19 Thread Eric Blake
Several times in the past, qemu has implemented a new event, but libvirt has not yet caught up to reporting that event to the user applications. While it is possible to track libvirt logs to see that an unknown event was received and ignored, it would be nicer to copy what 'virsh

Re: [libvirt] [PATCH 2/2] maint: improve debug of libvirt-{qemu, lxc} apis

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 08:13:36AM -0700, Eric Blake wrote: diff --git a/src/libvirt_internal.h b/src/libvirt_internal.h index 115d8d1..b8c842d 100644 --- a/src/libvirt_internal.h +++ b/src/libvirt_internal.h @@ -27,6 +27,113 @@ # include internal.h +/* Helper macros to implement

Re: [libvirt] [PATCH 1/2] docs: improve event-related documentation

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 08:13:35AM -0700, Eric Blake wrote: While looking at event code, I noticed that the documentation was trying to refer me to functions that don't exist. Also fix some typos and poor formatting. * src/libvirt.c (virConnectDomainEventDeregister)

Re: [libvirt] [RFC PATCH] qemu: new API for tracking arbitrary monitor events

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 08:15:09AM -0700, Eric Blake wrote: diff --git a/src/libvirt-qemu.c b/src/libvirt-qemu.c index db52c65..849932d 100644 --- a/src/libvirt-qemu.c +++ b/src/libvirt-qemu.c @@ -237,3 +237,126 @@ error: virDispatchError(conn); return NULL; } + + +/** + *

[libvirt] [PATCH] virkeycode: Allow ANSI_A

2013-12-19 Thread Michal Privoznik
https://bugzilla.redhat.com/show_bug.cgi?id=1044806 Currently, sending the ANSI_A keycode from os_x codepage doesn't work as it has a special value of 0x0. Our internal code handles that no different to other not defined keycodes. Hence, in order to allow it we must change all the undefined

Re: [libvirt] [PATCH 1/1] virsh nodecpustats erroneously returns stats for offline cpus on Linux

2013-12-19 Thread Bing Bu Cao
On 12/19/2013 12:40 PM, Pradipta Kr. Banerjee wrote: From: Pradipta Kr. Banerjee bpra...@in.ibm.com virsh nodecpustats erroneously returns stats for offline cpus on Linux To retrieve node cpu statistics on Linux system, the linuxNodeGetCPUstats function performs a minimal match of the input

Re: [libvirt] JNA Error Callback could cause core dump.

2013-12-19 Thread Claudio Bley
At Thu, 18 Oct 2012 15:48:22 +, Benjamin Wang (gendwang) wrote: Hi, When I changed code as following: public class Connect { // Load the native part static { Libvirt.INSTANCE.virInitialize(); try { ErrorHandler.processError(Libvirt.INSTANCE);

Re: [libvirt] [RFC PATCH] qemu: new API for tracking arbitrary monitor events

2013-12-19 Thread Michal Privoznik
On 19.12.2013 16:15, Eric Blake wrote: Several times in the past, qemu has implemented a new event, but libvirt has not yet caught up to reporting that event to the user applications. While it is possible to track libvirt logs to see that an unknown event was received and ignored, it would

Re: [libvirt] [PATCH] virkeycode: Allow ANSI_A

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 04:46:50PM +0100, Michal Privoznik wrote: https://bugzilla.redhat.com/show_bug.cgi?id=1044806 Currently, sending the ANSI_A keycode from os_x codepage doesn't work as it has a special value of 0x0. Our internal code handles that no different to other not defined

Re: [libvirt] [RFC PATCH] qemu: new API for tracking arbitrary monitor events

2013-12-19 Thread Daniel P. Berrange
On Thu, Dec 19, 2013 at 04:59:28PM +0100, Michal Privoznik wrote: On 19.12.2013 16:15, Eric Blake wrote: Several times in the past, qemu has implemented a new event, but libvirt has not yet caught up to reporting that event to the user applications. While it is possible to track libvirt