From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Carstens Subject: Re: [PATCH/RFC 00/16] compat: convert to COMPAT_SYSCALL_DEFINE Date: Thu, 13 Mar 2014 08:32:30 +0100 Message-ID: <20140313073230.GA4268@osiris> References: <1394121083-2507-1-git-send-email-heiko.carstens@de.ibm.com> <20140312194526.GA3755@osiris> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from e06smtp14.uk.ibm.com ([195.75.94.110]:57582 "EHLO e06smtp14.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751499AbaCMHcg (ORCPT ); Thu, 13 Mar 2014 03:32:36 -0400 Received: from /spool/local by e06smtp14.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 13 Mar 2014 07:32:34 -0000 Received: from b06cxnps4074.portsmouth.uk.ibm.com (d06relay11.portsmouth.uk.ibm.com [9.149.109.196]) by d06dlp03.portsmouth.uk.ibm.com (Postfix) with ESMTP id BB8B11B08051 for ; Thu, 13 Mar 2014 07:32:18 +0000 (GMT) Received: from d06av05.portsmouth.uk.ibm.com (d06av05.portsmouth.uk.ibm.com [9.149.37.229]) by b06cxnps4074.portsmouth.uk.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id s2D7WJN847775976 for ; Thu, 13 Mar 2014 07:32:19 GMT Received: from d06av05.portsmouth.uk.ibm.com (localhost [127.0.0.1]) by d06av05.portsmouth.uk.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id s2D7WUdw009001 for ; Thu, 13 Mar 2014 01:32:31 -0600 Content-Disposition: inline In-Reply-To: <20140312194526.GA3755@osiris> Sender: linux-next-owner@vger.kernel.org List-ID: To: Mark Brown Cc: linux-next@vger.kernel.org Hi Mark, I obviously missed that you are currently handling linux-next, so I sent my request to Stephen yesterday :) Could you add the git://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git compat to linux-next, please? Thanks, Heiko (sorry for top-posting) On Wed, Mar 12, 2014 at 08:45:26PM +0100, Heiko Carstens wrote: > Hi Stephen, > > since nobody objected to the compat changes I described below, could you > please add the tree > > git://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git compat > > to linux-next? > > Thanks, > Heiko > > [full quote below] > > On Thu, Mar 06, 2014 at 04:51:07PM +0100, Heiko Carstens wrote: > > Hi all, > > > > this patch series converts all common code compat syscalls to use the new > > COMPAT_SYSCALL_DEFINE macros introduced by Al Viro. > > My main motivation is to get rid of the 1400+ line s390 specific assembler > > file which contains code for each and every compat system call to perform > > proper sign, zero and pointer conversion. > > > > Eventually the conversion is pretty trivial, except that for a couple of > > compat syscalls argument types needed to be changed from e.g. 'long' to > > 'compat_long_t' so that the COMPAT_SYSCALL_DEFINE macro can perform > > proper sign extension from the 32 bit compat system call argument like > > it was orginally passed in from user space. > > > > The patch series keeps the "noop" conversions (5-10) and those with type > > changes (11-15) separate. > > > > Patches 1-4 are just additional cleanup patches. > > > > Especially I'd like to know from somebody familiar with the x32 ABI if > > any of the type changes break x32? > > I intentionally did not touch the preadv64 and pwritev64 compat system > > calls since they have been explicitly added to allow x32 pass 64 bit > > arguments. > > However it's not obvious to me if any of my other compat system call > > changes break x32 or not. > > > > FWIW, this patch series omits the whole s390 patches, since I assume > > they are not very interesting to non-s390 people ;) > > > > The complete series (including the s390 only patches) is available at > > > > git://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git compat > > > > If nobody objects I'd like to have the branch integrated into linux-next > > to get some additional testing. > > > > So, please let me know if I screwed up something! :) > > > > Thanks, > > Heiko > > > > Heiko Carstens (16): > > compat: let architectures define __ARCH_WANT_COMPAT_SYS_GETDENTS64 > > compat: add COMPAT_SYSCALL_DEFINE0 macro > > ipc/compat_sys_msgrcv: change msgtyp type from long to compat_long_t > > fs/compat: optional preadv64/pwrite64 compat system calls > > kernel/compat: convert to COMPAT_SYSCALL_DEFINE > > net/compat: convert to COMPAT_SYSCALL_DEFINE > > mm/compat: convert to COMPAT_SYSCALL_DEFINE > > security/compat: convert to COMPAT_SYSCALL_DEFINE > > fs/compat: convert to COMPAT_SYSCALL_DEFINE > > ipc/compat: convert to COMPAT_SYSCALL_DEFINE > > fs/compat: convert to COMPAT_SYSCALL_DEFINE with changing parameter types > > ipc/compat: convert to COMPAT_SYSCALL_DEFINE with changing parameter types > > net/compat: convert to COMPAT_SYSCALL_DEFINE with changing parameter types > > kexec/compat: convert to COMPAT_SYSCALL_DEFINE with changing parameter types > > mm/compat: convert to COMPAT_SYSCALL_DEFINE with changing parameter types > > s390/compat: build error for large compat syscall args > > > > (diffstat below includes the s390 only patches) > > > > arch/arm64/include/asm/unistd.h | 1 + > > arch/mips/include/asm/unistd.h | 1 - > > arch/s390/include/asm/compat.h | 6 +- > > arch/s390/kernel/compat_linux.c | 116 +-- > > arch/s390/kernel/compat_linux.h | 81 +-- > > arch/s390/kernel/compat_signal.c | 4 +- > > arch/s390/kernel/compat_wrapper.S | 1425 ------------------------------------- > > arch/s390/kernel/compat_wrapper.c | 209 ++++++ > > arch/s390/kernel/entry.h | 6 +- > > arch/s390/kernel/syscalls.S | 504 ++++++------- > > arch/x86/include/asm/unistd.h | 3 + > > fs/compat.c | 121 ++-- > > fs/compat_ioctl.c | 5 +- > > fs/exec.c | 6 +- > > fs/read_write.c | 36 +- > > include/linux/compat.h | 62 +- > > include/linux/kexec.h | 6 - > > include/linux/syscalls.h | 2 + > > include/uapi/asm-generic/unistd.h | 1 + > > ipc/compat.c | 13 +- > > ipc/compat_mq.c | 32 +- > > kernel/compat.c | 100 +-- > > kernel/kexec.c | 8 +- > > kernel/ptrace.c | 4 +- > > mm/mempolicy.c | 18 +- > > mm/process_vm_access.c | 26 +- > > net/compat.c | 32 +- > > security/keys/compat.c | 4 +- > > 28 files changed, 828 insertions(+), 2004 deletions(-) > > > > -- > > 1.8.4.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-next" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html