From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM01-BY2-obe.outbound.protection.outlook.com (mail-by2nam01on0065.outbound.protection.outlook.com [104.47.34.65]) by dpdk.org (Postfix) with ESMTP id 755581AF03 for ; Mon, 9 Oct 2017 07:46:37 +0200 (CEST) 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=jCnivyuAjfH1SQEAZhngG/kqK0bqufpwSSgRee7WPIY=; b=nazi56kvWjMj+HgHj8PDIE+dhpwEP4p0AKXZ5pnofT9PGI7PPdMgOxHOirhJHAg5U07GFILfJdxoqSF0bTGH4dUdSYgltyf2tLTauHROGMiAwPDq19iJwiEyWSKMmPpNet+ThxrJWNvebDW5YH2GRaFpgVgx7v1sxmLT/stBt30= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; Received: from [192.168.0.105] (103.76.56.167) by BN6PR07MB3090.namprd07.prod.outlook.com (10.172.105.8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Mon, 9 Oct 2017 05:46:32 +0000 To: Thomas Monjalon References: <20170831063719.19273-1-santosh.shukla@caviumnetworks.com> <20171008124011.1577-1-santosh.shukla@caviumnetworks.com> <20171008124011.1577-11-santosh.shukla@caviumnetworks.com> <2261239.kuHBKoApgZ@xps> <0d07a0ca-5fb3-f275-9d5f-2f30a8503caa@caviumnetworks.com> Cc: dev@dpdk.org, olivier.matz@6wind.com, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com, John McNamara , ferruh.yigit@intel.com From: santosh Message-ID: Date: Mon, 9 Oct 2017 11:16:14 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <0d07a0ca-5fb3-f275-9d5f-2f30a8503caa@caviumnetworks.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [103.76.56.167] X-ClientProxiedBy: MAXPR0101CA0042.INDPRD01.PROD.OUTLOOK.COM (10.174.62.156) To BN6PR07MB3090.namprd07.prod.outlook.com (10.172.105.8) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 4f2f50e9-b962-4883-9028-08d50ed919a3 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:BN6PR07MB3090; X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB3090; 3:lb5C1e8uBdO6Gc8u0TyD00JsbOAXWbrpmdSrL9jpSh4NlFaCZxhwqK5Uombc9pIcL9KRNxT/j9Ih8c+jWkQzeUon4SGzy+9FGlOaU7PEuGJS9H/1sVu+tUsd+paoNaRyPPQx10Ve8SV3o/sgyf0cPRMY0NW71/4XsilK9YasDI2t6+WPuTkWoRtTxgRGbTBj24GTXpWA6Uot/j+NfhRastnVGZqijaSguCGdTPqz0IUV+SNM0OjZ4mOZZ8crdDR7; 25:QALcFRF0/o3RG+qbW+EWGvlrCuJmazxLv/PPkDTuy0TF36RoXulQME7j5d6QzmBiTcLtoSdHj14NMDPs1Dzi9PyjOA+8ykdnDEEUvByPAc9pexbU8rkIsAwFfStSmF6PXPMJ4Df8+h+6C03SSikCw6+8Mu7MaLZslXL01DB5r+OMYDk133dxdb7oS3BYD/sVsYSIDXBt9Q5Pmt8lnYlSodtjgiVzhg+32rZnWCDth9vIPyZN5WBBlv/ZQXLoauabpddwK3Ocu4Cxor1aGVJMde/sTdEUBQI3PZSP7BDlWNwhInAwAfTGM5rNZH37jwet1SQ8GuGftt5iV/jx349vpw==; 31:B7XVU2mWPVPWZodrivwXtjpHH4qE4zUqvc+mbHo2Fa6MMOCGjRcExUO55PtAtP3h9ocdwz4ErMq1xoYYrgXG3dxH7xHPNV5kddczsw4H0+Ddwj+3JrKy8Rg0PT9SXwrN8xvYJ5e69NC+W+J3b+roBRZ2C035zAE8T8HfMD+fl6Q8/ZCAlP/Ul7f+LVjT2WH1i2OiNxBA5e7dS4aIOypHhxzJD32lbzNSU731hQD/s0U= X-MS-TrafficTypeDiagnostic: BN6PR07MB3090: X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB3090; 20:sLIGRcvhxaCLMOTe7Qy8SGiP9g9ety+hQnFhi1iLA/sRTcjan9PX9Z93qWFzfmcVtcWxusCO0Y1QagchdP9DzMKNlEaxlhnMBE2+AK49UzrvCOK9ObRDI8jjcjApW/c/QIBy198twlkXnP8nw5AiV7u28ilUs833fV502EerdBzzDSTiXV+9DleTZdz9NvtiK+Ra6oHPdT07edP75xCGayHx1ft7Do7sLKvPDNIcQbKDIzuIiS/7UdEGMa/6rQRVPGXV89vHM+GkCyFtL3bjFFoK8CZCnPY7yAt0mg4Tv1RoMjT0uQRnFiQCf+g8KOrFICyZhD49EGKVH/Wxz39+Alczb4QIjdXVAdxdGwMmUBY6ZRO7COu2XPT8sOAagdSLwxlUL48sD30ejeP6BVYbHBj9Zhw9lsEhNSApWJn0WCyeRC0LqxuIXp4SeVkjs5tFxKR8l8gwv0L4e6cUUEYOEM+DW+jspXPFkYWsPjpMgqUqm4jWpOdRQo8WfWT0OPBdh8/GH1lYojSc2faits+FbWH8Tjwz0AX6+tEL0CXlX0V7i/mW+sQIW1Zwh//ae2AfRuIWloHz8h0Ij1T+S6QfUNX+DwAtl+7556g+3et2IaM=; 4:UvPgurBnnjXILBKxcLfUPDzPe226L+wk/gb0vYeHJocdye56Oz7K1JnPe5TjDShv1lAdDDs5gUIZIHGMrv2+rpylKwLB/e1pwnoDgSILvYK+ouBJPYhzqs9u0pq1z3jYAYzwZ9AaRBBrbOrIJAQouXCqCEXIWy9alRctJ2Wi/e757qhSkHnUoLCABPboj/2SGgdfy6d89SwfWuGgdiP1r6CUdyWvaAtHFGaTxgzNQ/AVN7hq2DL+vNKdTObq7wwL2cRI6JO4a7s86tpdJXItEpGAdmk0WW02dYK2+BxNB6Y= X-Exchange-Antispam-Report-Test: UriScan:(228905959029699); X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(100000703101)(100105400095)(93006095)(10201501046)(3002001)(6041248)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123564025)(20161123562025)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:BN6PR07MB3090; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:BN6PR07MB3090; X-Forefront-PRVS: 045584D28C X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(6049001)(346002)(376002)(377454003)(189002)(24454002)(199003)(305945005)(6666003)(6306002)(117156002)(65826007)(8656003)(6246003)(478600001)(53376002)(25786009)(72206003)(53936002)(966005)(86152003)(47776003)(33646002)(65956001)(65806001)(6916009)(36756003)(66066001)(5660300001)(54356999)(76176999)(101416001)(2950100002)(42882006)(97736004)(16576012)(316002)(8676002)(83506001)(81156014)(81166006)(3846002)(6116002)(68736007)(31686004)(50986999)(90366009)(16526018)(93886005)(229853002)(50466002)(2906002)(58126008)(7736002)(4326008)(64126003)(77096006)(6486002)(105586002)(31696002)(23746002)(8936002)(106356001)(230700001)(189998001); DIR:OUT; SFP:1101; SCL:1; SRVR:BN6PR07MB3090; H:[192.168.0.105]; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1; BN6PR07MB3090; 23:H5LXcoDTYBrtf5762c2g2OlpNURMFcrka5pGM?= =?Windows-1252?Q?SRO6ITj515ckpTxY7vOoFz7Q2MjD6SdAWfxQ1xXgbkmOql3MAjGMHEfc?= =?Windows-1252?Q?siKtAgjuaAA2UNafTU914/oCjE1/xUQKt5oOxBiCkN+607kIFrB3Zxhv?= =?Windows-1252?Q?iSDTqrl/IOu8a4Nxj5baJckmWDepmiOzyp6j9RjlJZhqaS9JiE5JeGhh?= =?Windows-1252?Q?IiCyqD7fHxT8PMiJMk+ab7f6GFqnhl5cBMQrW1h7/QtdDyoqMw5rrsKl?= =?Windows-1252?Q?QV0nb5PypGqPuQ2bNxJepePbIR/B7A1quS6o0EqISlkDyG162j4LI9iS?= =?Windows-1252?Q?xr4KgMLIjHlbHvY7rzYmdRvXdJxS0IzZT4lbFwl9Qu6ScYZ32BViHprn?= =?Windows-1252?Q?A0jLX/2ujKSsERRqX4ydUT3GLK3+pqOo8s2lZQrqc4vEhi3dbAm+Z/YD?= =?Windows-1252?Q?ZiHboXg+3P/t5rEoe39ULJRmZrxT301tf6eUjADaxpXArraq5XSwkYLF?= =?Windows-1252?Q?w5IKIhvHUGUC9WDA/9XXHo29R1Lav//gzbNs+dAubxkI0uuKe2OTAp5i?= =?Windows-1252?Q?T3/tMDYr4f0nzIPLMSMZ1/SPl96jfAL7p6DEhOhrXagyTI70JJLycjz0?= =?Windows-1252?Q?Yez1mS3YczJodO8hqFSmUNefUu74J5e9PNGdFwsNym1E+k5ZrjmzF1ca?= =?Windows-1252?Q?YW8Og3ZTWF3wPqcR4LQIdM9cnMJG6Hk5Dz7HYbAjA/VVkk1yf2P4yInB?= =?Windows-1252?Q?Ld+kHHfsFdiFHfZxE3wvmPf4NSNIfavzObpwx/ehf/1yJFhL1zmwE/xs?= =?Windows-1252?Q?cpVdWIWtBzuN0PyoG+nyAgrmL3Rdwmk7hBXIlE2GnjAMOMpyvVytyQuP?= =?Windows-1252?Q?LHImQcw/ql9cjubzh85ycJZfFrsRH1rJGcG76Wo/eEHoQ6cP933GSBmJ?= =?Windows-1252?Q?AG28kUu7lo9WMN2AaQ9OJqbYWPlADxgVSL2pcSdBYoB1bNXjd4obWpQI?= =?Windows-1252?Q?oAxWRIYot9KimqyGM05GsfNRrwgmiTfuTXyrw+FCYTHhz6LrBpiRWKbg?= =?Windows-1252?Q?YotsTbGKui2wctCO41WFqHHVNWB2D+Shv3BYiFfG6W5mV7/mWTCC/1vX?= =?Windows-1252?Q?q7OkA0jJLM9/rWLZX3rY68CldEJVUjYlkymlJDuczf3FtRKH1Z8gg8xT?= =?Windows-1252?Q?DBXfOptuGlsDvoPjwCbO8IvpW5yboehgJzi2/xRDxwzU/9nR46BiqbMq?= =?Windows-1252?Q?dDpRBB3T7st9IZRulUDFRGS2y5Slmq4bECyydGFhpTzjGxqVcp/k3kAI?= =?Windows-1252?Q?HCmjBg3LZVCUWTDqVFxV3odHEjEk0YpYKYqO32CRLKs/SQ30Moi5KQ1c?= =?Windows-1252?Q?xu5PX6TvKpkKMXhz7Gm1NhKICC9mO4UOF60oGMnYyRFzs9p7M38VZTUm?= =?Windows-1252?Q?oyG3XtPBxItsumr1kPe4Xt7Xg2b6ZIhNHCekn8PTt0D4kgghsBVgXACa?= =?Windows-1252?Q?8SB5t8WCItdONnODlfjKAi2SvSq1/wBICKNR40I1SJizw9MY0WBjByOP?= =?Windows-1252?Q?XclN7+jmsAx/spGyQU5LTdGonJeFvlffsOM/1shTK2im5+3U/ZLO0E38?= =?Windows-1252?Q?JsQTLAPlo5ugnkaKmav1mo=3D?= X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB3090; 6:6mQ5WzbqhVB1n3yeH97XHcKxMClXi0jiyKRbioUEovElE+A8KLvh3q/3XDnQ1nd022kP0sHCm5O2/5R1A4tJ18L+R6p0y1eMpP1+c5AOHxPu05ap2GE+kFcuw54FHhw9V134xhCp1Ey1D9Yu1EsRc9PUrjfNvSIpu+hIMq/p5aOSZZ6jvf71eRM6rmdl/4xhP85P7K+CyDMXt9ZS6AEgNQmQLefFd7vmhdRgA+5vPEYQoc8QaC/rP+GRmNAqZDDS1s3tZpA35+h6ecDgw5Po1kE0mRSAQPIUpqjCJHt9Y0F77zdjUOTsO4Meh/YpBm3ALunl+6Q7jAHXsb43RFH4Bw==; 5:4qJ5pNz/H4WIlh+lHXSBmr/m3csx3Hw8bNQZFq4mnnSobyOgwUf42qOR1wp0VHjjOR1uZfDfndu/GZjZsxPIWG3Atvt85n/jZ9R8qdXWGALzMkS7xtX0gKXgHufnKJM9RFUUfhs9sYZfHnizN9o7nIPNhyz+9YUJxqtMyYetpmk=; 24:kCsSsiYbYNcsXuWZHzm0rqD/3aTprCVjRRSnze+0Tks1dYOW1D2PCBwc4P+hPx5Nh3Zi7BU766JntkuvtPEewFyjheyqY0pJx5vuMbUGpNs=; 7:UrAAknYWrARRBNWx35tVGbEBzZ7P2mGLQqjdUoY/GKj+84FBP93GLQXh6ITwQeJUgl6Te3nSn7GM6doFwIXPaJ06ESxfH1/AS10uYQ2b+qei4dQHF69nHdwD1aYM+CWKGwmYjLoL5/fAr+J9ooWqkQiy8SXGib4Ii0ltE6/GBGvX23GmWbttd3LKxRBLPDOg1WeIpdpP2F1Fk/NYnfx9hYI1J/8qokZeDhD7Dyulo64= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Oct 2017 05:46:32.0978 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR07MB3090 Subject: Re: [dpdk-dev] [PATCH v3 10/10] doc: add mempool and octeontx mempool device 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: Mon, 09 Oct 2017 05:46:38 -0000 On Monday 09 October 2017 10:31 AM, santosh wrote: > Hi Thomas, > > > On Sunday 08 October 2017 10:13 PM, Thomas Monjalon wrote: >> 08/10/2017 14:40, Santosh Shukla: >>> This commit adds a section to the docs listing the mempool >>> device PMDs available. >> It is confusing to add a mempool guide, given that we already have >> a mempool section in the programmer's guide: >> http://dpdk.org/doc/guides/prog_guide/mempool_lib.html >> >> And we will probably need also some doc for bus drivers. >> >> I think it would be more interesting to create a platform guide >> where you can describe the bus and the mempool. >> OK for doc/guides/platform/octeontx.rst ? > No Strong opinion, > > But IMO, purpose of introducing mempool PMD was inspired from > eventdev, Which I find pretty organized. > > Yes, we have mempool_lib guide but that is more about common mempool > layer details like api, structure layout etc.. I wanted > to add guide which tells about mempool PMD's and their capability > if any, thats why included octeontx as strarter and was thinking > that other external-mempool PMDs like dpaa/dpaa2 , sw ring pmd may come > later. > > If above said does not make sense then will follow Thomas proposition > and propose a patch. > > Thoughts? > Additional input: mempool PMD logically can work across nics.. could be a reason to not to mention under platform/octeontx or platform/dpaa ..etc.. IMO, Its worth adding a new section for mempool PMD. Thoughts? Regards, >> I choose to integrate this series without this last patch. >> I mark this patch as rejected. >> Please submit a new one separately. >> >>> It then adds the octeontx fpavf mempool PMD to the listed mempool >>> devices. >>> >>> Cc: John McNamara >>> >>> Signed-off-by: Santosh Shukla >>> Signed-off-by: Jerin Jacob >>> Reviewed-by: John McNamara >>> --- >> [...] >>> --- a/MAINTAINERS >>> +++ b/MAINTAINERS >>> @@ -340,6 +340,13 @@ F: drivers/net/liquidio/ >>> F: doc/guides/nics/liquidio.rst >>> F: doc/guides/nics/features/liquidio.ini >>> >>> +Cavium Octeontx Mempool >>> +M: Santosh Shukla >>> +M: Jerin Jacob >>> +F: drivers/mempool/octeontx >> A slash is missing at the end of the directory. >> >> Until now, the mempool and bus drivers are listed with net drivers. >> We could move them in a platform section later. >> For now, let's put it as "Cavium OcteonTX" in net drivers. >> >> I fixed and merged it with the first patch. > Thanks. > > IMO, for MAINTAINERS file: > Just like we have entry for "Eventdev Driver" and underneath > to that- all vendor specific PMD sits, I was thinking to > introduce "Mempool Drivers" such that we place all > external mempool PMDs + s/w PMD (example: Ring) sits underneath. > > thoughts? > >>> +F: doc/guides/mempool/index.rst >> The index must not be part of Octeontx section. >> >>> +F: doc/guides/mempool/octeontx.rst