Re: [libvirt] [PATCH] qemu: bulk stats: typo in monitor handling

2014-12-11 Thread Peter Krempa
On 12/11/14 08:44, Francesco Romani wrote: > A typo in qemuConnectGetAllDomainStats makes the code > mark the monitor as available when qemuDomainObjBeginJob > fails, instead of when it succeeds, as the correct flow > requires. > > This patch fixes the check and updates the code documentation > ac

Re: [libvirt] [PATCH] qemu: bulk stats: typo in monitor handling

2014-12-11 Thread Martin Kletzander
On Thu, Dec 11, 2014 at 08:44:09AM +0100, Francesco Romani wrote: A typo in qemuConnectGetAllDomainStats makes the code mark the monitor as available when qemuDomainObjBeginJob fails, instead of when it succeeds, as the correct flow requires. This patch fixes the check and updates the code docum

[libvirt] [PATCH] qemu: bulk stats: typo in monitor handling

2014-12-10 Thread Francesco Romani
A typo in qemuConnectGetAllDomainStats makes the code mark the monitor as available when qemuDomainObjBeginJob fails, instead of when it succeeds, as the correct flow requires. This patch fixes the check and updates the code documentation accordingly. Signed-off-by: Francesco Romani --- src/qem