From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from NAM03-CO1-obe.outbound.protection.outlook.com (mail-co1nam03on0046.outbound.protection.outlook.com [104.47.40.46]) by dpdk.org (Postfix) with ESMTP id 62DE91AFE8 for ; Mon, 9 Oct 2017 07:01:56 +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=0c09+3fJmiaNkNADt5wvqggR1oX6iTkf5fyHhjDrxQE=; b=emCGocIwNEalb1CEraZamIWthfYZy5p0bSEGmXCfGmD54xbjZNCSj1+TnBc6KiHiJwlLYbC5zXYiA37RzwZRnAzsKWUKozYhg8hBWVuQvok23heuShFQVG1DCwtp7csAp3xcRIzvJ5Ve/smqaYhDokMLdOKCEBASYWBswldo2PE= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Santosh.Shukla@cavium.com; Received: from [192.168.0.105] (103.76.56.167) by BN6PR07MB3089.namprd07.prod.outlook.com (10.172.105.7) 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:01:51 +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> 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: <0d07a0ca-5fb3-f275-9d5f-2f30a8503caa@caviumnetworks.com> Date: Mon, 9 Oct 2017 10:31:38 +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: <2261239.kuHBKoApgZ@xps> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Originating-IP: [103.76.56.167] X-ClientProxiedBy: PN1PR01CA0084.INDPRD01.PROD.OUTLOOK.COM (10.174.144.152) To BN6PR07MB3089.namprd07.prod.outlook.com (10.172.105.7) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 5583e0bf-e106-4130-217c-08d50ed2dbd5 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:BN6PR07MB3089; X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB3089; 3:hS8QaxdnZS/qXkJODmYc4+G9QndbO/XOhrDTOavl2y54KnmqlBqa72pr++20+FwSsLQnGd866UmxKsBqG6IM3kQ2Qe1GddbZSPPrgM/bZerFozC8cB0xKoNNUpNfM+Yi+cv11W5fipNZronYBJa3DhsPpyt28j3TwNiZKsBtHCACMkqlujX6MYaBq4tbcq/B66jlsROIDImQOPu8FdaywkVy3r6Xe6qNw2S5ru7uhq3gkARNxbVJKBX4E7i7R9Yt; 25:Cd9wfA9WufxfvCmRwrDzgRMpQqwDWPOVP/wAAg83grnq3Q1vVK9gExyYCOUxSJAdO7Up3UtPWiakaJOSc9PST31aeehANJ6IXtJdIH2HbMjPQmhjNYbyoC7+BeB8O5PqRDKzri3zbbfv4UHOB43Qn0tQ8tPdgY5G0bjqlWDge8J3d6VNVtjbZ5JSr7ni3cVvybhZrTfVavfA1woy7JidoyAUZzbqOxqfP/1kvHtfE2vmIY867Hs+6da8d8GLiSCBZobNnJlPsAjzA3dh4scyVaRORSYUSNX5f1O2Afmq7x3XHjXfhYkd8UR2KjWdaf+LXzFcCHcC20c150Zej/09+w==; 31:1boCRaZZFyPzDfoFc1KYqB14F8NT97BDE8i1QeWeTtM7WGQWnbCnaiGGdoCqdJu3/mR2bbBVhGKY8F5R8xEZYLp7/NF46Mtd9wiN0AZiQMRDDIfxN2dCTAJUIFakI8/3PL3rRcgzCrwQEHcmFdE5XMyFYERnk3GcGFCmZF7GfUj+rjR5z+/HfKPBKl1cQeylMnEsVx+k0hn7k82iXr8SB5mPl3aO3Hw08ieiNVkO9/w= X-MS-TrafficTypeDiagnostic: BN6PR07MB3089: X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB3089; 20:5Qsj3hLa4eFG2kxvN3KY6efOu18z0dSMvsWSjwuQcsKAGgzYMaSriKku1EcS0frkTWLG8d9fCNGaSwFKQ0+MMNZkd9x+K/JJuQMCdCQBcVNiggzFghcyUQdpO4XLM22MWr3Q6Qgpw05fhhzf+OSSUhXH1C53Kku/c7o39U5NCxawIvTg9hi7T+CZp3+eHJ365hgRWCxfXmNG2SZHdSuQj2kgd0DW2Ajb5p7ypAVe+CAKqJ3xd4eOVjFPOeFGibutpcBs9Xi/on8QW8nX3XMQP5nWnOwZdXGMfq/Eyt0Y73VpBo5gdg7CvN6S5QhHFEP4xqdZU66g2OH8hLMbN8j9JW5RtPn3U92Yeh3K2eAJnYyxkElzr0asvxrdnIr0F62DqRdhTRqRUj9viNfkgIb9ZvWGcHUKnZBDc8sgKHVif8aMtgN1He58qoIkH+0dywvoX2GQaopW9C1Eob6/wMV4jDE9OakX9WCW6vtjsANNhswwcSgbbB+lYRuCM9W17mfegrOwMeHbFj5Y/5NZwkc41d8z4dLfErkwd98dGT5yxT3k0obnn2msAPOw3YXNgw3VfNcY8Tx23xMma7lMlIwAKTjc8BxV6Re37z5zEwvVMSM=; 4:FQZQct/VC/ZJwRWONtNcjaTWE1ta8TaaDEOgCdLloShWnrp/wITgjTJN6ivQ4nEVUe7RJ+LZF3JEZuFwgM7x1xPIC1jI0XdHMpw4CFeUKVAYsdVzDpEnAZA7Az4nwvHHpLpy8urSADfgEjApPIeiRyn/P5DOAByezcJ5gyinuheqEEA0UkOfj3/fFb1SEQKmmCEnR6+/NxiqPqRwiuvSOeghjMf7TthePvNHJU7nOPGedj+YNAN60NNTE+fSptiLn8GshC/2C88s1ZnaFoAV3d56iwn/BefWizl1zyTN+/8= 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)(8121501046)(5005006)(10201501046)(93006095)(3002001)(100000703101)(100105400095)(6041248)(20161123562025)(20161123558100)(20161123560025)(20161123555025)(20161123564025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:BN6PR07MB3089; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:BN6PR07MB3089; X-Forefront-PRVS: 045584D28C X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6049001)(6009001)(346002)(376002)(199003)(189002)(24454002)(377454003)(65826007)(76176999)(58126008)(50986999)(53936002)(65806001)(64126003)(2906002)(54356999)(86152003)(5660300001)(4326008)(66066001)(65956001)(47776003)(6246003)(53376002)(72206003)(966005)(117156002)(478600001)(36756003)(97736004)(25786009)(68736007)(101416001)(33646002)(6306002)(50466002)(93886005)(16576012)(77096006)(189998001)(6666003)(16526018)(31686004)(6486002)(8936002)(316002)(2950100002)(42882006)(6916009)(81166006)(23746002)(229853002)(81156014)(83506001)(305945005)(8656003)(31696002)(105586002)(106356001)(8676002)(6116002)(230700001)(90366009)(7736002)(3846002); DIR:OUT; SFP:1101; SCL:1; SRVR:BN6PR07MB3089; 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; BN6PR07MB3089; 23:p0aiWfAfzxgcWg6ciUh+/gu8seZzVUUou++Lo?= =?Windows-1252?Q?fftLCwWX9PL5IEL6L0k8oiUCHQoLoP47v27PBR8M3VgV5YI0HPDkatTZ?= =?Windows-1252?Q?+cp5M6qkCo0MsB8+o3DHEtg7rFz8JpQ9xlML6geCn+iy+PeTNuMshTrT?= =?Windows-1252?Q?qlWbC4kY39WvwZXN53OW1uwXIDvWjMB8flZKo2H5q7nlNEmXkh2z4GBT?= =?Windows-1252?Q?B18HDejr9ffmUG+2k2jTa68lksLwZbilSfT0MPa/Xya5ygN8ZfJV6Df7?= =?Windows-1252?Q?pwtBKlG8gfWQ81j1rLHUiwAxcG+uF+BrkBl+GzofUrPQKjunodStI2zT?= =?Windows-1252?Q?b6hGVOta74wHARQlsdnihBnA8/aYxMi6MsqlZtPe7GegluF1vrlrvpkG?= =?Windows-1252?Q?nQfG74y8E5IcFJjiUJXufcAHt3S9hP4hvBTCH67qK/4Rwy80emtwQ64m?= =?Windows-1252?Q?U/QDGl8Lcya5ixMLCwKFt45FoH3qSGWznNePBJpjraMhIq/XGWYDD0Hd?= =?Windows-1252?Q?N5bv4QG9mKE9cG3TWAk5vtPy7XEYO2bk+uKPyfkpDzW7p/i37ow1n9Vi?= =?Windows-1252?Q?pd6LNy7k9s5hGj1XCtGwOk/SkGdvM+Ns0zDvDczttzTSHow1IQRsi8sO?= =?Windows-1252?Q?hVEtWfuJZ7VdnNA2HUj0vr1Y6n2H7Imy8+cFV0bP8X2f8XRmbwDsPIgZ?= =?Windows-1252?Q?M4PwZjSwnZN1EUAszgGgLpIzUvZx8MU73qYyS86kLcrx2FMT1BQmTwww?= =?Windows-1252?Q?0lApi492RaxopK0cYRajL8u9dK8DK7qQUpjD1mrIyggZ5eaNMQzOVLh0?= =?Windows-1252?Q?mvW42eIwINAZq7mkgh3gyV2CT2j8/tbpp8CbHWem5eGix2A9vIUJesVF?= =?Windows-1252?Q?f5liBRO2qqm0cTUWdA5pGLFI7TB2KDSnmWU8SkJ38xHYQkeztdTXzJpH?= =?Windows-1252?Q?13uv4fxPVn4qbulShNT69tWCaPZ0Pap1Y6rQsYhfwOdDkzeJQ+C9TzXh?= =?Windows-1252?Q?9tzsinkrTmtnTr4YJb3GuDtvCbbj2A+J0y/Uw3hxU6FLKrVf9iOT/L/P?= =?Windows-1252?Q?pCC4ID6ZdQcXYm92Yp98EGtxIgAUbNH2QPbfTvdrafgOvHfNxIu1H+DH?= =?Windows-1252?Q?RGHyX1caA9SmswCyeKasjbOlRlel3H+f2lIj1Vr4+5DFKBDeHPiq4Ay+?= =?Windows-1252?Q?RPNkZ3psCHtKujl+Zo8dySgQQha96eVAHjdvfF19l651UgP6IWVCsy90?= =?Windows-1252?Q?z93gzGAuI45tXJd2v8EH7YBX3Ag6r0hNzozxb/ybM7mVkGQ5EsVesa/H?= =?Windows-1252?Q?cFpzzy9fvlQvSc00nhQW+NJ0cW1VV3vl7WTuD+maQNzvzZztO7ZCWEll?= =?Windows-1252?Q?xuXMz0vSlJcadsTJ4ZlVbMRSd0FLSVrzkbZlN5g83fEBA17f6499Fkp7?= =?Windows-1252?Q?W0iUAeXIb+jGR/Ps3F3VnqhTfg+N5Ok9VEPp/MnvJ7d/nRvYC7s/SdYg?= =?Windows-1252?Q?C6KF4p7PmUb8k9x9F1ieU6N92KxNT5Jlh+YJKeWVlwRm4Zxec513bhho?= =?Windows-1252?Q?i8S4soZ/CQCgXxOblGG+b3Lm07L7y5nruVPwQLw6SII+AirlW7OI2sj2?= =?Windows-1252?Q?xlVnFQAMHz28pKviZgDss0=3D?= X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB3089; 6:CCVfSneaqN/1KOU4SP5JLI6FkpHtZfuiAdo3wLmbD1+/jwN7FeLiJ1KisInvLj/km16IIQUXH0fdr0DH/geyNolOz9/y4SqQEEngquEH2sM8QE5kLg1u1vZD7A0zmnhulZC2L0JcD9Gzh4onwnOV/cxZRB9Dye1GHT+tiQwyk6CqrernhuoXUTX4BiF79+2YVAwsC7iovlfGXhMPqV4teF3vdHMe8Y1PqmKkwriiL23Le2DJYpCxNp62shmWJWdu/xT9Gi7np9mDpEQhLhWBt34PiQNg18LJxRg85glje+o9R4aclSpbJ7vq++PApcI9K8/xq1PfbwiYYkW0IdOdgA==; 5:8/qycE7x1bHB2Yx9c4O4PzkkNsCGfoYLn1hrKLoXTv7Q6c0GarICZjJCpg8qzuHGaVYwSdruwjX9u+ITd8GxVFoFS0PAoC1JoF2kGb7FVr2gKwn4LTZ7UtG6cL4PRdjyAN6sNw5Lnl+fr0oX8Zyga96ffEGqBOnvZbrq+1Fcu8E=; 24:OESC4G8EDUH0E+0xuNxoi+VwGf8lI685VeMAChlyeWHQu9i1Zh8mH9bt3HfDz0THffvUvS7vG0MsaSkvhTGxbJRWHsRLZFNGmQ6jEpajQPQ=; 7:oRM/Qajss+NJg37q/TCrTUyKJzSKvAsSP5kablaFOM8Zbu4WWCVEGcVIOR1xGLuG2kwEJuGWuywiqfOxuDRLB6Tqkfx6ePRazBCkdqAQ8gxYKfWVhlD4OuT1myh+5k8uaqRQOwYwX2ANd9eoTlKwG+xVIMOnMpzTIK4lXMmRmM6w018qgUrsEwWIoQZahgRwN36KoBEL9EJ6No6GqzfAIXjkaaFqQRvVTey1VnCCiL0= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Oct 2017 05:01:51.7517 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR07MB3089 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:01:57 -0000 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? > 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