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 67F582C37 for ; Fri, 23 Jun 2017 05:29:49 +0200 (CEST) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jun 2017 20:29:48 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.39,376,1493708400"; d="scan'208";a="118315742" Received: from tanjianf-mobl.ccr.corp.intel.com (HELO [10.239.201.82]) ([10.239.201.82]) by fmsmga006.fm.intel.com with ESMTP; 22 Jun 2017 20:29:47 -0700 To: Gaetan Rivet , dev@dpdk.org References: From: "Tan, Jianfeng" Message-ID: Date: Fri, 23 Jun 2017 11:29:46 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit 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 03:29:50 -0000 Hi Gaetan, Do you (or anyone else) have plan to move vdev bus into drivers/bus/. 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 >