From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH 00/50] LiquidIO PMD Date: Thu, 23 Feb 2017 09:56:01 +0000 Message-ID: References: <1487669225-30091-1-git-send-email-shijith.thotton@caviumnetworks.com> <20170221122247.132b1b86@xeon-e3> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Shijith Thotton , Stephen Hemminger Return-path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 72BD15911 for ; Thu, 23 Feb 2017 10:56:04 +0100 (CET) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 2/22/2017 4:56 AM, Shijith Thotton wrote: > > > On Wednesday 22 February 2017 01:52 AM, Stephen Hemminger wrote: >> On Tue, 21 Feb 2017 14:56:15 +0530 >> Shijith Thotton wrote: >> >>> The patch series provides initial version of virtual function poll mode >>> driver for Cavium LiquidIO II server adapters. This version adds support >>> for LiquidIO II CN2350 210SV adapter. >>> >>> Patch series includes driver documentation doc/guides/nics/liquidio.rst >>> and list of supported features doc/guides/nics/features/liquidio.ini. >>> Updated release notes to notify the addition of new PMD. >>> >>> Shijith Thotton (50): >>> net/liquidio/base: hardware register definitions >>> config: liquidio PMD configuration >>> net/liquidio: added PMD version map file >>> net/liquidio: definitions for log >>> maintainers: claim responsibility for LiquidIO PMD >>> net/liquidio: liquidio VF PMD Driver registration >>> net/liquidio: added Makefile >>> net/liquidio/base: macros to read and write register >>> net/liquidio: liquidio device init >>> net/liquidio: add API to disable io queues >>> net/liquidio: add API to setup io queue registers >>> net/liquidio: add mbox APIs for PF/VF communication >>> net/liquidio: add API to setup mbox registers >>> net/liquidio: add API for VF/PF handshake >>> net/liquidio: add API for VF FLR >>> net/liquidio: add APIs to allocate and free IQ >>> net/liquidio: add API to setup instruction queue >>> net/liquidio: add API to allocate and free command pool >>> net/liquidio: add API to allocate and free soft command >>> net/liquidio: add APIs for response list >>> net/liquidio: add APIs to send packet to device >>> net/liquidio: add API to configure device >>> net/liquidio: add API to setup Rx queue >>> net/liquidio: initialize Rx queue >>> net/liquidio: add Rx data path >>> net/liquidio: add API to release Rx queue >>> net/liquidio: add API to setup Tx queue >>> net/liquidio: add APIs for sg list >>> net/liquidio: add API to enable and disable IO queues >>> net/liquidio: add Tx data path for single segment >>> net/liquidio: add Tx data path for multiple segments >>> net/liquidio: add APIs to flush IQ and free buffers >>> net/liquidio: add API to release Tx queue >>> net/liquidio: add API to start device and check link >>> net/liquidio: add API for link update >>> net/liquidio: add API to alloc and send command >>> net/liquidio: add API to control Rx >>> net/liquidio: add RSS support >>> net/liquidio: add API to get device info >>> net/liquidio: add API to set MTU >>> net/liquidio: add API to enable and disable multicast >>> net/liquidio: add API to set link up and down >>> net/liquidio: add API to configure udp tunnel port >>> net/liquidio: add support for Rx stats >>> net/liquidio: add support for Tx stats >>> net/liquidio: add APIs for hardware stats >>> net/liquidio: add API for dev stop >>> net/liquidio: add API for dev close >>> net/liquidio: add API to add and remove VLAN port >>> doc: added documents >>> >>> MAINTAINERS | 7 + >>> config/common_base | 11 + >>> doc/guides/nics/features/liquidio.ini | 29 + >>> doc/guides/nics/index.rst | 1 + >>> doc/guides/nics/liquidio.rst | 269 ++++ >>> doc/guides/rel_notes/release_17_05.rst | 3 + >>> drivers/net/Makefile | 1 + >>> drivers/net/liquidio/Makefile | 62 + >>> drivers/net/liquidio/base/lio_23xx_reg.h | 194 +++ >>> drivers/net/liquidio/base/lio_23xx_vf.c | 586 ++++++++ >>> drivers/net/liquidio/base/lio_23xx_vf.h | 97 ++ >>> drivers/net/liquidio/base/lio_hw_defs.h | 249 ++++ >>> drivers/net/liquidio/base/lio_mbox.c | 275 ++++ >>> drivers/net/liquidio/base/lio_mbox.h | 131 ++ >>> drivers/net/liquidio/lio_ethdev.c | 2040 ++++++++++++++++++++++++++ >>> drivers/net/liquidio/lio_ethdev.h | 204 +++ >>> drivers/net/liquidio/lio_logs.h | 91 ++ >>> drivers/net/liquidio/lio_rxtx.c | 1885 ++++++++++++++++++++++++ >>> drivers/net/liquidio/lio_rxtx.h | 769 ++++++++++ >>> drivers/net/liquidio/lio_struct.h | 689 +++++++++ >>> drivers/net/liquidio/rte_pmd_lio_version.map | 4 + >>> mk/rte.app.mk | 1 + >>> 22 files changed, 7598 insertions(+) >>> create mode 100644 doc/guides/nics/features/liquidio.ini >>> create mode 100644 doc/guides/nics/liquidio.rst >>> create mode 100644 drivers/net/liquidio/Makefile >>> create mode 100644 drivers/net/liquidio/base/lio_23xx_reg.h >>> create mode 100644 drivers/net/liquidio/base/lio_23xx_vf.c >>> create mode 100644 drivers/net/liquidio/base/lio_23xx_vf.h >>> create mode 100644 drivers/net/liquidio/base/lio_hw_defs.h >>> create mode 100644 drivers/net/liquidio/base/lio_mbox.c >>> create mode 100644 drivers/net/liquidio/base/lio_mbox.h >>> create mode 100644 drivers/net/liquidio/lio_ethdev.c >>> create mode 100644 drivers/net/liquidio/lio_ethdev.h >>> create mode 100644 drivers/net/liquidio/lio_logs.h >>> create mode 100644 drivers/net/liquidio/lio_rxtx.c >>> create mode 100644 drivers/net/liquidio/lio_rxtx.h >>> create mode 100644 drivers/net/liquidio/lio_struct.h >>> create mode 100644 drivers/net/liquidio/rte_pmd_lio_version.map >>> >> >> >> Splitting into small patches is great. >> Is this still bisectable? Does each step build? >> > > Hi Stephen, > > Each step can be build independently. I confirm overall build and patch by patch build is fine.