From nobody Sat Nov 1 07:47:13 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) client-ip=208.118.235.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zoho.com: domain of gnu.org designates 208.118.235.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=gmail.com Return-Path: Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) by mx.zohomail.com with SMTPS id 152631773934381.17746514969292; Mon, 14 May 2018 10:08:59 -0700 (PDT) Received: from localhost ([::1]:47328 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fIGy2-0003NL-Es for importer@patchew.org; Mon, 14 May 2018 13:08:58 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33678) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fIGlR-00005F-42 for qemu-devel@nongnu.org; Mon, 14 May 2018 12:55:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fIGlM-00062s-9p for qemu-devel@nongnu.org; Mon, 14 May 2018 12:55:57 -0400 Received: from mail-pl0-x22a.google.com ([2607:f8b0:400e:c01::22a]:46484) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fIGlM-00062b-3K for qemu-devel@nongnu.org; Mon, 14 May 2018 12:55:52 -0400 Received: by mail-pl0-x22a.google.com with SMTP id 30-v6so2871319pld.13 for ; Mon, 14 May 2018 09:55:51 -0700 (PDT) Received: from localhost.localdomain (109.227.199.104.bc.googleusercontent.com. [104.199.227.109]) by smtp.gmail.com with ESMTPSA id u69-v6sm631448pgb.42.2018.05.14.09.55.46 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 14 May 2018 09:55:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=RDzK/IZdtuxv+giKDkRCJEMEUmWV17NczqputA7O5qY=; b=sXKsiAvHDlzXlLeUeoIaqy4D9XjKRqKUNEci7ahMytJTWRBo2gBwUUz5ZoeJX4bBkZ kopekERu4Ti7LFrORo6421ndAyYqkdwNGZJOCwwhiuNnAQbOwbqSf+w51O1nq6t7hKHN wQwcuyBkrfg18ZilnoB8DR9FAvZLQZndQnyxFLViIh97r/dsk0Xgo6F2O6Qxu3rOArSZ hAqr8Vm39ECFzsv0lLYPitdke31rK0siehWgpvwwNlAPaOhXdDuz6rTKcC4wu5rdAToD pwnxB/lovGYauJzwaQq9zqdZT+EiuIHvh2ZDiQ4uj6gB9/7nu1hpcA6UJlXQu1ppdKdf ZLjg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=RDzK/IZdtuxv+giKDkRCJEMEUmWV17NczqputA7O5qY=; b=iwTxv14U0pRSE4dJ4FG17pE73dURGVyW/VJ9EyhmJvICnziJF3kSPReNh/N5oKrN+u nLgQsmnh89OLbgzpkslnp7a1T4mfuNxZxEy6pRAN8YAT9/4mtpJg7h/ecYhOvsuF+shB IL5LzKnNM+q95zTPHgm+7/Fn3yal+oyAhurrACN/TvXlonjMMzGiJo40HcR9Q872TP8H jCfCzu+uBYa5iJwMa2oTMYWacXy3bgwPhQdFtHoEpVoNqepikSNhcQBrfrqr4ycitRYd +liSaR0TTYF3bX5sVlfUKy8fP8vzO+avpqiaEMalEjdxx5Lb4AigHD5tgeIhT9fkl00S oKFQ== X-Gm-Message-State: ALKqPweXIoQhLnTbipLTQkpv1bJeXjliRzpUMLvC/10GcIwmC1a2OYRW 0/H+8Yla6qlK+zuz4/Wm0szYJQ== X-Google-Smtp-Source: AB8JxZoAkCr6mr0GdI/hePw8GPuy2gBtCkC1F2Wnubmb6rYsU5oRFCIXWX96Wr3DO+u0fQGxzGQ/vw== X-Received: by 2002:a17:902:683:: with SMTP id 3-v6mr7270174plh.291.1526316950750; Mon, 14 May 2018 09:55:50 -0700 (PDT) From: Zhang Chen To: qemu-devel@nongnu.org Date: Tue, 15 May 2018 00:54:22 +0800 Message-Id: <20180514165424.12884-16-zhangckid@gmail.com> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180514165424.12884-1-zhangckid@gmail.com> References: <20180514165424.12884-1-zhangckid@gmail.com> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:400e:c01::22a Subject: [Qemu-devel] [PATCH V7 RESEND 15/17] filter-rewriter: handle checkpoint and failover event X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: zhanghailiang , Jason Wang , Markus Armbruster , "Dr . David Alan Gilbert" , Zhang Chen , Paolo Bonzini Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_0 Z_629925259 SPT_0 Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" After one round of checkpoint, the states between PVM and SVM become consistent, so it is unnecessary to adjust the sequence of net packets for old connections, besides, while failover happens, filter-rewriter needs to check if it still needs to adjust sequence of net packets. Signed-off-by: zhanghailiang Signed-off-by: Zhang Chen --- migration/colo.c | 13 +++++++++++++ net/filter-rewriter.c | 40 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 53 insertions(+) diff --git a/migration/colo.c b/migration/colo.c index 5b055f79f1..3dfd84d897 100644 --- a/migration/colo.c +++ b/migration/colo.c @@ -31,6 +31,7 @@ #include "qapi/qapi-events-migration.h" #include "qapi/qmp/qerror.h" #include "sysemu/cpus.h" +#include "net/filter.h" =20 static bool vmstate_loading; static Notifier packets_compare_notifier; @@ -82,6 +83,11 @@ static void secondary_vm_do_failover(void) if (local_err) { error_report_err(local_err); } + /* Notify all filters of all NIC to do checkpoint */ + colo_notify_filters_event(COLO_EVENT_FAILOVER, &local_err); + if (local_err) { + error_report_err(local_err); + } =20 if (!autostart) { error_report("\"-S\" qemu option will be ignored in secondary side= "); @@ -786,6 +792,13 @@ void *colo_process_incoming_thread(void *opaque) goto out; } =20 + /* Notify all filters of all NIC to do checkpoint */ + colo_notify_filters_event(COLO_EVENT_CHECKPOINT, &local_err); + if (local_err) { + qemu_mutex_unlock_iothread(); + goto out; + } + vmstate_loading =3D false; vm_start(); trace_colo_vm_state_change("stop", "run"); diff --git a/net/filter-rewriter.c b/net/filter-rewriter.c index 0909a9a8af..f3c306cc89 100644 --- a/net/filter-rewriter.c +++ b/net/filter-rewriter.c @@ -20,6 +20,8 @@ #include "qemu/main-loop.h" #include "qemu/iov.h" #include "net/checksum.h" +#include "net/colo.h" +#include "migration/colo.h" =20 #define FILTER_COLO_REWRITER(obj) \ OBJECT_CHECK(RewriterState, (obj), TYPE_FILTER_REWRITER) @@ -277,6 +279,43 @@ static ssize_t colo_rewriter_receive_iov(NetFilterStat= e *nf, return 0; } =20 +static void reset_seq_offset(gpointer key, gpointer value, gpointer user_d= ata) +{ + Connection *conn =3D (Connection *)value; + + conn->offset =3D 0; +} + +static gboolean offset_is_nonzero(gpointer key, + gpointer value, + gpointer user_data) +{ + Connection *conn =3D (Connection *)value; + + return conn->offset ? true : false; +} + +static void colo_rewriter_handle_event(NetFilterState *nf, int event, + Error **errp) +{ + RewriterState *rs =3D FILTER_COLO_REWRITER(nf); + + switch (event) { + case COLO_EVENT_CHECKPOINT: + g_hash_table_foreach(rs->connection_track_table, + reset_seq_offset, NULL); + break; + case COLO_EVENT_FAILOVER: + if (!g_hash_table_find(rs->connection_track_table, + offset_is_nonzero, NULL)) { + object_property_set_str(OBJECT(nf), "off", "status", errp); + } + break; + default: + break; + } +} + static void colo_rewriter_cleanup(NetFilterState *nf) { RewriterState *s =3D FILTER_COLO_REWRITER(nf); @@ -332,6 +371,7 @@ static void colo_rewriter_class_init(ObjectClass *oc, v= oid *data) nfc->setup =3D colo_rewriter_setup; nfc->cleanup =3D colo_rewriter_cleanup; nfc->receive_iov =3D colo_rewriter_receive_iov; + nfc->handle_event =3D colo_rewriter_handle_event; } =20 static const TypeInfo colo_rewriter_info =3D { --=20 2.17.0