Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933974AbbLWScV (ORCPT ); Wed, 23 Dec 2015 13:32:21 -0500 Received: from mail-bl2on0061.outbound.protection.outlook.com ([65.55.169.61]:10976 "EHLO na01-bl2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754536AbbLWScT (ORCPT ); Wed, 23 Dec 2015 13:32:19 -0500 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Yuri.Norov@caviumnetworks.com; Date: Wed, 23 Dec 2015 21:31:45 +0300 From: Yury Norov To: Arnd Bergmann CC: , , , , , , , , , Andrew Pinski , , Andrew Pinski , , , , , , Subject: Re: [PATCH v6 12/20] arm64:ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it Message-ID: <20151223183145.GA27193@yury-N73SV> References: <1450215766-14765-1-git-send-email-ynorov@caviumnetworks.com> <1450215766-14765-13-git-send-email-ynorov@caviumnetworks.com> <201512212331.58152.arnd@arndb.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <201512212331.58152.arnd@arndb.de> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [95.143.213.121] X-ClientProxiedBy: AM2PR09CA0013.eurprd09.prod.outlook.com (25.161.22.151) To BLUPR07MB612.namprd07.prod.outlook.com (10.141.207.27) X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB612;2:+BG0/gMdNbs3Vmt6daZyUxGGnigAiiRDGeq5BavT7KmRuX3P4CLI439H5dAy5cxeWgQze+gO/RtENRlsDQiCZdHPRv/mcFs+RKYTrTi9tw7wiMePLOhoPwxVgBB9sdW678kBMdaFawE8aa+sluiJwQ==;3:N7jfUZoTfq4ZtVznTC7I/Y9dhSQbFneCoOyzIYDWGQVqSU7+MPF/+rheif8catlAMDQfwBJSA9cPUMLpdH8MfdBLwbld84f/SDzq492O9A/GKRC++SZL9Pkvt5XvltGW;25:TFY27Ejv8ZrTVF7WtyOAJ2dVyK15EQSpYzfpF9CP0tOm03LLcu8aP4VwqTQluxHBNyYv4My3m1UxVSKy0FP+haxfmMVX1lP4BOYoCmOuKmQjUQfsLTRQ5ErS6gSMsHK785R8op2h/hs7KzS+4vS8gv9wqUBHNBqmp+2DTg5hUnqjYgidJxXigX2FxRsMnW904OR5j2UEMzQ6YfyTApLvf9lFGaiyXIShDQRZUWWvi4eGEQUC62MAa5np9I+LhkjyoMXLSH+e8fA55PjBEkPr4g== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BLUPR07MB612; X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB612;20:CwFARt4U+FMjaUYrVGJ1FLxYfNnqVnUnMZFflGYvxQGqTzZhIZbyScFxL7b05u2DSeHykTHZw6cZrHVbTpI8aPG0bSEWTAwOKFnjVFZTQLo92xbuKWdHsjxx79bvhzbouqeZ101moy3BQAs4svo1/D97HdpKJCq57FIA8BUM1qcBxSjSeY5CgpcJa4TuabLVaT4HwIW79AgeJq4knZqQjXB0/5sApEAil0IgdXd/YHO+k32VQTe/feKJJREeAE0rm/K+V8YjeMfzf5XX3wKqZvuooSZ6xZVn5FwmkgqcdBNXg3o+ug6lM2hLYdE95zN6SjFk8oNckOyGWW5qgCuMp4aDYvOb+Kzxh7/kJMzcV6ohsWL67RBQXYDQ8Z3KzmMcku/sRfPjB+YX2QNfFMF9LV3CFcX/DM/WwckZqZ7TWwit69xaG7X4oKh68tRqUtJu4vKLY115qkXMdd59/AXxi0bsB21sRMA6vFsrR2iKw/OBeoQ3cSSt2vtMYptYoTunUr8os34eCUZJ15yvqjKyS1nLOpxSNfEyd0UxlwNkGLKRDn3blsQHh1PVep3O98NsH+l2DktoX53+8zT5eRNUMJDav7PxRDmpdYhKnHxKUaI= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(520078)(8121501046)(10201501046)(3002001);SRVR:BLUPR07MB612;BCL:0;PCL:0;RULEID:;SRVR:BLUPR07MB612; X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB612;4:Clfz3CNI0X/UBqE144DZFUhcrUQM+K8RqhmR428AtYE3KuSZJDXN2l/+MSb8RJYf2eKOZZDRfdrlpvxjebzVr152nSRbrEXMSqeps8f27qs4MmOr7kTS5kI+a6OwKp3M758ALlqS2CZOz6RZmEz4MNvaseLRZm0Tb3y6VKznu1A0uWMEy+RHpXqFWIxE0l9/8FVbppuE5Vb14/C6qfpRT/c4SGK1ujoYltNBrOTgwKUguQFwnjLqZJSRftEXj5JfsnuXk4W3UW8JNbSVvBgOwHTToZKzwdkVfHVzghms22dYGsSkECL8cH3ARSsmkAUubLelynQuG1B1B0+Ogl7wGsEZfqNx+zgF852BUdH3zT6ojXCE6XN8rHStyTdgM7BP X-Forefront-PRVS: 0799B1B2D7 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(6009001)(6069001)(52034003)(24454002)(189002)(199003)(3846002)(5008740100001)(23726003)(6116002)(1076002)(1096002)(97756001)(101416001)(2950100001)(586003)(15975445007)(50466002)(1720100001)(92566002)(5004730100002)(47776003)(77096005)(66066001)(105586002)(42186005)(76506005)(40100003)(122386002)(54356999)(46406003)(76176999)(4001350100001)(50986999)(5001960100002)(189998001)(110136002)(33656002)(33716001)(81156007)(97736004)(83506001)(19580395003)(106356001)(19580405001)(87976001);DIR:OUT;SFP:1101;SCL:1;SRVR:BLUPR07MB612;H:localhost;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BLUPR07MB612;23:6K1Dg94TsMgwdQJR/E4pnPJD75acX7718p50zQg2Um?= =?us-ascii?Q?yAp/bjUxdurXXQlol/JEvicV5zaBQ3X2Y7lbWI+BNQeg1mH2mBwPmNPXiToE?= =?us-ascii?Q?kGF5Z4ft1kFw/YKIGDjz5ZYicEZxr/p8VkZBnUo5kBUQNnrahSYSumOndMzO?= =?us-ascii?Q?Q3SvLwJ5P04GQAm58KRCLeOxffrKak1gpcNqYcw9jUDu5MCxrauB8c6sAmWT?= =?us-ascii?Q?HuCFfJXH4Fs60LB/QwX3OM4/SGoVeuNU003f8fn6WaauAL++Qh36Wtp/oIHJ?= =?us-ascii?Q?+1vjEAzqQgm0eE9Xs0bRveU5LsSsyKTA0PxD7E3ivFXHdOQ/TtIDv7yPaPbH?= =?us-ascii?Q?UfEmnDbmCcDM+S0sB5LJPCNd7BuLGjG+sqA0XO7Piz5XJaFIjNVCHYQvgPON?= =?us-ascii?Q?jVbxvWUshFY9HsbZIH3wlj10/OMQlcQc+N1mG0adx1EMPjRWg0h1s44+J6ht?= =?us-ascii?Q?ZFbkKFNPMBJYTWAUD/DIHUV/kAmQaRfIgTrx19VuX/ACXxKe5pkNs7kDoyJW?= =?us-ascii?Q?Yu9p98Ryt8A28eAvj0cGvoFElmO1iufiwM4tMNTH9BuUbNactdZ+QbGT/AKY?= =?us-ascii?Q?YUebfI9GXgL5NIjWHo9uCi/OxWvoTbp12W0r+f7nGGEnVi8XaDbXqIjK9OQT?= =?us-ascii?Q?BBNyeu1DWi6tpzdRudsk9PJuu14qCMFDg5WW2N5InFrazvf31y20gJJ8H2PU?= =?us-ascii?Q?SmCi63cx36mSaw8KKl9q6xm38hQ7O62cOlkGAfDQG/JED0IhK9kfAzHt7GL4?= =?us-ascii?Q?Jy1aEO7xwN8Z4yGz7bziRuMWbo55fYFvnyKJLqzaZ8u+wHMys5qfaK7A/9AL?= =?us-ascii?Q?z3y6TN4MfWeAqMiLATXCDDMTMRqK2Xsp1uQczSAJtnQ2lN5De0oLPS9Lm/2n?= =?us-ascii?Q?9d6tUTgRAs8W2Rs+plsPRghPwxi5lFCgD4AjsE7qaEDrAJ2i+i2QlL5NMf+Y?= =?us-ascii?Q?CajJ0Z5Lu+3oW3EJ7zlWMs0mdSdSHst4mQ4IwtGt5/Shttxdc5hk7+fM6x4F?= =?us-ascii?Q?g1XaoIkgK870ZVrNNtACaLX+C0llLvpoA0kMZLGZb//HeBnnbtQlkPRksL84?= =?us-ascii?Q?mZXSRCdT1v28MUkpU5J0o8vX7nFuF4gHYI6GFp2dx2w4Vps3DR9F0ML8fI5Y?= =?us-ascii?Q?SOnZVeig9wcgMNBO3G6zwt96kxinvnWfYHoAT4Cifkwvkk3jQ9nMga1Z51ua?= =?us-ascii?Q?b+ZnM8xT4e4OW11lZwulwX4cGft+vgiOf0?= X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB612;5:IBQSZpX7rPnM9MKHdF8vsGv5t8MsoxxleTApxhagNq0KygQ3xjBqK7Tdxm6SFzFC8Z2eXBMV7mOaG19fEUWa0IalYNVM4tZDrwSJ5wpbq0w1agosISd03B9DBum3ZIAG3olyptVEON7rDBwVQaC9Zw==;24:sanNsJwje6JkjObpF0ip7nFjiA6v/pN0n0o9kGGKBXrhB7Sfmb++Fdd6tTd7dAP0bGhFyWNkTfE+fryL6BuZ2Dku4nG3pn68p74WkqxN3zQ= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Dec 2015 18:32:15.8836 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR07MB612 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2839 Lines: 67 On Mon, Dec 21, 2015 at 11:31:57PM +0100, Arnd Bergmann wrote: > On Tuesday 15 December 2015, Yury Norov wrote: > > + > > +#define compat_sys_open_by_handle_at sys_open_by_handle_at > > +#define compat_sys_openat sys_openat > > + > > One more thing I just remembered: I think we want this behavior for all new > 32-bit architectures, it was a bug to call compat_sys_openat for the generic > syscall table, as we don't support 32-bit off_t. > > Could you split this out into a separate patch that does these changes: > > - change the default asm-generic/unistd.h to use sys_openat/sys_open_by_handle_at > - change tile to override those two to keep the current (suboptimal) behavior > - change the force_o_largefile() definition so it defaults to true for all future > architectures. The easiest way is probably to add a Kconfig symbol for this > that gets selected by all 32-bit architectures, so we can use > > #define force_o_largefile() ((BITS_PER_LONG != 32) || !IS_ENABLED(CONFIG_ARCH_32BIT_OFF_T)) > Hi Arnd, First two items are OK. The last one... Do you need it to remove compat_sys_openat and compat_sys_open_by_handle_at? The patch that introduces CONFIG_ARCH_32BIT_OFF_T will affect all architectures, and so I need to get ack from each maintainer. I will also have to ask them to test that change, because I have no access to all the hardware. (Even with QEMU, I cannot test them all.) I think the only man who is able to success with it is Linus :)... Some arches has more than one compat mode. For example ARM64: aarch32 requires this config enabled, but ilp32 needs it disabled. Now we can enable both features, but this will make them mutual exclusive. We can instead (un)define __ARCH_WANT_32BIT_OFF_T for each ABI of each platform, but it's even more work. And we'd think twice how to do it because it's not mechanical work. For example, on aarch64 it will look like: #define __ARCH_WANT_32BIT_OFF_T is_a32_compat_task() I have no idea how it will look on x86 or ppc. In your previous email (Nov 18) you write that tile is the only user of asm-generic/unistd.h that needs compat behaviour. If so, why not just to turn it around, as you initially suggested, and fix tile. And do nothing with force_o_largefile()? By the way, is there a comprehensive list of linux platforms/abis, or at least ones that use asm-generic/unistd.h? BR, Yury. > Arnd > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/