From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:44431) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gwiZc-0005bW-Ih for qemu-devel@nongnu.org; Thu, 21 Feb 2019 02:15:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gwiZb-0005gj-9a for qemu-devel@nongnu.org; Thu, 21 Feb 2019 02:15:12 -0500 Received: from hqemgate16.nvidia.com ([216.228.121.65]:7583) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gwiZa-0005fn-Nm for qemu-devel@nongnu.org; Thu, 21 Feb 2019 02:15:11 -0500 Date: Wed, 20 Feb 2019 23:10:03 -0800 From: Neo Jia Message-ID: <20190221071003.GA10501@nvidia.com> References: <1550611400-13703-1-git-send-email-kwankhede@nvidia.com> <20190220102252.GB2608@work-vm> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH v3 0/5] Add migration support for VFIO device List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Tian, Kevin" Cc: Kirti Wankhede , "Dr. David Alan Gilbert" , "alex.williamson@redhat.com" , "Yang, Ziye" , "Liu, Changpeng" , "Liu, Yi L" , "mlevitsk@redhat.com" , "eskultet@redhat.com" , "cohuck@redhat.com" , "jonathan.davies@nutanix.com" , "eauger@redhat.com" , "aik@ozlabs.ru" , "pasic@linux.ibm.com" , "felipe@nutanix.com" , "Zhengxiao.zx@alibaba-inc.com" , "shuangtai.tst@alibaba-inc.com" , "Ken.Xue@amd.com" , "Wang, Zhi A" , "Zhao, Yan Y" , "yulei.zhang@intel.com" , "qemu-devel@nongnu.org" On Thu, Feb 21, 2019 at 05:52:53AM +0000, Tian, Kevin wrote: > > From: Kirti Wankhede [mailto:kwankhede@nvidia.com] > > Sent: Thursday, February 21, 2019 1:25 PM > > > > On 2/20/2019 3:52 PM, Dr. David Alan Gilbert wrote: > > > * Kirti Wankhede (kwankhede@nvidia.com) wrote: > > >> Add migration support for VFIO device > > > > > > Hi Kirti, > > > Can you explain how this compares and works with Yan Zhao's > > > set? > > > > This patch set is incremental version of my previous patch set: > > https://patchwork.ozlabs.org/cover/1000719/ > > This takes care of the feedbacks received on previous version. > > > > This patch set is different than Yan Zhao's set. > > > > I can help give some background about Yan's work: > > There was a big gap between Kirti's last version and the overall review > comments, especially this one: > https://www.mail-archive.com/qemu-devel@nongnu.org/msg576652.html Hi Kevin, > > Then there was no reply from Kirti whether she agreed with the comments > and was working on a new version. Sorry, we should ack on those comments when we have received them last time. > > Then we think we should jump in to keep the ball moving, based on > a fresh implementation according to recommended direction, i.e. focusing > on device state management instead of sticking to migration flow in kernel > API design. > > and also more importantly we provided kernel side implementation based > on Intel GVT-g to give the whole picture of both user/kernel side changes. > That should give people a better understanding of how those new APIs > are expected to be used by Qemu, and to be implemented by vendor driver. > > That is why Yan just shared her work. Really glad to see the v2 version works for you guys, appreciate for the driver side changes. > > Now it's great to see that Kirti is still actively working on this effort and is > also moving toward the right direction. Let's have a close look at two > implementations and then choose a cleaner one as base for future > enhancements. :-) Yes, the v3 has addressed all the comments / concerns raised in the v2, I think we should take a look and keep moving. Just a quick thought - would be possible / better to have Kirti focus on the QEMU patches and Yan take care GVT-g kernel driver side changes? This will give us the best testing coverage. Hope I don't step on anybody's toes here. ;-) Thanks, Neo > > Thanks > Kevin