From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 0135E7CFD for ; Fri, 22 Sep 2017 00:10:54 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 9B9F72140B; Thu, 21 Sep 2017 18:10:54 -0400 (EDT) Received: from frontend1 ([10.202.2.160]) by compute1.internal (MEProxy); Thu, 21 Sep 2017 18:10:54 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=mesmtp; bh=QfSVeE/tAIs3d0U Jcgo1H9eQORmMCwkU/YptoeR0i20=; b=M0XjqlTpCH9FkH3AHRMVNQXiKJCytos cKHNZS2XY1fnzUulkTpbTAodHZFs+xiwwskpnjIF5fRsFWZQd4YAeA+EdyDB20jR xAAE+Twup23Blv0Rnsw5Lin4eOQgwL0ghgsq58DRNXwqmUNiA6acRR1XYQDCmaCz v8QUdgFiD7Hk= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc:x-sasl-enc; s= fm1; bh=QfSVeE/tAIs3d0UJcgo1H9eQORmMCwkU/YptoeR0i20=; b=QQClIGKf 9qKIV9Ntit3XECkgfkZaHVYMkw9LHOeNl3hGKcW5KFpcCXbJEd1+6eQiGBsQwSH+ DfnBjZB0F3dNcPcD0uc58c0z4iFFm5p3mG03cKhz3vqiCa9JmKzJbuDm5KvFvaVf vDuyK+HMINW3t2LuAcuRHF08o75fxqrUN7ywL9LVBOZCkeZjTlejWure8hZm87WT 3d5lfhdeUnSbT4Ow95jimcbpb2+PqPkKJIg3NdpcK62CR1GNjOq/AKfWbNujNbH6 ey6dZA0hzxOytpMJa0pPKPJ8wkzKeLnp9EScAXOEXHNKFpXj61cGvx3dq5T0hzYh GY6EshYv0c6+QQ== X-ME-Sender: X-Sasl-enc: ox7TtEKsCbFPAby1cOZ6JWLkjgiHItNweKao7l/sTqif 1506031854 Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 4F26D7E675; Thu, 21 Sep 2017 18:10:54 -0400 (EDT) From: Thomas Monjalon To: Shreyansh Jain Cc: dev@dpdk.org, ferruh.yigit@intel.com, hemant.agrawal@nxp.com Date: Fri, 22 Sep 2017 00:10:53 +0200 Message-ID: <25531756.sobbYYJFMz@xps> In-Reply-To: <20170909112132.13936-1-shreyansh.jain@nxp.com> References: <20170823141213.25476-1-shreyansh.jain@nxp.com> <20170909112132.13936-1-shreyansh.jain@nxp.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v4 00/41] Introduce NXP DPAA Bus, Mempool and PMD 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: Thu, 21 Sep 2017 22:10:55 -0000 09/09/2017 13:20, Shreyansh Jain: > DPAA, or Datapath Acceleration Architecture [R2], is a set of hardware > components designed for high-speed network packet processing. This > architecture provides the infrastructure to support simplified sharing of > networking interfaces and accelerators by multiple CPU cores, and the > accelerators themselves. > > This patchset introduces the following: > 1. DPAA Bus (drivers/bus/dpaa) > The core of DPAA bus is implemented using 3 main hardware blocks: QMan, > or Queue Manager; BMan, or Buffer Manager and FMan, or Frame Manager. > The patches introduce necessary layers to expose the DPAA hardware > blocks for interfacing with RTE framework. I guess these are the same blocks as for DPAA2? They are in drivers/bus/fslmc/ Why introducing yet another bus driver? The fslmc one was supposed to cover any Freescale (NXP (Qualcomm)) SoC. > 2. DPAA Mempool (drivers/mempool/dpaa) > BMan, or Buffer Manager, block of DPAA features a hardware offloaded > mempool. These patches add support for a driver to manage the BMan > block. This driver allows for mempool creation, deletion, buffer > acquire and release, as per the RTE APIs. > > 3. DPAA PMD (drivers/net/dpaa) > The Poll Mode Driver for DPAA NIC Interfaces. > > Patch Layout > ============ > > 01: Add DPAA SoC build configuration > 02~16: Add DPAA Bus support and features, incrementally > 17: Add Documentation > 18~21: Add DPAA Mempool support > 22~41: Add PMD and its various features, incrementally It is a very long series introducing 3 different subsystems. I think everybody was scared about reviewing it. Why you did not split it?