[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events
From: |
Yoshiaki Tamura |
Subject: |
Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events |
Date: |
Thu, 10 Jun 2010 07:07:10 +0900 |
2010/6/10 Juan Quintela <address@hidden>:
> Yoshiaki Tamura <address@hidden> 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 have a concern on relation between newly introduced
>> QMP Migration events and existing MIG_STATE_*. Are they supposed to
>> be 1 to 1 mapping?
>
> Good question. In my tree I had info migrate being the same on
> source and destination, but I didn't have async migration. I have to
> check with yours.
>
>> If so, instead of calling
>> monitor_protocol_event() everywhere, how about introducing a common
>> function in migration that sets s->mig_state and emits QMP Migration
>> events at once?
>
> I did that locally, just that this is in the middle of a big cleanup.
>
> I am not sure that is still enough. We have:
>
> - qemu status: running/stopped
> - we have another state "incoming" migration that needs to be same level
> than running/stopped
> - I think MIG_STATE_* should also be promotted to this level, but
> haven't fully thought how to do it.
I think this problem isn't limited to migration. We need an central
table that contains the relation between states in qemu and
corresponding QMP events that must be emitted. Otherwise, we can get
into troubles like missing to emit necessary QMP events, or duplicated
QMP events.
Thanks,
Yoshi
>
> Later, Juan.
>
>
- Re: [Qemu-devel] [PATCH] win32: Add define for missing EPROTONOSUPPORT, (continued)
- [Qemu-devel] [PATCH v3 2/5] Factorize common migration incoming code, Juan Quintela, 2010/06/09
- [Qemu-devel] [PATCH v3 4/5] QMP: Emit migration events on incoming migration, Juan Quintela, 2010/06/09
- [Qemu-devel] [PATCH v3 5/5] QMP: Emit migration events on outgoing migration, Juan Quintela, 2010/06/09
- Re: [Qemu-devel] [PATCH v3 0/5] Add QMP migration events, Yoshiaki Tamura, 2010/06/09
- Re: [Qemu-devel] [PATCH v3 0/5] Add QMP migration events, Luiz Capitulino, 2010/06/09
- Re: [Qemu-devel] [PATCH v3 0/5] Add QMP migration events, Yoshiaki Tamura, 2010/06/09
- [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Juan Quintela, 2010/06/10
- [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Luiz Capitulino, 2010/06/11
- Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Anthony Liguori, 2010/06/11
- Re: [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Luiz Capitulino, 2010/06/11
- [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Juan Quintela, 2010/06/12
- [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Luiz Capitulino, 2010/06/14
- [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Juan Quintela, 2010/06/14
- [Qemu-devel] Re: [PATCH v3 0/5] Add QMP migration events, Juan Quintela, 2010/06/12