From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47128) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ceYg1-0001UN-PU for qemu-devel@nongnu.org; Thu, 16 Feb 2017 21:53:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ceYfy-0005Pa-Jd for qemu-devel@nongnu.org; Thu, 16 Feb 2017 21:53:41 -0500 Received: from szxga02-in.huawei.com ([119.145.14.65]:33291) by eggs.gnu.org with esmtps (TLS1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.71) (envelope-from ) id 1ceYfy-0005M9-1E for qemu-devel@nongnu.org; Thu, 16 Feb 2017 21:53:38 -0500 From: zhanghailiang Date: Fri, 17 Feb 2017 10:53:10 +0800 Message-ID: <1487299994-98348-1-git-send-email-zhang.zhanghailiang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset="y" Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PATCH v3 0/4] colo-compare: fix some bugs List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: jasowang@redhat.com, zhangchen.fnst@cn.fujitsu.com, lizhijian@cn.fujitsu.com Cc: qemu-devel@nongnu.org, xuquan8@huawei.com, pss.wulizhen@huawei.com, zhanghailiang This series includes two parts: codes optimization and bug fix. patch 1 tries to move timer process into colo compare thread as a new coroutine. patch 2 ~ 4 fixe some bugs of colo compare. v2->v3: - change the definition of remove_fd_in_watch() instead of introducing a function (Marc-André Lureau's suggestion) v1->v2: - Squash patch 3 of last version into patch 2. (ZhangChen's suggestion) zhanghailiang (4): colo-compare: use g_timeout_source_new() to process the stale packets colo-compare: kick compare thread to exit after some cleanup in finalization char: remove the right fd been watched in qemu_chr_fe_set_handlers() colo-compare: Fix removing fds been watched incorrectly in finalization chardev/char-fd.c | 6 +-- chardev/char-io.c | 8 ++-- chardev/char-io.h | 2 +- chardev/char-pty.c | 2 +- chardev/char-socket.c | 4 +- chardev/char-udp.c | 6 +-- chardev/char.c | 2 +- net/colo-compare.c | 115 ++++++++++++++++++++++++++------------------------ 8 files changed, 74 insertions(+), 71 deletions(-) -- 1.8.3.1