qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Jason Wang <jasowang@redhat.com>
To: Zhang Chen <chen.zhang@intel.com>
Cc: qemu-dev <qemu-devel@nongnu.org>, Zhang Chen <zhangckid@gmail.com>
Subject: Re: [PATCH V2 0/7] Latest COLO tree queued patches
Date: Mon, 25 May 2020 10:46:59 +0800	[thread overview]
Message-ID: <ba019b1f-8201-b17b-3a66-b22a945aa213@redhat.com> (raw)
In-Reply-To: <20200522075357.19883-1-chen.zhang@intel.com>


On 2020/5/22 下午3:53, Zhang Chen wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> Hi Jason, this series include latest COLO related patches.
> I have finish basic test and review.
> If no other comments, please check and merge this series.
>
> Derek Su (1):
>    colo-compare: Fix memory leak in packet_enqueue()
>
> Lukas Straub (6):
>    net/colo-compare.c: Create event_bh with the right AioContext
>    chardev/char.c: Use qemu_co_sleep_ns if in coroutine
>    net/colo-compare.c: Fix deadlock in compare_chr_send
>    net/colo-compare.c: Only hexdump packets if tracing is enabled
>    net/colo-compare.c: Check that colo-compare is active
>    net/colo-compare.c: Correct ordering in complete and finalize
>
>   chardev/char.c     |   7 +-
>   net/colo-compare.c | 277 +++++++++++++++++++++++++++++++++------------
>   net/colo.c         |   7 ++
>   net/colo.h         |   1 +
>   net/trace-events   |   1 +
>   5 files changed, 222 insertions(+), 71 deletions(-)Philippe
>

Applied with the following tweaks:

- reword the commit log of patch 2 as  Philippe suggested
- a missing ack from Philippe in patch 7

Thanks



      parent reply	other threads:[~2020-05-25  2:47 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-22  7:53 [PATCH V2 0/7] Latest COLO tree queued patches Zhang Chen
2020-05-22  7:53 ` [PATCH V2 1/7] net/colo-compare.c: Create event_bh with the right AioContext Zhang Chen
2020-05-22  7:53 ` [PATCH V2 2/7] chardev/char.c: Use qemu_co_sleep_ns if in coroutine Zhang Chen
2020-05-22  7:53 ` [PATCH V2 3/7] net/colo-compare.c: Fix deadlock in compare_chr_send Zhang Chen
2020-05-22  7:53 ` [PATCH V2 4/7] net/colo-compare.c: Only hexdump packets if tracing is enabled Zhang Chen
2020-05-22  7:53 ` [PATCH V2 5/7] net/colo-compare.c: Check that colo-compare is active Zhang Chen
2020-05-22  7:53 ` [PATCH V2 6/7] net/colo-compare.c: Correct ordering in complete and finalize Zhang Chen
2020-05-22  7:53 ` [PATCH V2 7/7] colo-compare: Fix memory leak in packet_enqueue() Zhang Chen
2020-05-25  2:46 ` Jason Wang [this message]

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=ba019b1f-8201-b17b-3a66-b22a945aa213@redhat.com \
    --to=jasowang@redhat.com \
    --cc=chen.zhang@intel.com \
    --cc=qemu-devel@nongnu.org \
    --cc=zhangckid@gmail.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).