From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-BY2-obe.outbound.protection.outlook.com (mail-by2nam03on0076.outbound.protection.outlook.com [104.47.42.76]) by dpdk.org (Postfix) with ESMTP id 27D40325D for ; Wed, 18 Oct 2017 16:03:04 +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=91UfznoRt5Sr8Ln1t6Qew+xIc3jVtIPJGYlvxt1opTw=; b=AjeNB5TFugBb72tR/6qONY0YVGDXVpVyVMzjMySfcsryBeuTfFaHCpool4p2UiXr2gWM/PTkUR3d9UT7nIAjbMP+AftuPE7sf7zk5uewisvUoZmi6sVI7G+/LVjhNA61ToE8pWUhJGJGXqb4MhLa9q7aSiNbMRjn+jjs0vo4D64= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; Received: from [192.168.0.105] (103.76.56.167) by MWHPR07MB3101.namprd07.prod.outlook.com (10.172.95.7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Wed, 18 Oct 2017 14:02:59 +0000 To: Thomas Monjalon , John McNamara References: <20170831063719.19273-1-santosh.shukla@caviumnetworks.com> <2b50074a-f964-d1de-d3b9-0abcd9df68a8@caviumnetworks.com> <2831928.n80VB9rmku@xps> Cc: dev@dpdk.org, olivier.matz@6wind.com, jerin.jacob@caviumnetworks.com, hemant.agrawal@nxp.com, ferruh.yigit@intel.com From: santosh Message-ID: <80cd844a-511e-5b27-4bc0-ea796611cb28@caviumnetworks.com> Date: Wed, 18 Oct 2017 19:32:44 +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: <2831928.n80VB9rmku@xps> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [103.76.56.167] X-ClientProxiedBy: PN1PR0101CA0037.INDPRD01.PROD.OUTLOOK.COM (10.174.150.23) To MWHPR07MB3101.namprd07.prod.outlook.com (10.172.95.7) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0b585d80-dfb4-4d39-49a3-08d51630f1f6 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:MWHPR07MB3101; X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3101; 3:9gClbBy/8bAZW9YwrU8CZy6eLeN9jbPVbT353ugijk5YZM+uYjFlbH6PsoX4zkZOyAcIpkSEsH2wz3n3o5IVgtcLTTcAtAmiCejUrg5QlMlM0ow24Ptf5bxNGfrUZAPOpAbOJV7lgHGOx6pF2k1/mqZEwPO5x5ZoHsHBQrrxIqjnsYM3UihoGbZbbVDTlT9uHcf9tYNpH5N0vpCsnwfIm2E1fYtTNezgjiZDChdoPLUZVc+yCoQW5PjwIO9wFKnh; 25:GXDvJtlyG8oTdHRy90UCDwsUxj5tuAE60TziCsipibgGmdC11EKsShnCh8T4SEMEFoUPihE25rjNYk2+N+yuustcuIHlzeETIoIff7SH4NceedaNjLgmE7SUlCLM5tfDk++5su8idutp/g57zO83XW0ASGoC3UVcZRCrIqklZnLtyi+atOYqZxkKq6/CCKQtMdtbJFFS2tiFt3xIVIFmdtgRfQy3aApVH5WTZ+3Ma0uaXkcBBuHittAM0PxsN3Hnyr875UvzoZdSjjT6Hw5iflwGd3MjMobOvmRetcNL2UjQLJD16ITazlpyNkr8tIl5789OlL2+IDsW3I8VKj3+hQ==; 31:3cAQ3PsdKyWsjJHmYLjIA4YOGzPdRK62IG3EiyUJhjx3fqDHT/hPROpbolU1rudUBioVGMFwsqZWuwirwOcOd4o8dH3ITZFn2dEmllaj2wpwxiacQXE+KIArcJFck29YeDUqabuXWyOOiMlBtbC+UssmLwiksiMsfX9vW7U0s66LQE4gZYQK1jfLV2xbbGy6RgLHaWGTceMFcIEeerVtq58yp4SK1zNGKknTopqnvbg= X-MS-TrafficTypeDiagnostic: MWHPR07MB3101: X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3101; 20:ZaRHLKQXTETp1++OdDpvYa6QjtIN5J6bJw5LllfMe1Ppsk5GDPLUE0MgTHQZ7cglat+JNHfgSZkO0GvyyHoaghkkKRF2mY2GZFFYyzhfxYujgmTazLgqEpEABNqZkj+QWeBONalJ651isc1qhWjjj8ivlF0X5t7OdpwORrEDLUlpMleLdIpM8oB9QgQz6I3SBFPsYLFVF5tx73YOGBinMEFDqWJIqaicSEHyEY0owLvGMXJVnLtjHCQxNxxr+nm9HDlI02lE7MBEL0J9EP+i4FK0G30UF3KTBoLY6IUG1o7RlCBqcsMjguwKcB1i58TIv4QqFeauPF0PLHkLUqjU9AT/XyvaYbPPBe45xF2qXbERG4anZTg7DRbZuW63xVDOtBWq1n/YdjpWBUutJ5TJkIPtC96ynR+grtkknKsYOWqHdpXBme1nchQNmxUbpsLZNM+80Kp9NhisPFjhOQNj/V66lDMv+ed6p7bwdy1Joryw3CDKn58k89LDGp+w2ouuw+/30FTuFVxKg9qOOqqsunIFpNetzelMjtBoc46Ivp1zuhITXeB7k0WyDxXjwzc806KXX+Sv8NoxVy+Yr0XfZyiKeAu3x9LNpMNMSmJlLa8=; 4:rUmsKeai6z5Ceflb0aIXtrXHwS0rDdc8M1LLIKfwcK2vhPIF9idWcYH67Y15tWY9q1Fow1dpPKGf5UWRGZSNKKVXdMFc846Tek/Tfl7N+ytOqdNUrOGhMsDM8/NeHMFxv/FeZVJ13njQZgNtQSPNrnj3rDd65kNewCIYpKqzcca3l71HrlnxJOt1cRZ2IWS5OwGjzKemX0w9RA1kY+yyb9eN+aSF2CUPF+aDZoyOBh5IkYl0WRKia2cbbZPK/LDN X-Exchange-Antispam-Report-Test: UriScan:; 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)(10201501046)(3002001)(93006095)(100000703101)(100105400095)(6041248)(20161123560025)(20161123562025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:MWHPR07MB3101; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:MWHPR07MB3101; X-Forefront-PRVS: 0464DBBBC4 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6049001)(6009001)(346002)(376002)(24454002)(199003)(377454003)(189002)(77096006)(16576012)(90366009)(65956001)(66066001)(65806001)(6306002)(86152003)(23746002)(47776003)(230700001)(6116002)(16526018)(33646002)(6486002)(316002)(3846002)(105586002)(478600001)(42882006)(106356001)(68736007)(65826007)(93886005)(110136005)(83506001)(25786009)(50466002)(2950100002)(229853002)(5660300001)(8676002)(305945005)(31686004)(72206003)(189998001)(53376002)(2906002)(561944003)(966005)(58126008)(101416001)(81156014)(54356999)(81166006)(53936002)(6246003)(117156002)(50986999)(76176999)(31696002)(97736004)(6666003)(7736002)(64126003)(36756003)(8936002)(4326008)(8656005)(8656005); DIR:OUT; SFP:1101; SCL:1; SRVR:MWHPR07MB3101; H:[192.168.0.105]; 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: =?Windows-1252?Q?1; MWHPR07MB3101; 23:GLpKfI0GHtAMoyJuXSrnh4ooBL8OHvjW3x7Mc?= =?Windows-1252?Q?iuGt3XS3xeaccA2qYNkh6bICx9Kylg76C3ANBE46qHhXKVayaqMZm3R6?= =?Windows-1252?Q?Llbe9D6HsHRnIhoi08SVb879hCa4YqDsE6VNQKU+QJH/+PKQCuOluiBV?= =?Windows-1252?Q?JDeLeGxEbRAyZFScUqXivWf00EJKw7jt4uLJsIs3HqBLeH/D3mIShbFX?= =?Windows-1252?Q?2BT6GEEbQm2yIt32ThJDSlavMFWv+OjAR5LeaP02dXwdm6VbkAbPQE/2?= =?Windows-1252?Q?Y/g+ED61gLEsSMF46TzOoEjLI5CdosA1+7e5IVEsJCf5C9rN/gYUPjnr?= =?Windows-1252?Q?qeLXf3n+oI8rm92SpS6T55nVeUGJMRcV70qPcC/0ut4NqHSaClidj4xs?= =?Windows-1252?Q?4j9RBYfJC3DIkpQRkJZaJBtbkzFHOs/LpQAWI2XYNYQka7UWoJJtgIhT?= =?Windows-1252?Q?hxRRut5J6SyJjtidfm4WRHTdwoMLRxNkL5av/0baTt98H4qXhmvJYyJ5?= =?Windows-1252?Q?oMK6j3obc/N6T1HN4+0tV8EN545lg1aCY/v4T0AWpZvdR6M6FD9/pnbN?= =?Windows-1252?Q?42vWXCa8OTMWWFPiq299pehtv0KTIW91ARiyUKfS95qAgqyxaVd1WCV9?= =?Windows-1252?Q?MBOQoQt/epRGwsDb0gDWmjhK0ebB9OTpAoObZBjno6g/HF/ho5g0Yy8Q?= =?Windows-1252?Q?PBQfKH+CXHgzZ2IZDT8g13WdWxAZZvC6Pi5OSvyCQbJ+RuJB+RihuSnU?= =?Windows-1252?Q?WRpnSO4PbS58gxRq+i68S1+J2LqeKJ7w3xn/obvcZKMRq9+bDbOwfTZ1?= =?Windows-1252?Q?arvy5hdHhJ7XB1Ve81bFSmCOdGD/rGzrdHtLb4VpLhVA9J/rj3UaKHoG?= =?Windows-1252?Q?XWi+OHr0hum7nv/dbvfffEDKJgc1qt7ZmJDyR0gJCmSQvB6+2FS2z+r5?= =?Windows-1252?Q?3v2egeNNPNy0dWfoCuRJ54poIEPinkhzDoMjYTu9MXOLKQ6iRMr+Yu8U?= =?Windows-1252?Q?urAPwkOui3ev47Gg6JDuD5/LmTofb0C6WZNkwPFRrzljE747XCTa7hqe?= =?Windows-1252?Q?pyrUPEEC8/DsYbn6uFsGeo3LGY3Zv2BJkJjjVVkAtkenxeabuiD55Ma6?= =?Windows-1252?Q?Qiy88ETwl6OQll7YKiXDqinlSN3YERTo9oN50sxAFtilO7z9vrp4XmAb?= =?Windows-1252?Q?AAAfD0gHe1xJebHwJyEXR2CCb9vKEKtABYHpWuvg6tGhYTHSGLUv8hJd?= =?Windows-1252?Q?FFKO1Ge3rk3/MqYWnB9dyVUNG2NAGr/CjdcvxfA+tXckF6BTIjBP02+y?= =?Windows-1252?Q?SIcOlHhpkdf91aoPBxr3nroa7om6TIzyWw9edvgCbeEXWfqh4UV9fDtB?= =?Windows-1252?Q?VhqxF4VTLSIlFEmeTm1EyafyRSiALzszmDncqyp3o1cnHSdUhxbFkpEr?= =?Windows-1252?Q?tA5P+EmNxR8bcx2zQGsnQtwTTBNHuQhPYCuHeDC7GAJ4gpKab/OVhAJx?= =?Windows-1252?Q?W5sCllO6s8kSG9PrsLyjbOIuYqmwlnvbqq24441tVvF388z+zPIsSmlY?= =?Windows-1252?Q?TpBKULgLoJUGaK7vHXqbRoD+5I+AOSBmWnGw83MEsRghYRuRjC9Ae2g4?= =?Windows-1252?Q?l3O/E/6k9NjLwvMTG+RZSHf0VYYdYzHUIWuVZvd21ovjrXGA3sx10o5Q?= =?Windows-1252?Q?TPAxuZ2Lw=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; MWHPR07MB3101; 6:o/jTd03aRUlv/KwvujAKyTSFwmyRVumLfpP8VD+GdqDostL+ezECrlq0w8VFIcob5Mm/pAXkUwBAG2O+CqB+Nyp572KZIB7P93uFfxIG/IjRmljtsPNq/QtWxRJ5Qo75PexX9R50iXsGIi9oMsSilpgNIs3MJXk6iqwyyDDI6n+XyRkX9j9yjJiKKVuASnE/irmLF2t+AT0H2srL/hx64CicGVdQamgl8JQSr/KpUJCNpYaSu2JKgJsgdSiKQjese6+sgdVeEoSNHVaQkdAjTUW92AILRgIeSppDVqnmONRcr8IaCxh0et8X5631VuBJh8Pih5TBdXt6RjhwWuujuw==; 5:EHPLBYROghvQaUBNRtTgJeDN49oOvbm+2HiMVts9YZfXNn7pWfrp0BWGb7xr0JuqMBFIPTM8GffI9xtjbK/vSnx15r/ihSxosRTPHzQ40HGV6hQuNXn9isDp4WxoTwaaTGkCTIHgcSe/qSNlJucXKL6rom07juvYRjzNxZf8Fp0=; 24:m/zCQJAMhf6uEa089QdmxmDHnKeFJpZM5UL2a0p5TVbm4RSyjqD2FpvEwjDEOJj+zzfuXfWQZt82iS0AoYjWFhDhjPBvz0o7BFVnvRdqZvQ=; 7:IJcujJKMiEBatQYzFYUQQUueREOFJU2rYRb0cWNczMkSllSBt++TSTqs+2o9+ERFwDFZiyJHsz3i3jYY0ELijpS2JkktWcUzlWDxarYiAcdaL7zcKnqHuvutw85L+DRM7aiP8+KrEypjFAIMKjacjhhR8wF5fZHivMK0KjSc+L/KxLK5caM4KjFXrMYsHTreHfeoMIEeG/cknd6nChSNl3bfydh3BpU6TRLiW4oWxco= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2017 14:02:59.4921 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR07MB3101 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: Wed, 18 Oct 2017 14:03:04 -0000 On Wednesday 18 October 2017 07:15 PM, Thomas Monjalon wrote: > 18/10/2017 14:17, santosh: >> Hi Thomas, >> >> >> On Monday 09 October 2017 02:49 PM, santosh wrote: >>> On Monday 09 October 2017 02:18 PM, Thomas Monjalon wrote: >>>> 09/10/2017 07:46, santosh: >>>>> 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. >>>> Yes sure it is interesting. >>>> The question is to know if mempool drivers make sense in their own guide >>>> or if it's better to group them with all related platform specifics. >>> I vote for keeping them just like Eventdev/cryptodev, >>> has vendor specific PMD's under one roof.. (has both s/w and hw). >> To be clear and move on to v3 for this patch: >> * Your proposition to mention about mempool block in dir struct like >> doc/guides/platform/octeontx.rst. >> And right now we have more than one reference for octeontx.rst in dpdk >> example: >> ./doc/guides/nics/octeontx.rst --> NIC >> ./doc/guides/eventdevs/octeontx.rst --> eventdev device >> >> Keeping above order in mind: My current proposal was to introduce doc like eventdev for mempool block. >> >> So now, I am in two mind, Whether I opt your path If so then that should I remove all octeontx.rst reference from dpdk? > I think we must keep octeontx.rst in nics and eventdevs. > > My proposal was to have a platform guide to give more explanations > about the common hardware and bus design. That way, event device also a common hw block.. just like mempool block is for octeontx platform. Also PCI bus is octeontx bus.. we don;t have platform specific bus like dpaa has, so bus stuff not applicable to octeontx doc(imo). > Some infos for tuning Intel platforms are in the quick start guide, > and could be moved later in such a platform guide. > > With this suggestion, we can include mempool drivers in the > platform guide as mempool is really specific to the platform. > > I thought you agreed on it when talking on IRC. yes, we did discussed on IRC. But I'm still unsure about scope of that guide from octeontx perspective: That new platform entry has info about only one block which is mempool and for other common block or specific blocks : user has to look around at different directories.. >> and bundle them under one roof OR go by my current proposal. >> >> Who'll take a call on that? > If you strongly feel that mempool driver is better outside, I don't have strong opinion on doc.. I'm just asking for more opinions here.. as I'm not fully convinced with your proposition. > you can make it outside in a mempool guide. > John do you have an opinion? >