From mboxrd@z Thu Jan 1 00:00:00 1970 From: santosh Subject: Re: [PATCH v3 10/10] doc: add mempool and octeontx mempool device Date: Mon, 9 Oct 2017 11:16:14 +0530 Message-ID: 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> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org, olivier.matz@6wind.com, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com, John McNamara , ferruh.yigit@intel.com To: Thomas Monjalon 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) In-Reply-To: <0d07a0ca-5fb3-f275-9d5f-2f30a8503caa@caviumnetworks.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 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