From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id CA408C5A2 for ; Fri, 29 Jan 2016 08:33:44 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga101.fm.intel.com with ESMTP; 28 Jan 2016 23:33:44 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,362,1449561600"; d="scan'208";a="891688442" Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.66.49]) by fmsmga001.fm.intel.com with ESMTP; 28 Jan 2016 23:33:42 -0800 Date: Fri, 29 Jan 2016 15:34:29 +0800 From: Yuanhan Liu To: Santosh Shukla Message-ID: <20160129073429.GS4257@yliu-dev.sh.intel.com> References: <1453203972-24855-1-git-send-email-sshukla@mvista.com> <1453203972-24855-10-git-send-email-sshukla@mvista.com> <20160129071717.GQ4257@yliu-dev.sh.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v5 09/11] virtio_pci: do not parse if interface is vfio-noiommu 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: Fri, 29 Jan 2016 07:33:45 -0000 On Fri, Jan 29, 2016 at 12:52:53PM +0530, Santosh Shukla wrote: > On Fri, Jan 29, 2016 at 12:47 PM, Yuanhan Liu > wrote: > > Two minor nits. > > > > Firstly about your title, you should be consistent: sometimes you > > use virtio_pci, and sometimes you use virtio_pic.h. And for virtio > > pmd driver, "virtio: " prefix is pretty enough, no need another > > extra "vfio: " or "pci: " prefix. > > > > And the same to your EAL changes. EAL is a bigger, having more > > components, thus sometimes 2 prefixs are used. And if you are > > not sure how to add prefix, dig the git history to get the answer. > > > > On Tue, Jan 19, 2016 at 05:16:10PM +0530, Santosh Shukla wrote: > >> If virtio interface attached to vfio-noiommu driver then > >> do not parse for virtio resource. Instead exit with return 0; > >> > >> Note: Applicable for virtio spec 0.95. > > > > And this is not necessary: io port stuff is for virtio 0.95 only. > > virtio 1.0 won't use that, at all. > > > > Okay, > > I removed [08/11] patch from v5 series and modified this patch > accordingly [1]. So, ignore this patch and pl. review provided link. > > [1] http://dpdk.org/dev/patchwork/patch/10143/ This patch actually looks good to me, despites the two minor nits. Another note is that while sending just one single update patch, it'd be better if you could link it to the old patch, to make it in the same email thread, otherwise, it's difficult for me to notice that single-alone patch: it's easily get lost. There is another option for that: the git scissors option; you could check the git format-patch man page for more detailed info (by searching "scissors" keyword). I'm just not quite sure Thomas like it or not. --yliu