From nobody Mon May 20 14:38:34 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=linux.intel.com ARC-Seal: i=1; a=rsa-sha256; t=1560127464; cv=none; d=zoho.com; s=zohoarc; b=U/zRtR8INF7rqVs54vlmohO7FFX9K1IO6ztlCEyH/XRJIjLtZu5BYzOXOf0HV01NJdMvUqKGLjXvV4mASeaekKEHKpdawkSk0XejyDgniDQhhxcAClMwNghZ3PtEkIsxIevzGJApn+QO0nvGEWvl97c8Kcp0PDodQqZtvz0GXbQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1560127464; h=Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To:ARC-Authentication-Results; bh=Vb3XbcO8ZYy0YzjjykZKHNYp83QHGAozL9J0QQeGuUY=; b=Ztls3IJhlVyGEuCkd0/Fc9L0NhpBGb/FXpsTT7NHL5KdI1Tu38uj/EbRWAMSIjUoI6r/N6TosL5l0ceNvkpqDIoOdMJ0quMwYhun2y4Wqr+WZ76Mj0s+C4hTQ92DQAMPRMeUqpZCRl2hBhH1n5vWYfqB0aXXdCtSAwTwNM8ugaA= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (209.51.188.17 [209.51.188.17]) by mx.zohomail.com with SMTPS id 1560127464652402.662151544102; Sun, 9 Jun 2019 17:44:24 -0700 (PDT) Received: from localhost ([::1]:39310 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1ha8Q2-0004nV-8w for importer@patchew.org; Sun, 09 Jun 2019 20:44:14 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:39475) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1ha8OR-0003jD-ER for qemu-devel@nongnu.org; Sun, 09 Jun 2019 20:42:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ha8OQ-0002mF-DI for qemu-devel@nongnu.org; Sun, 09 Jun 2019 20:42:35 -0400 Received: from mga18.intel.com ([134.134.136.126]:55869) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1ha8OQ-0002ji-4b for qemu-devel@nongnu.org; Sun, 09 Jun 2019 20:42:34 -0400 Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 09 Jun 2019 17:42:27 -0700 Received: from richard.sh.intel.com (HELO localhost) ([10.239.159.54]) by fmsmga002.fm.intel.com with ESMTP; 09 Jun 2019 17:42:26 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.63,573,1557212400"; d="scan'208";a="183246801" From: Wei Yang To: qemu-devel@nongnu.org Date: Mon, 10 Jun 2019 08:41:59 +0800 Message-Id: <20190610004159.20966-1-richardw.yang@linux.intel.com> X-Mailer: git-send-email 2.19.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 134.134.136.126 Subject: [Qemu-devel] [PATCH v2] migration/xbzrle: update cache and current_data in one place X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Wei Yang , dgilbert@redhat.com, quintela@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" When we are not in the last_stage, we need to update the cache if page is not the same. Currently this procedure is scattered in two places and mixed with encoding status check. This patch extract this general step out to make the code a little bit easy to read. Signed-off-by: Wei Yang Reviewed-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela --- v2: give more comment on the behavior --- migration/ram.c | 25 +++++++++++++++---------- 1 file changed, 15 insertions(+), 10 deletions(-) diff --git a/migration/ram.c b/migration/ram.c index e9b40d636d..17cc9b2b44 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -1594,25 +1594,30 @@ static int save_xbzrle_page(RAMState *rs, uint8_t *= *current_data, encoded_len =3D xbzrle_encode_buffer(prev_cached_page, XBZRLE.current_= buf, TARGET_PAGE_SIZE, XBZRLE.encoded_bu= f, TARGET_PAGE_SIZE); + + /* + * Update the cache contents, so that it corresponds to the data + * sent, in all cases except where we skip the page. + */ + if (!last_stage && encoded_len !=3D 0) { + memcpy(prev_cached_page, XBZRLE.current_buf, TARGET_PAGE_SIZE); + /* + * In the case where we couldn't compress, ensure that the caller + * sends the data from the cache, since the guest might have + * changed the RAM since we copied it. + */ + *current_data =3D prev_cached_page; + } + if (encoded_len =3D=3D 0) { trace_save_xbzrle_page_skipping(); return 0; } else if (encoded_len =3D=3D -1) { trace_save_xbzrle_page_overflow(); xbzrle_counters.overflow++; - /* update data in the cache */ - if (!last_stage) { - memcpy(prev_cached_page, *current_data, TARGET_PAGE_SIZE); - *current_data =3D prev_cached_page; - } return -1; } =20 - /* we need to update the data in the cache, in order to get the same d= ata */ - if (!last_stage) { - memcpy(prev_cached_page, XBZRLE.current_buf, TARGET_PAGE_SIZE); - } - /* Send XBZRLE based compressed page */ bytes_xbzrle =3D save_page_header(rs, rs->f, block, offset | RAM_SAVE_FLAG_XBZRLE); --=20 2.19.1