From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0095.outbound.protection.outlook.com [207.46.100.95]) by dpdk.org (Postfix) with ESMTP id BDEC958DB for ; Wed, 2 Dec 2015 18:38:32 +0100 (CET) Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.Jacob@caviumnetworks.com; Received: from localhost.localdomain (122.167.201.210) by BY1PR0701MB1721.namprd07.prod.outlook.com (10.162.111.140) with Microsoft SMTP Server (TLS) id 15.1.331.20; Wed, 2 Dec 2015 17:38:27 +0000 Date: Wed, 2 Dec 2015 23:08:06 +0530 From: Jerin Jacob To: Thomas Monjalon Message-ID: <20151202173805.GB2452@localhost.localdomain> References: <1448995276-9599-1-git-send-email-jianbo.liu@linaro.org> <2275492.7Tn0tJ2v06@xps13> <20151202165302.GA2452@localhost.localdomain> <2257776.TI734NhvVv@xps13> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <2257776.TI734NhvVv@xps13> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [122.167.201.210] X-ClientProxiedBy: BM1PR01CA0006.INDPRD01.PROD.OUTLOOK.COM (25.163.198.141) To BY1PR0701MB1721.namprd07.prod.outlook.com (25.162.111.140) X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1721; 2:r/BvxetGEt6T3LA3OlgCNEw2TglJ5nquBucmVoa0Vas68i6SEevAKWWfWhLMoOs5Z9bME7TtYp5Lvrh71EW+gzZHXiO1ZOou4ryLbzVElMihrCxbsSZPcKQXUW6GOTMyUCm94hffCFzS1NDt6aG9iw==; 3:6PFmo5J595ahjrELzp+OAamsLAc81AGFBDOWdMdr3ZtZvbh5uztMIptyZ0IUkLUWLiK9BVl8emzvfwfxZSZp62HeGZbsSkkcDdnLt+wYs+7uN8cmUqa3vdyE5mfH15XD; 25:ztQE7xFCIJBLiOu1AUE/oXSguv3BdrmaRnffnQdwZG69tmv3o3x/z9PSYDqBK6HxUl1hC3Q23K6R2l11rKxr3hC30lGEtbvBJI4DrRU8X0OavUpO201T6vvyPtOra7IUBoVhKvnNH0txclbkFN/wnlQz/Qd6T+a+2cAVWpZblF3/TD8WYS2/QBFfF7EpqDvF4X8w4YR98sR6ucZaIn7dYIMblVVNiwBGYZJWd7MIjWlNt3LGwD8FCh+Wg5/JW6hZdA0y1zTGli+vu1yLMsz8pQ== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY1PR0701MB1721; X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1721; 20:9rosostmnKbBPLvETaAq3/W68WTdrPx17oz+CDPHij86xTz3Ejk09uniZYe9ZC1BStwbYn55yjoOBVUimfLvGpFy82QIJeeCrJo0/hR99vzLJb45VSs/a9dAWdmOc+IuqXpFt9KY52fx7hZpf/2+OGMgwYts07IyZPe33fk7uautXxxZwx+WjbKy6qamSI3dvS2Fhd1PfcjMMyCj2tV8GGhiln1Oc7EiFp+I2gJlPheJDKfavsjwNuQLW7W0rF30lwVMazNxiH9LUyEHxF97Mny53M9awDBjoBONtpfD//l34mgnxAQrlDFvT+UKI8KMqCiov5ZXSoP0DGbGj4za/02NRIMaemBixUI5Vf8jkDZbwnqHJlPK4lcEOU/1T/QAA+/8zU+ZvXxOmCPfyFCa6IJ/rTQDbAbEY/xwC3N6ixxCNbzOjMACuKi36/MNjKcBKjnxGPU/JgH/VRk/UwAPNkQuVa76W241H0ppc/vpV9Igwc444k5755f9anxQC6CfpfRxpqdAY5LWBhjWPCW6ety9faxEzk06idCxUBPkW5BxW3+jrm3hlu6rFSfd7GrZKtY8xZaltCgJxH5tGuUGqNxqML6FA0IX0NKMKqWReYo= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(236414709691187); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(8121501046)(520078)(5005006)(3002001)(10201501046); SRVR:BY1PR0701MB1721; BCL:0; PCL:0; RULEID:; SRVR:BY1PR0701MB1721; X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1721; 4:l8npmpSywLviS4xyUWE+eul3B3ZKwuUmCjTiXwDuefRvUzGNQUXQDTLm/ldhjuLP8mlC0GRLelJRtuIOpwDuVTeVV0PcLZxGiOnIVNcqN8qS4iGN1WQkb4nvJ/7vG/T07EU+h/mTQeKtT8+8MEaUeCAp2fmvmbIBi9sLbKaI3VAb5NmH3drpgUNgaxOPUx/+pUznDROYfEocTYwjLjW1nhChcMOSXutDfI59BsUjRwURTzpY+cv+UISCh5OWxfT0S8/aMkxDfG8gHMBF7MAJM+rlo+nPhYUK0YUW/pVkt0Fod+DHCcYhMbA70dnAlq+T/NtkyrW5tNLfru+BnMr13SypDsxcvkLyoHq7u91TKSRROHfUEfXrVqjzg8pGJp/RKNRszREAVHm7D01u9C5CFjosp1eyvjEed6CWrzCAvgvNsq5rFe1/+PL0MpNS0HhU X-Forefront-PRVS: 077884B8B5 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6069001)(6009001)(24454002)(189002)(199003)(377424004)(2950100001)(40100003)(5004730100002)(42186005)(47776003)(122386002)(4001350100001)(1076002)(46406003)(3846002)(586003)(77096005)(23726003)(6116002)(5008740100001)(66066001)(92566002)(81156007)(87976001)(50986999)(189998001)(19580395003)(5001960100002)(105586002)(61506002)(19580405001)(106356001)(1096002)(86362001)(97756001)(83506001)(54356999)(97736004)(93886004)(110136002)(101416001)(76176999)(33656002)(50466002)(7099028); DIR:OUT; SFP:1101; SCL:1; SRVR:BY1PR0701MB1721; H:localhost.localdomain; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: caviumnetworks.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BY1PR0701MB1721; 23:awhcd62Nf1AJeattA4t345JDdx4Y26NJ3d4JsU6?= =?us-ascii?Q?bfgcROWsEIJlerp9dv9n4pYoyqf9w5nUUw4sAYwCvUpt6Q9/9S3i5difGf8m?= =?us-ascii?Q?TabWtSs8Ev2FziIrIOzEaCnbi0e7fjCXE1E81GhyMfi8uF2uSic9MpkOWdZS?= =?us-ascii?Q?7Gxle/roFAaqzMynAq6k7VFaNdT5nSOXH9YynF6VbpnPP1gKZKz/F4JOy4T/?= =?us-ascii?Q?q7/BEVUG8sFCP+0PQYs+iTVKeMtM7PrUXl5L8jeClNm9at3asqsLv+gGkEAb?= =?us-ascii?Q?IrH/GQ4C6GdPvrx3CcrvGpIM6RmncRelfA09CqwQ27S2vcJViMNGeo1clFI1?= =?us-ascii?Q?Ej04I8JYQFC/TPylZHkc+PDuihyuMprUzdv9jhq8Q9ipDsrqE15xmkXziGqR?= =?us-ascii?Q?dClzD7u1sYRM5FlwVT0r8XW+Vrh/vOigoAMSWzUtHH9caZ6tf/VfkqAzhJGO?= =?us-ascii?Q?YSfzgxq7TE/3KZpd0GBXVexe9r2CeauL+rP0yp3A8vgjoWD2Mwg8DyyMmNDW?= =?us-ascii?Q?XkAPVVkaRcfOta+MkO5WzhujtU27t1D979O7AcZk72n+TQ6lLChybxyLARc7?= =?us-ascii?Q?EvUHUwtwbNe+QxTV6ZsCzesBDbdfoj+vzbWNOZoRKO0m/HCgs2iBq+nd/2l/?= =?us-ascii?Q?FsBVG2FRm9xzYWGs0l400+jOhqciZF9R95Az6uOPRFbV/to4pm9+8JxRqKdh?= =?us-ascii?Q?+c3f5m/CpvIzNtE5OgZD194wJ+TyJSQIF6ZY1g60ExtpEnX66/O4pl8jrj9R?= =?us-ascii?Q?0VvnPEM5JKEmeVnbgUb9r64dqMFAOR399ItOSXbsRzBm22fVT83pwf4Dd7it?= =?us-ascii?Q?Tuy+TTABDUS55KHJ8fGk9YzyjLYrHrIPPU/m7AGlKm/8p6J593eIMqeAJFu7?= =?us-ascii?Q?mRLYzL6dLfD7VWJWkNqZORMBF5I+5HbG65uNQPgiOgnIQabrav2y+5VPFg3T?= =?us-ascii?Q?MUzuQI7VZYbIgIxh0rVAWRmnQgAPBArUkIUty+UwnW5+/WgL2v0/m+tv/K6K?= =?us-ascii?Q?AVYQbveGZ+6KVGAkc2lzheQ55ZoJHO+VfjEzSTWLkNGnGwp302GJq7ReRtVL?= =?us-ascii?Q?SJSYJjshj4ljlvVYX4NkQK5YTkFLiU9wmY6ZHhVmyI7I8phyFffRWFDvEhOK?= =?us-ascii?Q?vppD1xnAOdapYtsgVzeH3Hfkib3JHBjaH8+7A2E1mRTkCmWXB2niHfAEs0Wz?= =?us-ascii?Q?9H2VOLMhcUgzf/38=3D?= X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1721; 5:tMQboqyc60X4tfb7ickiMf2oRnShCAzEQ9q7gzoVvTsA5nWKtMvqnys3MPWFidQkAl2hu7pIA4zq2IPhhtNDL+so0Q7FnHTkMyOpY8KCcOQXEovNVRauVY46k4mZWjoF4Sx2JofnkY5RCfX17CtOQw==; 24:HyD0wdhUMPb9WParQPIxdnMEoUuE68FLOJ2deZa5mPqXbkx+PJcmoN6ZcTUHb8DsZ73mEiMXVzlwOmQW4+D5EiK16UopxIRXerIHNQKwtHE= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Dec 2015 17:38:27.1385 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR0701MB1721 Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH 3/4] eal/arm: Enable lpm/table/pipeline libs 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: Wed, 02 Dec 2015 17:38:33 -0000 On Wed, Dec 02, 2015 at 05:57:10PM +0100, Thomas Monjalon wrote: > 2015-12-02 22:23, Jerin Jacob: > > On Wed, Dec 02, 2015 at 05:40:13PM +0100, Thomas Monjalon wrote: > > > 2015-12-02 20:04, Jerin Jacob: > > > > On Wed, Dec 02, 2015 at 09:13:51PM +0800, Jianbo Liu wrote: > > > > > On 2 December 2015 at 18:39, Jerin Jacob wrote: > > > > > > AND they include "rte_lpm.h"(it internally includes rte_vect.h) > > > > > > that lead to multiple definition and its not good. > > > > > > > > > > > But you will have similar issue since "typedef int32x4_t __m128i" > > > > > appears in both your patch and this header file. > > > > > > > > I just tested it, it won't break, back to back "typedef int32x4_t __m128i" > > > > is fine(unlike inline function). > > > > > > > > my intention to keep __m128i "as is" because changing the __m128i to rte_??? > > > > something would break the ABI. > > > > > > Isn't it already broken in 2.2? > > > > Does it mean, You would like to have rte_128i(or similar) kind of > > abstraction to represent 128bit SIMD variable in DPDK? > > If you are convinced that it is the best way to write a generic code, yes. > I think the most important question is to know what is the best solution > for performance and maintainability. The API/ABI questions will be considered IMO, a true portable platform-independent library may need rte_128i kind of abstracttion to represent a 128bit SIMD variable. I can send an RFC patch to see the changes required across the DPDK. > after. > > Thanks for your involvement guys.