[Qemu-devel] [PATCH V3 0/3] COLO-compare: Make COLO-compare support remote COLO-frame

Zhang Chen posted 3 patches 6 years, 10 months ago
Failed in applying to current master (apply log)
net/colo-compare.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
qemu-options.hx    | 41 ++++++++++++++++++++----
2 files changed, 124 insertions(+), 8 deletions(-)
[Qemu-devel] [PATCH V3 0/3] COLO-compare: Make COLO-compare support remote COLO-frame
Posted by Zhang Chen 6 years, 10 months ago
This series focus on COLO-proxy remote colo-frame support.
Xen COLO-frame is the first user. We add a new chardev socket
in colo-compare as the way of communicate with remote COLO-frame.
And remote COLO-frame notify colo-proxy part depend on this serise:
https://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg03904.html

I will send another part of this series after depend patchset have
been merged.

V3:
 - Fix codestyle.

V2:
 - Rename this series.
 - Change communication way to remote colo-frame.
 - Some bugfix.
 - Split the main function, anther part wait depend patchset.


Zhang Chen (3):
  COLO-compare: Add new parameter for communicate with remote colo-frame
  COLO-compare: Add remote checkpoint notify chardev socket handler
    frame
  COLO-compare: Add remote initialization and checkpoint notification

 net/colo-compare.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
 qemu-options.hx    | 41 ++++++++++++++++++++----
 2 files changed, 124 insertions(+), 8 deletions(-)

-- 
2.7.4




Re: [Qemu-devel] [PATCH V3 0/3] COLO-compare: Make COLO-compare support remote COLO-frame
Posted by Dr. David Alan Gilbert 6 years, 10 months ago
* Zhang Chen (zhangchen.fnst@cn.fujitsu.com) wrote:
> This series focus on COLO-proxy remote colo-frame support.
> Xen COLO-frame is the first user. We add a new chardev socket
> in colo-compare as the way of communicate with remote COLO-frame.
> And remote COLO-frame notify colo-proxy part depend on this serise:
> https://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg03904.html

Can you explain a bit more about how the 'remote colo-frame' works?
Is this the comparison that's separate or what?

Dave

> I will send another part of this series after depend patchset have
> been merged.
> 
> V3:
>  - Fix codestyle.
> 
> V2:
>  - Rename this series.
>  - Change communication way to remote colo-frame.
>  - Some bugfix.
>  - Split the main function, anther part wait depend patchset.
> 
> 
> Zhang Chen (3):
>   COLO-compare: Add new parameter for communicate with remote colo-frame
>   COLO-compare: Add remote checkpoint notify chardev socket handler
>     frame
>   COLO-compare: Add remote initialization and checkpoint notification
> 
>  net/colo-compare.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
>  qemu-options.hx    | 41 ++++++++++++++++++++----
>  2 files changed, 124 insertions(+), 8 deletions(-)
> 
> -- 
> 2.7.4
> 
> 
> 
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK

Re: [Qemu-devel] [PATCH V3 0/3] COLO-compare: Make COLO-compare support remote COLO-frame
Posted by Zhang Chen 6 years, 10 months ago

On 06/29/2017 06:36 PM, Dr. David Alan Gilbert wrote:
> * Zhang Chen (zhangchen.fnst@cn.fujitsu.com) wrote:
>> This series focus on COLO-proxy remote colo-frame support.
>> Xen COLO-frame is the first user. We add a new chardev socket
>> in colo-compare as the way of communicate with remote COLO-frame.
>> And remote COLO-frame notify colo-proxy part depend on this serise:
>> https://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg03904.html
> Can you explain a bit more about how the 'remote colo-frame' works?
> Is this the comparison that's separate or what?

COLO consists of three modules, colo-frame, colo-proxy, block-replication.
If we use qemu/kvm COLO, the three parts all in qemu side.
If we use Xen COLO, the colo-frame in Xen side(the remote COLO-frame),
and colo-proxy,block-replication in qemu side. So, in this case,
remote colo-frame need use qmp command and socket connect to 
block-replication
and colo-proxy finish COLO job.
KVM COLO and Xen COLO use the same colo-proxy to compare net packet.

Thanks
Zhang Chen

>
> Dave
>
>> I will send another part of this series after depend patchset have
>> been merged.
>>
>> V3:
>>   - Fix codestyle.
>>
>> V2:
>>   - Rename this series.
>>   - Change communication way to remote colo-frame.
>>   - Some bugfix.
>>   - Split the main function, anther part wait depend patchset.
>>
>>
>> Zhang Chen (3):
>>    COLO-compare: Add new parameter for communicate with remote colo-frame
>>    COLO-compare: Add remote checkpoint notify chardev socket handler
>>      frame
>>    COLO-compare: Add remote initialization and checkpoint notification
>>
>>   net/colo-compare.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
>>   qemu-options.hx    | 41 ++++++++++++++++++++----
>>   2 files changed, 124 insertions(+), 8 deletions(-)
>>
>> -- 
>> 2.7.4
>>
>>
>>
>>
> --
> Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
>
>
>

-- 
Thanks
Zhang Chen




Re: [Qemu-devel] [PATCH V3 0/3] COLO-compare: Make COLO-compare support remote COLO-frame
Posted by Jason Wang 6 years, 10 months ago

On 2017年06月06日 16:12, Zhang Chen wrote:
> This series focus on COLO-proxy remote colo-frame support.
> Xen COLO-frame is the first user. We add a new chardev socket
> in colo-compare as the way of communicate with remote COLO-frame.
> And remote COLO-frame notify colo-proxy part depend on this serise:
> https://lists.nongnu.org/archive/html/qemu-devel/2017-04/msg03904.html
>
> I will send another part of this series after depend patchset have
> been merged.
>
> V3:
>   - Fix codestyle.
>
> V2:
>   - Rename this series.
>   - Change communication way to remote colo-frame.
>   - Some bugfix.
>   - Split the main function, anther part wait depend patchset.
>
>
> Zhang Chen (3):
>    COLO-compare: Add new parameter for communicate with remote colo-frame
>    COLO-compare: Add remote checkpoint notify chardev socket handler
>      frame
>    COLO-compare: Add remote initialization and checkpoint notification
>
>   net/colo-compare.c | 91 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
>   qemu-options.hx    | 41 ++++++++++++++++++++----
>   2 files changed, 124 insertions(+), 8 deletions(-)
>

Thanks for the series.

To speed up things, I would like to see IOThread conversion first. Then 
we can have this on top.