Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-16 Thread Andres Lagar-Cavilla
On Jan 15, 2013, at 5:13 PM, Eric Blake wrote: > On 01/07/2013 02:25 PM, Andres Lagar-Cavilla wrote: >> Perform all the appropriate plumbing. >> >> When qemu/KVM VMs are paused manually through a monitor not-owned by libvirt, >> libvirt will think of them as "paused" event after they are resumed

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-15 Thread Eric Blake
On 01/07/2013 02:25 PM, Andres Lagar-Cavilla wrote: > Perform all the appropriate plumbing. > > When qemu/KVM VMs are paused manually through a monitor not-owned by libvirt, > libvirt will think of them as "paused" event after they are resumed and > effectively running. With this patch the discrep

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-09 Thread Andres Lagar-Cavilla
On Jan 9, 2013, at 4:26 AM, Michal Privoznik wrote: > On 08.01.2013 21:08, Andres Lagar-Cavilla wrote: >> On Jan 8, 2013, at 1:30 PM, "Daniel P. Berrange" wrote: > > > >>> >>> ACK to your patch anyway. >> >> Cool, thanks. >> >> Not sure about the process around here. More ACKs needed? >>

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-09 Thread Michal Privoznik
On 08.01.2013 21:08, Andres Lagar-Cavilla wrote: > On Jan 8, 2013, at 1:30 PM, "Daniel P. Berrange" wrote: >> >> ACK to your patch anyway. > > Cool, thanks. > > Not sure about the process around here. More ACKs needed? > > Andres No, I've pushed the patch now. Michal -- libvir-list mailin

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-08 Thread Andres Lagar-Cavilla
On Jan 8, 2013, at 1:30 PM, "Daniel P. Berrange" wrote: > On Tue, Jan 08, 2013 at 01:28:10PM -0500, Andres Lagar-Cavilla wrote: >> On Jan 7, 2013, at 5:44 PM, "Daniel P. Berrange" wrote: >> >>> On Mon, Jan 07, 2013 at 04:25:01PM -0500, Andres Lagar-Cavilla wrote: Perform all the appropriat

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-08 Thread Daniel P. Berrange
On Tue, Jan 08, 2013 at 01:28:10PM -0500, Andres Lagar-Cavilla wrote: > On Jan 7, 2013, at 5:44 PM, "Daniel P. Berrange" wrote: > > > On Mon, Jan 07, 2013 at 04:25:01PM -0500, Andres Lagar-Cavilla wrote: > >> Perform all the appropriate plumbing. > >> > >> When qemu/KVM VMs are paused manually t

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-08 Thread Andres Lagar-Cavilla
On Jan 7, 2013, at 5:44 PM, "Daniel P. Berrange" wrote: > On Mon, Jan 07, 2013 at 04:25:01PM -0500, Andres Lagar-Cavilla wrote: >> Perform all the appropriate plumbing. >> >> When qemu/KVM VMs are paused manually through a monitor not-owned by libvirt, >> libvirt will think of them as "paused" e

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-08 Thread Dave Allan
On Mon, Jan 07, 2013 at 10:44:41PM +, Daniel P. Berrange wrote: > On Mon, Jan 07, 2013 at 04:25:01PM -0500, Andres Lagar-Cavilla wrote: > > Perform all the appropriate plumbing. > > > > When qemu/KVM VMs are paused manually through a monitor not-owned by > > libvirt, > > libvirt will think of

Re: [libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-07 Thread Daniel P. Berrange
On Mon, Jan 07, 2013 at 04:25:01PM -0500, Andres Lagar-Cavilla wrote: > Perform all the appropriate plumbing. > > When qemu/KVM VMs are paused manually through a monitor not-owned by libvirt, > libvirt will think of them as "paused" event after they are resumed and > effectively running. With this

[libvirt] [PATCH] Add RESUME event listener to qemu monitor.

2013-01-07 Thread Andres Lagar-Cavilla
Perform all the appropriate plumbing. When qemu/KVM VMs are paused manually through a monitor not-owned by libvirt, libvirt will think of them as "paused" event after they are resumed and effectively running. With this patch the discrepancy goes away. This is meant to address bug 892791. Signed-