Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753761AbcD2NOM (ORCPT ); Fri, 29 Apr 2016 09:14:12 -0400 Received: from mail-by2on0083.outbound.protection.outlook.com ([207.46.100.83]:64880 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753439AbcD2NOH (ORCPT ); Fri, 29 Apr 2016 09:14:07 -0400 Authentication-Results: arndb.de; dkim=none (message not signed) header.d=none;arndb.de; dmarc=none action=none header.from=caviumnetworks.com; Date: Fri, 29 Apr 2016 16:13:38 +0300 From: Yury Norov To: Arnd Bergmann CC: Catalin Marinas , , , , , , , , , , , , , Andrew Pinski , , Andrew Pinski , , , , Subject: Re: [PATCH 20/25] arm64:ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it Message-ID: <20160429131338.GB32007@yury-N73SV> References: <1459894127-17698-1-git-send-email-ynorov@caviumnetworks.com> <2760348.qmm1L3yQch@wuerfel> <20160428222137.GB18227@yury-N73SV> <4271094.ei7kj5zJxn@wuerfel> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <4271094.ei7kj5zJxn@wuerfel> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [95.143.213.121] X-ClientProxiedBy: HE1PR03CA0017.eurprd03.prod.outlook.com (10.163.170.155) To SN1PR07MB2239.namprd07.prod.outlook.com (10.164.47.145) X-MS-Office365-Filtering-Correlation-Id: b2e259b5-d6f1-4781-fee2-08d370302249 X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2239;2:ZMNRpCj7lrWxRLQhvUoP/ahdyRUunpanp9+53xMRSALsQ1R/v11j21NvEac3CArzsu4i2Cnn/De403AZ2nDGbMp2rf7nHhfj0eMNfHP1SzbHnV8vKeTvx2RanCTeFyN4qUAGlGjE2Q5ZU2DmP/VuQ8aTmAmGJCFEup1ZUS2Xet6eqIQpuns39XHjfVzXQ/4a;3:vJbXkJz9031mw0fYmwODPdwe++IHTf2upCEiJ/PoLaG1GbtenXMzP5K9ZSKfziXm82KfD5M0qSO9lP97MMs1Xydm4OMaGq/+50ZQUxO66AVBfAh/scKL8hRJX12cTwGw;25:4R/MHnRNGeLjllODLEuTpgIQsXVO4I9kRxpOKViNbBD3p15E5VPlwOhW5p1Gx5TFWH2X+ydLCdPMAWBvR7lEuHPKKpGP6J24WnQpFrnAcB6HAKeB/Fsv+bCrIn8GLMhRzziwwTyspapWAcdHJ9D8GJE2Gtc1sIz67TluZu9YxWsS+CK3iArvdqS3x01JuR0a2qhW1E884jrm8igknv+P98hGYTdxIUy4e3UqylN8USzjZYR4G/OwIPX6n+S3mnbcJyRecHkg5jw4v+MVsFOMcDqWsXVLbYmAndDFK3bXRw0+T01CxCxIHe2DwjpqGbK9rTotKSnBj08//b1iPDWtxg== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:SN1PR07MB2239; X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2239;20:G2lSTGySXZUI+7K11uIoQ+xknCRNCKyYUW+r6C+wXMSRN9F/dt5yQhPpeKYqq97RW/U2n4wxgLJrMVLyTDeAygmqtZ0Ztgt/d+N4L/XQ9JMWgFAUIrdUxUXmBjI6R7YrvYCcRal5lWc2NnJFmnG+/0Yi2S8n4rQTYF1S/hn8x9+Djo+kTJaN/luaoIULSA2lE18ibuqONSdZZZSo+ONH2Qce4vY1aLP7Cishr55Flg2okhkr2IIhAwX1RqW4HvginAApNVV6i6bbGln6vEVeRSQVvbmCVUnRACViCySlVjA+Rlv4prAnz8NPLYBlIrEnTEMvNsU1nOu5D2fd1P4No4vnbTBbYfk2xYfkZgUY4lkpY7xag0TMLCEHGqEoKQj/DyiI4DbdxLFYbwbknvGi2Mdv43+gYd6JQJPPgYHDRGglaHMi9ufwiteFLg3gaxIvzsxZDICo28kkPTO4xf5mUvO3DgkgHpLqMsWMD5ZRynhOBB3kxiUQhitTc9/tiUOF737ulcDvESgvD882oBhDiZH7JA1kd/YPme3FozgOFogkv4ymomlKpcatZ8nBo3n+Wid7FK0cUpy2z1fEvBNTCKsjwvbhsYfhM0SdEMvf/ag= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(9101521072)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046);SRVR:SN1PR07MB2239;BCL:0;PCL:0;RULEID:;SRVR:SN1PR07MB2239; X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2239;4:jneuNZQ3xz6Qw0w3xh8oUIuJx8kTk3jYA2+1c5PIswgMrY1GMGe+04P5n8dLoeSbboqymWSfYKy8WatbZVi/IoN3awjYwvCTVwMALF+83Vuyn/IXi67BjRnVQwP+VWkKXbYroCnA8oaNSQccWYnrioavMbQCd6kCsL3QWOcuMO+aIYyBvsqRPAFhL/L2hhfzfIwfVGkepolmk8eGBpKP1Y590CTgEbDL/L+nT/JVB6UJblssd3v/h4TNuQSEND5rHfSRfNPvTV23WDO5loOUeJDPRNcuzkYpZ+dTSSRlDu4BIQ98dyn3z23UOXYCSuIDfnbCe9kPhQR3bEv/0W3LVTjhURLd4MWu+s/R79Vgcpf9d4mkCTR07qNzi8bkfXBXlPQ6F60wQfjC907yysULsQ== X-Forefront-PRVS: 0927AA37C7 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(6069001)(24454002)(46406003)(33716001)(83506001)(19580395003)(15975445007)(189998001)(66066001)(110136002)(77096005)(97756001)(9686002)(4001350100001)(4326007)(93886004)(5004730100002)(54356999)(6116002)(76176999)(3846002)(50986999)(1076002)(81166005)(23726003)(76506005)(92566002)(2906002)(586003)(42186005)(47776003)(33656002)(50466002)(5008740100001)(2950100001)(1096002);DIR:OUT;SFP:1101;SCL:1;SRVR:SN1PR07MB2239;H:localhost;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;SN1PR07MB2239;23:RsRvghLCpRvBCfVEy0LShtuJ8qOCfphNAiJWuzBkj?= =?us-ascii?Q?/Nre4jhPRi5ubRoxuVMmUVP7X8GomhTdif9NcW5xAfYqyyn8I2GyDHfL3WEc?= =?us-ascii?Q?ldmc2Pigg3j1vS7lUm2mZzd+iMYLc/ooWHAEOXKT4aIDlJWcPGqsPPKVUmS2?= =?us-ascii?Q?/omMARzC0ysOZdXUEIpdWv7OfRAuM5CAGFpG7SfhPHCFKmyVCz8vP8d/iiw7?= =?us-ascii?Q?9EKdExfkXFxssa7zwPKg6CMaRBDAwWpDJ8UHHkiwDMLuf9cT63BJbOOQykyu?= =?us-ascii?Q?Ncp1gsw22BFpKdlPFV3I3JbWZOcGUnW4OBAhxFSts+Vg58J02DgEn0bM6T2a?= =?us-ascii?Q?1FjR/frrzwj2/R9OmEzBqE/fuc4L0rbjSWy6KCPU4558/pd9XnK3KLRSs7fH?= =?us-ascii?Q?doJyS0pdXCLvCuQPUeVE6CExJx/wXDiqSwsQw8nkgdj5Dikg8CA6DpChBeeH?= =?us-ascii?Q?GIxfR5ThMLSduM344OFWF7tBvuWcQ53hv+2Qok+g/SERnJDEvmGcH4r3vECn?= =?us-ascii?Q?3mGofbrdo1Jt+8cMll2+nPb9SHD4i4R9ZrwH5o0Llfto/yZHOH0BebdMJf/+?= =?us-ascii?Q?LCPglCNyC5ejirSO7ooi/tMNxxP1Uymg4Bv/67kSB43/OPLf2cTB2mvz5kS5?= =?us-ascii?Q?jmQlgqpjcy8UNphXobyebsMblQgWSrd9etVjwVhZ5RmovtI/GCFY1TR/pesj?= =?us-ascii?Q?SBQ0qOlCP8+jj0ulSy2cZ1vDZI4kwCNsrgp9+UYv7/xsJ6lUSfVJpTvw66Rx?= =?us-ascii?Q?Mwx3q+I9opHs05/TQfxYIM/GNI1VYg0UMmuOMbavP/BiKTuez/jxED7N4vjk?= =?us-ascii?Q?IkhRoM5ij6KuJcNN8kI7NSvgqFiQ3yZa46dzaXgbt9IgOotPkbD/yq5+hfaq?= =?us-ascii?Q?YGbU0JXUFNu2FTGN4rxkP9plr9wHPndI0TeXvxWk690qCY4Ay8JRm173YNou?= =?us-ascii?Q?vIgCx51WdQTMhzJ9FdwiI+Nj8Lo++/2lWt7x6rJ2KPNvvWbo6bvs1BVXYBu4?= =?us-ascii?Q?Sg9/vKnOZP14pggwn0U8/rB?= X-Microsoft-Exchange-Diagnostics: 1;SN1PR07MB2239;5:87/V4oKooeY5Uhsao2yOt+E8SS/zcfyhAAYOybzixuKnd7GV/VNI6KFMdWE9lp6c1cR10LGmHiEuO154pJY5vdRbsHTJE4SPbev86dELJEdzEJ6JDpMx8PemEiLyiuzX34fBjAVcCCT7D0vkW9RfOQ==;24:bY5Std7gSrhHdkchyBJKS1L1XTDrqh16DWOYvc/laIDezuaaRnR2lkEqIyJw1NENW+pVWFnkx/b74BnzfrIHAybJa+KpSXPBdAEKD6HXvog=;7:CJkj1F7pP97TNp7PaqsgsbN0a8AI7RfkjXPStvHu2+AYi2yriu0Zf/LcZLhYNoPptUIi4ZTUqSM1n+f1xwWgCsnO+SCLaGbvlWSboaJqqJo6Oq28G8HtWtvoy1FiurBAgJwV3Aa5zKHKMO6UndHgiMXilr444i8FR9LULe58SQrTiArTXNYPOzIUiXE+ztyb SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Apr 2016 13:14:02.0543 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN1PR07MB2239 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1407 Lines: 48 On Fri, Apr 29, 2016 at 12:43:41AM +0200, Arnd Bergmann wrote: > On Friday 29 April 2016 01:21:37 Yury Norov wrote: > > index 1458ad7..410d817 100644 > > --- a/arch/arm64/kernel/sys_ilp32.c > > +++ b/arch/arm64/kernel/sys_ilp32.c > > @@ -17,6 +17,8 @@ > > * along with this program. If not, see > > * . > > */ > > > > +#define __SYSCALL_COMPAT > > + > > #include > > #include > > #include > > @@ -48,13 +50,12 @@ asmlinkage long > > ilp32_sys_rt_sigreturn_wrapper(void); > > > > #include > > > > -#undef __SYSCALL > > -#undef __SC_COMP > > -#undef __SC_WRAP > > -#undef __SC_3264 > > -#undef __SC_COMP_3264 > > > > -#define __SYSCALL_COMPAT > > #define __SYSCALL(nr, sym) [nr] = sym, > > #define __SC_WRAP(nr, sym) [nr] = compat_##sym, > > > > This patch makes gcc warn about redefinition. > > > > arch/arm64/kernel/sys_ilp32.c:59:0: warning: "__SYSCALL" redefined > > #define __SYSCALL(nr, sym) [nr] = sym, > > ^ > > In file included from include/asm-generic/unistd.h:1:0, > > > > Ok, I think I see it now. Can you #undef the two symbols at the > end of arch/arm64/include/uapi/asm/unistd.h I think it doesn't look better than what we have now, but not worse as well. If you like it, I'll change. > or possibly > include/uapi/asm-generic/unistd.h? > > Arnd