From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f174.google.com (mail-wr0-f174.google.com [209.85.128.174]) by dpdk.org (Postfix) with ESMTP id 6F8402C00 for ; Fri, 23 Jun 2017 10:20:02 +0200 (CEST) Received: by mail-wr0-f174.google.com with SMTP id 77so55230516wrb.1 for ; Fri, 23 Jun 2017 01:20:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=fFzjf9NwSuREvtoAbvsN5KOZLnyAEEgseojLyo07d+Y=; b=l2jAHQUGdEAPQJFNLv5x1ZYXfsR55Dtr92buQo0nsL88lc5TJXtbuQwFOq0KlFxr9i QzOgO76d8+mL4qMCd2X/yyDnHeRNKYrNgOdcqEme4fUMcSotZx7SjXQqBTsIMAsxazUz AQQ5r0wqzqt7Ycsp5Q9Rk9Hmz1RdxVHigxCla7GfMcXs1AUva4xt503i6M3YCvFau560 1CYtIaoI5xLpzsyMPzdyV2RqUH4N9g6oIjG1xaS8hObi9PGZMZXGwPdX+9pzPU45Uv1A bbYSSwfLmMHcQUNgT33WiD5m+uq5LuobfGKUgtHr5wxgvoWGJ2Y+ZvZyfbOGVCbw/zVK xqMw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=fFzjf9NwSuREvtoAbvsN5KOZLnyAEEgseojLyo07d+Y=; b=Np6tVHVHZMIV6Rx6W+oeBfaOhfmOHbWZM2d6SoZve2eYjHClz7eyD917oBlWNtJJyB W+die1foJNyUyBoVY0hoOSVsqewQbB8dxdNl7nipgvC8AFUtUZ4ij1Gqzqq0AvS+q39u LR47c2HCPGtY9Mn9m4UDwQf7o7TxmN4obfXyl8ceMT/JVCW3BCEJ+Z315jwA7RJMwcdf uex1t/yXnFc+hLtIJeBX+F3v/RSHI6yeFC9BH3u8m60NyuL822mEOInm6DD3M/tydHG7 KDXGCROnFwnEAuyDBrytCFyn8EVXPDqpXYRwWLDcwirwfp9KoO8JsxLVYvpRaCdPG7xD 75HQ== X-Gm-Message-State: AKS2vOw/8XgPJfSryfPg6zb40FGYrCMQ2UbjF3S2QMr/InaWECuG5AIT LQyEU1YFrupsVki1 X-Received: by 10.28.137.206 with SMTP id l197mr4243911wmd.41.1498206001870; Fri, 23 Jun 2017 01:20:01 -0700 (PDT) Received: from bidouze.vm.6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id r142sm3590199wmg.24.2017.06.23.01.20.00 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 23 Jun 2017 01:20:01 -0700 (PDT) Date: Fri, 23 Jun 2017 10:19:52 +0200 From: =?iso-8859-1?Q?Ga=EBtan?= Rivet To: "Tan, Jianfeng" Cc: dev@dpdk.org Message-ID: <20170623081952.GI2344@bidouze.vm.6wind.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) Subject: Re: [dpdk-dev] [PATCH v3 0/9] bus/pci: remove PCI bus from EAL 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: Fri, 23 Jun 2017 08:20:02 -0000 Hi Jianfeng, On Fri, Jun 23, 2017 at 11:29:46AM +0800, Tan, Jianfeng wrote: > Hi Gaetan, > > Do you (or anyone else) have plan to move vdev bus into drivers/bus/. > It is in the roadmap for the DPDK, but I haven't planned to do it. > Thanks, > Jianfeng > > On 6/21/2017 7:36 AM, Gaetan Rivet wrote: > >This patchset moves the PCI bus out of the EAL to the drivers/bus > >subdirectory. > > > >Almost all dependencies have been worked out, only remains KNI to be made > >fully independent from the PCI bus in its lib section. > > > >This patchset includes a patch disabling it, that should be removed before > >integration, once all dependencies have been fixed. > > > >The pmdinfogen app has been moved to the drivers subdirectory. This > >allows using the dependency syntax from the build system, as this app > >also depends on the PCI bus. The dependency graph is as follows: > > > >drivers/bus/pci > > | > > + drivers/pmdinfogen > > | > > + drivers/net > > > >The compilation has been tested on debian 8, Redhat 7.2 and FreeBSD 10.3 > > > >This patchset depends on: > > > >eal: complete attach / detach support > >http://dpdk.org/ml/archives/dev/2017-June/067057.html > >http://dpdk.org/dev/patchwork/patch/24969/ > > > >v1 -> v2: > > > >This patchet now depends also on the eventdev PCI dependency fix: > > > >[PATCH v2 0/4] Remove PCI and VDEV dependency from eventdev library > >http://dpdk.org/ml/archives/dev/2017-June/067402.html > >http://dpdk.org/dev/patchwork/patch/25092/ > > > > * Fixes a few PCI dependency left > > * While wholes files are moved, fixed a few checkpatch warnings to > > start anew. > > * Updated eventdev and cryptodev build dependencies > > > >People who want to evaluate / test the whole system can clone: > >https://github.com/grivet/dpdk/tree/master-net > > > >v3: > > > >Further checkpatch fixes have been applied upon the new bus. > >Both cryptodev and pdump have been fixed. The patches disabling them have been > >removed. > >This patchset now depends on both the cryptodev and pdump dependency > >fixes: > > > >[dpdk-dev] [PATCH 00/12] Remove cryptodev driver > >http://dpdk.org/ml/archives/dev/2017-May/066382.html > > > >[dpdk-dev] [PATCH] pdump: remove unnecessary header file > >http://dpdk.org/ml/archives/dev/2017-June/067785.html > > > >My tree has been updated to follow the latest state of the patchsets > >currently live on the ML: > >https://github.com/grivet/dpdk/tree/master-net > > > >Gaetan Rivet (9): > > kni: disabled by default > > eal: expose rte_eal_using_phys_addrs > > ethdev: remove useless PCI dependency > > bus: properly include rte_debug > > pmdinfogen: move to drivers subdirectory > > bus/pci: introduce pci bus > > bus/pci: follow checkpatch > > drivers: update eventdev dependencies > > drivers: update cryptodev dependencies > > > > GNUmakefile | 2 +- > > MAINTAINERS | 2 +- > > buildtools/Makefile | 36 -- > > buildtools/pmdinfogen/Makefile | 47 -- > > buildtools/pmdinfogen/pmdinfogen.c | 422 -------------- > > buildtools/pmdinfogen/pmdinfogen.h | 125 ---- > > config/common_base | 10 + > > config/common_linuxapp | 2 +- > > drivers/Makefile | 7 +- > > drivers/bus/Makefile | 2 + > > drivers/bus/pci/Makefile | 60 ++ > > drivers/bus/pci/bsd/Makefile | 32 ++ > > drivers/bus/pci/bsd/rte_bus_pci_version.map | 21 + > > drivers/bus/pci/bsd/rte_pci.c | 671 ++++++++++++++++++++++ > > drivers/bus/pci/include/rte_pci.h | 656 +++++++++++++++++++++ > > drivers/bus/pci/linux/Makefile | 37 ++ > > drivers/bus/pci/linux/rte_bus_pci_version.map | 21 + > > drivers/bus/pci/linux/rte_pci.c | 722 +++++++++++++++++++++++ > > drivers/bus/pci/linux/rte_pci_init.h | 97 ++++ > > drivers/bus/pci/linux/rte_pci_uio.c | 571 +++++++++++++++++++ > > drivers/bus/pci/linux/rte_pci_vfio.c | 684 ++++++++++++++++++++++ > > drivers/bus/pci/linux/rte_vfio_mp_sync.c | 424 ++++++++++++++ > > drivers/bus/pci/private.h | 169 ++++++ > > drivers/bus/pci/rte_pci_common.c | 601 ++++++++++++++++++++ > > drivers/bus/pci/rte_pci_common_uio.c | 234 ++++++++ > > drivers/pmdinfogen/Makefile | 47 ++ > > drivers/pmdinfogen/pmdinfogen.c | 422 ++++++++++++++ > > drivers/pmdinfogen/pmdinfogen.h | 125 ++++ > > lib/librte_eal/bsdapp/eal/Makefile | 3 - > > lib/librte_eal/bsdapp/eal/eal.c | 1 - > > lib/librte_eal/bsdapp/eal/eal_pci.c | 672 ---------------------- > > lib/librte_eal/bsdapp/eal/rte_eal_version.map | 15 - > > lib/librte_eal/common/Makefile | 2 +- > > lib/librte_eal/common/eal_common_bus.c | 1 + > > lib/librte_eal/common/eal_common_pci.c | 603 -------------------- > > lib/librte_eal/common/eal_common_pci_uio.c | 233 -------- > > lib/librte_eal/common/eal_private.h | 138 ----- > > lib/librte_eal/common/include/rte_pci.h | 601 -------------------- > > lib/librte_eal/linuxapp/eal/Makefile | 12 +- > > lib/librte_eal/linuxapp/eal/eal.c | 1 - > > lib/librte_eal/linuxapp/eal/eal_interrupts.c | 1 - > > lib/librte_eal/linuxapp/eal/eal_memory.c | 3 +- > > lib/librte_eal/linuxapp/eal/eal_pci.c | 723 ------------------------ > > lib/librte_eal/linuxapp/eal/eal_pci_init.h | 97 ---- > > lib/librte_eal/linuxapp/eal/eal_pci_uio.c | 567 ------------------- > > lib/librte_eal/linuxapp/eal/eal_pci_vfio.c | 674 ---------------------- > > lib/librte_eal/linuxapp/eal/eal_vfio_mp_sync.c | 424 -------------- > > lib/librte_eal/linuxapp/eal/rte_eal_version.map | 16 +- > > lib/librte_eal/linuxapp/eal/rte_memory_linux.h | 64 +++ > > lib/librte_ether/rte_ethdev.c | 1 - > > mk/rte.app.mk | 2 + > > 51 files changed, 5687 insertions(+), 5416 deletions(-) > > delete mode 100644 buildtools/Makefile > > delete mode 100644 buildtools/pmdinfogen/Makefile > > delete mode 100644 buildtools/pmdinfogen/pmdinfogen.c > > delete mode 100644 buildtools/pmdinfogen/pmdinfogen.h > > create mode 100644 drivers/bus/pci/Makefile > > create mode 100644 drivers/bus/pci/bsd/Makefile > > create mode 100644 drivers/bus/pci/bsd/rte_bus_pci_version.map > > create mode 100644 drivers/bus/pci/bsd/rte_pci.c > > create mode 100644 drivers/bus/pci/include/rte_pci.h > > create mode 100644 drivers/bus/pci/linux/Makefile > > create mode 100644 drivers/bus/pci/linux/rte_bus_pci_version.map > > create mode 100644 drivers/bus/pci/linux/rte_pci.c > > create mode 100644 drivers/bus/pci/linux/rte_pci_init.h > > create mode 100644 drivers/bus/pci/linux/rte_pci_uio.c > > create mode 100644 drivers/bus/pci/linux/rte_pci_vfio.c > > create mode 100644 drivers/bus/pci/linux/rte_vfio_mp_sync.c > > create mode 100644 drivers/bus/pci/private.h > > create mode 100644 drivers/bus/pci/rte_pci_common.c > > create mode 100644 drivers/bus/pci/rte_pci_common_uio.c > > create mode 100644 drivers/pmdinfogen/Makefile > > create mode 100644 drivers/pmdinfogen/pmdinfogen.c > > create mode 100644 drivers/pmdinfogen/pmdinfogen.h > > delete mode 100644 lib/librte_eal/bsdapp/eal/eal_pci.c > > delete mode 100644 lib/librte_eal/common/eal_common_pci.c > > delete mode 100644 lib/librte_eal/common/eal_common_pci_uio.c > > delete mode 100644 lib/librte_eal/common/include/rte_pci.h > > delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci.c > > delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci_init.h > > delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci_uio.c > > delete mode 100644 lib/librte_eal/linuxapp/eal/eal_pci_vfio.c > > delete mode 100644 lib/librte_eal/linuxapp/eal/eal_vfio_mp_sync.c > > create mode 100644 lib/librte_eal/linuxapp/eal/rte_memory_linux.h > > > -- Gaëtan Rivet 6WIND