From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0073.outbound.protection.outlook.com [207.46.100.73]) by dpdk.org (Postfix) with ESMTP id 663CD5AA0 for ; Mon, 30 Nov 2015 06:48:20 +0100 (CET) Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.Jacob@caviumnetworks.com; Received: from localhost.localdomain (122.167.201.210) by BY1PR0701MB1722.namprd07.prod.outlook.com (10.162.111.141) with Microsoft SMTP Server (TLS) id 15.1.331.20; Mon, 30 Nov 2015 05:48:16 +0000 Date: Mon, 30 Nov 2015 11:17:52 +0530 From: Jerin Jacob To: Jianbo Liu Message-ID: <20151130054749.GA11512@localhost.localdomain> References: <1448631268-10692-1-git-send-email-jerin.jacob@caviumnetworks.com> <1448631268-10692-3-git-send-email-jerin.jacob@caviumnetworks.com> <20151129234829.GA2913@qq.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20151129234829.GA2913@qq.com> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [122.167.201.210] X-ClientProxiedBy: PN1PR01CA0016.INDPRD01.PROD.OUTLOOK.COM (25.164.137.23) To BY1PR0701MB1722.namprd07.prod.outlook.com (25.162.111.141) X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722; 2:kI3CrzbijjlXYMfFCo8inn/lZOaK0KZN1GWNsCj7fTWDTgAaWZNmmpfJXT9G/lL4JS/WZkBgWQ+7HXXXmycY/UTZr0mp7asoDCrmNeTWziAmNddD6o2F6bSdjXeD/+Ll3xSugf8wMEBLjQG+ycjGSg==; 3:hQonEKs0amrZr8Xa+qtLNWtRLYjNX/8e8pf4TcIYtjNkxNoLK9l1eB5Egah6KqxkktYgFuaxhosO/OW3rfrbRq1igAHTEAv5QtsRhCCRoH0E4Gvq0pvVf28ZieN2L0LB; 25:WVuyN0KUtxp4j1ikWQB6BceOdwe4Y1RXctqL7pqjXK2dKDtTJb9zGlHboI+5Azt7rz0UVSGp3zkRDIXryzocjBpduGU0dUJ3Of5lAW19abVvgzEPcDrHTGuOcV7EfT7I1O50ZPjQ6NmGPWCXamc2M51slkle+rb8eIqV7RXT6vr24DiXUVU1QqqwqKV+kAa6Rjt1bKzfcNnDhOfNBPT9Nt4tvX5EGBw7vuV35aw8qDZPQDdVCmDitAcBFDx952dlxrJfhDfhpwP3tIltSUSw+A== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BY1PR0701MB1722; X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722; 20:3FeOpmqTNb2P91PnqJOAR+8Ei0PZSKIpZTIKv7TQNq85yufkS6eTfSOpWkpJDSqyuwDfhhX7BfjBoe4yy6iXZa5mr4YAH8W+QAH1H4KIWgunPad5R8TMguRwvVY5t6BQWyciFhjZ1LjkLrpnV45QxJBv4+/7nWGb1Oy+6d6vCWdGKckVdScCYnYI9pHcpUWKwmQWmDPIv6ulqcVvvFYshkMdsnY136lAO47bBXV8iw5hQo9DSQml8sD4GU9Ygjr9wZsMPlU0qASoWjQPURFHs9P3uX38nDunAabCLJrpAIZ8HPj5Kut+eDwlwO6tXX7ohZWwXiqZcfzQkyW7E7YgMwc0NHy1u49a71YQ5dgzodlhC7bninra/F3RFvclPQENXh+HJVVaMlTHLGJ5vi2Fm8kdViE5rj8LkVQjeoMyAp7ZU2azAP3lRri+9Pj5/gLiqfLP2ZoAEJX/IC+f3tQMtdxuD3GuaphwBlrKeXoGQFd8pVvgtr415kwh/wFviBUrSyNCXxpt/EtLIFa3foQj4sgBA+ToOrjalZNsW3Kp6X5xJuGR9+Lx/FbVlZzPqiAiVelSe4kjCIhwpGYehJC2cwNpE1f0kfwRSzGus0t47T4= 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)(5005006)(520078)(10201501046)(3002001); SRVR:BY1PR0701MB1722; BCL:0; PCL:0; RULEID:; SRVR:BY1PR0701MB1722; X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722; 4:uQLk2lCY8zvBxVT2WV2k/5AAVTmU6CsNYqLjGqV6SWKdG3Vhrximl2dP0tlD9RMqAH1WZP4sGLAFqIDnXVgtixRvb4J9r6Vlc72c0amzhvGupPEEMbFFmahFTcR9S8r05+4PSeN3oQSYX9+G3sRTYA8Szf4T1XAVU9/wIkd21G39j1Er/ETM6Ad2P0HMHtBX5QTKk9XvLWWK/J1k+Lr/mrMMT+F0R8kr2Z6L+15QCZcNFPrRGDDEx82TXyorES6U3khiNRJI+CBE1vKfzP686dZVG0CLbLHv1RJS0bItTUV7bd7gZtz6KdGhU20ZpQhI4afWA9hN9qX/SY9L5SDJcTc64ZUpk+VA/Te+ywz5pAwkqd2BmYAc6jOXaSZFIqd1NArgx4zUMdcY2f7A4MVUxLUeabgtOCxdnJLzJDaXGpSX+V9hOF/asHCIxTBeltAC X-Forefront-PRVS: 0776C39A48 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(6069001)(199003)(189002)(24454002)(51914003)(97756001)(101416001)(77096005)(50466002)(5004730100002)(92566002)(33656002)(87976001)(42186005)(2950100001)(5001960100002)(19580395003)(110136002)(1076002)(19580405001)(50986999)(5008740100001)(40100003)(122386002)(76176999)(6116002)(586003)(4001350100001)(3846002)(23726003)(1096002)(106356001)(83506001)(97736004)(54356999)(105586002)(81156007)(189998001)(47776003)(86362001)(61506002)(66066001)(46406003)(7099028); DIR:OUT; SFP:1101; SCL:1; SRVR:BY1PR0701MB1722; H:localhost.localdomain; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX: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; BY1PR0701MB1722; 23:fkvfr2ZJN3NvkUFuKbcpJ165E+5zPr1vzKxKXvh?= =?us-ascii?Q?802WQhrJtY9duRcss+fKcXajJ7rnTTi4T0U3nAQ8r2ajT0ygBUD+n2fDEgQU?= =?us-ascii?Q?/tq1MXbHyIikCVKitIaC4defy/Lc8LDs0Gxnr7nqLDE/fTUgkLjQ2GJBVvd9?= =?us-ascii?Q?gj+VFq/ax67QOXKTmOxSqvVchklBka2pjReQyT9i8feQGjoRcVa28BC3XRwS?= =?us-ascii?Q?4z+4NG+O5WgRyisfUyNVfcbpK0Fh8fEu0tGHZ79tpY5hAfXeAFVgP3duAU6F?= =?us-ascii?Q?kwHJLtaHfFzgbIRgzHCEIB0h7LeSVIcj55/JxKFs4H20QEc6xHDOe/AaZlJU?= =?us-ascii?Q?M1wmIBBVUoWST7ha23TKuQtKVAonp9/TmFZc1Bl1yeEr6WRinE2RR5SWsXkD?= =?us-ascii?Q?Uaf9gwo0frNsl09YNhDmepNsj8nFN0vzYGDQia6i6K8vos8V0gDtHwm2AJn1?= =?us-ascii?Q?vXhwiASld9q3lqmhdIkw9Pts/rgjIC0Fz3MvnOARI9VjayNUletDkhbZwNi+?= =?us-ascii?Q?yNcg+i6HwxjB6bUhEpJ1wAnZpKPg8JO8inxCiwpCfPsU8rqHLuzVbVOui8xF?= =?us-ascii?Q?2r/ohV7VArgEmyIV4PoZnnTwQ+zk0T8t1IySCMvRL+rhgut5bQtxV4HHH6uO?= =?us-ascii?Q?cr1w36MrYqHmfyJpyHllNJWk53T7zLo4JrOVsBfLR3HJKzymnFdoXxekL2DE?= =?us-ascii?Q?uy0oSCio8NbS5LsSUhX/L6eSe8lx61SMM6lbnBvHs/dbk31wbT8mr5C6xjjD?= =?us-ascii?Q?p3U8MZkuSLLo5fy+gAvk4iSOmA0eDGtWgUA73RY3KVgJV/cJzH8fq/zlFulV?= =?us-ascii?Q?1L6n1/CgYL+U4CJRp0PxvQ5bIHIjl/k+iCoFxjd7vvz6z/AgL+BabzfIf+2A?= =?us-ascii?Q?StSuHyZkXGjuiqAiqs7JiA8E4VHaIpKlQRyEVF+xuz3I/7CwIDhc0L6MzORP?= =?us-ascii?Q?nfcKDJAona03c1bL+6/Mh6LU92b8WmX5j7W1YxgUZ02u74XzbXqf7LBzIWuK?= =?us-ascii?Q?nYI6Gg2sHuunahVY0kVsQDHj/0XrZs+q6Pyb4LD7IJA9sPCWWRVh6FwagbtV?= =?us-ascii?Q?yIkZpgFObIXYhlC60303gO2wF8KQDHBMbCSBZ1Upoi8OWz++KkoWiZSqPTfM?= =?us-ascii?Q?qo3z3EuivXT60408PWt/0z88o5y8d0rhCPEE+44hLTuJQxNfiXoGNTQ=3D?= =?us-ascii?Q?=3D?= X-Microsoft-Exchange-Diagnostics: 1; BY1PR0701MB1722; 5:OHIIHDugGPZdwwxbTKpO/CL6kwiylgO3Eeiu3g2C8y9ejF0+e0lyWm6ARPrgGz+uDnEV+AgD2QKynmxfUpoNJXgvln4HQscj/McUrytUxxUOawlNIqr/ZLueV37LoCvhInQCOWUJd9cf0NPPS1qg5Q==; 24:umihFueXQcLRktyBPB0w2HrH8w+uumAuUICBq6R01N1TWtDqKoLIU5u5W1YKHME184pIN4ynIg2zqGPOH1dvqEepxKIZOUPlVcHp3VguUOQ= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Nov 2015 05:48:16.0722 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY1PR0701MB1722 Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v3 2/2] config: disable CONFIG_RTE_SCHED_VECTOR for arm 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, 30 Nov 2015 05:48:21 -0000 On Sun, Nov 29, 2015 at 06:48:29PM -0500, Jianbo Liu wrote: > On Fri, Nov 27, 2015 at 07:04:28PM +0530, Jerin Jacob wrote: > > Commit 42ec27a0178a causes compiling error on arm, as RTE_SCHED_VECTOR > > does support only SSE intrinsic, so disable it till we have neon support. > > > > Fixes: 42ec27a0178a ("sched: enable SSE optimizations in config") > > > > Signed-off-by: Jerin Jacob > > --- > > config/common_arm64 | 1 + > > config/defconfig_arm-armv7a-linuxapp-gcc | 1 + > > 2 files changed, 2 insertions(+) > > > > diff --git a/config/common_arm64 b/config/common_arm64 > > index 5e5e303..d6a9cb9 100644 > > --- a/config/common_arm64 > > +++ b/config/common_arm64 > > @@ -46,3 +46,4 @@ CONFIG_RTE_LIBRTE_I40E_PMD=n > > CONFIG_RTE_LIBRTE_LPM=n > > CONFIG_RTE_LIBRTE_TABLE=n > > CONFIG_RTE_LIBRTE_PIPELINE=n > > +CONFIG_RTE_SCHED_VECTOR=n > > diff --git a/config/defconfig_arm-armv7a-linuxapp-gcc b/config/defconfig_arm-armv7a-linuxapp-gcc > > index 82143af..9924ff9 100644 > > --- a/config/defconfig_arm-armv7a-linuxapp-gcc > > +++ b/config/defconfig_arm-armv7a-linuxapp-gcc > > @@ -57,6 +57,7 @@ CONFIG_RTE_LIBRTE_ACL=n > > CONFIG_RTE_LIBRTE_LPM=n > > CONFIG_RTE_LIBRTE_TABLE=n > > CONFIG_RTE_LIBRTE_PIPELINE=n > > +CONFIG_RTE_SCHED_VECTOR=n > > > > # cannot use those on ARM > > CONFIG_RTE_KNI_KMOD=n > > -- > > 2.1.0 > > > > Hi Jerin, Hi Jianbo, Thanks for the review. Looking forward to seeing contributions to DPDK-ARM. We definitely need more hands to make best DPDK-ARM port. > In this way, we still have to modify two files each time a new feature > is added but not verified on ARM architectures. > Since disabling those drivers and libs are common for both armv7 and > armv8, can you put them in one config file, for example: common_arm? I initially thought of making it a single common_arm file, Then later I realized that it may not be worth as, 1) If a new feature added to DPDK which has the dependency on SSE then implementer has to disable on "n" platforms(tile, powerpc..).By unifying single arm config will make it "n-1" so it's like "n" vs "n-1" not "n" vs "2n" 2) AFAIK, PCI NIC PMD's are not yet supported in ARMv7 platform yet unlike ARMv8. Till we have PCI NIC PMD support, armv7 config needs to be updated for each and every new PMD inclusion. 3) neon capabilities are bit different in ARMv7 and ARMv8. For instance, "vqtbl1q_u8" neon intrinsics is not defined in ARMv7 which used in implementing ACL-NEON. i.e Need additional efforts to extend the armv8 neon code to armv7(or vice versa).So it's better to have fine control on the config file to enable selective features 3) anyway we may need common_armv8 file to address the "IMPLEMENTATION DEFINED" parts of the armv8 specific in future, like frequency at cntvct_el0 runs ? optional features like armv8 crypto instruction support or not? It's armv8 v1 or v2 ? atomic instruction support for not? its a long list 4)I would like to see ARM configs as different config like i686, X86_64 in DPDK > It is not like common_arm64, which is solely for armv8 platform. > Actually, the arm64 common config is defconfig_arm64-armv8a-linuxapp-gcc I thought so, Then I realized that we may have FreeBSD, arm compiler, clang, llvm support in future. > you can include it in the thunderx or xgene1 config files respectively, > and overriding some special config if needed. Agree. existing patch addresses this > > On the other hand, If we support the features in the future by > replacing SSE intrinsic with NEON, we just need to remove the lines in one place. See point 3 above, I feel rather than coming with the framework to fix the exceptions it's better to fix the exceptions its self. I am planning to send out next patch by today for supporting CONFIG_RTE_LIBRTE_LPM,CONFIG_RTE_LIBRTE_TABLE,CONFIG_RTE_LIBRTE_PIPELINE. i.e only a few entries will be common. Please find below the list, the reason for setting as "n" for armv7 and armv8 is different. lack of PCI PMD supports vs SIMD support. CONFIG_RTE_IXGBE_INC_VECTOR=n CONFIG_RTE_LIBRTE_VIRTIO_PMD=n CONFIG_RTE_LIBRTE_IVSHMEM=n CONFIG_RTE_LIBRTE_FM10K_PMD=n CONFIG_RTE_LIBRTE_I40E_PMD=n - Jerin > > Regards, > Jianbo