From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <Santosh.Shukla@cavium.com>
Received: from NAM02-SN1-obe.outbound.protection.outlook.com
(mail-sn1nam02on0058.outbound.protection.outlook.com [104.47.36.58])
by dpdk.org (Postfix) with ESMTP id D09DE2B95
for <dev@dpdk.org>; Thu, 29 Dec 2016 08:08:57 +0100 (CET)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com;
h=From:Date:Subject:Message-ID:Content-Type:MIME-Version;
bh=lBnYFBWvoDrrLnDGXmZrs8heZVvj0AEYN+cRqTLJSMk=;
b=L6xbcQ2f/70836U6eT3wbsUiZu88v01Msggu7vRSklqpnwlt/Tt86Ukx8tTfwRvJebCU9dG7vcdyoAI3jSr2uRbyv3clSs41rhUSWN8LEmVZg4cZM+lgfQhRO1mNgakCYgzI0ufQ/APPST90zIolBNNyXEy0Yl1Pi/V7DzZii9I=
Authentication-Results: spf=none (sender IP is )
smtp.mailfrom=Santosh.Shukla@cavium.com;
Received: from santosh-Latitude-E5530-non-vPro (111.93.218.67) by
BY1PR0701MB1722.namprd07.prod.outlook.com (10.162.111.141) with Microsoft
SMTP Server (version=TLS1_2,
cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.803.11; Thu, 29
Dec 2016 07:08:49 +0000
Date: Thu, 29 Dec 2016 12:38:25 +0530
From: Santosh Shukla <santosh.shukla@caviumnetworks.com>
To: Shreyansh Jain <shreyansh.jain@nxp.com>
CC: <dev@dpdk.org>, <thomas.monjalon@6wind.com>, <bruce.richardson@intel.com>,
<john.mcnamara@intel.com>, <ferruh.yigit@intel.com>,
<jerin.jacob@caviumnetworks.com>, Hemant Agrawal <hemant.agrawal@nxp.com>
Message-ID: <20161229070807.GA7823@santosh-Latitude-E5530-non-vPro>
References: <1482180853-18823-1-git-send-email-hemant.agrawal@nxp.com>
<1482988612-6638-1-git-send-email-shreyansh.jain@nxp.com>
<1482988612-6638-17-git-send-email-shreyansh.jain@nxp.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <1482988612-6638-17-git-send-email-shreyansh.jain@nxp.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
X-Originating-IP: [111.93.218.67]
X-ClientProxiedBy: DM3PR14CA0081.namprd14.prod.outlook.com (10.166.156.177) To
BY1PR0701MB1722.namprd07.prod.outlook.com (10.162.111.141)
X-MS-Office365-Filtering-Correlation-Id: c68062b0-7b08-4873-2eff-08d42fb98c92
X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001);
SRVR:BY1PR0701MB1722;
X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722;
3:HvoErXkPovq7XHuIFs8Nt8rb82rPuym7YRN2x65ydh710BntcswKQTkJF7ZRGL31FahgWxfMc1rPaCRN2Fo7y1rCDHYRtHcf9tWcmcjK+z/v7KqMl0p4vPssh9MFcUn2mJVMvyZfObPEOQzNrEx01LdsZHdqQrbImn4MqgbZpTG9yHgMDPVXz9VgIErXNdAhPRh/+GZWP0SPpytjx2888GCHAqLn7jwg+e3XCKMQp2VBsUlcJhZ9ha4u8+AwNvoOvtRjGGuEUz6rgTAMsNUFRg==;
25:FC+5adfPOvs6tSz+WZTO/i1g2Oo5Ku4na/dLGeTPKpnlD4OIvSvAgizRwp5IYshnv/0VGxFq5LrKZ07V9MwPeQSabGVbZ9dAOi+AsrBouRIURugbedsAimbIfxOwlEBOm643aPtQ152Fs21LqMseWHwMHL4FdqHByzSZC/Y5l2xtPyc98Nd6WksiJZtcOYgQ63iiL/m5bSZ83mzKqQBkhUpPXXky+LQS4K4A3FMBeIg5lVltK1CC54FwJAljPp7JVVhV3V2Gc9hcgws2lhM8IsQI/brJ2I7+Y0Ltjf8a08WDotBY2IFkvMQigdnWXmWnN3UJi0zI4pNnOEGE3WjtGarvBNPwBwxIL4bLD5KsIJ67qWpTVEEkcA6gpnRWiVFxXFz8EZ2xKLlZmV5kM9ilpRfzEYvn7vz1y4e298w1M11ihj+obeeiS8DH5Pn+AzxOe4AwnWlymQfB58hIAmDAzQ==
X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722;
31:WCnhzRdCyKuhJUoMMWLaZLnMrzWVvxQlYGA5fBpCG/JnILUxn6tidSo9bZ5mWqjEO4P8fFy7SBr0wlfg1U+gv5niNmfPIDxX97jEaDhdPw1+uVgILnTlmF58/Aguafhy2X7sxJpuSs01jImyPAZbg797bIXJpPRipABEAqMGvVe2/DmgPzBiSxo99WWnOO6PV/6tnbTKzNqZ/XuKAgxXOHT2d4hDgsXM2Q6Sda9WaXHHRZtfDTEX1MU6AHrnSBoO;
20:jZCYT+rkcX85oTDJhJbDDoAAADLZhOQ/ptTKyHE9Jsi2p1dtjwbeK1vQbfkY9AePcKZx/mfDyk2OonWbNwiQJG9V+B9l8mP1RRA4Hi79ZjUM8fmKpI6EhqHhb79yjFc9xseUMOKp9EbEypTuFWn9qf46Z9OHIJSuHBDvZFnPRvmXMR82eweoAHjzg9xj8D2jgrqbTUU8ZEaBvO3YxacHkOJb/IfL/IwF5PaAHA/802eKL9VeCojCkfCxR6jI63uOfnElpQpdn96ispiD8sVKiVYepk6aQWFu7nQpaDgLg0utElPMFlO1Y10Iz9sS86PQ/lQ30qg5Wdg5exnuAIu9P6OJ/c4Et5sHnUm/DV93YqSFg67/yox+8Zv2oJtag0gJfRcWaydTXF02nnaZDah9UD/6jL+4m1RithPAqJ2iakbD2TMToAhk8bWh7L4eOmcxeV4Ab9C+VxAP5Fl5MO4x8UPsk6d11K5L3jq1XI3mBAz/aKyyieGCdGHOwWGN6tC0/oZ7F6PT/B1vd2sv6FCKEw2Ca+Flipu9C9+dSiYsVpLhwNfEOmJCkbdpfxwdeXdiUj/P8GUzvPjNR9bUN12YEqcXxpFjJguG+L3BMkXZY/k=
X-Microsoft-Antispam-PRVS: <BY1PR0701MB172299095629DD4A2372D332EA6B0@BY1PR0701MB1722.namprd07.prod.outlook.com>
X-Exchange-Antispam-Report-Test: UriScan:(185117386973197)(275809806118684);
X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0;
RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6041248)(20161123555025)(20161123560025)(20161123558021)(20161123564025)(20161123562025)(6072148);
SRVR:BY1PR0701MB1722; BCL:0; PCL:0; RULEID:; SRVR:BY1PR0701MB1722;
X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722;
4:uUSmhsHcRElwQmynYQcLH5rCBS7aXyHqs1eq8klEtzwUm20jo/aIbXLPX6Qyd21Yx4wjur1oiV16eGamtfpDiieg5RBBKl4ZC9QOUU/u1LdMLAvKRADuLtHM84qCMpTTsmMpqfXw0cJENUVhInP2Xrw4V1p9pfZ8XPw78C7FW0QCZupkKQCpQktnaFX1iszJj8IkQYt4OCXAYqjJeBRs0fX1h7FrTNGuC1a8mhMVN7W1P+Se+rcVLNLPSvWTSqTcwoDT1F15WJGYWykW4lL5uoOKwW8UCnMPyzwndxM0+TfQgF6bULBgAcUa/vyuzSYtxu1E9U/o8Pwc/VwqsWTPsxEyhEM8tGYH1IHUYRcLV1oGGR7XyFtYtpPNTB6wq/nOy54j9HmYhObeITMdHVcYh9WZt2WYNm2AOL4va4PD0K6L94ZiGoPeK3Bo0f6BI0OvlDr9TSRHo5H7A99zgeEMvWbNUC3qqevNU4PmpUOmBUTHRxaClyDete0WwB1lXegsHgmna/FN6pXjlh7k9MfvYNlOuOEWmY4qZlLb8oPYxA0/rZ/euF6xINdJ2VGY52i2BnyUteUBMACbJ1HWYKMz1VKhfjwYSLsTU+OmXwbJQEISKGptnxUebPba8qFjkp+uF5KGgnqzYDf6TqWbXC6X8FFMrph9j0MF0mRPIju/vJKQ/BKigIbSzOx09lDByZTI
X-Forefront-PRVS: 01713B2841
X-Forefront-Antispam-Report: SFV:NSPM;
SFS:(10009020)(4630300001)(6009001)(7916002)(39450400003)(24454002)(199003)(189002)(5009440100003)(105586002)(4001350100001)(106356001)(54356999)(42186005)(8676002)(575784001)(97736004)(83506001)(76176999)(189998001)(33716001)(97756001)(33656002)(4326007)(5660300001)(229853002)(23726003)(47776003)(38730400001)(7736002)(92566002)(1076002)(9686002)(50986999)(68736007)(8656002)(46406003)(6116002)(6496003)(2950100002)(2906002)(3846002)(81166006)(81156014)(55016002)(110136003)(6666003)(305945005)(50466002)(101416001)(25786008)(6916009)(66066001)(42882006)(7099028)(18370500001)(7059030);
DIR:OUT; SFP:1101; SCL:1; SRVR:BY1PR0701MB1722;
H:santosh-Latitude-E5530-non-vPro; FPR:; SPF:None; PTR:InfoNoRecords; A:1;
MX:1; LANG:en;
Received-SPF: None (protection.outlook.com: cavium.com does not designate
permitted sender hosts)
X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BY1PR0701MB1722;
23:sLQ0dIwITR+G8+UbeNj3+vOmaiJ4ekjAzC4le92?=
=?us-ascii?Q?b4pzNBW4rumpLVU3sXZr5zHuXvAzj17q1R7/FgPSg6wnmHWmoItxbuCmjzKn?=
=?us-ascii?Q?O6xzXRsPvO+9BtXXVeeBDAlkXpoMtNJpJZ1W/1/iEOJQygD8EzbHKYLM7IFb?=
=?us-ascii?Q?mQAWiYySazjcU18U7ADeILYy9NYUs4d9fg8sdUQDMU8MK1+7GofMDqVS5Jzi?=
=?us-ascii?Q?vbRy0YQxzyCoWX8S39lD+iII1HLnmsgOi3EWU/uibkc0UX0OydPKZZraUPyq?=
=?us-ascii?Q?YYKyNftcUtxNzioTDwpApdgL4xZoQtMfDtUCjjF3glyaA4i0reFJn+3OG4ZY?=
=?us-ascii?Q?BdYz2dnRhV+Mp59ZFN4lgLHRhUjcBmsgV1CFj5M8lBiaTCet52u7KeuA+srx?=
=?us-ascii?Q?hXlmzQI4rHpY172vlR3VXF6wn+1PHIGJWXe4H9sAt9MBmWVBCYKYMNW11bsu?=
=?us-ascii?Q?E8dy99ZzjUeiRfqYuVI+8qRcyxPuQ0LGsmyh7lpmbuEVSmx9XjHpD4yuzMQ3?=
=?us-ascii?Q?dmZsDDZSoqylUw1YMg5Bnvk4S3RVGuj7qEF9gDqvDfsHlBpAZCsVGsVl7XPV?=
=?us-ascii?Q?gfosUhs7delZIMrk/VZrVEA0vW52TJwOi5zXPSWJgNGOcYEVCLrjkKeqxgUi?=
=?us-ascii?Q?J52I3e1Z98enmDWJO8286VGMTmbhbuxF/76O9ocM/ALLioSU08Pzw/HIkSMN?=
=?us-ascii?Q?Ix7ycigkprC5pYtOwXnVTRKYUrYmtsPMI7/n6OgNqmciHBe45mccCSW0HSl9?=
=?us-ascii?Q?nMT799qyAM2OaCITKxuen5jh9HTJ1w6Tc/gW/BG1SxpnYRnMSMMTp8S9Azr1?=
=?us-ascii?Q?ST9R9LP0Hh0cvqkbUC4yMyw8yqPewBeQyhHGOhG8WhZOWbJfqw8K1mVCT5k8?=
=?us-ascii?Q?37id/pIenBZVkxm66ggSbcIa17DfICQMYYcR6CpS1C3VcjRqKy52s04I2E+7?=
=?us-ascii?Q?/LdD0rxcTiesQ40vKgusCufWf2eOOSTWziG9gTjYQiG4x5yNI2ijJAp5tII+?=
=?us-ascii?Q?4mCKhaJwmnh1o3A731Mk+9sAVxaY6/LZPDEHYOBADHZ3AQOJ6EG4clO0tOJw?=
=?us-ascii?Q?PEzRsM04ptshVWShJ4wbAETalqrY1x0JvT0BkI0GP69pQIpdhCW2zApgZbt+?=
=?us-ascii?Q?52sC8kPn/dYJwpVgcnfqoNz+07cZ4XndZsOOcq0G+iGKxZx4aqPtSEXTj0ek?=
=?us-ascii?Q?Pkm53KoveAQqnaxsLZTe2vPNwuI6sHw5mvylrgyx32OTMqfGADI26VYbmLlt?=
=?us-ascii?Q?kGrpGEYGYJ7e0lV3piE9mTypPL0TqxSZIaL3gygi+wDlgTCyE2P1/DGCaKpz?=
=?us-ascii?Q?aCLIoAa1lVZZVVjtELeACoEQuc92OA46D6/9JDUbKyBzl?=
X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722;
6:ycUgkdXk1EaDp1qtW2f9bF2X2k9V4eNlBE5e0UR+CNBlZOlG4BkxrABGjdz3bBJ1eVJ9h67OAM9Zf5aOu22Wbb+CkJ3ghQTgKOyb9FfBbjm9KO+SWwELderFKxUFggMYFeO5vuN/COG1rjs8opqPZ64a8klXRrliz5dDM0mTzwKhmYHromO7Ot2pjT+GOupHDKbiqmTXOPlqYs2kTXK4gkx5zpHDdM/0ZyjyFTzH7d7Fgx+xReHVTXX/czOJt2APDHIqYQDfjjKnaO0RlmfEom/V0h+/vgsnbEG6PYOQjEBpGnEUrmXI/mRMA4EOAd8VV7THESe6IQIFQg6Fvwyq50Q0luZizmRJWUXhEOAKcsxMSu+AdYd6trFhbxqYBqk30WWzRLvdUHKIrEZkLc9CEcZH027n8aYxfCs+bz1/JLg=;
5:ewfm/iw61fpYerYXtfNbJRQadeNAYkDruEbIyV2fhOs0CHOj8ul/egEmfeYeny26APbVeizP9C7wvjAm/YAHf0GQJ2WERfDSb3RIqPdzmBatboPa1nfDwkktaqhGj08TB6MfxAzn+U9MiAgC8vdnKQ==;
24:HgnoLR1c48nbeUSpJtxA7WE6nJ+gGWoKwPhsz/19lIhmoDsn/02cMPWPIjI2R7O5hIcwzQWm2fW5uvYiQ3w3sgYtqoiKoE22ZErwem9ZAOM=
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722;
7:GbfJw1QIdL5QzV0hMbMyjXvCU7QHhtoRntni1jQuk6p9pD5CRjX7SkY0QN5CuxTOp5g2Tbk297Juvodd/Sa6epXnxCd0+ng0vCdhke+552DtnGHCCRT8LkujRMhO/L7kGf+THkIlDymhV4HO2r+IlrQuWlH/QOkMF1Cs5r3m1/kXWQhWCdk1ITo/id/pTZLN3sZlWU6aRb1QQpxPl/vQZywhc9mKY+07ZReCbbXczZyaNJt9Ak4lFom5kyakIgZv9Bms3JB3gSJoPqWz3ovrOvAMeuA4fHWxU/Sv/iFyzlLCgXrg1MOOVVRjiI4CYp7XEEkhyh1JRj/DlQxJwfMZ54WrSIinE/Tzu8IoH4r3L2RK6rZ5uIZxaZKPcq31j4R2hxt6wbbVV4evd5VDz9etz06nDvbrG1Ia4KmbdS5apfSzHtah043LOt7LLrt+/SE+pCRcdtW4GhZ2UnqAhiQL9A==
X-OriginatorOrg: caviumnetworks.com
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Dec 2016 07:08:49.8424 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR0701MB1722
Subject: Re: [dpdk-dev] [PATCH v3 16/33] drivers/pool/dpaa2: adding hw
offloaded mempool
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
<mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
<mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Thu, 29 Dec 2016 07:08:58 -0000
Hi Shreyansh,
On Thu, Dec 29, 2016 at 10:46:35AM +0530, Shreyansh Jain wrote:
> From: Hemant Agrawal <hemant.agrawal@nxp.com>
>
> Adding NXP DPAA2 architecture specific mempool support
> Each mempool instance is represented by a DPBP object
> from the FSL-MC bus.
>
> This patch also registers a dpaa2 type MEMPOOL OPS
>
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
> ---
> config/common_base | 1 +
> config/defconfig_arm64-dpaa2-linuxapp-gcc | 4 +
> drivers/Makefile | 1 +
> drivers/bus/fslmc/Makefile | 2 +
> drivers/bus/fslmc/fslmc_vfio.c | 9 +-
> drivers/bus/fslmc/fslmc_vfio.h | 2 +
> drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c | 137 +++++++++
> drivers/bus/fslmc/portal/dpaa2_hw_pvt.h | 19 ++
> drivers/bus/fslmc/rte_pmd_fslmcbus_version.map | 2 +
> drivers/common/Makefile | 3 +
> drivers/pool/Makefile | 38 +++
> drivers/pool/dpaa2/Makefile | 67 +++++
> drivers/pool/dpaa2/dpaa2_hw_mempool.c | 339 ++++++++++++++++++++++
> drivers/pool/dpaa2/dpaa2_hw_mempool.h | 95 ++++++
> drivers/pool/dpaa2/rte_pmd_dpaa2_pool_version.map | 8 +
> mk/rte.app.mk | 1 +
> 16 files changed, 727 insertions(+), 1 deletion(-)
> create mode 100644 drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c
> create mode 100644 drivers/pool/Makefile
> create mode 100644 drivers/pool/dpaa2/Makefile
> create mode 100644 drivers/pool/dpaa2/dpaa2_hw_mempool.c
> create mode 100644 drivers/pool/dpaa2/dpaa2_hw_mempool.h
> create mode 100644 drivers/pool/dpaa2/rte_pmd_dpaa2_pool_version.map
>
> diff --git a/config/common_base b/config/common_base
> index d605e85..493811f 100644
> --- a/config/common_base
> +++ b/config/common_base
> @@ -276,6 +276,7 @@ CONFIG_RTE_LIBRTE_THUNDERX_NICVF_DEBUG_MBOX=n
> # Compile Support Libraries for NXP DPAA2
> #
> CONFIG_RTE_LIBRTE_DPAA2_COMMON=n
> +CONFIG_RTE_LIBRTE_DPAA2_POOL=n
>
> #
> # Compile NXP DPAA2 FSL-MC Bus
> diff --git a/config/defconfig_arm64-dpaa2-linuxapp-gcc b/config/defconfig_arm64-dpaa2-linuxapp-gcc
> index d3bc9d8..7665912 100644
> --- a/config/defconfig_arm64-dpaa2-linuxapp-gcc
> +++ b/config/defconfig_arm64-dpaa2-linuxapp-gcc
> @@ -42,10 +42,14 @@ CONFIG_RTE_ARCH_ARM_TUNE="cortex-a57+fp+simd"
> CONFIG_RTE_MAX_LCORE=8
> CONFIG_RTE_MAX_NUMA_NODES=1
>
> +CONFIG_RTE_PKTMBUF_HEADROOM=256
> +
> #
> # Compile Support Libraries for DPAA2
> #
> CONFIG_RTE_LIBRTE_DPAA2_COMMON=y
> +CONFIG_RTE_LIBRTE_DPAA2_POOL=n
> +CONFIG_RTE_MBUF_DEFAULT_MEMPOOL_OPS="dpaa2"
>
> #
> # Compile NXP DPAA2 FSL-MC Bus
> diff --git a/drivers/Makefile b/drivers/Makefile
> index bdae63b..9fd268e 100644
> --- a/drivers/Makefile
> +++ b/drivers/Makefile
> @@ -33,6 +33,7 @@ include $(RTE_SDK)/mk/rte.vars.mk
>
> DIRS-y += common
> DIRS-y += bus
> +DIRS-y += pool
> DIRS-y += net
> DIRS-$(CONFIG_RTE_LIBRTE_CRYPTODEV) += crypto
>
> diff --git a/drivers/bus/fslmc/Makefile b/drivers/bus/fslmc/Makefile
> index 1b815dd..35f30ad 100644
> --- a/drivers/bus/fslmc/Makefile
> +++ b/drivers/bus/fslmc/Makefile
> @@ -47,6 +47,7 @@ CFLAGS += "-Wno-strict-aliasing"
> CFLAGS += -I$(RTE_SDK)/drivers/bus/fslmc
> CFLAGS += -I$(RTE_SDK)/drivers/bus/fslmc/mc
> CFLAGS += -I$(RTE_SDK)/drivers/common/dpaa2/qbman/include
> +CFLAGS += -I$(RTE_SDK)/drivers/pool/dpaa2
> CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linuxapp/eal
>
> # versioning export map
> @@ -63,6 +64,7 @@ SRCS-$(CONFIG_RTE_LIBRTE_FSLMC_BUS) += \
> mc/mc_sys.c
>
> SRCS-$(CONFIG_RTE_LIBRTE_FSLMC_BUS) += portal/dpaa2_hw_dpio.c
> +SRCS-$(CONFIG_RTE_LIBRTE_FSLMC_BUS) += portal/dpaa2_hw_dpbp.c
> SRCS-$(CONFIG_RTE_LIBRTE_FSLMC_BUS) += fslmc_vfio.c
> SRCS-$(CONFIG_RTE_LIBRTE_FSLMC_BUS) += fslmc_bus.c
>
> diff --git a/drivers/bus/fslmc/fslmc_vfio.c b/drivers/bus/fslmc/fslmc_vfio.c
> index ed0a8b9..4e47ec8 100644
> --- a/drivers/bus/fslmc/fslmc_vfio.c
> +++ b/drivers/bus/fslmc/fslmc_vfio.c
> @@ -268,7 +268,7 @@ int fslmc_vfio_process_group(struct rte_bus *bus)
> char path[PATH_MAX];
> int64_t v_addr;
> int ndev_count;
> - int dpio_count = 0;
> + int dpio_count = 0, dpbp_count = 0;
> struct fslmc_vfio_group *group = &vfio_groups[0];
> static int process_once;
>
> @@ -418,6 +418,11 @@ int fslmc_vfio_process_group(struct rte_bus *bus)
> if (!ret)
> dpio_count++;
> }
> + if (!strcmp(object_type, "dpbp")) {
> + ret = dpaa2_create_dpbp_device(object_id);
> + if (!ret)
> + dpbp_count++;
> + }
> }
> closedir(d);
>
> @@ -425,6 +430,8 @@ int fslmc_vfio_process_group(struct rte_bus *bus)
> if (ret)
> FSLMC_VFIO_LOG(DEBUG, "Error in affining qbman swp %d", ret);
>
> + FSLMC_VFIO_LOG(DEBUG, "DPAA2: Added dpbp_count = %d dpio_count=%d\n",
> + dpbp_count, dpio_count);
> return 0;
>
> FAILURE:
> diff --git a/drivers/bus/fslmc/fslmc_vfio.h b/drivers/bus/fslmc/fslmc_vfio.h
> index e89d980..9bf69d4 100644
> --- a/drivers/bus/fslmc/fslmc_vfio.h
> +++ b/drivers/bus/fslmc/fslmc_vfio.h
> @@ -76,4 +76,6 @@ int dpaa2_create_dpio_device(struct fslmc_vfio_device *vdev,
> struct vfio_device_info *obj_info,
> int object_id);
>
> +int dpaa2_create_dpbp_device(int dpbp_id);
> +
> #endif /* _FSLMC_VFIO_H_ */
> diff --git a/drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c b/drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c
> new file mode 100644
> index 0000000..16d5b24
> --- /dev/null
> +++ b/drivers/bus/fslmc/portal/dpaa2_hw_dpbp.c
> @@ -0,0 +1,137 @@
> +/*-
> + * BSD LICENSE
> + *
> + * Copyright (c) 2016 Freescale Semiconductor, Inc. All rights reserved.
> + * Copyright (c) 2016 NXP. All rights reserved.
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions
> + * are met:
> + *
> + * * Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + * * Redistributions in binary form must reproduce the above copyright
> + * notice, this list of conditions and the following disclaimer in
> + * the documentation and/or other materials provided with the
> + * distribution.
> + * * Neither the name of Freescale Semiconductor, Inc nor the names of its
> + * contributors may be used to endorse or promote products derived
> + * from this software without specific prior written permission.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> + */
> +
> +#include <unistd.h>
> +#include <stdio.h>
> +#include <sys/types.h>
> +#include <string.h>
> +#include <stdlib.h>
> +#include <fcntl.h>
> +#include <errno.h>
> +
> +#include <rte_malloc.h>
> +#include <rte_memcpy.h>
> +#include <rte_string_fns.h>
> +#include <rte_cycles.h>
> +#include <rte_kvargs.h>
> +#include <rte_dev.h>
> +#include <rte_ethdev.h>
> +
> +#include <fslmc_logs.h>
> +#include <fslmc_vfio.h>
> +#include <mc/fsl_dpbp.h>
> +#include "portal/dpaa2_hw_pvt.h"
> +#include "portal/dpaa2_hw_dpio.h"
> +
> +TAILQ_HEAD(dpbp_device_list, dpaa2_dpbp_dev);
> +static struct dpbp_device_list *dpbp_dev_list; /*!< DPBP device list */
> +
> +int
> +dpaa2_create_dpbp_device(
> + int dpbp_id)
> +{
> + struct dpaa2_dpbp_dev *dpbp_node;
> + int ret;
> +
> + if (!dpbp_dev_list) {
> + dpbp_dev_list = malloc(sizeof(struct dpbp_device_list));
> + if (!dpbp_dev_list) {
> + PMD_INIT_LOG(ERR, "Memory alloc failed in DPBP list\n");
> + return -1;
> + }
> + /* Initialize the DPBP List */
> + TAILQ_INIT(dpbp_dev_list);
> + }
> +
> + /* Allocate DPAA2 dpbp handle */
> + dpbp_node = (struct dpaa2_dpbp_dev *)
> + malloc(sizeof(struct dpaa2_dpbp_dev));
> + if (!dpbp_node) {
> + PMD_INIT_LOG(ERR, "Memory allocation failed for DPBP Device");
> + return -1;
> + }
> +
> + /* Open the dpbp object */
> + dpbp_node->dpbp.regs = mcp_ptr_list[MC_PORTAL_INDEX];
> + ret = dpbp_open(&dpbp_node->dpbp,
> + CMD_PRI_LOW, dpbp_id, &dpbp_node->token);
> + if (ret) {
> + PMD_INIT_LOG(ERR, "Resource alloc failure with err code: %d",
> + ret);
> + free(dpbp_node);
> + return -1;
> + }
> +
> + /* Clean the device first */
> + ret = dpbp_reset(&dpbp_node->dpbp, CMD_PRI_LOW, dpbp_node->token);
> + if (ret) {
> + PMD_INIT_LOG(ERR, "Failure cleaning dpbp device with"
> + " error code %d\n", ret);
> + return -1;
> + }
> +
> + dpbp_node->dpbp_id = dpbp_id;
> + rte_atomic16_init(&dpbp_node->in_use);
> +
> + TAILQ_INSERT_HEAD(dpbp_dev_list, dpbp_node, next);
> +
> + PMD_INIT_LOG(DEBUG, "Buffer pool resource initialized %d", dpbp_id);
> +
> + return 0;
> +}
> +
> +struct dpaa2_dpbp_dev *dpaa2_alloc_dpbp_dev(void)
> +{
> + struct dpaa2_dpbp_dev *dpbp_dev = NULL;
> +
> + /* Get DPBP dev handle from list using index */
> + TAILQ_FOREACH(dpbp_dev, dpbp_dev_list, next) {
> + if (dpbp_dev && rte_atomic16_test_and_set(&dpbp_dev->in_use))
> + break;
> + }
> +
> + return dpbp_dev;
> +}
> +
> +void dpaa2_free_dpbp_dev(struct dpaa2_dpbp_dev *dpbp)
> +{
> + struct dpaa2_dpbp_dev *dpbp_dev = NULL;
> +
> + /* Match DPBP handle and mark it free */
> + TAILQ_FOREACH(dpbp_dev, dpbp_dev_list, next) {
> + if (dpbp_dev == dpbp) {
> + rte_atomic16_dec(&dpbp_dev->in_use);
> + return;
> + }
> + }
> +}
> diff --git a/drivers/bus/fslmc/portal/dpaa2_hw_pvt.h b/drivers/bus/fslmc/portal/dpaa2_hw_pvt.h
> index ef3eb71..3b846a0 100644
> --- a/drivers/bus/fslmc/portal/dpaa2_hw_pvt.h
> +++ b/drivers/bus/fslmc/portal/dpaa2_hw_pvt.h
> @@ -41,6 +41,13 @@
> #define MC_PORTAL_INDEX 0
> #define NUM_DPIO_REGIONS 2
>
> +#define MEMPOOL_F_HW_PKT_POOL 0x8000 /**< mpool flag to check offloaded pool */
> +
> +/* Maximum release/acquire from QBMAN */
> +#define DPAA2_MBUF_MAX_ACQ_REL 7
> +
> +#define MAX_BPID 256
> +
> struct dpaa2_dpio_dev {
> TAILQ_ENTRY(dpaa2_dpio_dev) next;
> /**< Pointer to Next device instance */
> @@ -63,6 +70,18 @@ struct dpaa2_dpio_dev {
> int32_t hw_id; /**< An unique ID of this DPIO device instance */
> };
>
> +struct dpaa2_dpbp_dev {
> + TAILQ_ENTRY(dpaa2_dpbp_dev) next;
> + /**< Pointer to Next device instance */
> + struct fsl_mc_io dpbp; /** handle to DPBP portal object */
> + uint16_t token;
> + rte_atomic16_t in_use;
> + uint32_t dpbp_id; /*HW ID for DPBP object */
> +};
> +
> /*! Global MCP list */
> extern void *(*mcp_ptr_list);
> +struct dpaa2_dpbp_dev *dpaa2_alloc_dpbp_dev(void);
> +void dpaa2_free_dpbp_dev(struct dpaa2_dpbp_dev *dpbp);
> +
> #endif
> diff --git a/drivers/bus/fslmc/rte_pmd_fslmcbus_version.map b/drivers/bus/fslmc/rte_pmd_fslmcbus_version.map
> index 4236377..76029b9 100644
> --- a/drivers/bus/fslmc/rte_pmd_fslmcbus_version.map
> +++ b/drivers/bus/fslmc/rte_pmd_fslmcbus_version.map
> @@ -2,6 +2,8 @@ DPDK_17.02 {
> global:
>
> dpaa2_affine_qbman_swp;
> + dpaa2_alloc_dpbp_dev;
> + dpaa2_free_dpbp_dev;
> dpbp_disable;
> dpbp_enable;
> dpbp_get_attributes;
> diff --git a/drivers/common/Makefile b/drivers/common/Makefile
> index 434280f..0a6d8db 100644
> --- a/drivers/common/Makefile
> +++ b/drivers/common/Makefile
> @@ -34,6 +34,9 @@ include $(RTE_SDK)/mk/rte.vars.mk
> CONFIG_RTE_LIBRTE_DPAA2_COMMON = $(CONFIG_RTE_LIBRTE_DPAA2_PMD)
>
> ifneq ($(CONFIG_RTE_LIBRTE_DPAA2_COMMON),y)
> +CONFIG_RTE_LIBRTE_DPAA2_COMMON = $(CONFIG_RTE_LIBRTE_DPAA2_POOL)
> +endif
> +ifneq ($(CONFIG_RTE_LIBRTE_DPAA2_COMMON),y)
> CONFIG_RTE_LIBRTE_DPAA2_COMMON = $(CONFIG_RTE_LIBRTE_FSLMC_BUS)
> endif
>
> diff --git a/drivers/pool/Makefile b/drivers/pool/Makefile
> new file mode 100644
> index 0000000..4325edd
> --- /dev/null
> +++ b/drivers/pool/Makefile
> @@ -0,0 +1,38 @@
> +# BSD LICENSE
> +#
> +# Copyright(c) 2016 NXP. All rights reserved.
> +# All rights reserved.
> +#
> +# Redistribution and use in source and binary forms, with or without
> +# modification, are permitted provided that the following conditions
> +# are met:
> +#
> +# * Redistributions of source code must retain the above copyright
> +# notice, this list of conditions and the following disclaimer.
> +# * Redistributions in binary form must reproduce the above copyright
> +# notice, this list of conditions and the following disclaimer in
> +# the documentation and/or other materials provided with the
> +# distribution.
> +# * Neither the name of NXP nor the names of its
> +# contributors may be used to endorse or promote products derived
> +# from this software without specific prior written permission.
> +#
> +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> +# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> +# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> +# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> +# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> +# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> +# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> +# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> +# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> +# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> +
> +include $(RTE_SDK)/mk/rte.vars.mk
> +
> +CONFIG_RTE_LIBRTE_DPAA2_POOL = $(CONFIG_RTE_LIBRTE_DPAA2_PMD)
> +
> +DIRS-$(CONFIG_RTE_LIBRTE_DPAA2_COMMON) += dpaa2
> +
> +include $(RTE_SDK)/mk/rte.subdir.mk
> diff --git a/drivers/pool/dpaa2/Makefile b/drivers/pool/dpaa2/Makefile
> new file mode 100644
> index 0000000..9494756
> --- /dev/null
> +++ b/drivers/pool/dpaa2/Makefile
> @@ -0,0 +1,67 @@
> +# BSD LICENSE
> +#
> +# Copyright(c) 2016 NXP. All rights reserved.
> +# All rights reserved.
> +#
> +# Redistribution and use in source and binary forms, with or without
> +# modification, are permitted provided that the following conditions
> +# are met:
> +#
> +# * Redistributions of source code must retain the above copyright
> +# notice, this list of conditions and the following disclaimer.
> +# * Redistributions in binary form must reproduce the above copyright
> +# notice, this list of conditions and the following disclaimer in
> +# the documentation and/or other materials provided with the
> +# distribution.
> +# * Neither the name of NXP nor the names of its
> +# contributors may be used to endorse or promote products derived
> +# from this software without specific prior written permission.
> +#
> +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> +# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> +# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> +# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> +# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> +# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> +# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> +# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> +# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> +# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> +
> +include $(RTE_SDK)/mk/rte.vars.mk
> +
> +#
> +# library name
> +#
> +LIB = librte_pmd_dpaa2_pool.a
> +
> +ifeq ($(CONFIG_RTE_LIBRTE_DPAA2_DEBUG_INIT),y)
> +CFLAGS += -O0 -g
> +CFLAGS += "-Wno-error"
> +else
> +CFLAGS += -O3
> +CFLAGS += $(WERROR_FLAGS)
> +endif
> +
> +CFLAGS += -I$(RTE_SDK)/drivers/bus/fslmc
> +CFLAGS += -I$(RTE_SDK)/drivers/common/dpaa2/qbman/include
> +CFLAGS += -I$(RTE_SDK)/lib/librte_eal/linuxapp/eal
> +
> +# versioning export map
> +EXPORT_MAP := rte_pmd_dpaa2_pool_version.map
> +
> +# Lbrary version
> +LIBABIVER := 1
> +
> +# all source are stored in SRCS-y
> +#
> +SRCS-$(CONFIG_RTE_LIBRTE_DPAA2_COMMON) += dpaa2_hw_mempool.c
> +
> +# library dependencies
> +DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_COMMON) += lib/librte_eal
> +DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_COMMON) += lib/librte_mempool
> +DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_COMMON) += lib/librte_pmd_dpaa2_qbman
> +DEPDIRS-$(CONFIG_RTE_LIBRTE_DPAA2_COMMON) += lib/librte_pmd_fslmcbus
> +
> +include $(RTE_SDK)/mk/rte.lib.mk
> diff --git a/drivers/pool/dpaa2/dpaa2_hw_mempool.c b/drivers/pool/dpaa2/dpaa2_hw_mempool.c
> new file mode 100644
> index 0000000..f36e909
> --- /dev/null
> +++ b/drivers/pool/dpaa2/dpaa2_hw_mempool.c
> @@ -0,0 +1,339 @@
> +/*-
> + * BSD LICENSE
> + *
> + * Copyright (c) 2016 Freescale Semiconductor, Inc. All rights reserved.
> + * Copyright (c) 2016 NXP. All rights reserved.
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions
> + * are met:
> + *
> + * * Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + * * Redistributions in binary form must reproduce the above copyright
> + * notice, this list of conditions and the following disclaimer in
> + * the documentation and/or other materials provided with the
> + * distribution.
> + * * Neither the name of Freescale Semiconductor, Inc nor the names of its
> + * contributors may be used to endorse or promote products derived
> + * from this software without specific prior written permission.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> + */
> +
> +#include <unistd.h>
> +#include <stdio.h>
> +#include <sys/types.h>
> +#include <string.h>
> +#include <stdlib.h>
> +#include <fcntl.h>
> +#include <errno.h>
> +
> +#include <rte_mbuf.h>
> +#include <rte_ethdev.h>
> +#include <rte_malloc.h>
> +#include <rte_memcpy.h>
> +#include <rte_string_fns.h>
> +#include <rte_cycles.h>
> +#include <rte_kvargs.h>
> +#include <rte_dev.h>
> +#include <rte_ethdev.h>
> +
> +#include <fslmc_logs.h>
> +#include <mc/fsl_dpbp.h>
> +#include <portal/dpaa2_hw_pvt.h>
> +#include <portal/dpaa2_hw_dpio.h>
> +#include "dpaa2_hw_mempool.h"
> +
> +struct dpaa2_bp_info bpid_info[MAX_BPID];
> +static struct dpaa2_bp_list *h_bp_list;
> +
> +static int
> +hw_mbuf_create_pool(struct rte_mempool *mp)
> +{
> + struct dpaa2_bp_list *bp_list;
> + struct dpaa2_dpbp_dev *avail_dpbp;
> + struct dpbp_attr dpbp_attr;
> + uint32_t bpid;
> + int ret;
> +
> + avail_dpbp = dpaa2_alloc_dpbp_dev();
> +
> + if (!avail_dpbp) {
> + PMD_DRV_LOG(ERR, "DPAA2 resources not available");
> + return -1;
> + }
> +
> + if (unlikely(!DPAA2_PER_LCORE_DPIO)) {
> + ret = dpaa2_affine_qbman_swp();
> + if (ret) {
> + RTE_LOG(ERR, PMD, "Failure in affining portal\n");
> + return 0;
> + }
> + }
> +
> + ret = dpbp_enable(&avail_dpbp->dpbp, CMD_PRI_LOW, avail_dpbp->token);
> + if (ret != 0) {
> + PMD_INIT_LOG(ERR, "Resource enable failure with"
> + " err code: %d\n", ret);
> + return -1;
> + }
> +
> + ret = dpbp_get_attributes(&avail_dpbp->dpbp, CMD_PRI_LOW,
> + avail_dpbp->token, &dpbp_attr);
> + if (ret != 0) {
> + PMD_INIT_LOG(ERR, "Resource read failure with"
> + " err code: %d\n", ret);
> + ret = dpbp_disable(&avail_dpbp->dpbp, CMD_PRI_LOW,
> + avail_dpbp->token);
> + return -1;
> + }
> +
> + /* Allocate the bp_list which will be added into global_bp_list */
> + bp_list = (struct dpaa2_bp_list *)malloc(sizeof(struct dpaa2_bp_list));
> + if (!bp_list) {
> + PMD_INIT_LOG(ERR, "No heap memory available");
> + return -1;
> + }
> +
> + /* Set parameters of buffer pool list */
> + bp_list->buf_pool.num_bufs = mp->size;
> + bp_list->buf_pool.size = mp->elt_size
> + - sizeof(struct rte_mbuf) - rte_pktmbuf_priv_size(mp);
> + bp_list->buf_pool.bpid = dpbp_attr.bpid;
> + bp_list->buf_pool.h_bpool_mem = NULL;
I am guessing that vfio-platform takes care of allocating memory to bman for
buffer management (Beside mapping the platform resource) Or bman hw internally
takes care and return a bpid handle to the application, right. Thus you don;t
care for allocating mz area for this mempool.
> + bp_list->buf_pool.mp = mp;
> + bp_list->buf_pool.dpbp_node = avail_dpbp;
> + bp_list->next = h_bp_list;
> +
> + bpid = dpbp_attr.bpid;
> +
> +
> + bpid_info[bpid].meta_data_size = sizeof(struct rte_mbuf)
> + + rte_pktmbuf_priv_size(mp);
> + bpid_info[bpid].bp_list = bp_list;
> + bpid_info[bpid].bpid = bpid;
> +
> + mp->pool_data = (void *)&bpid_info[bpid];
> +
> + PMD_INIT_LOG(DEBUG, "BP List created for bpid =%d", dpbp_attr.bpid);
> +
> + h_bp_list = bp_list;
> + /* Identification for our offloaded pool_data structure
> + */
> + mp->flags |= MEMPOOL_F_HW_PKT_POOL;
> + return 0;
> +}
> +
> +static void
> +hw_mbuf_free_pool(struct rte_mempool *mp)
> +{
> + struct dpaa2_bp_info *bpinfo;
> + struct dpaa2_bp_list *bp;
> + struct dpaa2_dpbp_dev *dpbp_node;
> +
> + if (!mp->pool_data) {
> + PMD_DRV_LOG(ERR, "Not a valid dpaa22 pool");
> + return;
> + }
> +
> + bpinfo = (struct dpaa2_bp_info *)mp->pool_data;
> + bp = bpinfo->bp_list;
> + dpbp_node = bp->buf_pool.dpbp_node;
> +
> + dpbp_disable(&(dpbp_node->dpbp), CMD_PRI_LOW, dpbp_node->token);
> +
> + if (h_bp_list == bp) {
> + h_bp_list = h_bp_list->next;
> + } else { /* if it is not the first node */
> + struct dpaa2_bp_list *prev = h_bp_list, *temp;
> + temp = h_bp_list->next;
> + while (temp) {
> + if (temp == bp) {
> + prev->next = temp->next;
> + free(bp);
why not use rte_free()?
> + break;
> + }
> + prev = temp;
> + temp = temp->next;
> + }
> + }
> +
> + dpaa2_free_dpbp_dev(dpbp_node);
> +}
> +
> +static
> +void dpaa2_mbuf_release(struct rte_mempool *pool __rte_unused,
> + void * const *obj_table,
> + uint32_t bpid,
> + uint32_t meta_data_size,
> + int count)
> +{
> + struct qbman_release_desc releasedesc;
> + struct qbman_swp *swp;
> + int ret;
> + int i, n;
> + uint64_t bufs[DPAA2_MBUF_MAX_ACQ_REL];
> +
> + if (unlikely(!DPAA2_PER_LCORE_DPIO)) {
> + ret = dpaa2_affine_qbman_swp();
> + if (ret != 0) {
> + RTE_LOG(ERR, PMD, "Failed to allocate IO portal");
> + return;
> + }
> + }
> + swp = DPAA2_PER_LCORE_PORTAL;
> +
> + /* Create a release descriptor required for releasing
> + * buffers into QBMAN
> + */
> + qbman_release_desc_clear(&releasedesc);
> + qbman_release_desc_set_bpid(&releasedesc, bpid);
> +
> + n = count % DPAA2_MBUF_MAX_ACQ_REL;
> +
> + /* convert mbuf to buffers for the remainder*/
> + for (i = 0; i < n ; i++)
> + bufs[i] = (uint64_t)obj_table[i] + meta_data_size;
> +
> + /* feed them to bman*/
> + do {
> + ret = qbman_swp_release(swp, &releasedesc, bufs, n);
> + } while (ret == -EBUSY);
> +
> + /* if there are more buffers to free */
> + while (n < count) {
> + /* convert mbuf to buffers */
> + for (i = 0; i < DPAA2_MBUF_MAX_ACQ_REL; i++)
> + bufs[i] = (uint64_t)obj_table[n + i] + meta_data_size;
> +
> + do {
> + ret = qbman_swp_release(swp, &releasedesc, bufs,
> + DPAA2_MBUF_MAX_ACQ_REL);
> + } while (ret == -EBUSY);
> + n += DPAA2_MBUF_MAX_ACQ_REL;
> + }
> +}
> +
> +int hw_mbuf_alloc_bulk(struct rte_mempool *pool,
> + void **obj_table, unsigned int count)
> +{
> +#ifdef RTE_LIBRTE_DPAA2_DEBUG_DRIVER
> + static int alloc;
> +#endif
> + struct qbman_swp *swp;
> + uint32_t mbuf_size;
> + uint16_t bpid;
> + uint64_t bufs[DPAA2_MBUF_MAX_ACQ_REL];
> + int i, ret;
> + unsigned int n = 0;
> + struct dpaa2_bp_info *bp_info;
> +
> + bp_info = mempool_to_bpinfo(pool);
> +
> + if (!(bp_info->bp_list)) {
> + RTE_LOG(ERR, PMD, "DPAA2 buffer pool not configured\n");
> + return -2;
> + }
> +
> + bpid = bp_info->bpid;
> +
> + if (unlikely(!DPAA2_PER_LCORE_DPIO)) {
> + ret = dpaa2_affine_qbman_swp();
> + if (ret != 0) {
> + RTE_LOG(ERR, PMD, "Failed to allocate IO portal");
> + return -1;
> + }
> + }
> + swp = DPAA2_PER_LCORE_PORTAL;
> +
> + mbuf_size = sizeof(struct rte_mbuf) + rte_pktmbuf_priv_size(pool);
> +
> + while (n < count) {
> + /* Acquire is all-or-nothing, so we drain in 7s,
> + * then the remainder.
> + */
> + if ((count - n) > DPAA2_MBUF_MAX_ACQ_REL) {
> + ret = qbman_swp_acquire(swp, bpid, bufs,
> + DPAA2_MBUF_MAX_ACQ_REL);
> + } else {
> + ret = qbman_swp_acquire(swp, bpid, bufs,
> + count - n);
> + }
> + /* In case of less than requested number of buffers available
> + * in pool, qbman_swp_acquire returns 0
> + */
> + if (ret <= 0) {
> + PMD_TX_LOG(ERR, "Buffer acquire failed with"
> + " err code: %d", ret);
> + /* The API expect the exact number of requested bufs */
> + /* Releasing all buffers allocated */
> + dpaa2_mbuf_release(pool, obj_table, bpid,
> + bp_info->meta_data_size, n);
> + return -1;
> + }
> + /* assigning mbuf from the acquired objects */
> + for (i = 0; (i < ret) && bufs[i]; i++) {
> + /* TODO-errata - observed that bufs may be null
> + * i.e. first buffer is valid,
> + * remaining 6 buffers may be null
> + */
> + obj_table[n] = (struct rte_mbuf *)(bufs[i] - mbuf_size);
> + rte_mbuf_refcnt_set((struct rte_mbuf *)obj_table[n], 0);
> + PMD_TX_LOG(DEBUG, "Acquired %p address %p from BMAN",
> + (void *)bufs[i], (void *)obj_table[n]);
> + n++;
> + }
> + }
> +
> +#ifdef RTE_LIBRTE_DPAA2_DEBUG_DRIVER
> + alloc += n;
> + PMD_TX_LOG(DEBUG, "Total = %d , req = %d done = %d",
> + alloc, count, n);
> +#endif
> + return 0;
> +}
> +
> +static int
> +hw_mbuf_free_bulk(struct rte_mempool *pool,
> + void * const *obj_table, unsigned int n)
> +{
> + struct dpaa2_bp_info *bp_info;
> +
> + bp_info = mempool_to_bpinfo(pool);
> + if (!(bp_info->bp_list)) {
> + RTE_LOG(ERR, PMD, "DPAA2 buffer pool not configured");
> + return -1;
> + }
> + dpaa2_mbuf_release(pool, obj_table, bp_info->bpid,
> + bp_info->meta_data_size, n);
> +
> + return 0;
> +}
> +
> +static unsigned
> +hw_mbuf_get_count(const struct rte_mempool *mp __rte_unused)
> +{
> + return 0;
> +}
> +
> +struct rte_mempool_ops dpaa2_mpool_ops = {
> + .name = "dpaa2",
> + .alloc = hw_mbuf_create_pool,
> + .free = hw_mbuf_free_pool,
> + .enqueue = hw_mbuf_free_bulk,
> + .dequeue = hw_mbuf_alloc_bulk,
> + .get_count = hw_mbuf_get_count,
> +};
> +
> +MEMPOOL_REGISTER_OPS(dpaa2_mpool_ops);
> diff --git a/drivers/pool/dpaa2/dpaa2_hw_mempool.h b/drivers/pool/dpaa2/dpaa2_hw_mempool.h
> new file mode 100644
> index 0000000..2cd2564
> --- /dev/null
> +++ b/drivers/pool/dpaa2/dpaa2_hw_mempool.h
> @@ -0,0 +1,95 @@
> +/*-
> + * BSD LICENSE
> + *
> + * Copyright (c) 2016 Freescale Semiconductor, Inc. All rights reserved.
> + * Copyright (c) 2016 NXP. All rights reserved.
> + *
> + * Redistribution and use in source and binary forms, with or without
> + * modification, are permitted provided that the following conditions
> + * are met:
> + *
> + * * Redistributions of source code must retain the above copyright
> + * notice, this list of conditions and the following disclaimer.
> + * * Redistributions in binary form must reproduce the above copyright
> + * notice, this list of conditions and the following disclaimer in
> + * the documentation and/or other materials provided with the
> + * distribution.
> + * * Neither the name of Freescale Semiconductor, Inc nor the names of its
> + * contributors may be used to endorse or promote products derived
> + * from this software without specific prior written permission.
> + *
> + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> + */
> +
> +#ifndef _DPAA2_HW_DPBP_H_
> +#define _DPAA2_HW_DPBP_H_
> +
> +#define DPAA2_MAX_BUF_POOLS 8
> +
> +struct buf_pool_cfg {
> + void *addr; /*!< The address from where DPAA2 will carve out the
> + * buffers. 'addr' should be 'NULL' if user wants
> + * to create buffers from the memory which user
> + * asked DPAA2 to reserve during 'nadk init'
> + */
> + phys_addr_t phys_addr; /*!< corresponding physical address
> + * of the memory provided in addr
> + */
> + uint32_t num; /*!< number of buffers */
> + uint32_t size; /*!< size of each buffer. 'size' should include
> + * any headroom to be reserved and alignment
> + */
> + uint16_t align; /*!< Buffer alignment (in bytes) */
> + uint16_t bpid; /*!< The buffer pool id. This will be filled
> + *in by DPAA2 for each buffer pool
> + */
> +};
> +
> +struct buf_pool {
> + uint32_t size;
> + uint32_t num_bufs;
> + uint16_t bpid;
> + uint8_t *h_bpool_mem;
> + struct rte_mempool *mp;
> + struct dpaa2_dpbp_dev *dpbp_node;
> +};
> +
> +/*!
> + * Buffer pool list configuration structure. User need to give DPAA2 the
> + * valid number of 'num_buf_pools'.
> + */
> +struct dpaa2_bp_list_cfg {
> + struct buf_pool_cfg buf_pool; /* Configuration of each buffer pool*/
> +};
> +
isn't dpaa2_bp_list_cfg{} thus buf_pool_cfg{} redundant? I couldn't find
struct used in this patch Or perhaps someother patch using them, If so then
pl. ignore my comment.
> +struct dpaa2_bp_list {
> + struct dpaa2_bp_list *next;
> + struct rte_mempool *mp;
> + struct buf_pool buf_pool;
> +};
> +
> +struct dpaa2_bp_info {
> + uint32_t meta_data_size;
> + uint32_t bpid;
> + struct dpaa2_bp_list *bp_list;
> +};
> +
> +#define mempool_to_bpinfo(mp) ((struct dpaa2_bp_info *)(mp)->pool_data)
> +#define mempool_to_bpid(mp) ((mempool_to_bpinfo(mp))->bpid)
> +
> +extern struct dpaa2_bp_info bpid_info[MAX_BPID];
> +
> +int hw_mbuf_alloc_bulk(struct rte_mempool *pool,
> + void **obj_table, unsigned int count);
> +
> +#endif /* _DPAA2_HW_DPBP_H_ */
> diff --git a/drivers/pool/dpaa2/rte_pmd_dpaa2_pool_version.map b/drivers/pool/dpaa2/rte_pmd_dpaa2_pool_version.map
> new file mode 100644
> index 0000000..289ab10
> --- /dev/null
> +++ b/drivers/pool/dpaa2/rte_pmd_dpaa2_pool_version.map
> @@ -0,0 +1,8 @@
> +DPDK_17.02 {
> + global:
> +
> + bpid_info;
> + hw_mbuf_alloc_bulk;
> +
> + local: *;
> +};
> diff --git a/mk/rte.app.mk b/mk/rte.app.mk
> index 438fa2c..1bfb804 100644
> --- a/mk/rte.app.mk
> +++ b/mk/rte.app.mk
> @@ -112,6 +112,7 @@ _LDLIBS-$(CONFIG_RTE_LIBRTE_CXGBE_PMD) += -lrte_pmd_cxgbe
> ifeq ($(CONFIG_RTE_LIBRTE_DPAA2_COMMON),y)
> _LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += -lrte_pmd_dpaa2
> _LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += -lrte_pmd_dpaa2_qbman
> +_LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += -lrte_pmd_dpaa2_pool
> _LDLIBS-$(CONFIG_RTE_LIBRTE_DPAA2_PMD) += -lrte_pmd_fslmcbus
> endif
> _LDLIBS-$(CONFIG_RTE_LIBRTE_E1000_PMD) += -lrte_pmd_e1000
> --
> 2.7.4
>