qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Jason Wang <jasowang@redhat.com>
To: Zhang Chen <zhangchen.fnst@cn.fujitsu.com>,
	qemu devel <qemu-devel@nongnu.org>
Cc: Li Zhijian <lizhijian@cn.fujitsu.com>,
	Wen Congyang <wency@cn.fujitsu.com>,
	zhanghailiang <zhang.zhanghailiang@huawei.com>,
	"eddie . dong" <eddie.dong@intel.com>,
	"Dr . David Alan Gilbert" <dgilbert@redhat.com>
Subject: Re: [Qemu-devel] [PATCH V15 00/12] Introduce COLO-Proxy
Date: Tue, 27 Sep 2016 16:38:18 +0800	[thread overview]
Message-ID: <4ee8b552-d55b-a8a4-ba63-6b7055443dc9@redhat.com> (raw)
In-Reply-To: <1474942956-27182-1-git-send-email-zhangchen.fnst@cn.fujitsu.com>



On 2016年09月27日 10:22, Zhang Chen wrote:
> COLO-proxy is a part of COLO project. COLO project is
> composed of COLO-frame, COLO-proxy and block-replication.
> It is used to compare the network package to help COLO
> decide whether to do checkpoint. With COLO-proxy's help,
> COLO greatly improves the performance.
>
> The filter-redirector, filter-mirror, colo-compare
> and filter-rewriter compose the COLO-proxy.
>
> COLO-compare
> It is used to compare the network package to help COLO decide
> whether to do checkpoint.
>
> Filter-rewriter
> It will rewrite some of secondary packet to make
> secondary guest's connection established successfully.
> In this module we will rewrite tcp packet's ack to the secondary
> from primary,and rewrite tcp packet's seq to the primary from
> secondary.
>
> The full version in this github:
> https://github.com/zhangckid/qemu/tree/colo-v2.7-proxy-mode-compare-and-rewriter-sep27
>
> v15:
>    - change "ConnectionKey key = { 0 };" to
>      "ConnectionKey key = {{0},};", fix typo.

This causes  build failure for mingw@fedora docker test.

I tend to fix by not using the initializer but call a memset() in 
fill_connection_key().

If you agree, I will fix it in my tree and no need to repost a new version.

  parent reply	other threads:[~2016-09-27  8:38 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-27  2:22 [Qemu-devel] [PATCH V15 00/12] Introduce COLO-Proxy Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 01/12] qemu-char: Add qemu_chr_add_handlers_full() for GMaincontext Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 02/12] colo-compare: introduce colo compare initialization Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 03/12] net/colo.c: add colo.c to define and handle packet Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 04/12] Jhash: add linux kernel jhashtable in qemu Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 05/12] colo-compare: track connection and enqueue packet Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 06/12] colo-compare: introduce packet comparison thread Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 07/12] colo-compare: add TCP, UDP, ICMP packet comparison Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 08/12] filter-rewriter: introduce filter-rewriter initialization Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 09/12] filter-rewriter: track connection and parse packet Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 10/12] filter-rewriter: rewrite tcp packet to keep secondary connection Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 11/12] MAINTAINERS: add maintainer for COLO-proxy Zhang Chen
2016-09-27  2:22 ` [Qemu-devel] [PATCH V15 12/12] docs: Add documentation " Zhang Chen
2016-09-27  8:38 ` Jason Wang [this message]
2016-09-27  8:45   ` [Qemu-devel] [PATCH V15 00/12] Introduce COLO-Proxy Zhang Chen
2016-09-30 14:04 ` Paolo Bonzini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4ee8b552-d55b-a8a4-ba63-6b7055443dc9@redhat.com \
    --to=jasowang@redhat.com \
    --cc=dgilbert@redhat.com \
    --cc=eddie.dong@intel.com \
    --cc=lizhijian@cn.fujitsu.com \
    --cc=qemu-devel@nongnu.org \
    --cc=wency@cn.fujitsu.com \
    --cc=zhang.zhanghailiang@huawei.com \
    --cc=zhangchen.fnst@cn.fujitsu.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).