From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757814AbcBYUYX (ORCPT ); Thu, 25 Feb 2016 15:24:23 -0500 Received: from mail-by2on0080.outbound.protection.outlook.com ([207.46.100.80]:2304 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1758126AbcBYUYQ (ORCPT ); Thu, 25 Feb 2016 15:24:16 -0500 X-Greylist: delayed 92871 seconds by postgrey-1.27 at vger.kernel.org; Thu, 25 Feb 2016 15:24:16 EST Authentication-Results: de.ibm.com; dkim=none (message not signed) header.d=none;de.ibm.com; dmarc=none action=none header.from=caviumnetworks.com; Date: Thu, 25 Feb 2016 23:23:50 +0300 From: Yury Norov To: Heiko Carstens CC: , , , , , , , , , , , , , , , Subject: Re: [PATCH 3/5] all: s390: move wrapper infrastructure to generic headers Message-ID: <20160225202350.GB16123@yury-N73SV> References: <1456338854-14901-1-git-send-email-ynorov@caviumnetworks.com> <1456338854-14901-4-git-send-email-ynorov@caviumnetworks.com> <20160225085140.GC3980@osiris> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20160225085140.GC3980@osiris> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [95.143.213.121] X-ClientProxiedBy: VI1PR05CA0013.eurprd05.prod.outlook.com (25.162.33.151) To BLUPR07MB609.namprd07.prod.outlook.com (10.141.207.12) X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB609;2:B+ETVduC8iJ9mXGqXRpgx7pDxXmMZAXaaNbvDw38fUBxK8Rwnm+GwG77vVHvjSMiQ/y22rzkbYJ8E6mJHFqJ+/JrN62mh3Z4KqCc1Mghh2y0XcWsnZRbpHyZIsDv5iriY8aWWF+D8Vm0FwcToaOG0w==;3:+D0IodIod4ZmtJVUZGiR5i2h2ZhaFjcLgUbFv9YU8lPe7LiynEaWGwlH7z5cNe+9RNF/A07iXhmV+ejwPAfLwNRo6P6ilDRx9l2DEBU9imehOvaHnKaFXK7u3PxTe42e;25:bPzzI2EZc2k1UPF2bShwQnjtK62Uiu+n/ObHZzvUey3I6Xp/zoTXS6BlBkrScRYwgXARAZQz/mizNihbfpKR5iGD/z5FWcr67iuUSFs2AxTGw3gdq/di/DceNTvS6munjJuof4CGWVzvhVLQbZZjJB45aYIDmPagQoNj4FmOqOpepXSM0Ggfz2SWCoWvcKVE2ev5jbS4+15I0BX+aa0EXwWToZqTeqhrzu0p2gHQ+qeNRNj+WimMQbGsuc6kbQbCvsjoDjB47Gt8grIMj8yvTmA9OjDG9DxwbUUNm0CiPrbOss1kOGTQbuwst/ZtRtb7 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BLUPR07MB609; X-MS-Office365-Filtering-Correlation-Id: e0787f00-e337-4ca3-f0f7-08d33e219f7a X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB609;20:fFEKlWPUJHt9BWiUKauGJMQGTFyDoKrMRDq0ruPwh4AfEGed+thBI3n6Wy4prLwQLRikD9tNpGMa2ueN0fg5PTWjtVoYnKHVTmvBT7EVjhNRMvLdIMY9+Ysvw/KFwj2tWdnHO/bvP5lEPEAh3E35NT09JqmOFQ+H1bLV6NeRU4m5ShajsRsLolR22b7Dncf8aY8pgnLhssj7R76MUyc1G2cDIa//2N9vcwzfsU+3QJr3OjJrZ7YfjWN2mAdI/HyggBjqGvLN4O0frA7MTTQWLpYNEJ29EOaIq8L9eVeARFos690n+Uun4BYqwUY8hEhrgqa4BRJK2LrH0/nYv/w67a0P7JvqIbzZ28SI5BXgvhfJ7RSW0QLp97bitR7BcJ79PmtKZ3+Hi48JoFUkk/w0Cu9bH1lZWdbw0U6qeFsgW+eRoxb1XxlAVGS2snY4slUh3uS4b/Y4TQPCRTMPWugi6ItGKp359pfd+tRqDBljr/iEpGKVaim+4809dKcImuUrjt0fQJQ/1MayVmNN/NzS5OqV+6MOsAvCN6FJRHBMCKVNTNrvMbZrwHrW3FG4S+vNfx5rfT80RxMskjdqf4I99J7/rCFL0IN1ed8Dga3M0F0= 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)(3002001)(10201501046);SRVR:BLUPR07MB609;BCL:0;PCL:0;RULEID:;SRVR:BLUPR07MB609; X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB609;4:B3BRgqrxMVgkoqaf/yXakToYwNWNcS10ocvGnKOADCO/FRSEZuoP+92g4nuaTtWJC39/IRRpTTwzNITfDem1HNRVyTio4Zm/8NdTHsJxxhQ2wbavWlLDWG0VcyUVDzUGuUZpORKdLCuI0K/meWd8saalQfzkjfvHFE3/xPopvILjov+DeiKw8rPPU1GFVOjzB3izf3vHqPSO2F8U4lssq58w/1Keu/Ksng7CfFnpq03HkACyNM0ThYobTR+1dXzd4yi1qAekJ8ONBERTLqF+gWCNT4BzIqNFx7H2E6gUdQdOxuQUMfXEEJMz6TlAbDPjton0IkRPtdVByGdFjbxxtBIqhPs5zuQDpN1L0WAy3yO3YDh8XuxvTQfdjiSqdYV7 X-Forefront-PRVS: 08635C03D4 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6069001)(6009001)(24454002)(83506001)(33716001)(4326007)(87976001)(122386002)(76506005)(40100003)(76176999)(54356999)(189998001)(50986999)(46406003)(97756001)(47776003)(42186005)(6116002)(5008740100001)(5004730100002)(3846002)(66066001)(586003)(23726003)(1096002)(33656002)(1076002)(4001350100001)(92566002)(50466002)(5001960100002)(2906002)(2950100001)(110136002)(77096005)(41533002);DIR:OUT;SFP:1101;SCL:1;SRVR:BLUPR07MB609;H:localhost;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BLUPR07MB609;23:/omwZfnTDCEa2zSYFHZ0S8pfzqzmtLFCu4yxiTVh0V?= =?us-ascii?Q?xzHgo9VBYiI2iJyPPAbp3VO8AExuPKq9UifTb/wMQUieg/BTiYWEhBohn1/3?= =?us-ascii?Q?yXaf8cLdBm9X7zKtszKXodRK0QU9lN+9GP+/rmyPd9gu4Gh68ddnJTUvDP/r?= =?us-ascii?Q?0INsKikkygDf8a3meja0N8YMGpWfR6gdr46dNwA/foQADwFrL0+QEfv0jaO9?= =?us-ascii?Q?vdhp7jyIOGXSBUTf9vz8LfU43Wiu1KTvIq6jbTIZVOSGGDjqwGPshCkLI6dD?= =?us-ascii?Q?Lo6k27VuqbMm86xUK4UcHDVh6R9xnaN4fu3gvSmu1oFSs9Aie1V9qOuDEDje?= =?us-ascii?Q?CK+TWBGKAbeEA34zxwDih8raZfwDRMrxQWjECDGOdwbaNF2q3gtzQMQUHrtQ?= =?us-ascii?Q?YVwiGa99dVPr3IYlKwQAH0/k7awRUo6tUYb6d5XDYFRU7z17faNkGNi2uuFD?= =?us-ascii?Q?ep6Z35NcpAuQsCs+GK5pofny2gAywEIZNPKWsufXZsEhKHj4a2ucmsBMBmb8?= =?us-ascii?Q?OQgIh7PufN5MEtOtAAinw7n54qx6StjcFQ2CBLwNo2Yfhh9WGzvDwDDXdGla?= =?us-ascii?Q?/SGbuA/P+RmRmXLOVjr8EVGBAmYlHmkQ37HcqsCqCfgk1v6oElf9SeqOIDUo?= =?us-ascii?Q?xG7keQpUsBcPl0J6jcRd9N8Yn7KN8onMbrfU+PN9tV7AqSFTfeF+TOJao96C?= =?us-ascii?Q?DWh5y7vMdOLEwkbfuxITQOv5uK6YX4dD1jdCvDibJ04tjlbr1dQnt+VMKahW?= =?us-ascii?Q?s3tbBh89Q8X9ebHyLQP1+qnR4PWiHDJMN3WucLYehaUR3O7L6ryLjn0P+VN9?= =?us-ascii?Q?HVvW+qakya9/SEmUr0+F+kyZfyP1NPWA0IoLaVkRqBo0iFV45tcnMWcCzZkz?= =?us-ascii?Q?lMQmLB97PBbgLc4myeE5ZvsQb0hL+p1wg1ocUsHrLP6bPnkjNrZ4ZVBdhCL+?= =?us-ascii?Q?beHgGTMJKrUccg1JIWZGQLGrAUoVV5Ikf58354mqRi8XaljnR3rV4MkDbqA5?= =?us-ascii?Q?VBy6c/ynHzeNOxC+fnkpOc?= X-Microsoft-Exchange-Diagnostics: 1;BLUPR07MB609;5:m7SGYanrOB90TnHcMC9qgr0LN+YTu9PYOJqDwXbVuaSB0y3UwKRRm1CABaQv3G5naCiCBCBPhzGxyBjj1USNmKIRBkA7EuKGQq+7C6+ZY6lvX3UFKR1qq1XCrMUoEZSCPiqRijF+cl8pQutHDYPT7g==;24:64N5LbNpW5C+M/Vo79UZvqmzwLcvZTFYAM0r/8t6GFlLuL1h0uqQSkTsm+gD1zBwwlUjO42jpxbNMG002nd9oN7XiFhPoQrvGq72d9Pwh9E= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 25 Feb 2016 20:24:11.6255 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR07MB609 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 25, 2016 at 09:51:40AM +0100, Heiko Carstens wrote: > On Wed, Feb 24, 2016 at 09:34:12PM +0300, Yury Norov wrote: > > diff --git a/include/linux/syscalls_structs.h b/include/linux/syscalls_structs.h > > new file mode 100644 > > index 0000000..a920cbc > > --- /dev/null > > +++ b/include/linux/syscalls_structs.h > > @@ -0,0 +1,60 @@ > > +#ifndef _LINUX_SYSCALL_STRUCTS_H > > +#define _LINUX_SYSCALL_STRUCTS_H > > + > > +struct epoll_event; > > +struct iattr; > > +struct inode; > > +struct iocb; > > +struct io_event; > > +struct iovec; > > +struct itimerspec; > > +struct itimerval; > > This is not needed for s390, right? So might be worth a separate patch > which moves the forward declarations? This patch also introduces __SC_WRAP, not needed as well. The idea of this patch is to introduce all the tricky machinery at once. If you think we need split it, I'm OK, but maybe it's enough to add more detailed description... What do you think? From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yury Norov Subject: Re: [PATCH 3/5] all: s390: move wrapper infrastructure to generic headers Date: Thu, 25 Feb 2016 23:23:50 +0300 Message-ID: <20160225202350.GB16123@yury-N73SV> References: <1456338854-14901-1-git-send-email-ynorov@caviumnetworks.com> <1456338854-14901-4-git-send-email-ynorov@caviumnetworks.com> <20160225085140.GC3980@osiris> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: <20160225085140.GC3980@osiris> Sender: linux-doc-owner@vger.kernel.org List-Archive: List-Post: To: Heiko Carstens Cc: arnd@arndb.de, catalin.marinas@arm.com, linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arch@vger.kernel.org, linux-doc@vger.kernel.org, Prasun.Kapoor@caviumnetworks.com, pinskia@gmail.com, agraf@suse.de, broonie@kernel.org, joseph@codesourcery.com, christoph.muellner@theobroma-systems.com, Nathan_Lynch@mentor.com, klimov.linux@gmail.com, bamvor.zhangjian@huawei.com List-ID: On Thu, Feb 25, 2016 at 09:51:40AM +0100, Heiko Carstens wrote: > On Wed, Feb 24, 2016 at 09:34:12PM +0300, Yury Norov wrote: > > diff --git a/include/linux/syscalls_structs.h b/include/linux/syscalls_structs.h > > new file mode 100644 > > index 0000000..a920cbc > > --- /dev/null > > +++ b/include/linux/syscalls_structs.h > > @@ -0,0 +1,60 @@ > > +#ifndef _LINUX_SYSCALL_STRUCTS_H > > +#define _LINUX_SYSCALL_STRUCTS_H > > + > > +struct epoll_event; > > +struct iattr; > > +struct inode; > > +struct iocb; > > +struct io_event; > > +struct iovec; > > +struct itimerspec; > > +struct itimerval; > > This is not needed for s390, right? So might be worth a separate patch > which moves the forward declarations? This patch also introduces __SC_WRAP, not needed as well. The idea of this patch is to introduce all the tricky machinery at once. If you think we need split it, I'm OK, but maybe it's enough to add more detailed description... What do you think? From mboxrd@z Thu Jan 1 00:00:00 1970 From: ynorov@caviumnetworks.com (Yury Norov) Date: Thu, 25 Feb 2016 23:23:50 +0300 Subject: [PATCH 3/5] all: s390: move wrapper infrastructure to generic headers In-Reply-To: <20160225085140.GC3980@osiris> References: <1456338854-14901-1-git-send-email-ynorov@caviumnetworks.com> <1456338854-14901-4-git-send-email-ynorov@caviumnetworks.com> <20160225085140.GC3980@osiris> Message-ID: <20160225202350.GB16123@yury-N73SV> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Feb 25, 2016 at 09:51:40AM +0100, Heiko Carstens wrote: > On Wed, Feb 24, 2016 at 09:34:12PM +0300, Yury Norov wrote: > > diff --git a/include/linux/syscalls_structs.h b/include/linux/syscalls_structs.h > > new file mode 100644 > > index 0000000..a920cbc > > --- /dev/null > > +++ b/include/linux/syscalls_structs.h > > @@ -0,0 +1,60 @@ > > +#ifndef _LINUX_SYSCALL_STRUCTS_H > > +#define _LINUX_SYSCALL_STRUCTS_H > > + > > +struct epoll_event; > > +struct iattr; > > +struct inode; > > +struct iocb; > > +struct io_event; > > +struct iovec; > > +struct itimerspec; > > +struct itimerval; > > This is not needed for s390, right? So might be worth a separate patch > which moves the forward declarations? This patch also introduces __SC_WRAP, not needed as well. The idea of this patch is to introduce all the tricky machinery at once. If you think we need split it, I'm OK, but maybe it's enough to add more detailed description... What do you think?