From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0055.outbound.protection.outlook.com [207.46.100.55]) by dpdk.org (Postfix) with ESMTP id 1666F8E8D for ; Mon, 2 Nov 2015 13:58:06 +0100 (CET) Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.Jacob@caviumnetworks.com; Received: from localhost.localdomain (122.167.52.198) by CY1PR0701MB1978.namprd07.prod.outlook.com (10.163.141.20) with Microsoft SMTP Server (TLS) id 15.1.312.18; Mon, 2 Nov 2015 12:58:03 +0000 Date: Mon, 2 Nov 2015 18:27:46 +0530 From: Jerin Jacob To: "Hunt, David" Message-ID: <20151102125743.GA9506@localhost.localdomain> References: <1446212959-19832-1-git-send-email-david.hunt@intel.com> <1446212959-19832-2-git-send-email-david.hunt@intel.com> <20151102045728.GB16413@localhost.localdomain> <56375597.8070805@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <56375597.8070805@intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [122.167.52.198] X-ClientProxiedBy: MAXPR01CA0013.INDPRD01.PROD.OUTLOOK.COM (25.164.147.20) To CY1PR0701MB1978.namprd07.prod.outlook.com (25.163.141.20) X-Microsoft-Exchange-Diagnostics: 1; CY1PR0701MB1978; 2:M2AYoxqGD5AzHwXPGUPVBKLjFXB22z5FR8XXkGml1x0DVjVTITsHSuc1W8eIMSUviERif9EqT9Eg+DkFoLINTQMlGhnYIaP7jwC2rlOUFgzGNKECbz3zdkGubRNFjAafT1U/sAijzERyuPtKK0VuGowNLrDcpChr7TB+alataJc=; 3:Pa0aKQYw1/zJ+LX7OXuLT+RPDqs0Oi6SP9HCKrdpgTfPWZo8VhdPv9f0wViwMboxX2AxsflEtBO+tA3OTQ//0k1oC2fMmMonnFLM3aI20Jr6c0CUft/ZixTYHnAJKEqVKxNbywUbTT/gJTO6znSnRA==; 25:mqWFPsKAktdy3zlJwVhmxRFFQX/J7Dh8svqepMQmXCa/pN4kVfAb+2NCfN1GCUq4TQcWXfMhThr0l6AltSInbFk/1oNIHEWW/MumcYgzVlcDaJASzqLo4lgSRaQQ+/tPO1wZVY/zNaCvCDXZToDOGGTOAe539qZpslLA2s0z38SN+9MWNzeXZjCoUJZJL5pDk72m8+p3VvHMV5p98g+HxkQB3ifMKvPaCYUsWS3C63pX5E2Mj1plnDole3L1Ojpy X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:CY1PR0701MB1978; X-Microsoft-Exchange-Diagnostics: 1; CY1PR0701MB1978; 20:LpnjCzUeMfJcBX6cpkI9nNcmVFwfj32vtE2+b3u3bHDbKhcFs+GqcnHDiQVT6aQBcTBFqtOagYjnru3Ityf9dVW+d6fvT0twMc58AbRGP5Fhsbwg3KRlzMDQVKAfnTX0MOaMdUNyVfUJ9i40ZiQVKFsC8YJmjKwvkwiJO6DkwWHNWYuu+kpLzxB05QkhmglfDrkNmu4pG+QGZcFOLoYVyqheFueZF91e/A4aFCQ/ZZKvR/HJeJ2ah09YZ2QDiu04NcCy0s/wWQZ6dLF5lPteiKKZ7JreCyODtmuaf217MwnCNkbVki+Xs11oDHyv2nM5Uj/M6FwfpkHMvL96J8IAnv+VJFZMeFPLXZxZDVk2ZzP5QlFIiZSy7Ri0tcCQL5FKe8iRIaY7a3GV5Q+eqTtXmogpmmTaTZUCG6hUgGeGcZ2QV6I3t+u4qr8PoQBqCZ5XllFKR2lilYxyOwbi9iE/TiwtB7z0Spi0lvkzrLo+IgWTgx/yHPJBgGFdWOIIGQRF8yThGC80+eA7zpsfw3Fd2sIEbfAsdo9dKr01tV2+77XyLiy+btJI3pizuCuZelD3XdJdly59Mb7qQ5DanLUxafDIrD0p7gbuOcs8wO08wGo= 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)(520078)(3002001)(10201501046); SRVR:CY1PR0701MB1978; BCL:0; PCL:0; RULEID:; SRVR:CY1PR0701MB1978; X-Microsoft-Exchange-Diagnostics: 1; CY1PR0701MB1978; 4:GdBiOVvMNTwW0g8DWI+bbQA3JkbaDJQY1yr0neUbMHZSBP1JRnRlaklTQ8BJ81JT6iwUbyOCdirzlHnS/x4JbcZ1Y5K3bUmco4m3Kbr0uxH2/v+BQrjORUEGByYi/DobATQ2SSNnYJG9i1QI03iKSGbmtLndTLqLl627+Yl53sLmKa4QKVu3Jkhi8xXKWBtvL58qY7tbwVRXTL3kAS33YQd9kJNYgkf7szp1i2uzV0M7bS63NC5OqghdoMO9eJr/b/2eU413nWxHN/0WbPUhDHRoubAWevvQ+AVcN4/lwg87hRdwRm0I0yAE/GMpo1oq1QlDwOsEJV07dm9jvhn86atmUyJ/15hDhFMe5026F7LTKGBvSP/hOVUlZDXt2KPQ X-Forefront-PRVS: 0748FF9A04 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(6069001)(189002)(199003)(479174004)(24454002)(23726002)(47776003)(77096005)(87976001)(83506001)(86362001)(19580405001)(19580395003)(92566002)(54356999)(76176999)(93886004)(50986999)(101416001)(61506002)(66066001)(189998001)(5007970100001)(110136002)(5004730100002)(81156007)(5001920100001)(97736004)(5008740100001)(40100003)(4001350100001)(5001960100002)(2950100001)(42186005)(46406003)(97756001)(50466002)(106356001)(122386002)(33656002)(105586002)(7099028); DIR:OUT; SFP:1101; SCL:1; SRVR:CY1PR0701MB1978; 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; CY1PR0701MB1978; 23:QqDOfZ4P2NzuDcDRxScbJjyF87OG4EUVUj/psOi?= =?us-ascii?Q?1b6BZ1L54WrmUelTCBOkzM/TLmZ1lqcgJvhsQtvFLg024hci9pwZZZK+qOj5?= =?us-ascii?Q?oTzqIA6NmH1NBULzR2ooTU5cVSQKXr2UqigvhfkuOHnNxMvBrx69cB96LGtw?= =?us-ascii?Q?C6REyLftD1OCGiTIXG3a26irm88EL1qTKpdUy5/msHOnCVsN8HoM7/YGyPB4?= =?us-ascii?Q?7n14ozHhkMH0Wf0o1+Zffyd08ZNIR+7sivU80QVfKaWW8UaiSDdtDCyagr3Q?= =?us-ascii?Q?zYnQDmqtveYEaYCnSXVH2pLZsmC3SPKLBZFr6MtnusP/P6Jno1ZKHHwzdyAF?= =?us-ascii?Q?Ve8F6sp6TOBkWsD7qRyEgYuedAvR2Sfv3GvsyNfKfyPTKqkpniyguLSgA+DH?= =?us-ascii?Q?ZR5xv7EyrBLkdgXfalQYPr9WMMAO+VHB6WV1ETXpaCmNN4uyHsVBf+w5cTtN?= =?us-ascii?Q?md5JAsWKO+0wskrphDs32JIR6vzuD6UTRqcIeOdmcFD8sKt7BFhfWDGyd7X/?= =?us-ascii?Q?9Eu6DziIvU2hXoOHKe63Sev4ZnWLqe3QikcCsZhDFMGzhE7SlQoRS/PV14IR?= =?us-ascii?Q?4Ecz13jm1XuSjxv8ZsW0u99sJfj+UklUWZfN8UHDKRNWRBQIPAYCYEPCZ7td?= =?us-ascii?Q?zk2mpIz/DSjURsyU/i4rcohwhRvfRd8E0RZ5r5OnSkka6EimAw/ZCDyyrXxP?= =?us-ascii?Q?s8Fuod0uDrciBNtMMGXGU67qsQgR5EnHX9y4HnUIPoITr1UQ6bnrAIjCmwnP?= =?us-ascii?Q?3N8XsnkoJPM5rEyPZPsgPmQnrr+K5EO4PmEloa4xp9fKyklN/y6kQnmqkg+4?= =?us-ascii?Q?hombfVLGvIkYuFydn/zpvFrfDq2fzEdCjSx+U2ui1kWJ35HogMaRuolEBrxV?= =?us-ascii?Q?Y5TAQXXDG6msvIPOhWskID9e38nCrbCgO/Eqj6GjbGvXA0s+PxoCuRh5etu8?= =?us-ascii?Q?QlO0GwPZqwAzdPc//ZSDBmKMY9IlgOBqh1GbwrhfI7FX7PNqQWz0o4+fMUmN?= =?us-ascii?Q?rwm/BNvSAxD9d7LK7BAy6CwE/eGShcAtXWYwjn8IcAoLJGlxNswhkamm+SIC?= =?us-ascii?Q?efDMupfyyIUilYBHFYDaXCRFXPZ/Bcj0VlOqrfROE0uY1lF8/Kr8cJHLDPAF?= =?us-ascii?Q?9eUjuUT/nSaK0Lxp9uycJBemaiKQwfW4nRCkEq34ax81QCV+4+edUbA=3D?= =?us-ascii?Q?=3D?= X-Microsoft-Exchange-Diagnostics: 1; CY1PR0701MB1978; 5:YNWm+OkEY3swWFohcsfTWKZOmp8jJsrZEUXyXK+tjJfa+JP+d2f+LlTaKotWdBJCSZ//brbAI/VlSKGzmoUYlgfJrXJP6vALkK+h3eGoqQyuvwD33gmz5u0yo//EHtOPG2DEUj03d8Q1MK+/doN4bg==; 24:gAh91b+AqkLd/tnLDkdnfhPHBnhVMUP/oYBI0IROXl4nyhnKjpASdQ7xDByKN0rJhgQz5OHyyOnm1QS7IITDHm417+JzI+i8XiB7eA3JXXg=; 20:TQMQiEza6wG881UZuZAWte/1ZcMk67fYbBZQUs9F81499yGmK1fKE1xQqTQkooJziQr06U4j4S3ez5LBKIiAmg== SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Nov 2015 12:58:03.2324 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR0701MB1978 Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v3 1/6] eal/arm: add 64-bit armv8 version of rte_memcpy.h 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: Mon, 02 Nov 2015 12:58:06 -0000 On Mon, Nov 02, 2015 at 12:22:47PM +0000, Hunt, David wrote: > On 02/11/2015 04:57, Jerin Jacob wrote: > >On Fri, Oct 30, 2015 at 01:49:14PM +0000, David Hunt wrote: > >>Signed-off-by: David Hunt > --snip-- > >>+#ifndef _RTE_MEMCPY_ARM_64_H_ > >>+#define _RTE_MEMCPY_ARM_64_H_ > >>+ > >>+#include > >>+#include > >>+ > >>+#ifdef __cplusplus > >>+extern "C" { > >>+#endif > >>+ > >>+#include "generic/rte_memcpy.h" > >>+ > >>+#ifdef __ARM_NEON_FP > > > >SIMD is not optional in armv8 spec.So every armv8 machine will have > >SIMD instruction unlike armv7.More over LDP/STP instruction is > >not part of SIMD.So this check is not required or it can > >be replaced with a check that select memcpy from either libc or this specific > >implementation > > Jerin, > I've just benchmarked the libc version against the hand-coded version of > the memcpy routines, and the libc wins in most cases. This code was just an > initial attempt at optimising the memccpy's, so I feel that with the current > benchmark results, it would better just to remove the assembly versions, and > use the libc version for the initial release on ARMv8. > Then, in the future, the ARMv8 experts are free to submit an optimised > version as a patch in the future. Does that sound reasonable to you? Make sense. Based on my understanding, other blocks are also not optimized for arm64. So better to revert back to CONFIG_RTE_FORCE_INTRINSICS and libc for initial version. BTW: I just tested ./arm64-armv8a-linuxapp-gcc/app/test and "byteorder_autotest" is broken. I think existing arm64 code is not optimized beyond CONFIG_RTE_FORCE_INTRINSICS. So better to use verified CONFIG_RTE_FORCE_INTRINSICS scheme. if you guys are OK with arm and arm64 as two different platform then I can summit the complete working patch for arm64.(as in my current source code "arm64" is a different platform(lib/librte_eal/common/include/arch/arm64/) > Rgds, > Dave. > > > --snip-- > > >