From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id B8F361B219 for ; Tue, 31 Oct 2017 17:58:30 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 31 Oct 2017 09:58:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,324,1505804400"; d="scan'208";a="915904497" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.241.225.190]) ([10.241.225.190]) by FMSMGA003.fm.intel.com with ESMTP; 31 Oct 2017 09:58:29 -0700 To: Bruce Richardson Cc: Anatoly Burakov , dev@dpdk.org, Jerin Jacob , Gaetan Rivet References: <8492fe4cf1a8d6f15d389c9148e4942f1d060a40.1509381300.git.gaetan.rivet@6wind.com> <20171030223227.8852-1-ferruh.yigit@intel.com> <20171031120338.GA6052@bricha3-MOBL3.ger.corp.intel.com> From: Ferruh Yigit Message-ID: Date: Tue, 31 Oct 2017 09:58:29 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20171031120338.GA6052@bricha3-MOBL3.ger.corp.intel.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v4] bus/pci: fix vfio mode X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 31 Oct 2017 16:58:31 -0000 On 10/31/2017 5:03 AM, Bruce Richardson wrote: > On Mon, Oct 30, 2017 at 10:32:27PM +0000, Ferruh Yigit wrote: >> Revert back to using VFIO_PRESENT as a marker to enable compilation >> of VFIO-related segments. >> >> VFIO_PRESENT is the combination of user configuration RTE_EAL_VFIO and >> kernel version support check. >> >> eal_vfio.h VFIO_PRESENT related check ordered to be compatible with >> rte_vfio.h one, no functional modification. >> >> Fixes: 279b581c897d ("vfio: expose functions") >> >> Signed-off-by: Jerin Jacob >> Signed-off-by: Gaetan Rivet >> Signed-off-by: Ferruh Yigit >> --- >> v4: >> >> * Alternative to v3, not superseding it, need to decide which one to >> get. This one: >> * fixes vfio for new kernels >> * fixes build for old kernels for vfio enabled in config case >> * print warning if vfio enabled in config but not supported by kernel >> >> * Independent from this patch, following may need fixing later >> * Kernel version check for vfio is broken for RTE_KERNELDIR >> * Kernel VFIO support check can be improved >> --- >> drivers/bus/pci/linux/pci_init.h | 2 +- >> drivers/bus/pci/linux/pci_vfio.c | 2 +- >> lib/librte_eal/common/include/rte_vfio.h | 14 ++++++++++++++ >> lib/librte_eal/linuxapp/eal/eal_interrupts.c | 1 - >> lib/librte_eal/linuxapp/eal/eal_vfio.h | 14 ++++++++++---- >> 5 files changed, 26 insertions(+), 7 deletions(-) >> > Hi, > > just to confirm that applying this patch fixes DPDK on my system at > least. > > From the discussion, I understood that this was a "compilation problem", > but in practice it manifests itself as compile working ok but your app > won't work with ports bound to vfio-pci. So please apply some fix for > this soon. The main reason of the patch is vfio issue. All three of patches from Jerin, Gaetan, and this are fixing that issue. Difference is: - Create or not dependency to eal_vfio.h again - Break or not build for old kernels > > Thanks for the work on this, all. > > Tested-by: Bruce Richardson >