[v10] Add new qmp commands to suppurt Xen COLO

[Qemu-devel] [PATCH V10 0/2] Add new qmp commands to suppurt Xen COLO

Posted by Zhang Chen 7 years, 2 months ago

Xen COLO depend on qemu COLO replication function.
So, We need new qmp commands for Xen to use qemu replication.

Corresponding libxl patches already in xen.git.
Commit ID:

ed37ef1f91c20f0ab162ce60f8c38400b917fa64
COLO: introduce new API to prepare/start/do/get_error/stop replication

a0ddc0b359375b2418213966dfbdbfab593ecc6f
tools/libxl: introduction of libxl__qmp_restore to load qemu state

V10:
   - Fix ReplicationStatus error comments.
   - Fix query-xen-replication-status example.

Zhang Chen (2):
  Add a new qmp command to start/stop replication
  Add a new qmp command to do checkpoint, query xen replication status

 migration/colo.c | 49 +++++++++++++++++++++++++++++++++++++
 qapi-schema.json | 73 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 122 insertions(+)

-- 
2.7.4

Re: [Qemu-devel] [PATCH V10 0/2] Add new qmp commands to suppurt Xen COLO

Posted by Stefano Stabellini 7 years, 1 month ago

Eric, are you OK with this series going upstream? If so, do you want me
to send the pull request for it or are you going to handle it?

On Sat, 25 Feb 2017, Zhang Chen wrote:
> Xen COLO depend on qemu COLO replication function.
> So, We need new qmp commands for Xen to use qemu replication.
> 
> Corresponding libxl patches already in xen.git.
> Commit ID:
> 
> ed37ef1f91c20f0ab162ce60f8c38400b917fa64
> COLO: introduce new API to prepare/start/do/get_error/stop replication
> 
> a0ddc0b359375b2418213966dfbdbfab593ecc6f
> tools/libxl: introduction of libxl__qmp_restore to load qemu state
> 
> V10:
>    - Fix ReplicationStatus error comments.
>    - Fix query-xen-replication-status example.
> 
> Zhang Chen (2):
>   Add a new qmp command to start/stop replication
>   Add a new qmp command to do checkpoint, query xen replication status
> 
>  migration/colo.c | 49 +++++++++++++++++++++++++++++++++++++
>  qapi-schema.json | 73 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 122 insertions(+)
> 
> -- 
> 2.7.4
> 
> 
>

Re: [Qemu-devel] [PATCH V10 0/2] Add new qmp commands to suppurt Xen COLO

Posted by Eric Blake 7 years, 1 month ago

On 02/27/2017 04:31 PM, Stefano Stabellini wrote:
> Eric, are you OK with this series going upstream? If so, do you want me
> to send the pull request for it or are you going to handle it?

Both patches have my R-b, but MAINTAINERS suggests the pull request
should go through COLO Framework (zhanghailiang) or maybe Xen (you).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Re: [Qemu-devel] [PATCH V10 0/2] Add new qmp commands to suppurt Xen COLO

Posted by Stefano Stabellini 7 years, 1 month ago

On Mon, 27 Feb 2017, Eric Blake wrote:
> On 02/27/2017 04:31 PM, Stefano Stabellini wrote:
> > Eric, are you OK with this series going upstream? If so, do you want me
> > to send the pull request for it or are you going to handle it?
> 
> Both patches have my R-b, but MAINTAINERS suggests the pull request
> should go through COLO Framework (zhanghailiang) or maybe Xen (you).

I am fine either way. zhanghailiang?

Re: [Qemu-devel] [PATCH V10 0/2] Add new qmp commands to suppurt Xen COLO

Posted by Hailiang Zhang 7 years, 1 month ago

On 2017/2/28 6:52, Stefano Stabellini wrote:
> On Mon, 27 Feb 2017, Eric Blake wrote:
>> On 02/27/2017 04:31 PM, Stefano Stabellini wrote:
>>> Eric, are you OK with this series going upstream? If so, do you want me
>>> to send the pull request for it or are you going to handle it?
>>
>> Both patches have my R-b, but MAINTAINERS suggests the pull request
>> should go through COLO Framework (zhanghailiang) or maybe Xen (you).
>
> I am fine either way. zhanghailiang?
>

Yes, I'm fine if you could give a pull request for this series.

Thanks.

> .
>