* [PATCH v2] ipc: provide generic compat versions of IPC syscalls [not found] ` <201112091602.31325.arnd@arndb.de> @ 2011-12-09 15:29 ` Chris Metcalf 2011-12-09 21:48 ` Andrew Morton 0 siblings, 1 reply; 6+ messages in thread From: Chris Metcalf @ 2011-12-09 15:29 UTC (permalink / raw) To: Arnd Bergmann, Ralf Baechle, Benjamin Herrenschmidt, Paul Mackerras, Martin Schwidefsky, Heiko Carstens, linux390, David S. Miller, Thomas Gleixner, Ingo Molnar, H. Peter Anvin, x86, Eric W. Biederman, Christoph Hellwig, Lucas De Marchi, Dmitry Torokhov, Andrew Morton, J. Bruce Fields, NeilBrown, linux-mips, linux-kernel, linuxppc-dev, linux-s390, sparclinux When using the "compat" APIs, architectures will generally want to be able to make direct syscalls to msgsnd(), shmctl(), etc., and in the kernel we would want them to be handled directly by compat_sys_xxx() functions, as is true for other compat syscalls. However, for historical reasons, several of the existing compat IPC syscalls do not do this. semctl() expects a pointer to the fourth argument, instead of the fourth argument itself. msgsnd(), msgrcv() and shmat() expect arguments in different order. This change adds an __ARCH_WANT_OLD_COMPAT_IPC define that can be set in <asm/compat.h> to preserve this behavior for ports that use it (x86, sparc, powerpc, s390, and mips). No actual semantics are changed for those architectures, and there is only a minimal amount of code refactoring in ipc/compat.c. Newer architectures like tile (and perhaps future architectures such as arm64 and unicore64) should not supply this define, and thus can avoid having any IPC-specific code at all in their architecture-specific compat layer. In the same vein, if this define is omitted, IPC_64 mode is assumed, since that's what the <asm-generic> headers expect. The workaround code in "tile" for msgsnd() and msgrcv() is removed with this change; it also fixes the bug that shmat() and semctl() were not being properly handled. Signed-off-by: Chris Metcalf <cmetcalf@tilera.com> --- The first version used an "__ARCH_WANT_GENERIC_COMPAT_IPC" define that was set by tile, and presumably would be set by all future architectures. Arnd Bergmann observed: > I like the patch, but I think the __ARCH_WANT_GENERIC_COMPAT_IPC > should be defined as the opposite, so all "old" architectures > have to set it while tile (and future architectures like arm64 > and unicore64) just get the default. [...] > We have powerpc, mips, s390, sparc and > x86 using the legacy method, while only parisc and tile get it right > and use the syscalls directly. arch/mips/include/asm/compat.h | 3 ++ arch/powerpc/include/asm/compat.h | 3 ++ arch/s390/include/asm/compat.h | 3 ++ arch/sparc/include/asm/compat.h | 3 ++ arch/tile/include/asm/compat.h | 11 ------ arch/tile/kernel/compat.c | 43 ---------------------- arch/x86/include/asm/compat.h | 3 ++ include/linux/compat.h | 12 ++++++- ipc/compat.c | 70 +++++++++++++++++++++++++++++++++--- 9 files changed, 90 insertions(+), 61 deletions(-) diff --git a/arch/mips/include/asm/compat.h b/arch/mips/include/asm/compat.h index b77df03..41a57cb 100644 --- a/arch/mips/include/asm/compat.h +++ b/arch/mips/include/asm/compat.h @@ -8,6 +8,9 @@ #include <asm/page.h> #include <asm/ptrace.h> +/* Use different 32-bit syscall convention than 64-bit for some syscalls. */ +#define __ARCH_WANT_OLD_COMPAT_IPC + #define COMPAT_USER_HZ 100 #define COMPAT_UTS_MACHINE "mips\0\0\0" diff --git a/arch/powerpc/include/asm/compat.h b/arch/powerpc/include/asm/compat.h index 88e602f..450a976 100644 --- a/arch/powerpc/include/asm/compat.h +++ b/arch/powerpc/include/asm/compat.h @@ -7,6 +7,9 @@ #include <linux/types.h> #include <linux/sched.h> +/* Use different 32-bit syscall convention than 64-bit for some syscalls. */ +#define __ARCH_WANT_OLD_COMPAT_IPC + #define COMPAT_USER_HZ 100 #define COMPAT_UTS_MACHINE "ppc\0\0" diff --git a/arch/s390/include/asm/compat.h b/arch/s390/include/asm/compat.h index 2e49748..9a38bae 100644 --- a/arch/s390/include/asm/compat.h +++ b/arch/s390/include/asm/compat.h @@ -7,6 +7,9 @@ #include <linux/sched.h> #include <linux/thread_info.h> +/* Use different 32-bit syscall convention than 64-bit for some syscalls. */ +#define __ARCH_WANT_OLD_COMPAT_IPC + #define PSW32_MASK_PER 0x40000000UL #define PSW32_MASK_DAT 0x04000000UL #define PSW32_MASK_IO 0x02000000UL diff --git a/arch/sparc/include/asm/compat.h b/arch/sparc/include/asm/compat.h index b8be20d..bfd3d8b 100644 --- a/arch/sparc/include/asm/compat.h +++ b/arch/sparc/include/asm/compat.h @@ -5,6 +5,9 @@ */ #include <linux/types.h> +/* Use different 32-bit syscall convention than 64-bit for some syscalls. */ +#define __ARCH_WANT_OLD_COMPAT_IPC + #define COMPAT_USER_HZ 100 #define COMPAT_UTS_MACHINE "sparc\0\0" diff --git a/arch/tile/include/asm/compat.h b/arch/tile/include/asm/compat.h index bf95f55..4b4b289 100644 --- a/arch/tile/include/asm/compat.h +++ b/arch/tile/include/asm/compat.h @@ -242,17 +242,6 @@ long compat_sys_fallocate(int fd, int mode, long compat_sys_sched_rr_get_interval(compat_pid_t pid, struct compat_timespec __user *interval); -/* Versions of compat functions that differ from generic Linux. */ -struct compat_msgbuf; -long tile_compat_sys_msgsnd(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, int msgflg); -long tile_compat_sys_msgrcv(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, long msgtyp, int msgflg); -long tile_compat_sys_ptrace(compat_long_t request, compat_long_t pid, - compat_long_t addr, compat_long_t data); - /* Tilera Linux syscalls that don't have "compat" versions. */ #define compat_sys_flush_cache sys_flush_cache diff --git a/arch/tile/kernel/compat.c b/arch/tile/kernel/compat.c index bf5e9d7..d67459b 100644 --- a/arch/tile/kernel/compat.c +++ b/arch/tile/kernel/compat.c @@ -16,7 +16,6 @@ #define __SYSCALL_COMPAT #include <linux/compat.h> -#include <linux/msg.h> #include <linux/syscalls.h> #include <linux/kdev_t.h> #include <linux/fs.h> @@ -95,52 +94,10 @@ long compat_sys_sched_rr_get_interval(compat_pid_t pid, return ret; } -/* - * The usual compat_sys_msgsnd() and _msgrcv() seem to be assuming - * some different calling convention than our normal 32-bit tile code. - */ - -/* Already defined in ipc/compat.c, but we need it here. */ -struct compat_msgbuf { - compat_long_t mtype; - char mtext[1]; -}; - -long tile_compat_sys_msgsnd(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, int msgflg) -{ - compat_long_t mtype; - - if (get_user(mtype, &msgp->mtype)) - return -EFAULT; - return do_msgsnd(msqid, mtype, msgp->mtext, msgsz, msgflg); -} - -long tile_compat_sys_msgrcv(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, long msgtyp, int msgflg) -{ - long err, mtype; - - err = do_msgrcv(msqid, &mtype, msgp->mtext, msgsz, msgtyp, msgflg); - if (err < 0) - goto out; - - if (put_user(mtype, &msgp->mtype)) - err = -EFAULT; - out: - return err; -} - /* Provide the compat syscall number to call mapping. */ #undef __SYSCALL #define __SYSCALL(nr, call) [nr] = (call), -/* The generic versions of these don't work for Tile. */ -#define compat_sys_msgrcv tile_compat_sys_msgrcv -#define compat_sys_msgsnd tile_compat_sys_msgsnd - /* See comments in sys.c */ #define compat_sys_fadvise64_64 sys32_fadvise64_64 #define compat_sys_readahead sys32_readahead diff --git a/arch/x86/include/asm/compat.h b/arch/x86/include/asm/compat.h index 30d737e..a72cbe4 100644 --- a/arch/x86/include/asm/compat.h +++ b/arch/x86/include/asm/compat.h @@ -8,6 +8,9 @@ #include <linux/sched.h> #include <asm/user32.h> +/* Use different 32-bit syscall convention than 64-bit for some syscalls. */ +#define __ARCH_WANT_OLD_COMPAT_IPC + #define COMPAT_USER_HZ 100 #define COMPAT_UTS_MACHINE "i686\0\0" diff --git a/include/linux/compat.h b/include/linux/compat.h index 66ed067..472e66c 100644 --- a/include/linux/compat.h +++ b/include/linux/compat.h @@ -224,6 +224,7 @@ struct compat_sysinfo; struct compat_sysctl_args; struct compat_kexec_segment; struct compat_mq_attr; +struct compat_msgbuf; extern void compat_exit_robust_list(struct task_struct *curr); @@ -234,13 +235,22 @@ asmlinkage long compat_sys_get_robust_list(int pid, compat_uptr_t __user *head_ptr, compat_size_t __user *len_ptr); +#ifdef __ARCH_WANT_OLD_COMPAT_IPC long compat_sys_semctl(int first, int second, int third, void __user *uptr); long compat_sys_msgsnd(int first, int second, int third, void __user *uptr); long compat_sys_msgrcv(int first, int second, int msgtyp, int third, int version, void __user *uptr); -long compat_sys_msgctl(int first, int second, void __user *uptr); long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, void __user *uptr); +#else +long compat_sys_semctl(int semid, int semnum, int cmd, int arg); +long compat_sys_msgsnd(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, int msgflg); +long compat_sys_msgrcv(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, long msgtyp, int msgflg); +long compat_sys_shmat(int shmid, compat_uptr_t shmaddr, int shmflg); +#endif +long compat_sys_msgctl(int first, int second, void __user *uptr); long compat_sys_shmctl(int first, int second, void __user *uptr); long compat_sys_semtimedop(int semid, struct sembuf __user *tsems, unsigned nsems, const struct compat_timespec __user *timeout); diff --git a/ipc/compat.c b/ipc/compat.c index 845a287..94ee136 100644 --- a/ipc/compat.c +++ b/ipc/compat.c @@ -27,6 +27,7 @@ #include <linux/msg.h> #include <linux/shm.h> #include <linux/syscalls.h> +#include <linux/ptrace.h> #include <linux/mutex.h> #include <asm/uaccess.h> @@ -117,6 +118,7 @@ extern int sem_ctls[]; static inline int compat_ipc_parse_version(int *cmd) { +#ifdef __ARCH_WANT_OLD_COMPAT_IPC int version = *cmd & IPC_64; /* this is tricky: architectures that have support for the old @@ -128,6 +130,10 @@ static inline int compat_ipc_parse_version(int *cmd) *cmd &= ~IPC_64; #endif return version; +#else + /* With the asm-generic APIs, we always use the 64-bit versions. */ + return IPC_64; +#endif } static inline int __get_compat_ipc64_perm(struct ipc64_perm *p64, @@ -232,10 +238,9 @@ static inline int put_compat_semid_ds(struct semid64_ds *s, return err; } -long compat_sys_semctl(int first, int second, int third, void __user *uptr) +static long do_compat_semctl(int first, int second, int third, u32 pad) { union semun fourth; - u32 pad; int err, err2; struct semid64_ds s64; struct semid64_ds __user *up64; @@ -243,10 +248,6 @@ long compat_sys_semctl(int first, int second, int third, void __user *uptr) memset(&s64, 0, sizeof(s64)); - if (!uptr) - return -EINVAL; - if (get_user(pad, (u32 __user *) uptr)) - return -EFAULT; if ((third & (~IPC_64)) == SETVAL) fourth.val = (int) pad; else @@ -305,6 +306,18 @@ long compat_sys_semctl(int first, int second, int third, void __user *uptr) return err; } +#ifdef __ARCH_WANT_OLD_COMPAT_IPC +long compat_sys_semctl(int first, int second, int third, void __user *uptr) +{ + u32 pad; + + if (!uptr) + return -EINVAL; + if (get_user(pad, (u32 __user *) uptr)) + return -EFAULT; + return do_compat_semctl(first, second, third, pad); +} + long compat_sys_msgsnd(int first, int second, int third, void __user *uptr) { struct compat_msgbuf __user *up = uptr; @@ -353,6 +366,37 @@ long compat_sys_msgrcv(int first, int second, int msgtyp, int third, out: return err; } +#else +long compat_sys_semctl(int semid, int semnum, int cmd, int arg) +{ + return do_compat_semctl(semid, semnum, cmd, arg); +} + +long compat_sys_msgsnd(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, int msgflg) +{ + compat_long_t mtype; + + if (get_user(mtype, &msgp->mtype)) + return -EFAULT; + return do_msgsnd(msqid, mtype, msgp->mtext, msgsz, msgflg); +} + +long compat_sys_msgrcv(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, long msgtyp, int msgflg) +{ + long err, mtype; + + err = do_msgrcv(msqid, &mtype, msgp->mtext, msgsz, msgtyp, msgflg); + if (err < 0) + goto out; + + if (put_user(mtype, &msgp->mtype)) + err = -EFAULT; + out: + return err; +} +#endif static inline int get_compat_msqid64(struct msqid64_ds *m64, struct compat_msqid64_ds __user *up64) @@ -470,6 +514,7 @@ long compat_sys_msgctl(int first, int second, void __user *uptr) return err; } +#ifdef __ARCH_WANT_OLD_COMPAT_IPC long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, void __user *uptr) { @@ -485,6 +530,19 @@ long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, uaddr = compat_ptr(third); return put_user(raddr, uaddr); } +#else +long compat_sys_shmat(int shmid, compat_uptr_t shmaddr, int shmflg) +{ + unsigned long ret; + long err; + + err = do_shmat(shmid, compat_ptr(shmaddr), shmflg, &ret); + if (err) + return err; + force_successful_syscall_return(); + return (long)ret; +} +#endif static inline int get_compat_shmid64_ds(struct shmid64_ds *s64, struct compat_shmid64_ds __user *up64) -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] ipc: provide generic compat versions of IPC syscalls 2011-12-09 15:29 ` [PATCH v2] ipc: provide generic compat versions of IPC syscalls Chris Metcalf @ 2011-12-09 21:48 ` Andrew Morton 2011-12-09 23:21 ` Arnd Bergmann 0 siblings, 1 reply; 6+ messages in thread From: Andrew Morton @ 2011-12-09 21:48 UTC (permalink / raw) To: Chris Metcalf Cc: linux-mips, NeilBrown, Heiko Carstens, Paul Mackerras, H. Peter Anvin, sparclinux, Christoph Hellwig, linux-s390, J. Bruce Fields, x86, Ingo Molnar, Arnd Bergmann, Lucas De Marchi, Thomas Gleixner, Dmitry Torokhov, linux-kernel, Ralf Baechle, Eric W. Biederman, Martin Schwidefsky, linux390, linuxppc-dev, David S. Miller On Fri, 9 Dec 2011 10:29:07 -0500 Chris Metcalf <cmetcalf@tilera.com> wrote: > When using the "compat" APIs, architectures will generally want to > be able to make direct syscalls to msgsnd(), shmctl(), etc., and > in the kernel we would want them to be handled directly by > compat_sys_xxx() functions, as is true for other compat syscalls. > > However, for historical reasons, several of the existing compat IPC > syscalls do not do this. semctl() expects a pointer to the fourth > argument, instead of the fourth argument itself. msgsnd(), msgrcv() > and shmat() expect arguments in different order. > > This change adds an __ARCH_WANT_OLD_COMPAT_IPC define that can be > set in <asm/compat.h> to preserve this behavior for ports that use it > (x86, sparc, powerpc, s390, and mips). No actual semantics are changed > for those architectures, and there is only a minimal amount of code > refactoring in ipc/compat.c. > > Newer architectures like tile (and perhaps future architectures such > as arm64 and unicore64) should not supply this define, and thus can > avoid having any IPC-specific code at all in their architecture-specific > compat layer. In the same vein, if this define is omitted, IPC_64 mode > is assumed, since that's what the <asm-generic> headers expect. > > The workaround code in "tile" for msgsnd() and msgrcv() is removed > with this change; it also fixes the bug that shmat() and semctl() were > not being properly handled. What would we need to do to get all architectures using the new interfaces, and remove __ARCH_WANT_OLD_COMPAT_IPC? Regarding the implementation: rather than patching the header files, it would be more conventional (and arguably better) to add select ARCH_WANT_OLD_COMPAT_IPC to arch/*/Kconfig, then use CONFIG_ARCH_WANT_OLD_COMPAT_IPC. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] ipc: provide generic compat versions of IPC syscalls 2011-12-09 21:48 ` Andrew Morton @ 2011-12-09 23:21 ` Arnd Bergmann 2011-12-09 15:29 ` [PATCH v3] " Chris Metcalf 0 siblings, 1 reply; 6+ messages in thread From: Arnd Bergmann @ 2011-12-09 23:21 UTC (permalink / raw) To: Andrew Morton Cc: linux-mips, NeilBrown, Heiko Carstens, Paul Mackerras, H. Peter Anvin, sparclinux, Christoph Hellwig, linux-s390, J. Bruce Fields, x86, Ingo Molnar, Lucas De Marchi, Chris Metcalf, Thomas Gleixner, Dmitry Torokhov, linux-kernel, Ralf Baechle, Eric W. Biederman, Martin Schwidefsky, linux390, linuxppc-dev, David S. Miller On Friday 09 December 2011 13:48:52 Andrew Morton wrote: > What would we need to do to get all architectures using the new > interfaces, and remove __ARCH_WANT_OLD_COMPAT_IPC? We would change the various compat_sys_ipc functions (mips, powerpc, s390, sparc, x86) to call the new functions instead of the existing ones, and test each architecture. For parisc, we might not actually need the either version, but I'd have to take a closer look to be sure. > Regarding the implementation: rather than patching the header > files, it would be more conventional (and arguably better) to add > > select ARCH_WANT_OLD_COMPAT_IPC > > to arch/*/Kconfig, then use CONFIG_ARCH_WANT_OLD_COMPAT_IPC. Yes. Arnd ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v3] ipc: provide generic compat versions of IPC syscalls 2011-12-09 23:21 ` Arnd Bergmann @ 2011-12-09 15:29 ` Chris Metcalf 2012-03-15 1:24 ` Chris Metcalf 0 siblings, 1 reply; 6+ messages in thread From: Chris Metcalf @ 2011-12-09 15:29 UTC (permalink / raw) To: Arnd Bergmann, Ralf Baechle, Benjamin Herrenschmidt, Paul Mackerras, Martin Schwidefsky, Heiko Carstens, linux390, David S. Miller, Thomas Gleixner, Ingo Molnar, H. Peter Anvin, x86, Eric W. Biederman, Christoph Hellwig, Lucas De Marchi, Dmitry Torokhov, Andrew Morton, J. Bruce Fields, NeilBrown, linux-mips, linux-kernel, linuxppc-dev, linux-s390, sparclinux When using the "compat" APIs, architectures will generally want to be able to make direct syscalls to msgsnd(), shmctl(), etc., and in the kernel we would want them to be handled directly by compat_sys_xxx() functions, as is true for other compat syscalls. However, for historical reasons, several of the existing compat IPC syscalls do not do this. semctl() expects a pointer to the fourth argument, instead of the fourth argument itself. msgsnd(), msgrcv() and shmat() expect arguments in different order. This change adds an ARCH_WANT_OLD_COMPAT_IPC config option that can be set to preserve this behavior for ports that use it (x86, sparc, powerpc, s390, and mips). No actual semantics are changed for those architectures, and there is only a minimal amount of code refactoring in ipc/compat.c. Newer architectures like tile (and perhaps future architectures such as arm64 and unicore64) should not select this option, and thus can avoid having any IPC-specific code at all in their architecture-specific compat layer. In the same vein, if this option is not selected, IPC_64 mode is assumed, since that's what the <asm-generic> headers expect. The workaround code in "tile" for msgsnd() and msgrcv() is removed with this change; it also fixes the bug that shmat() and semctl() were not being properly handled. Signed-off-by: Chris Metcalf <cmetcalf@tilera.com> --- arch/Kconfig | 3 ++ arch/mips/Kconfig | 1 + arch/powerpc/Kconfig | 1 + arch/s390/Kconfig | 1 + arch/sparc/Kconfig | 1 + arch/tile/include/asm/compat.h | 11 ------ arch/tile/kernel/compat.c | 43 ------------------------ arch/x86/Kconfig | 1 + include/linux/compat.h | 12 ++++++- ipc/compat.c | 70 ++++++++++++++++++++++++++++++++++++--- 10 files changed, 83 insertions(+), 61 deletions(-) diff --git a/arch/Kconfig b/arch/Kconfig index 4b0669c..dfb1e07 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -181,4 +181,7 @@ config HAVE_RCU_TABLE_FREE config ARCH_HAVE_NMI_SAFE_CMPXCHG bool +config ARCH_WANT_OLD_COMPAT_IPC + bool + source "kernel/gcov/Kconfig" diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig index d46f1da..ad2af82 100644 --- a/arch/mips/Kconfig +++ b/arch/mips/Kconfig @@ -2420,6 +2420,7 @@ config MIPS32_COMPAT config COMPAT bool depends on MIPS32_COMPAT + select ARCH_WANT_OLD_COMPAT_IPC default y config SYSVIPC_COMPAT diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig index 951e18f..e2be710 100644 --- a/arch/powerpc/Kconfig +++ b/arch/powerpc/Kconfig @@ -146,6 +146,7 @@ config COMPAT bool default y if PPC64 select COMPAT_BINFMT_ELF + select ARCH_WANT_OLD_COMPAT_IPC config SYSVIPC_COMPAT bool diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig index 373679b..2fc3bca 100644 --- a/arch/s390/Kconfig +++ b/arch/s390/Kconfig @@ -221,6 +221,7 @@ config COMPAT prompt "Kernel support for 31 bit emulation" depends on 64BIT select COMPAT_BINFMT_ELF + select ARCH_WANT_OLD_COMPAT_IPC help Select this option if you want to enable your system kernel to handle system-calls from ELF binaries for 31 bit ESA. This option diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig index f92602e..846cb5c 100644 --- a/arch/sparc/Kconfig +++ b/arch/sparc/Kconfig @@ -577,6 +577,7 @@ config COMPAT depends on SPARC64 default y select COMPAT_BINFMT_ELF + select ARCH_WANT_OLD_COMPAT_IPC config SYSVIPC_COMPAT bool diff --git a/arch/tile/include/asm/compat.h b/arch/tile/include/asm/compat.h index bf95f55..4b4b289 100644 --- a/arch/tile/include/asm/compat.h +++ b/arch/tile/include/asm/compat.h @@ -242,17 +242,6 @@ long compat_sys_fallocate(int fd, int mode, long compat_sys_sched_rr_get_interval(compat_pid_t pid, struct compat_timespec __user *interval); -/* Versions of compat functions that differ from generic Linux. */ -struct compat_msgbuf; -long tile_compat_sys_msgsnd(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, int msgflg); -long tile_compat_sys_msgrcv(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, long msgtyp, int msgflg); -long tile_compat_sys_ptrace(compat_long_t request, compat_long_t pid, - compat_long_t addr, compat_long_t data); - /* Tilera Linux syscalls that don't have "compat" versions. */ #define compat_sys_flush_cache sys_flush_cache diff --git a/arch/tile/kernel/compat.c b/arch/tile/kernel/compat.c index bf5e9d7..d67459b 100644 --- a/arch/tile/kernel/compat.c +++ b/arch/tile/kernel/compat.c @@ -16,7 +16,6 @@ #define __SYSCALL_COMPAT #include <linux/compat.h> -#include <linux/msg.h> #include <linux/syscalls.h> #include <linux/kdev_t.h> #include <linux/fs.h> @@ -95,52 +94,10 @@ long compat_sys_sched_rr_get_interval(compat_pid_t pid, return ret; } -/* - * The usual compat_sys_msgsnd() and _msgrcv() seem to be assuming - * some different calling convention than our normal 32-bit tile code. - */ - -/* Already defined in ipc/compat.c, but we need it here. */ -struct compat_msgbuf { - compat_long_t mtype; - char mtext[1]; -}; - -long tile_compat_sys_msgsnd(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, int msgflg) -{ - compat_long_t mtype; - - if (get_user(mtype, &msgp->mtype)) - return -EFAULT; - return do_msgsnd(msqid, mtype, msgp->mtext, msgsz, msgflg); -} - -long tile_compat_sys_msgrcv(int msqid, - struct compat_msgbuf __user *msgp, - size_t msgsz, long msgtyp, int msgflg) -{ - long err, mtype; - - err = do_msgrcv(msqid, &mtype, msgp->mtext, msgsz, msgtyp, msgflg); - if (err < 0) - goto out; - - if (put_user(mtype, &msgp->mtype)) - err = -EFAULT; - out: - return err; -} - /* Provide the compat syscall number to call mapping. */ #undef __SYSCALL #define __SYSCALL(nr, call) [nr] = (call), -/* The generic versions of these don't work for Tile. */ -#define compat_sys_msgrcv tile_compat_sys_msgrcv -#define compat_sys_msgsnd tile_compat_sys_msgsnd - /* See comments in sys.c */ #define compat_sys_fadvise64_64 sys32_fadvise64_64 #define compat_sys_readahead sys32_readahead diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index cb9a104..0e1f474 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -2131,6 +2131,7 @@ config IA32_AOUT config COMPAT def_bool y depends on IA32_EMULATION + select ARCH_WANT_OLD_COMPAT_IPC config COMPAT_FOR_U64_ALIGNMENT def_bool COMPAT diff --git a/include/linux/compat.h b/include/linux/compat.h index 66ed067..f295dae 100644 --- a/include/linux/compat.h +++ b/include/linux/compat.h @@ -224,6 +224,7 @@ struct compat_sysinfo; struct compat_sysctl_args; struct compat_kexec_segment; struct compat_mq_attr; +struct compat_msgbuf; extern void compat_exit_robust_list(struct task_struct *curr); @@ -234,13 +235,22 @@ asmlinkage long compat_sys_get_robust_list(int pid, compat_uptr_t __user *head_ptr, compat_size_t __user *len_ptr); +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC long compat_sys_semctl(int first, int second, int third, void __user *uptr); long compat_sys_msgsnd(int first, int second, int third, void __user *uptr); long compat_sys_msgrcv(int first, int second, int msgtyp, int third, int version, void __user *uptr); -long compat_sys_msgctl(int first, int second, void __user *uptr); long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, void __user *uptr); +#else +long compat_sys_semctl(int semid, int semnum, int cmd, int arg); +long compat_sys_msgsnd(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, int msgflg); +long compat_sys_msgrcv(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, long msgtyp, int msgflg); +long compat_sys_shmat(int shmid, compat_uptr_t shmaddr, int shmflg); +#endif +long compat_sys_msgctl(int first, int second, void __user *uptr); long compat_sys_shmctl(int first, int second, void __user *uptr); long compat_sys_semtimedop(int semid, struct sembuf __user *tsems, unsigned nsems, const struct compat_timespec __user *timeout); diff --git a/ipc/compat.c b/ipc/compat.c index 845a287..a6df704 100644 --- a/ipc/compat.c +++ b/ipc/compat.c @@ -27,6 +27,7 @@ #include <linux/msg.h> #include <linux/shm.h> #include <linux/syscalls.h> +#include <linux/ptrace.h> #include <linux/mutex.h> #include <asm/uaccess.h> @@ -117,6 +118,7 @@ extern int sem_ctls[]; static inline int compat_ipc_parse_version(int *cmd) { +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC int version = *cmd & IPC_64; /* this is tricky: architectures that have support for the old @@ -128,6 +130,10 @@ static inline int compat_ipc_parse_version(int *cmd) *cmd &= ~IPC_64; #endif return version; +#else + /* With the asm-generic APIs, we always use the 64-bit versions. */ + return IPC_64; +#endif } static inline int __get_compat_ipc64_perm(struct ipc64_perm *p64, @@ -232,10 +238,9 @@ static inline int put_compat_semid_ds(struct semid64_ds *s, return err; } -long compat_sys_semctl(int first, int second, int third, void __user *uptr) +static long do_compat_semctl(int first, int second, int third, u32 pad) { union semun fourth; - u32 pad; int err, err2; struct semid64_ds s64; struct semid64_ds __user *up64; @@ -243,10 +248,6 @@ long compat_sys_semctl(int first, int second, int third, void __user *uptr) memset(&s64, 0, sizeof(s64)); - if (!uptr) - return -EINVAL; - if (get_user(pad, (u32 __user *) uptr)) - return -EFAULT; if ((third & (~IPC_64)) == SETVAL) fourth.val = (int) pad; else @@ -305,6 +306,18 @@ long compat_sys_semctl(int first, int second, int third, void __user *uptr) return err; } +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC +long compat_sys_semctl(int first, int second, int third, void __user *uptr) +{ + u32 pad; + + if (!uptr) + return -EINVAL; + if (get_user(pad, (u32 __user *) uptr)) + return -EFAULT; + return do_compat_semctl(first, second, third, pad); +} + long compat_sys_msgsnd(int first, int second, int third, void __user *uptr) { struct compat_msgbuf __user *up = uptr; @@ -353,6 +366,37 @@ long compat_sys_msgrcv(int first, int second, int msgtyp, int third, out: return err; } +#else +long compat_sys_semctl(int semid, int semnum, int cmd, int arg) +{ + return do_compat_semctl(semid, semnum, cmd, arg); +} + +long compat_sys_msgsnd(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, int msgflg) +{ + compat_long_t mtype; + + if (get_user(mtype, &msgp->mtype)) + return -EFAULT; + return do_msgsnd(msqid, mtype, msgp->mtext, msgsz, msgflg); +} + +long compat_sys_msgrcv(int msqid, struct compat_msgbuf __user *msgp, + size_t msgsz, long msgtyp, int msgflg) +{ + long err, mtype; + + err = do_msgrcv(msqid, &mtype, msgp->mtext, msgsz, msgtyp, msgflg); + if (err < 0) + goto out; + + if (put_user(mtype, &msgp->mtype)) + err = -EFAULT; + out: + return err; +} +#endif static inline int get_compat_msqid64(struct msqid64_ds *m64, struct compat_msqid64_ds __user *up64) @@ -470,6 +514,7 @@ long compat_sys_msgctl(int first, int second, void __user *uptr) return err; } +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, void __user *uptr) { @@ -485,6 +530,19 @@ long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, uaddr = compat_ptr(third); return put_user(raddr, uaddr); } +#else +long compat_sys_shmat(int shmid, compat_uptr_t shmaddr, int shmflg) +{ + unsigned long ret; + long err; + + err = do_shmat(shmid, compat_ptr(shmaddr), shmflg, &ret); + if (err) + return err; + force_successful_syscall_return(); + return (long)ret; +} +#endif static inline int get_compat_shmid64_ds(struct shmid64_ds *s64, struct compat_shmid64_ds __user *up64) -- 1.6.5.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v3] ipc: provide generic compat versions of IPC syscalls 2011-12-09 15:29 ` [PATCH v3] " Chris Metcalf @ 2012-03-15 1:24 ` Chris Metcalf 2012-03-15 16:25 ` Arnd Bergmann 0 siblings, 1 reply; 6+ messages in thread From: Chris Metcalf @ 2012-03-15 1:24 UTC (permalink / raw) To: Arnd Bergmann, Ralf Baechle, Benjamin Herrenschmidt, Paul Mackerras, Martin Schwidefsky, Heiko Carstens, linux390, David S. Miller, Thomas Gleixner, Ingo Molnar, H. Peter Anvin, x86, Eric W. Biederman, Christoph Hellwig, Lucas De Marchi, Dmitry Torokhov, Andrew Morton, J. Bruce Fields, NeilBrown, linux-mips, linux-kernel, linuxppc-dev, linux-s390, sparclinux Somewhat belatedly, I'm looking to push this through the "tile" tree. Andrew Morton has been keeping it in his tree but I figure I should push it since it directly affects "tile" and currently no one else. However, it would be nice if anyone could provide a Reviewed-by or Acked-by on the patch. Thanks! On 12/9/2011 10:29 AM, Chris Metcalf wrote: > When using the "compat" APIs, architectures will generally want to > be able to make direct syscalls to msgsnd(), shmctl(), etc., and > in the kernel we would want them to be handled directly by > compat_sys_xxx() functions, as is true for other compat syscalls. > > However, for historical reasons, several of the existing compat IPC > syscalls do not do this. semctl() expects a pointer to the fourth > argument, instead of the fourth argument itself. msgsnd(), msgrcv() > and shmat() expect arguments in different order. > > This change adds an ARCH_WANT_OLD_COMPAT_IPC config option that can be > set to preserve this behavior for ports that use it (x86, sparc, powerpc, > s390, and mips). No actual semantics are changed for those architectures, > and there is only a minimal amount of code refactoring in ipc/compat.c. > > Newer architectures like tile (and perhaps future architectures such > as arm64 and unicore64) should not select this option, and thus can > avoid having any IPC-specific code at all in their architecture-specific > compat layer. In the same vein, if this option is not selected, IPC_64 > mode is assumed, since that's what the <asm-generic> headers expect. > > The workaround code in "tile" for msgsnd() and msgrcv() is removed > with this change; it also fixes the bug that shmat() and semctl() were > not being properly handled. > > Signed-off-by: Chris Metcalf <cmetcalf@tilera.com> > --- > arch/Kconfig | 3 ++ > arch/mips/Kconfig | 1 + > arch/powerpc/Kconfig | 1 + > arch/s390/Kconfig | 1 + > arch/sparc/Kconfig | 1 + > arch/tile/include/asm/compat.h | 11 ------ > arch/tile/kernel/compat.c | 43 ------------------------ > arch/x86/Kconfig | 1 + > include/linux/compat.h | 12 ++++++- > ipc/compat.c | 70 ++++++++++++++++++++++++++++++++++++--- > 10 files changed, 83 insertions(+), 61 deletions(-) > > diff --git a/arch/Kconfig b/arch/Kconfig > index 4b0669c..dfb1e07 100644 > --- a/arch/Kconfig > +++ b/arch/Kconfig > @@ -181,4 +181,7 @@ config HAVE_RCU_TABLE_FREE > config ARCH_HAVE_NMI_SAFE_CMPXCHG > bool > > +config ARCH_WANT_OLD_COMPAT_IPC > + bool > + > source "kernel/gcov/Kconfig" > diff --git a/arch/mips/Kconfig b/arch/mips/Kconfig > index d46f1da..ad2af82 100644 > --- a/arch/mips/Kconfig > +++ b/arch/mips/Kconfig > @@ -2420,6 +2420,7 @@ config MIPS32_COMPAT > config COMPAT > bool > depends on MIPS32_COMPAT > + select ARCH_WANT_OLD_COMPAT_IPC > default y > > config SYSVIPC_COMPAT > diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig > index 951e18f..e2be710 100644 > --- a/arch/powerpc/Kconfig > +++ b/arch/powerpc/Kconfig > @@ -146,6 +146,7 @@ config COMPAT > bool > default y if PPC64 > select COMPAT_BINFMT_ELF > + select ARCH_WANT_OLD_COMPAT_IPC > > config SYSVIPC_COMPAT > bool > diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig > index 373679b..2fc3bca 100644 > --- a/arch/s390/Kconfig > +++ b/arch/s390/Kconfig > @@ -221,6 +221,7 @@ config COMPAT > prompt "Kernel support for 31 bit emulation" > depends on 64BIT > select COMPAT_BINFMT_ELF > + select ARCH_WANT_OLD_COMPAT_IPC > help > Select this option if you want to enable your system kernel to > handle system-calls from ELF binaries for 31 bit ESA. This option > diff --git a/arch/sparc/Kconfig b/arch/sparc/Kconfig > index f92602e..846cb5c 100644 > --- a/arch/sparc/Kconfig > +++ b/arch/sparc/Kconfig > @@ -577,6 +577,7 @@ config COMPAT > depends on SPARC64 > default y > select COMPAT_BINFMT_ELF > + select ARCH_WANT_OLD_COMPAT_IPC > > config SYSVIPC_COMPAT > bool > diff --git a/arch/tile/include/asm/compat.h b/arch/tile/include/asm/compat.h > index bf95f55..4b4b289 100644 > --- a/arch/tile/include/asm/compat.h > +++ b/arch/tile/include/asm/compat.h > @@ -242,17 +242,6 @@ long compat_sys_fallocate(int fd, int mode, > long compat_sys_sched_rr_get_interval(compat_pid_t pid, > struct compat_timespec __user *interval); > > -/* Versions of compat functions that differ from generic Linux. */ > -struct compat_msgbuf; > -long tile_compat_sys_msgsnd(int msqid, > - struct compat_msgbuf __user *msgp, > - size_t msgsz, int msgflg); > -long tile_compat_sys_msgrcv(int msqid, > - struct compat_msgbuf __user *msgp, > - size_t msgsz, long msgtyp, int msgflg); > -long tile_compat_sys_ptrace(compat_long_t request, compat_long_t pid, > - compat_long_t addr, compat_long_t data); > - > /* Tilera Linux syscalls that don't have "compat" versions. */ > #define compat_sys_flush_cache sys_flush_cache > > diff --git a/arch/tile/kernel/compat.c b/arch/tile/kernel/compat.c > index bf5e9d7..d67459b 100644 > --- a/arch/tile/kernel/compat.c > +++ b/arch/tile/kernel/compat.c > @@ -16,7 +16,6 @@ > #define __SYSCALL_COMPAT > > #include <linux/compat.h> > -#include <linux/msg.h> > #include <linux/syscalls.h> > #include <linux/kdev_t.h> > #include <linux/fs.h> > @@ -95,52 +94,10 @@ long compat_sys_sched_rr_get_interval(compat_pid_t pid, > return ret; > } > > -/* > - * The usual compat_sys_msgsnd() and _msgrcv() seem to be assuming > - * some different calling convention than our normal 32-bit tile code. > - */ > - > -/* Already defined in ipc/compat.c, but we need it here. */ > -struct compat_msgbuf { > - compat_long_t mtype; > - char mtext[1]; > -}; > - > -long tile_compat_sys_msgsnd(int msqid, > - struct compat_msgbuf __user *msgp, > - size_t msgsz, int msgflg) > -{ > - compat_long_t mtype; > - > - if (get_user(mtype, &msgp->mtype)) > - return -EFAULT; > - return do_msgsnd(msqid, mtype, msgp->mtext, msgsz, msgflg); > -} > - > -long tile_compat_sys_msgrcv(int msqid, > - struct compat_msgbuf __user *msgp, > - size_t msgsz, long msgtyp, int msgflg) > -{ > - long err, mtype; > - > - err = do_msgrcv(msqid, &mtype, msgp->mtext, msgsz, msgtyp, msgflg); > - if (err < 0) > - goto out; > - > - if (put_user(mtype, &msgp->mtype)) > - err = -EFAULT; > - out: > - return err; > -} > - > /* Provide the compat syscall number to call mapping. */ > #undef __SYSCALL > #define __SYSCALL(nr, call) [nr] = (call), > > -/* The generic versions of these don't work for Tile. */ > -#define compat_sys_msgrcv tile_compat_sys_msgrcv > -#define compat_sys_msgsnd tile_compat_sys_msgsnd > - > /* See comments in sys.c */ > #define compat_sys_fadvise64_64 sys32_fadvise64_64 > #define compat_sys_readahead sys32_readahead > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index cb9a104..0e1f474 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -2131,6 +2131,7 @@ config IA32_AOUT > config COMPAT > def_bool y > depends on IA32_EMULATION > + select ARCH_WANT_OLD_COMPAT_IPC > > config COMPAT_FOR_U64_ALIGNMENT > def_bool COMPAT > diff --git a/include/linux/compat.h b/include/linux/compat.h > index 66ed067..f295dae 100644 > --- a/include/linux/compat.h > +++ b/include/linux/compat.h > @@ -224,6 +224,7 @@ struct compat_sysinfo; > struct compat_sysctl_args; > struct compat_kexec_segment; > struct compat_mq_attr; > +struct compat_msgbuf; > > extern void compat_exit_robust_list(struct task_struct *curr); > > @@ -234,13 +235,22 @@ asmlinkage long > compat_sys_get_robust_list(int pid, compat_uptr_t __user *head_ptr, > compat_size_t __user *len_ptr); > > +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC > long compat_sys_semctl(int first, int second, int third, void __user *uptr); > long compat_sys_msgsnd(int first, int second, int third, void __user *uptr); > long compat_sys_msgrcv(int first, int second, int msgtyp, int third, > int version, void __user *uptr); > -long compat_sys_msgctl(int first, int second, void __user *uptr); > long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, > void __user *uptr); > +#else > +long compat_sys_semctl(int semid, int semnum, int cmd, int arg); > +long compat_sys_msgsnd(int msqid, struct compat_msgbuf __user *msgp, > + size_t msgsz, int msgflg); > +long compat_sys_msgrcv(int msqid, struct compat_msgbuf __user *msgp, > + size_t msgsz, long msgtyp, int msgflg); > +long compat_sys_shmat(int shmid, compat_uptr_t shmaddr, int shmflg); > +#endif > +long compat_sys_msgctl(int first, int second, void __user *uptr); > long compat_sys_shmctl(int first, int second, void __user *uptr); > long compat_sys_semtimedop(int semid, struct sembuf __user *tsems, > unsigned nsems, const struct compat_timespec __user *timeout); > diff --git a/ipc/compat.c b/ipc/compat.c > index 845a287..a6df704 100644 > --- a/ipc/compat.c > +++ b/ipc/compat.c > @@ -27,6 +27,7 @@ > #include <linux/msg.h> > #include <linux/shm.h> > #include <linux/syscalls.h> > +#include <linux/ptrace.h> > > #include <linux/mutex.h> > #include <asm/uaccess.h> > @@ -117,6 +118,7 @@ extern int sem_ctls[]; > > static inline int compat_ipc_parse_version(int *cmd) > { > +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC > int version = *cmd & IPC_64; > > /* this is tricky: architectures that have support for the old > @@ -128,6 +130,10 @@ static inline int compat_ipc_parse_version(int *cmd) > *cmd &= ~IPC_64; > #endif > return version; > +#else > + /* With the asm-generic APIs, we always use the 64-bit versions. */ > + return IPC_64; > +#endif > } > > static inline int __get_compat_ipc64_perm(struct ipc64_perm *p64, > @@ -232,10 +238,9 @@ static inline int put_compat_semid_ds(struct semid64_ds *s, > return err; > } > > -long compat_sys_semctl(int first, int second, int third, void __user *uptr) > +static long do_compat_semctl(int first, int second, int third, u32 pad) > { > union semun fourth; > - u32 pad; > int err, err2; > struct semid64_ds s64; > struct semid64_ds __user *up64; > @@ -243,10 +248,6 @@ long compat_sys_semctl(int first, int second, int third, void __user *uptr) > > memset(&s64, 0, sizeof(s64)); > > - if (!uptr) > - return -EINVAL; > - if (get_user(pad, (u32 __user *) uptr)) > - return -EFAULT; > if ((third & (~IPC_64)) == SETVAL) > fourth.val = (int) pad; > else > @@ -305,6 +306,18 @@ long compat_sys_semctl(int first, int second, int third, void __user *uptr) > return err; > } > > +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC > +long compat_sys_semctl(int first, int second, int third, void __user *uptr) > +{ > + u32 pad; > + > + if (!uptr) > + return -EINVAL; > + if (get_user(pad, (u32 __user *) uptr)) > + return -EFAULT; > + return do_compat_semctl(first, second, third, pad); > +} > + > long compat_sys_msgsnd(int first, int second, int third, void __user *uptr) > { > struct compat_msgbuf __user *up = uptr; > @@ -353,6 +366,37 @@ long compat_sys_msgrcv(int first, int second, int msgtyp, int third, > out: > return err; > } > +#else > +long compat_sys_semctl(int semid, int semnum, int cmd, int arg) > +{ > + return do_compat_semctl(semid, semnum, cmd, arg); > +} > + > +long compat_sys_msgsnd(int msqid, struct compat_msgbuf __user *msgp, > + size_t msgsz, int msgflg) > +{ > + compat_long_t mtype; > + > + if (get_user(mtype, &msgp->mtype)) > + return -EFAULT; > + return do_msgsnd(msqid, mtype, msgp->mtext, msgsz, msgflg); > +} > + > +long compat_sys_msgrcv(int msqid, struct compat_msgbuf __user *msgp, > + size_t msgsz, long msgtyp, int msgflg) > +{ > + long err, mtype; > + > + err = do_msgrcv(msqid, &mtype, msgp->mtext, msgsz, msgtyp, msgflg); > + if (err < 0) > + goto out; > + > + if (put_user(mtype, &msgp->mtype)) > + err = -EFAULT; > + out: > + return err; > +} > +#endif > > static inline int get_compat_msqid64(struct msqid64_ds *m64, > struct compat_msqid64_ds __user *up64) > @@ -470,6 +514,7 @@ long compat_sys_msgctl(int first, int second, void __user *uptr) > return err; > } > > +#ifdef CONFIG_ARCH_WANT_OLD_COMPAT_IPC > long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, > void __user *uptr) > { > @@ -485,6 +530,19 @@ long compat_sys_shmat(int first, int second, compat_uptr_t third, int version, > uaddr = compat_ptr(third); > return put_user(raddr, uaddr); > } > +#else > +long compat_sys_shmat(int shmid, compat_uptr_t shmaddr, int shmflg) > +{ > + unsigned long ret; > + long err; > + > + err = do_shmat(shmid, compat_ptr(shmaddr), shmflg, &ret); > + if (err) > + return err; > + force_successful_syscall_return(); > + return (long)ret; > +} > +#endif > > static inline int get_compat_shmid64_ds(struct shmid64_ds *s64, > struct compat_shmid64_ds __user *up64) -- Chris Metcalf, Tilera Corp. http://www.tilera.com ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v3] ipc: provide generic compat versions of IPC syscalls 2012-03-15 1:24 ` Chris Metcalf @ 2012-03-15 16:25 ` Arnd Bergmann 0 siblings, 0 replies; 6+ messages in thread From: Arnd Bergmann @ 2012-03-15 16:25 UTC (permalink / raw) To: Chris Metcalf Cc: linux-mips, NeilBrown, Heiko Carstens, Paul Mackerras, H. Peter Anvin, sparclinux, Christoph Hellwig, linux-s390, J. Bruce Fields, x86, Ingo Molnar, Lucas De Marchi, Thomas Gleixner, Dmitry Torokhov, linux-kernel, Ralf Baechle, Eric W. Biederman, Martin Schwidefsky, linux390, Andrew Morton, linuxppc-dev, David S. Miller On Thursday 15 March 2012, Chris Metcalf wrote: > > Somewhat belatedly, I'm looking to push this through the "tile" tree. > Andrew Morton has been keeping it in his tree but I figure I should push it > since it directly affects "tile" and currently no one else. Sounds good. > However, it would be nice if anyone could provide a Reviewed-by or Acked-by > on the patch. Thanks! As the main author of ipc/compat.c in its current form, Reviewed-by: Arnd Bergmann <arnd@arndb.de> ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-03-15 16:25 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <201112091536.pB9Fa5f7002738@farm-0002.internal.tilera.com>
[not found] ` <201112091602.31325.arnd@arndb.de>
2011-12-09 15:29 ` [PATCH v2] ipc: provide generic compat versions of IPC syscalls Chris Metcalf
2011-12-09 21:48 ` Andrew Morton
2011-12-09 23:21 ` Arnd Bergmann
2011-12-09 15:29 ` [PATCH v3] " Chris Metcalf
2012-03-15 1:24 ` Chris Metcalf
2012-03-15 16:25 ` Arnd Bergmann
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).