From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 2F143F72 for ; Tue, 22 Dec 2015 08:59:55 +0100 (CET) Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (Postfix) with ESMTPS id AA4A1C0B7E21; Tue, 22 Dec 2015 07:59:53 +0000 (UTC) Received: from pxdev.xzpeter.org (vpn1-6-183.pek2.redhat.com [10.72.6.183]) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id tBM7xaJ9005740 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO); Tue, 22 Dec 2015 02:59:45 -0500 Date: Tue, 22 Dec 2015 15:59:35 +0800 From: Peter Xu To: "Xie, Huawei" Message-ID: <20151222075935.GD7532@pxdev.xzpeter.org> References: <1449027793-30975-1-git-send-email-yuanhan.liu@linux.intel.com> <1450321921-27799-1-git-send-email-yuanhan.liu@linux.intel.com> <1450321921-27799-4-git-send-email-yuanhan.liu@linux.intel.com> <20151222065552.GC7532@pxdev.xzpeter.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) X-Scanned-By: MIMEDefang 2.68 on 10.5.11.27 Cc: "Michael S. Tsirkin" , "dev@dpdk.org" , Victor Kaplansky Subject: Re: [dpdk-dev] [PATCH v2 3/6] vhost: log used vring changes X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Dec 2015 07:59:55 -0000 On Tue, Dec 22, 2015 at 07:07:25AM +0000, Xie, Huawei wrote: > On 12/22/2015 2:56 PM, Peter Xu wrote: > > Got a question here: > > > > I see that we are logging down changes when we are marking > > used_vring. Do we need to log down buffer copy in rte_memcpy() too? > > I am not sure whether I understand it correctly, it seems that this > > is part of DPDK API ops to deliver data to the guest (from, e.g., > > OVS?), when we do rte_memcpy(), we seems to be modifying guest > > memory too. Am I wrong? > > > > Peter > > desc buffer logging isn't included in v1, but in the patch 4 of this > patch set, and actually it is the major work in vhost live migration. Yes, it is. Thanks to point out. Peter > --huawei > > [...] >