From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <hemant.agrawal@nxp.com>
Received: from NAM03-BY2-obe.outbound.protection.outlook.com
(mail-by2nam03on0045.outbound.protection.outlook.com [104.47.42.45])
by dpdk.org (Postfix) with ESMTP id 5EAAC282
for <dev@dpdk.org>; Tue, 3 Jan 2017 09:22:17 +0100 (CET)
Received: from BLUPR0301CA0002.namprd03.prod.outlook.com (10.162.113.140) by
DM5PR03MB2476.namprd03.prod.outlook.com (10.168.233.22) with Microsoft SMTP
Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id
15.1.817.10; Tue, 3 Jan 2017 08:22:15 +0000
Received: from BY2FFO11FD048.protection.gbl (2a01:111:f400:7c0c::144) by
BLUPR0301CA0002.outlook.office365.com (2a01:111:e400:5259::12) with Microsoft
SMTP Server (version=TLS1_2,
cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.817.10 via
Frontend Transport; Tue, 3 Jan 2017 08:22:14 +0000
Authentication-Results: spf=fail (sender IP is 192.88.158.2)
smtp.mailfrom=nxp.com; caviumnetworks.com; dkim=none (message not signed)
header.d=none;caviumnetworks.com; dmarc=fail action=none
header.from=nxp.com;caviumnetworks.com; dkim=none (message not signed)
header.d=none;
Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not
designate 192.88.158.2 as permitted sender) receiver=protection.outlook.com;
client-ip=192.88.158.2; helo=az84smr01.freescale.net;
Received: from az84smr01.freescale.net (192.88.158.2) by
BY2FFO11FD048.mail.protection.outlook.com (10.1.15.176) with Microsoft SMTP
Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.803.8
via Frontend Transport; Tue, 3 Jan 2017 08:22:09 +0000
Received: from [127.0.0.1] ([10.232.133.65])
by az84smr01.freescale.net (8.14.3/8.14.0) with ESMTP id v038M0Kv018736;
Tue, 3 Jan 2017 01:22:06 -0700
To: Santosh Shukla <santosh.shukla@caviumnetworks.com>, Shreyansh Jain
<shreyansh.jain@nxp.com>
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>
<20161229070807.GA7823@santosh-Latitude-E5530-non-vPro>
CC: <dev@dpdk.org>, <thomas.monjalon@6wind.com>, <bruce.richardson@intel.com>,
<john.mcnamara@intel.com>, <ferruh.yigit@intel.com>,
<jerin.jacob@caviumnetworks.com>
From: Hemant Agrawal <hemant.agrawal@nxp.com>
Message-ID: <cad37039-1dfd-5c68-29ab-0fc804afffd8@nxp.com>
Date: Tue, 3 Jan 2017 13:52:00 +0530
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101
Thunderbird/45.5.1
MIME-Version: 1.0
In-Reply-To: <20161229070807.GA7823@santosh-Latitude-E5530-non-vPro>
Content-Type: text/plain; charset="windows-1252"; format=flowed
Content-Transfer-Encoding: 7bit
X-EOPAttributedMessage: 0
X-Matching-Connectors: 131279053329933422;
(91ab9b29-cfa4-454e-5278-08d120cd25b8); ()
X-Forefront-Antispam-Report: CIP:192.88.158.2; IPV:NLI; CTRY:US; EFV:NLI;
SFV:NSPM;
SFS:(10009020)(6009001)(7916002)(336005)(39410400002)(39860400002)(39450400003)(39400400002)(39380400002)(39840400002)(39850400002)(2980300002)(1109001)(1110001)(339900001)(377454003)(24454002)(199003)(189002)(92566002)(68736007)(69596002)(229853002)(97736004)(189998001)(230700001)(65806001)(85426001)(47776003)(5001770100001)(33646002)(93886004)(356003)(120886001)(54356999)(50986999)(65956001)(76176999)(36756003)(5660300001)(7126002)(104016004)(6636002)(65826007)(2906002)(31686004)(81166006)(38730400001)(106466001)(8676002)(105606002)(4326007)(2950100002)(6666003)(54906002)(8936002)(7246003)(23746002)(4001350100001)(64126003)(626004)(50466002)(31696002)(83506001)(77096006)(8656002)(81156014)(86362001)(305945005)(575784001);
DIR:OUT; SFP:1101; SCL:1; SRVR:DM5PR03MB2476; H:az84smr01.freescale.net; FPR:;
SPF:Fail; PTR:InfoDomainNonexistent; A:1; MX:1; LANG:en;
X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11FD048;
1:6uYILGVd0oqUZdkMf5LTXVuPmUe7Q6G+VTr639tbEt5yf5x3Eb5K3jEPduCOwVy7pTA/zSl+BygNLYK0IBoSwtGFJiqBgUK/BmAKYN/AKX/vun5GgLZP/NnXIDM0FJfWWO6fEWWRGQ+z9O1030h8CuOn+Rf5r/f1TKw3urrH4cM99vait29pTSSDGJvZwIWTRtBTGsTff7A9G1ayOXV+kcG2I7YUnYDS9h7GwNEu4Mmkmp7oEz5uMAQR4wN8kjRWLJynND66vikOLvraKuRF48MeSJTvdZHEzTTnBZhBhFejL66Pvm9e8MuyMbUkEW6csZ1jxFuBwdFxM52/BJAmD+wsd29KgqP8+/lEfOFBJsl0WPE+FC70kR0HQlc93a/GhNN5li4FithNCHjCJdS7fsceNhU73u31VWRcnseH0XyrmD497rLA74GdrbVmYhjsmqLu0Lcm5bdFhF8zLJdLqFm89wGRfIjGR8GXwcwOoqfpgDCH6KSUBLa4j1kv6dcf2NJ6WQ0/kZ4N0yQ+mO0OBZhe1FxfJ3GsJpXcjW6Q4URnGPPwCI1/x5CO9zOcFtbHwbapd39+FOAs6ghRx62BJ4ezS9Uc0au2hSZR/P8cb2KkLdMYv5zmXAUPuDR2rcdAJSnFXAGaCBdM0z63rF+Nhw==
X-MS-Office365-Filtering-Correlation-Id: f61d76cb-12b5-4b20-246e-08d433b19ea5
X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:DM5PR03MB2476;
X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2476;
3:5zuoETJHAXwwmkFP2yIJoSWIZ7l01aTCPoS+icUERzA+MRbb1rU6M6oYj2wHgjTTINOQ1caCBXxjVJ0fzHlLK2n/usLsC6Ejsd1cbv78Mkm5aRNBedKdLs6TPObELaxRO7zQ5Y3MQvvM64sCi8wnHCpiG9vI9SULXdbAu1iX9jylgL/WT6fQAyzmGK2BREH8GziVr37poIj6yWBxlFMz1taQMGPjYfLhnSuRbpDIL98y7K0AB8fsBM1fSWt78tukRGcQc7gjb4OUTrYgk9beEFMlYtWJTTmhgtiSTrJvNzi+8XTC3SDwbuLi8x4d392RBithG5SOU08cXfRbIC0OqwotQZeuyBpf+Yn30M/T30sH0NBEwvv93AxSX3eC1HJG
X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2476;
25:JK079+Dj9F8xwwKxtkV8ByoyTGOi1AInOhuuf54zjvvR+fO5MiTEhGtlvAfpV/T5uYakz7mFdi7ByT18SKMZ5TYLMof0idH+pmcSCXHmtawv3FoXXmi9ltOqRG8EVtaLci0JRso6XF67gsrhEsRkQTDfltyv5how71iOvZfIPWGMOvvDcanvsg7QR33r4AUyWYXl/pS4zHYqrWy3UJMTLKpYtQJXRdnbjJAmgwJt1hs1TMySO+v+F0NlJ9tZUodPKtMQCHM0F76MMcxiELhMO5NPRJj/r6pdMaaO48cQUMncgVhAEp2PdMwgRd54ZAbLaAMAPYEDJsCL99mryue7oQsxexn9jNshuFzwzscGBZU7sMkhnEgB5DQtdyeI8S9RRRY0gL1/TNwk2/CXtOfBhT+l6PRTQvzLRSrnAAr3aNpiFQXNrcn+CGvQl9ERx+Q7ykvjNKr6HQjzehEBMEe0kUUxwLXsTosvTXtkkTYeG8OwnTspwDZIOGY3E1q6PK3BYrlBA67CwHDRePWgni/uYiiCf3DsHXke7BeSkQKF23RytnE03pZRKWQIOicL+a/NtWFmfyY8roFXeNFPztymU3cNoOmuZiNom3CtMwU5xfJULMWRkMjt+lYrVgY0IvazpukdGQ1sk9TIsyrjUubmMuchBxsiYFQ66BrbRzNnp4WbX/Rb1edg7VtufPJSxyTLTkTRK0cJCnKq/OFUENABvQyF61AfpfLkMsqkiGTu1lgaMAGIkWSAVDYmAHontqiP+HUFzg1ofz8TxWiplVphTA==
X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2476;
31:syCiGchdBxKb6gmDDdllmlCfGOTHpYKGRSmw5UQDnobKJLpdBa/WePSG+/AA5fOZIAWarvK8VsA0kCIzCNNjoB9zspZ4zyDoA4gtDqiu0r1JfTvAhdLaVCUII32Nz/27hTDXREwmAiJmSYLLHyh9f7w0L9Iwix8ZMJMVFFIDlJ21k02DIcBIwrKeyo5MrK7G5Whr7xwXTJGql7o1e69T3M2GhmkUpnGADddZAS1n/IIaC6J6v/Npp5238vL/K+UxjPS4H8dacGlUAo0xWSB79vqOUPtes2k22C7z/15Wj4o=
X-Microsoft-Antispam-PRVS: <DM5PR03MB247670054FE52C3A554B103B896E0@DM5PR03MB2476.namprd03.prod.outlook.com>
X-Exchange-Antispam-Report-Test: UriScan:(185117386973197)(275809806118684);
X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0;
RULEID:(6095060)(601004)(2401047)(13024025)(13023025)(13015025)(13017025)(13018025)(8121501046)(5005006)(3002001)(10201501046)(6055026)(6096035)(20161123561025)(20161123559025)(20161123556025)(20161123565025)(20161123563025);
SRVR:DM5PR03MB2476; BCL:0; PCL:0; RULEID:(400006); SRVR:DM5PR03MB2476;
X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2476;
4:4dbWrfYsE68hIuquxopNBH0J6Xx3aQvR0tA95KUhrd3UML9kut63i0qh4mLppPQtT8eCq+5wxUg0F44kCmQSDCGpmtO2yUL5L/mV5zs3hQIENw0Fq+VwAEIX3ehNTITrcy8yr9BT+zEX03RrDzWbXsInHvlRmojGTHfLXKo89hCNQu79YxlbGjM5guZ4cgBQeA8vY4ylN7Jqqrh3W/vxgi36nFtBIUfJr/p6K3a/LKeTJ30E0dq41UkvW95E/yChNvg0fcjB+bIuWuAUlopGcPcK6QY5P+/6aAUC8/UpRc9nVgRXphMF4461L5QNI6dgbnHQMBjjxIURKxjuWrQIlSncDleURqvo/8VyInxRMU7PwXQA3ZrfgBVWbPzs2PYIYC0V8VlJpjc/0t4utLrBvNq93sZo+HBXbcdkDlRaejejf/OIPWZ36LWJ8zX6NIt75Z+t/X1V1n0C6EXlMm1VgJZM2M5X0N7fN3EPHIIVUZA78Y6Vx3sQLKfn7UCzFkp1WY9+08vw57MgvuTyUKTBtCbWiXnKN+zPyzqTV75Y28RMD0SdNep+MDswvDmgm85gufTOxgbEtgRSyYDlQnPSHACfok7fUPDeDnsXkfT/l1e8tBmIQJFbAvqO/icLKZffV1LsVTIdApHa4i1wWHuFn7/YJ+Krz8U/5pMR5EmvsntpuSk8aWegebjazvCNGiQvnT8sFX3O7vczjDbPab6OXjJdx6EWKXwx7hLfpW0lso8WZbJSh0eWzEmUK8oc9fvJ1cOrVw1mDadC5+r4BMbwNA==
X-Forefront-PRVS: 01762B0D64
X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1; DM5PR03MB2476;
23:GUbyFusvdYvVZhAKS+1ukh3O7SuhPjtTFNahE?=
=?Windows-1252?Q?MA4ySyU+Fp0GQ13oSTJEA/HxkTjKsYqq/uePDfinerR7X1b2NVqSoFk9?=
=?Windows-1252?Q?PI8XchkjAlLnEcz52F3VuQWlsL2FPir13UQmX24wMoG/lFEYP6pMSx8A?=
=?Windows-1252?Q?fqLA/2D9bvFi6/sj9USEczFe6pjI1hTEW9Uwo0ShYcRFSCtcCWIn6gDt?=
=?Windows-1252?Q?xAMNlWRUIeufL4kJlr0Y+ENKjLO7FOKHOj5Dy0wFbaPJ7o5JFzwoMc3v?=
=?Windows-1252?Q?RuDP1Wo9cYRK4N4iKvnDngtEIRz3bihjjUt9SDHO549DyDTJ2Ev5AK9n?=
=?Windows-1252?Q?79m64n1vgSE+PKKvRFwUgXSMSDY36/eUjA30l3QfxS1gx6+LnG9HPW0W?=
=?Windows-1252?Q?MXbekNQ7T0n2JNthPWwMK9ufF3u/Bv0aUIKk9MGnp3z6odMoQiJtd6oF?=
=?Windows-1252?Q?HfvEvqaMjRcxol6LVllvPZ+FFLnl6VXA/FIeGPunYt8AwrJOXjPYCsES?=
=?Windows-1252?Q?YlC4jqniOXSyDi40x16N1x36NX7q6hcEV+UmErYNcW/XdIcZmcNtuKc1?=
=?Windows-1252?Q?8fYoA9bIq4VcVBnE3d6eQ4w6V9waVfcR4QzEFft8HZGQcleo4+ErjFj+?=
=?Windows-1252?Q?AwJ4hrLTRXb1z/+CJKgnqCxrXxNH3QePnlLv/OAwLl7fnpIhStDra9ih?=
=?Windows-1252?Q?0VfsiqOOJK9MxQtDtkb3OaFbmd7dpe49a2M0KqK7OAyUBYomWmU59itE?=
=?Windows-1252?Q?5lCmVhmqR/BDmMQ59M38j5JUiaZaqd/mGPJAhYsOWt9cqdD5ybCfcHk1?=
=?Windows-1252?Q?/Sx9T0fOuPAN0WJo7CFXtPP9pENKuNRTGWeDQi6EXDsGqQW1/SaJTLCD?=
=?Windows-1252?Q?OH+Oi+S32D+D19GIwrSQUjb62oFF+ctk4t/g1qTCyaB/uur0j15XKFUg?=
=?Windows-1252?Q?CQnj5qp/oZN1z5cyXuf1kmj2P4VOi1vY1eg/LTNkH/sJ8OEABU4T9Lk7?=
=?Windows-1252?Q?EEiSiOpex/uMTCIdwcQsG/9On8kOVQUwabE5PmUhLwATYkT9iIvpOFud?=
=?Windows-1252?Q?63UFMgjN4lhVH6nvEk2bbhHhaBhsB6YfTEUXo1/GET7AN4B3JwR5/0zi?=
=?Windows-1252?Q?sJEDzlPE6jRMkWQMc75WYA2elUOGNakyNE0WvQNgTABzaGJrgiQwcEno?=
=?Windows-1252?Q?xskGLYWplNe+p1rV8WHKxn/STbRmDvM2ErS9DiT6p/wiqgXCMklVQFrB?=
=?Windows-1252?Q?Iarl4Qcwi85KSOu+7ILEwWnXRkep+Vy9q4As/J2JA08KpgakSXdhG4CI?=
=?Windows-1252?Q?EIaAa6NoqnumCRyaP2zypWtVZul/KvpjfWKcBA9FjOvmvs+Apyl4hYNN?=
=?Windows-1252?Q?AofR6qC9jBBMyy5gJwk3SToubCchUWfrkHctjQNpzEuqHtHtofr5dUoa?=
=?Windows-1252?Q?b6ixttRXUZbYkO4UmZn5tdMNwIHudlbtQ+PWB8i/5txsQJ0pmntrlVP0?=
=?Windows-1252?Q?B2eTuSRbRNi8vehoJCtqSId406w7ZKw1V6kAVngnUGJ5/Z8isY1w2EHI?=
=?Windows-1252?Q?4cKK+JZq1WrDRc24UVaFDXs95wyFYd6vU+TCafHO2i+fJgJ9vL6jRtpR?=
=?Windows-1252?Q?1XWblvBUsSEOt1fLP4rLP7cRTyxtOA9NSgXMYG0RZnZNTVGN5twQJwdl?=
=?Windows-1252?Q?UOM4ooSSEZRQy00CVzL6Uiw+F988wlrG3Hc5VYwEprmIyYiU5W7AFOlS?=
=?Windows-1252?Q?amnPiBfhVWBDBhlzpLf8ZBSUuaj4OJXRRfBWMQJyCb+hcxdogQXbtIOy?=
=?Windows-1252?Q?EQu?=
X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2476;
6:UhyDqGPm3HLyWzJcZr17KiNgtRdlpRXpBprxV69NVOJgcWd6Grz2gdIeYeH8BrBbUMiEXiV+o/YeQUWmeHzmDMpfVzW1+/ot6Z9TA/50+45M6PKkaNcTRgtRumdmA98mWTVsnoisH4p+zfYP1yXU8wxFd957UbUFuQQ6ataij+XJFdgqVmmgaAqASguHypUarWwmV42efb2dIOCQe1nRiAQXRCLD1fWP6A3pA5a20oK7XTOfC7Sj3yrztIrXgbhjOHIzMnywNz514TC2mdwvxWYwu8H3who2dIe654/wL6OvHC9LqOwCL3Xr+Nj42aELOs2DtCyGLz3zwG/HGkOhqMt98ssaX2TKF3kMKyScIs1pMaT7yz5UXJQk6Q08mfmCoWJgQ7bqxus5DK+PKZ4/c5Po7MXWLOCTPldNne72bDUSxyoqqDxmP+C2vppI9NPX;
5:JFKtPQGwiNWekKsDTGNqXZ+RFIGbxptMf6aMmPaXmkpdrc7aKDuabKfcsl7SodSNt/txyp6ruxXmZCmVXdhFd4uCyxe6vNyF1xUaWIrFJj37jXzOItdksC3omf+wElcyplBRZ6fq4tc90iGzH2+8VswRtBWb/r7ccl7XpDDyaqpq3b4cBYATt5LBbgyaRXYI;
24:dobIRmNaRX2HH+6PgoPSAhidgY2yS1aoW5sNwiKbZqWoKrRIshBEO2GHY0hhBboeZ6RdazYHKS3S9st4YAUFR8I/G5GBctnvPfc+wpFk8fM=
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-Microsoft-Exchange-Diagnostics: 1; DM5PR03MB2476;
7:jUrP7o3MnA8rox/ojLHGBlOKpxkugwf193qD9dEcx6wPudRnAgYaTbv02hC3kSVyqDVS6PlwBHPDDjVeB1FC7QX+erKW0bGDWVZ2V07AUeGajdF7j9kgIFv+GdbmIcSygvxZOlrREF0lMgA0fEE4zJujrI6Se/4AJ5pMWBCTtXie7rI2M8/pRAvruoDunHplX+9fteTA3O3DakklCIcmccrWpudWePkHVqfKFQ+Rg5cPUFF2Xo4yQ6xuq+07uli/N8LZTALx0R6nIQXi+7sA8KZgQXCmCrKjI/lMFRvswQyFfzjAvONYuXtOaTFVEfPZaNkvk08UhBOswMBDjAUYSoyTlHLZ4CbhTcO1+MsqPxoa06E/CTEU9Od0KymrfZTRhWVJB9vJfmRcQqYPPdygxT6kWsFmNW3LQz3XZXR7BUhNJRf4VntZkIAmsC807BCMlifJh6hnRmp0sJ60jv0GpQ==
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jan 2017 08:22:09.9982 (UTC)
X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.158.2];
Helo=[az84smr01.freescale.net]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR03MB2476
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: Tue, 03 Jan 2017 08:22:18 -0000
On 12/29/2016 12:38 PM, Santosh Shukla wrote:
> 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.
>
No, that is not the case.
vfio-fsl-mc bus only manages the dpbp pools (mainly pool ids) for dpaa2
platform.
dpaa2_bp_list and buffer memory are all managed by the driver.
>> + 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()?
I see that rte_free/alloc are mostly for dmable memory or for
portability, which is not required here.
>
>> + 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.
>
Your comment is right, they are reduntant. we will clean it up, if we
are spinning a new version.
>> +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
>>
>