From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id BE13D1B246 for ; Tue, 31 Oct 2017 13:03:44 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 31 Oct 2017 05:03:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,323,1505804400"; d="scan'208";a="1031602173" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.32]) by orsmga003.jf.intel.com with SMTP; 31 Oct 2017 05:03:39 -0700 Received: by (sSMTP sendmail emulation); Tue, 31 Oct 2017 12:03:39 +0000 Date: Tue, 31 Oct 2017 12:03:38 +0000 From: Bruce Richardson To: Ferruh Yigit Cc: Anatoly Burakov , dev@dpdk.org, Jerin Jacob , Gaetan Rivet Message-ID: <20171031120338.GA6052@bricha3-MOBL3.ger.corp.intel.com> References: <8492fe4cf1a8d6f15d389c9148e4942f1d060a40.1509381300.git.gaetan.rivet@6wind.com> <20171030223227.8852-1-ferruh.yigit@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171030223227.8852-1-ferruh.yigit@intel.com> Organization: Intel Research and Development Ireland Ltd. User-Agent: Mutt/1.9.1 (2017-09-22) 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 12:03:46 -0000 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. Thanks for the work on this, all. Tested-by: Bruce Richardson