[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-17 Thread Anthony Liguori
On 06/17/2010 11:45 AM, Luiz Capitulino wrote: On Thu, 17 Jun 2010 18:34:00 +0200 Juan Quintela wrote: Luiz Capitulino wrote: On Wed, 16 Jun 2010 21:10:04 +0200 Juan Quintela wrote: Luiz Capitulino wrote: On Tue, 15 Jun 2010 17:24:59 +0200 Juan Quintela wrot

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-17 Thread Juan Quintela
Luiz Capitulino wrote: > On Wed, 16 Jun 2010 21:10:04 +0200 > Juan Quintela wrote: > >> Luiz Capitulino wrote: >> > On Tue, 15 Jun 2010 17:24:59 +0200 >> > Juan Quintela wrote: >> >> >> > >> >> > I still don't see the need for MIGRATION_STARTED, it could be useful in >> >> > the target but I'

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-17 Thread Luiz Capitulino
On Thu, 17 Jun 2010 18:34:00 +0200 Juan Quintela wrote: > Luiz Capitulino wrote: > > On Wed, 16 Jun 2010 21:10:04 +0200 > > Juan Quintela wrote: > > > >> Luiz Capitulino wrote: > >> > On Tue, 15 Jun 2010 17:24:59 +0200 > >> > Juan Quintela wrote: > >> > >> >> > > >> >> > I still don't see t

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-17 Thread Luiz Capitulino
On Wed, 16 Jun 2010 21:10:04 +0200 Juan Quintela wrote: > Luiz Capitulino wrote: > > On Tue, 15 Jun 2010 17:24:59 +0200 > > Juan Quintela wrote: > > >> > > >> > I still don't see the need for MIGRATION_STARTED, it could be useful in > >> > the target but I'd like to understand the use case in

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-16 Thread Juan Quintela
Luiz Capitulino wrote: > On Tue, 15 Jun 2010 17:24:59 +0200 > Juan Quintela wrote: >> > >> > I still don't see the need for MIGRATION_STARTED, it could be useful in >> > the target but I'd like to understand the use case in more detail. >> >> At this point, if you are doing migration with tcp,

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-16 Thread Luiz Capitulino
On Tue, 15 Jun 2010 17:24:59 +0200 Juan Quintela wrote: > Luiz Capitulino wrote: > > On Tue, 15 Jun 2010 12:30:57 +0200 > > Juan Quintela wrote: > > > >> Anthony Liguori wrote: > >> > On 06/14/2010 02:54 PM, Juan Quintela wrote: > >> >> Anthony Liguori wrote: > >> > >> >>> What makes migrati

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-15 Thread Juan Quintela
Luiz Capitulino wrote: > On Tue, 15 Jun 2010 12:30:57 +0200 > Juan Quintela wrote: > >> Anthony Liguori wrote: >> > On 06/14/2010 02:54 PM, Juan Quintela wrote: >> >> Anthony Liguori wrote: >> >> >>> What makes migration important and not savevm? >> >>> >> >> That is the reason why I ins

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-15 Thread Luiz Capitulino
On Tue, 15 Jun 2010 12:30:57 +0200 Juan Quintela wrote: > Anthony Liguori wrote: > > On 06/14/2010 02:54 PM, Juan Quintela wrote: > >> Anthony Liguori wrote: > > >>> What makes migration important and not savevm? > >>> > >> That is the reason why I insist to have the events "both" in sou

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-15 Thread Juan Quintela
Anthony Liguori wrote: > On 06/14/2010 02:54 PM, Juan Quintela wrote: >> Anthony Liguori wrote: >>> What makes migration important and not savevm? >>> >> That is the reason why I insist to have the events "both" in source and >> destination. About how to integrate savevm on the whole pict

Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/14/2010 02:54 PM, Juan Quintela wrote: Anthony Liguori wrote: On 06/14/2010 01:35 PM, Juan Quintela wrote: Ok. lets stop here. My definitions: Event: this important thing happened (important has several meanings). Migration events fully enter in this definition. Further

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Juan Quintela
Anthony Liguori wrote: > On 06/14/2010 01:35 PM, Juan Quintela wrote: >> Ok. lets stop here. My definitions: >> >> Event: this important thing happened (important has several meanings). >> >> Migration events fully enter in this definition. Furthermore, migration >> events happens from actions

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/14/2010 01:35 PM, Juan Quintela wrote: Anthony Liguori wrote: On 06/14/2010 11:02 AM, Juan Quintela wrote: Anthony Liguori wrote: On 06/12/2010 06:05 AM, Juan Quintela wrote: Luiz Capitulinowrote: The monitor that did it knows

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Juan Quintela
Anthony Liguori wrote: > On 06/14/2010 11:02 AM, Juan Quintela wrote: >> Anthony Liguori wrote: >> >>> On 06/12/2010 06:05 AM, Juan Quintela wrote: >>> Luiz Capitulino wrote: >> The monitor that did it knows it, nobody else knows it. At destination tim

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/14/2010 11:02 AM, Juan Quintela wrote: Anthony Liguori wrote: On 06/12/2010 06:05 AM, Juan Quintela wrote: Luiz Capitulino wrote: The monitor that did it knows it, nobody else knows it. At destination time, I guess you agree this is important, i.e. the managemen

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Juan Quintela
Anthony Liguori wrote: > On 06/12/2010 06:05 AM, Juan Quintela wrote: >> Luiz Capitulino wrote: >> The monitor that did it knows it, nobody else knows it. At destination >> time, I guess you agree this is important, i.e. the management app knows >> that migration has started. >> > > Dual mo

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Juan Quintela
Luiz Capitulino wrote: > On Sat, 12 Jun 2010 13:20:54 +0200 > Juan Quintela wrote: >> Both. > > What does it report in the source? That migration has started :) Nothing else, nothing less. Think again multiple monitors and/or audit. >> Why can't we just tell them that migration has ended wi

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Luiz Capitulino
On Mon, 14 Jun 2010 09:35:52 -0500 Anthony Liguori wrote: > On 06/14/2010 09:24 AM, Luiz Capitulino wrote: > > On Mon, 14 Jun 2010 08:58:19 -0500 > > Anthony Liguori wrote: > > > > > >> For 0.13, we need to focus on introducing the least disruptive change > >> that addresses the fundamental

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Luiz Capitulino
On Sat, 12 Jun 2010 13:20:54 +0200 Juan Quintela wrote: > Luiz Capitulino wrote: > > On Fri, 11 Jun 2010 09:38:42 -0500 > > Anthony Liguori wrote: > > > >> > 1. QMP only returns the response when the command is finished, eg: > >> > > >> > C: { "execute": "migrate", "id": "foo" ... } > >>

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/14/2010 09:24 AM, Luiz Capitulino wrote: On Mon, 14 Jun 2010 08:58:19 -0500 Anthony Liguori wrote: For 0.13, we need to focus on introducing the least disruptive change that addresses the fundamental requirement--allow clients to avoid a polling loop for determining when migration en

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Luiz Capitulino
On Mon, 14 Jun 2010 08:58:19 -0500 Anthony Liguori wrote: > For 0.13, we need to focus on introducing the least disruptive change > that addresses the fundamental requirement--allow clients to avoid a > polling loop for determining when migration ends. Having a single event > with no payload

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/12/2010 06:05 AM, Juan Quintela wrote: Luiz Capitulino wrote: On Thu, 10 Jun 2010 12:44:55 +0200 Juan Quintela wrote: Luiz Capitulino wrote: On Wed, 9 Jun 2010 14:10:53 +0200 Juan Quintela wrote: This is a resent with what we agreed on yesterday call.

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-14 Thread Anthony Liguori
On 06/12/2010 06:14 AM, Juan Quintela wrote: Anthony Liguori wrote: On 06/11/2010 09:30 AM, Luiz Capitulino wrote: On Thu, 10 Jun 2010 12:44:55 +0200 Juan Quintela wrote: I think we've more or less agreed that MIGRATION_CONNECTED is really the event we want. Thi

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-12 Thread Juan Quintela
Luiz Capitulino wrote: > On Fri, 11 Jun 2010 09:38:42 -0500 > Anthony Liguori wrote: > >> > 1. QMP only returns the response when the command is finished, eg: >> > >> > C: { "execute": "migrate", "id": "foo" ... } >> > /* nothing is returned, other commands are issued, after several

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-12 Thread Juan Quintela
Anthony Liguori wrote: > On 06/11/2010 09:30 AM, Luiz Capitulino wrote: >> On Thu, 10 Jun 2010 12:44:55 +0200 >> Juan Quintela wrote: > > I think we've more or less agreed that MIGRATION_CONNECTED is really > the event we want. This had the problem of migrating to a file/whatever. MIGRATION_CO

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-12 Thread Juan Quintela
Luiz Capitulino wrote: > On Thu, 10 Jun 2010 12:44:55 +0200 > Juan Quintela wrote: > >> Luiz Capitulino wrote: >> > On Wed, 9 Jun 2010 14:10:53 +0200 >> > Juan Quintela wrote: >> > >> >> This is a resent with what we agreed on yesterday call. >> >> Migration events would be there for 0.13 unti

Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-11 Thread Luiz Capitulino
On Fri, 11 Jun 2010 09:38:42 -0500 Anthony Liguori wrote: > > 1. QMP only returns the response when the command is finished, eg: > > > > C: { "execute": "migrate", "id": "foo" ... } > > /* nothing is returned, other commands are issued, after several > > hours... */ > > S: { "re

Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-11 Thread Anthony Liguori
On 06/11/2010 09:30 AM, Luiz Capitulino wrote: On Thu, 10 Jun 2010 12:44:55 +0200 Juan Quintela wrote: Luiz Capitulino wrote: On Wed, 9 Jun 2010 14:10:53 +0200 Juan Quintela wrote: This is a resent with what we agreed on yesterday call. Migration events would be there f

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-11 Thread Luiz Capitulino
On Thu, 10 Jun 2010 12:44:55 +0200 Juan Quintela wrote: > Luiz Capitulino wrote: > > On Wed, 9 Jun 2010 14:10:53 +0200 > > Juan Quintela wrote: > > > >> This is a resent with what we agreed on yesterday call. > >> Migration events would be there for 0.13 until we get proper > >> async command

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-10 Thread Juan Quintela
Luiz Capitulino wrote: > On Wed, 9 Jun 2010 14:10:53 +0200 > Juan Quintela wrote: > >> This is a resent with what we agreed on yesterday call. >> Migration events would be there for 0.13 until we get proper >> async command support. > > Something which is not clear to me is the set of events we

Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-09 Thread Yoshiaki Tamura
2010/6/10 Juan Quintela : > Yoshiaki Tamura wrote: >> Hi Juan, >> > >> I don't know I have addressed the problem correctly, but here is my >> try to get "info migrate" on incoming side. >> >> http://lists.gnu.org/archive/html/qemu-devel/2010-06/msg00417.html > > I saw it, haven't had the time to l

[Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events

2010-06-09 Thread Juan Quintela
Yoshiaki Tamura wrote: > Hi Juan, > > I don't know I have addressed the problem correctly, but here is my > try to get "info migrate" on incoming side. > > http://lists.gnu.org/archive/html/qemu-devel/2010-06/msg00417.html I saw it, haven't had the time to look at it yet. > Apart from that I ha