From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-bn1-obe.outbound.protection.outlook.com (mail-bn1on0065.outbound.protection.outlook.com [157.56.110.65]) by dpdk.org (Postfix) with ESMTP id 76C012A5D for ; Fri, 3 Jun 2016 12:50:16 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:To:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=ezk0VOOrWek5qNuV7noNmlDCBF77ZrxihW1spx9c1H4=; b=Z/BcETLBqYKk9uafqsa58tL61SJKUm7CK068O8uUiiXrRuMRf+vbGci1OuhIpaAcppkgPqbEMXjWqJzxZRSknUwWtPudlcIUtjFvk7LLJfd/+OsmJ7UIryj8QK8e8lU12mo3X7baEaiw/tSQLdayhicGrE4cVUeJegDUyVirVys= Authentication-Results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=caviumnetworks.com; Received: from localhost.localdomain (122.167.39.39) by BY1PR0701MB1724.namprd07.prod.outlook.com (10.162.111.143) with Microsoft SMTP Server (TLS) id 15.1.506.9; Fri, 3 Jun 2016 10:50:13 +0000 Date: Fri, 3 Jun 2016 16:19:27 +0530 From: Jerin Jacob To: "Hunt, David" CC: , , Message-ID: <20160603104922.GA9490@localhost.localdomain> References: <1464797998-76690-1-git-send-email-david.hunt@intel.com> <1464874043-67467-1-git-send-email-david.hunt@intel.com> <1464874043-67467-2-git-send-email-david.hunt@intel.com> <20160603063755.GA5277@localhost.localdomain> <57515BBE.4070205@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <57515BBE.4070205@intel.com> User-Agent: Mutt/1.6.1 (2016-04-27) X-Originating-IP: [122.167.39.39] X-ClientProxiedBy: MA1PR01CA0053.INDPRD01.PROD.OUTLOOK.COM (10.164.116.153) To BY1PR0701MB1724.namprd07.prod.outlook.com (10.162.111.143) X-MS-Office365-Filtering-Correlation-Id: 1088cc85-5808-44f6-95cc-08d38b9cd85e X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1724; 2:m2AxitHpuJV2HzPZSv8rhwo10eeWC1YjDFe8dp08MK8YEEEvWzRlUv9mrkkIemWnS6wi2GyhtXHaqdBLqq0aCQ3Uwn3mlJ1OYPVAw2a+uk1F1nanDWtzNYJGpOe+WMeAtomsuxTecwYhJqp22hTnRMZRanxzriBZTdOjT9ai9ntOmVB9e0gfW5mphZsUmwXa; 3:H/7PrdvUmu8Y/D8FGEjM16/NFk9Ei3VVhC33ExZonb2mLfLvouwXL8BtRKal/37JPadJrrzNrZODeCViIgT2eXOm+pYxpWEy/H05kAn8gFtmSptvLl7F3U2nCNW+yQTT; 25:bzknuQKfyq+fynFll0k4E9d8zp2EdPX6dTIv4wAqPx9Yq8aaE4qDJVXLHJ+8Ptnh1DjJl7P6uO2YTstfRW6ujDvSRaVh+RUohkU/ixgQYozhXou3ba0urC8KGW+fBELP8DyxsBiTtVPfcPfH81iasz3/gpnm0XdexmY8uIHl0Tvjl9tSsCHh+QXLHGXvDs2XP9oWd3TfBBblJOS55zRYYsm0Bc1KmPHFBWJnQf678gYMf303L+BqGxjh09LgxDWfG28O3FSnF5E/AHNFjN667c164/XqYiD7qQOieHSJ95AjqL8ObWh+t5+6lkS7gQducrmuhRwTxHLYjTMHqTA6TsRiwujMOt2EjZ1RybYqlpmpggObhnGew2gWnu/ShDsK8aycwaWG95RvS94cARLP0jd5viPxDgbA6r1XUEX/ffY= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY1PR0701MB1724; X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1724; 20:TS1XRlzJe96FBcd0vLDlYPlYUo3W7HW4dwVKmAIdnWTt6hJCZ6h7IF01bD17Le5WCc1yLlAxwlm+DnlLPiesxbB0u4oiXru2mGKvE2lHxnylWtzaEqewthBKg8QlHmhJ4IyMuEH8tdurOGrazg6dsenwmcOlWmYNuojBAiBpeAE5nq4maDJY1aM6MZJhcO+7ORm3PRibaoWb9s+nzyhWAsyUXx9JHh6VjaLwPiWN9IuZcH0JMkJZlcHgmVc6FlWQdqbP4zPK7mQtRUhMRXhsIq/1Nc//Q4D1IdiYqwHDbTnM3uhKBi8mndcLywJPu4W4wTXdhjBctUmr3J5om1s769ww+17eceOPhHOhrcvgkFKh/9pEEShgxrxxcf87vO8tkWvnw9omPGDRaSiT2KMGDDQB1j1hMnDs3CJqXNoJ1DYRuXFrTwYQ26wWfATR7ZQsr/Ztvb5mgSXtzqe2WA+NB+aq8ofQqF0a3ZOdkMhkqZ8cjcu9njY0qBTIrnkvJZZIeENSH9Kxr3YN3+yu9cBf322z7jw088bepUMEwWsiluaeMOx6F6pqRjWfnsxihRJNEOeP4Dg8NFW2jSdzJYiKOeuxhc6xRutiQjkjJFxM7Vw= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(5005006)(8121501046)(3002001)(10201501046); SRVR:BY1PR0701MB1724; BCL:0; PCL:0; RULEID:; SRVR:BY1PR0701MB1724; X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1724; 4:1Cg6Sikw365cgoegb66PorjVOD3GG60pvDTRfDaccwYk4GlPqQJ72xMu/q1KERu9X0BKkthHRsRXcR8IjlAvMgMSe93o+yo9CWRV3C/JnpArh+T51pbBiF2Ik4Zs0yLuc10rzCAA6FFBhDwYJjMma7rCn2YM2sP4YwKbE0ip8KjUCYYF7GsWXMqKNcXKugrhy2ieXsIo9v8XYVxOKEpPkPxhRN0BiSjBcrKPIZNqzFUk5aHgM8X0aBuNHYEcKUBIUZD7hFciIQcKdoRpByuMKHn8HMkHbD3vwTnXJmEu9SAavvIN5SgH5XmedA6tWehUHJ44X2Wmo9/toGtBoCGuRd7Vmbw8CGabbIV4ivwwiwu1oDxtH4YIxac/12+l5+lq X-Forefront-PRVS: 0962D394D2 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6009001)(6069001)(24454002)(47776003)(5004730100002)(66066001)(93886004)(5008740100001)(9686002)(97756001)(2950100001)(83506001)(77096005)(92566002)(8676002)(81166006)(19580395003)(61506002)(42186005)(110136002)(586003)(3846002)(33656002)(2906002)(6116002)(50466002)(54356999)(50986999)(46406003)(4326007)(1076002)(23726003)(4001350100001)(189998001)(76176999)(7099028); DIR:OUT; SFP:1101; SCL:1; SRVR:BY1PR0701MB1724; H:localhost.localdomain; FPR:; SPF:None; MLV:sfv; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BY1PR0701MB1724; 23:x0YFTEFblBOsyo6Y13cQcdf03aq5RakByVLxqGp?= =?us-ascii?Q?j/DGmzdHjJ8ygA/aAY6BqdI9eI4/XqTfM4vpfigZIOiW1lnwP8BWblRWD0RL?= =?us-ascii?Q?aTVJYdYH+I4mUw39+hRGXgwRUl5LHMXke7GbTL+EYNsSCnG4Yz6z8V2IzC1q?= =?us-ascii?Q?Q3Xtvsg3DhvYpejgb4j7ljWZZtOpOm4K3CTKr4P7vcnZNuu9yrIMD7Bc1+Nw?= =?us-ascii?Q?a8OU9iiS7nk/z6jRljtJahneP41qUOJwR3QxiFvcSdBLFRz2pOImaQpLKN0h?= =?us-ascii?Q?dF7VNn4pzuO67rOdzfOAOwabA7h7owp9DcEQtpGuzXf6rpRVbDHxEGL9A/C8?= =?us-ascii?Q?8rZP4Bi1v1aIM5jxnZhJQ7T8m+8BekW/0G4nxn3iMGoy0bmDwvM97G9msFAN?= =?us-ascii?Q?9kDSIMtYjymAgPspDpftKoeCRj5/DR3Oa3pcSJkkeZBfID1alUAQtwS102fg?= =?us-ascii?Q?EL3nuV83X9yvUhw1fU1lt6e459l1Seow4OyTnte6I3D7/LaN5QDPp/TP98qK?= =?us-ascii?Q?3MDF/29rtfvws16oGhTIl1FJ3Mbd5fxGCye+id60pHEB1sBMmzbsxe9q2ceB?= =?us-ascii?Q?xzmqlGHdFjv1/v/GQ5R8w/jvjvBaZ7pgRKNxeMntmS/R6MM/3adGiZL7S3Z9?= =?us-ascii?Q?VAiTIBgM0V+WPoqf0GV/baMwWGPWYKxEOITF9j8/kpVfnzAchnAgyH/1jbzK?= =?us-ascii?Q?toUXqp8bkDl2c+jbxTfPB1VS5l/Jv2QC5qww2/VnfLWjMOOnQE31rKA5wPi3?= =?us-ascii?Q?Xp+m2Jl3o03Oy6DLnkCh933m0g1+qIulVNVakG20uP5LloAk6lyDt5sqDV/i?= =?us-ascii?Q?R0wnh+fMOVevu5233RSiIv3lgfWk8/es1CTrnSZTFgwmFW+L0873qfEy6CIX?= =?us-ascii?Q?DjWumizdRsl+PbPrJo+hGNEb7BhaJrm9wm6wTKFK1j9jstZvoV7V/+zSAiyI?= =?us-ascii?Q?LgvX8h535gr9jhFw7O8Z5BwSnl9omztu6K5V/1StK8ZYhBPiahtpZrCG/5BH?= =?us-ascii?Q?VOAo=3D?= X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1724; 5:c6D5EPhSQEmYvpsQCya2pXZ16Py/J7JmCms9owjFg8GCZsQ+zHIG57M9TpqwV+QlHmwnJSsLsa4JlJocy0rmaL/33ezF8kZKl4r/8IfIegrUYov3a8rPqKlvfg0p+R1EKX3Bwz2eMV3OivZMvAreLg==; 24:UwBrsO0y0iRODOoDq2oFzGh4FJEI7XuXDtaFZFKa2TIoNG0A4kDWA2UUthRi5KczEwW6FWmMFDZBWlP4/7c0Wha6oWEn8I0gRqe004r9d9g=; 7:ttmsMDOd/CQVWt7LnOQ6IqWMg3phrGPziazc9+8RPnK9A1W1VG8XNUqcAHZnOQP2buEr7oaEkKGBWL2a9FnZPs4NQjZIhuPs/Mkur3HUtfBCmlSIEquTIAM00fjxwVMSZ+2SvbsA7WP8v3ObqlqdaOHTMnAXGt3Jtob9R2xdCOXrAgXGuUoZB6zZbduvByDwVeh22Fxvh84nXJzc3vIpHVzqcNpVGh7vGSyg/cn7Ktg= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jun 2016 10:50:13.2983 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR0701MB1724 Subject: Re: [dpdk-dev] [PATCH v7 1/5] mempool: support external mempool operations X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 03 Jun 2016 10:50:17 -0000 On Fri, Jun 03, 2016 at 11:28:14AM +0100, Hunt, David wrote: > > > > > static inline struct rte_mempool_ops * > > > rte_mempool_ops_get(int ops_index) > > > > > > return &rte_mempool_ops_table.ops[ops_index]; > > |> 2) Considering "get" and "put" are the fast-path callbacks for > > |> pool-manger, Is it possible to avoid the extra overhead of the > > |> following > > |> _load_ and additional cache line on each call, > > |> rte_mempool_handler_table.handler[handler_idx] > > |> > > |> I understand it is for multiprocess support but I am thing can we > > |> introduce something like ethernet API support for multiprocess and > > |> resolve "put" and "get" functions pointer on init and store in > > |> struct mempool. Some thinking like > > |> > > |> file: drivers/net/ixgbe/ixgbe_ethdev.c > > |> search for if (rte_eal_process_type() != RTE_PROC_PRIMARY) { > > | > > |I'll look at this one before posting the next version of the patch > > |(soon). :) > > > > Have you checked the above comment, if it difficult then postpone it. But > > IMO it will reduce few cycles in fast-path and reduce the cache usage in > > fast path > > > > [snip] > > I looked at it, but I'd need to do some more digging into it to see how it > can be > done properly. I'm not seeing any performance drop at the moment, and it may > be a good way to improve further down the line. Is it OK to postpone? I am OK for fixing it later. Performance issue should come in the use cases where mempool "local cache" gets overflows and "get" and "put." function pointers being used. Like crypto and ethdev, fast path function pointers can accommodate in the main structure itself rather than one more indirection. Jerin