From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 5D1212BBB for ; Fri, 26 Feb 2016 03:23:39 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga104.fm.intel.com with ESMTP; 25 Feb 2016 18:23:38 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,498,1449561600"; d="scan'208";a="911532598" Received: from yliu-dev.sh.intel.com (HELO yliu-dev) ([10.239.66.49]) by fmsmga001.fm.intel.com with ESMTP; 25 Feb 2016 18:23:38 -0800 Date: Fri, 26 Feb 2016 10:23:58 +0800 From: Yuanhan Liu To: David Marchand Message-ID: <20160226022358.GG14300@yliu-dev.sh.intel.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] virtio PMD is not working with master version 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, 26 Feb 2016 02:23:39 -0000 Mauricio, thanks for the testing and report. On Thu, Feb 25, 2016 at 02:30:18PM +0100, David Marchand wrote: > On Thu, Feb 25, 2016 at 12:30 PM, Mauricio Vásquez > wrote: > > ... > > EAL: PCI device 0000:00:04.0 on NUMA socket -1 > > EAL: probe driver: 1af4:1000 rte_virtio_pmd > > EAL: PCI memory mapped at 0x7f892dc00000 > > PMD: virtio_read_caps(): [40] skipping non VNDR cap id: 11 > > PMD: virtio_read_caps(): no modern virtio pci device found. > > PMD: vtpci_init(): trying with legacy virtio pci. > > EAL: eal_parse_sysfs_value(): cannot open sysfs value > > /sys/bus/pci/devices/0000:00:04.0/uio/uio0/portio/port0/start > > EAL: pci_uio_ioport_map(): cannot parse portio start > > EAL: Error - exiting with code: 1 > > Cause: Requested device 0000:00:04.0 cannot be used > > ... > > [snip] > > > ... > > PMD: parse_sysfs_value(): parse_sysfs_value(): cannot open sysfs value > > /sys/bus/pci/devices/0000:00:04.0/uio/uio0/portio/port0/size > > PMD: virtio_resource_init_by_uio(): virtio_resource_init_by_uio(): cannot > > parse size > > PMD: virtio_resource_init_by_ioports(): PCI Port IO found start=0xc100 with > > size=0x20 > > [snip] > > > > > According to git bisect it appears to be that it does not work anymore after > > the b8f04520ad71 ("virtio: use PCI ioport API") commit. > > >From the logs, I would say I broke uio_pci_generic since we are in > "uio" case, but uio portio sysfs does not exist. > virtio pmd fell back to ioports discovery before my change. Maybe we can do same? --- diff --git a/lib/librte_eal/linuxapp/eal/eal_pci.c b/lib/librte_eal/linuxapp/eal/eal_pci.c index 4346973..579731c 100644 --- a/lib/librte_eal/linuxapp/eal/eal_pci.c +++ b/lib/librte_eal/linuxapp/eal/eal_pci.c @@ -685,12 +685,11 @@ int rte_eal_pci_ioport_map(struct rte_pci_device *dev, int bar, struct rte_pci_ioport *p) { - int ret; + int ret = -1; switch (dev->kdrv) { #ifdef VFIO_PRESENT case RTE_KDRV_VFIO: - ret = -1; if (pci_vfio_is_enabled()) ret = pci_vfio_ioport_map(dev, bar, p); break; @@ -700,14 +699,14 @@ rte_eal_pci_ioport_map(struct rte_pci_device *dev, int bar, ret = pci_uio_ioport_map(dev, bar, p); break; default: + break; + } + #if defined(RTE_ARCH_X86_64) || defined(RTE_ARCH_I686) - /* special case for x86 ... */ + /* special case for x86 ... */ + if (ret) ret = pci_ioport_map(dev, bar, p); -#else - ret = -1; #endif - break; - } if (!ret) p->dev = dev; ------------------------------------------------------- If that looks okay to you, I could send a formal patch. --yliu > Problem can be workaround for now by unbinding your device from > uio_pci_generic or on the contrary bind to igb_uio. > > I've been sick at home, all this week. > Will see next week for a fix unless someone sends one. > > > -- > David Marchand