From: Arnd Bergmann I have tested the code with the open posix test suite and found the same four failures for both 64-bit and compat mode, most tests pass. The patch is against -mc1, but I guess it also applies to the other trees around. What worries me more than mq_attr compatibility is the conversion of struct sigevent, which might turn out really hard when more fields in there are used. AFAICS, the only other part in the kernel ABI is sys_timer_create(), so maybe it's not too late to deprecate the current structure and create a structure that can be used properly for compat syscalls. --- 25-akpm/include/linux/compat.h | 16 +++ 25-akpm/include/linux/mqueue.h | 4 25-akpm/include/linux/posix_types.h | 1 25-akpm/include/linux/syscalls.h | 1 25-akpm/include/linux/types.h | 1 25-akpm/ipc/Makefile | 3 25-akpm/ipc/compat_mq.c | 185 ++++++++++++++++++++++++++++++++++++ 25-akpm/kernel/sys.c | 5 8 files changed, 212 insertions(+), 4 deletions(-) diff -puN include/linux/compat.h~compat_mq include/linux/compat.h --- 25/include/linux/compat.h~compat_mq 2004-04-08 13:43:22.288092504 -0700 +++ 25-akpm/include/linux/compat.h 2004-04-08 13:43:22.301090528 -0700 @@ -90,6 +90,22 @@ typedef union compat_sigval { compat_uptr_t sival_ptr; } compat_sigval_t; +typedef struct compat_sigevent { + compat_sigval_t sigev_value; + compat_int_t sigev_signo; + compat_int_t sigev_notify; + union { + compat_int_t _pad[SIGEV_PAD_SIZE]; + compat_int_t _tid; + + struct { + compat_uptr_t _function; + compat_uptr_t _attribute; + } _sigev_thread; + } _sigev_un; +} compat_sigevent_t; + + 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, diff -puN include/linux/mqueue.h~compat_mq include/linux/mqueue.h --- 25/include/linux/mqueue.h~compat_mq 2004-04-08 13:43:22.290092200 -0700 +++ 25-akpm/include/linux/mqueue.h 2004-04-08 13:43:22.301090528 -0700 @@ -18,9 +18,9 @@ #ifndef _LINUX_MQUEUE_H #define _LINUX_MQUEUE_H -#define MQ_PRIO_MAX 32768 +#include -typedef int mqd_t; +#define MQ_PRIO_MAX 32768 struct mq_attr { long mq_flags; /* message queue flags */ diff -puN include/linux/posix_types.h~compat_mq include/linux/posix_types.h --- 25/include/linux/posix_types.h~compat_mq 2004-04-08 13:43:22.291092048 -0700 +++ 25-akpm/include/linux/posix_types.h 2004-04-08 13:43:22.301090528 -0700 @@ -42,6 +42,7 @@ typedef void (*__kernel_sighandler_t)(in /* Type of a SYSV IPC key. */ typedef int __kernel_key_t; +typedef int __kernel_mqd_t; #include diff -puN include/linux/syscalls.h~compat_mq include/linux/syscalls.h --- 25/include/linux/syscalls.h~compat_mq 2004-04-08 13:43:22.293091744 -0700 +++ 25-akpm/include/linux/syscalls.h 2004-04-08 13:43:22.302090376 -0700 @@ -48,7 +48,6 @@ struct timex; struct timezone; struct tms; struct utimbuf; -typedef int mqd_t; struct mq_attr; #include diff -puN include/linux/types.h~compat_mq include/linux/types.h --- 25/include/linux/types.h~compat_mq 2004-04-08 13:43:22.295091440 -0700 +++ 25-akpm/include/linux/types.h 2004-04-08 13:43:22.303090224 -0700 @@ -31,6 +31,7 @@ typedef __kernel_key_t key_t; typedef __kernel_suseconds_t suseconds_t; typedef __kernel_timer_t timer_t; typedef __kernel_clockid_t clockid_t; +typedef __kernel_mqd_t mqd_t; #ifdef __KERNEL__ typedef __kernel_uid32_t uid_t; diff -puN /dev/null ipc/compat_mq.c --- /dev/null 2003-09-15 06:40:47.000000000 -0700 +++ 25-akpm/ipc/compat_mq.c 2004-04-08 13:43:22.304090072 -0700 @@ -0,0 +1,185 @@ +/* + * ipc/compat_mq.c + * 32 bit emulation for POSIX message queue system calls + * + * Copyright (C) 2004 IBM Deutschland Entwicklung GmbH, IBM Corporation + * Author: Arnd Bergmann + */ + +#include +#include +#include +#include +#include + +#include + +struct compat_mq_attr { + compat_long_t mq_flags; /* message queue flags */ + compat_long_t mq_maxmsg; /* maximum number of messages */ + compat_long_t mq_msgsize; /* maximum message size */ + compat_long_t mq_curmsgs; /* number of messages currently queued */ + compat_long_t __reserved[4]; /* ignored for input, zeroed for output */ +}; + +static inline int get_compat_mq_attr(struct mq_attr *attr, + const struct compat_mq_attr __user *uattr) +{ + if (verify_area(VERIFY_READ, uattr, sizeof *uattr)) + return -EFAULT; + + return __get_user(attr->mq_flags, &uattr->mq_flags) + | __get_user(attr->mq_maxmsg, &uattr->mq_maxmsg) + | __get_user(attr->mq_msgsize, &uattr->mq_msgsize) + | __get_user(attr->mq_curmsgs, &uattr->mq_curmsgs); +} + +static inline int put_compat_mq_attr(const struct mq_attr *attr, + struct compat_mq_attr __user *uattr) +{ + if (clear_user(uattr, sizeof *uattr)) + return -EFAULT; + + return __put_user(attr->mq_flags, &uattr->mq_flags) + | __put_user(attr->mq_maxmsg, &uattr->mq_maxmsg) + | __put_user(attr->mq_msgsize, &uattr->mq_msgsize) + | __put_user(attr->mq_curmsgs, &uattr->mq_curmsgs); +} + +asmlinkage long compat_sys_mq_open(const char __user *u_name, + int oflag, compat_mode_t mode, + struct compat_mq_attr __user *u_attr) +{ + struct mq_attr attr; + mm_segment_t oldfs; + char *name; + long ret; + + if (!u_attr) + return sys_mq_open(u_name, oflag, mode, 0); + + if (get_compat_mq_attr(&attr, u_attr)) + return -EFAULT; + + name = getname(u_name); + if (IS_ERR(name)) + return PTR_ERR(name); + + oldfs = get_fs(); + set_fs(KERNEL_DS); + ret = sys_mq_open(name, oflag, mode, &attr); + set_fs(oldfs); + + putname(name); + return ret; +} + +static struct timespec __user *compat_prepare_timeout( + const struct compat_timespec __user *u_abs_timeout) +{ + struct timespec ts, __user *u_ts; + + if (!u_abs_timeout) + return 0; + + u_ts = compat_alloc_user_space(sizeof(*u_ts)); + if (get_compat_timespec(&ts, u_abs_timeout) + || copy_to_user(u_ts, &ts, sizeof(*u_ts))) + return ERR_PTR(-EFAULT); + + return u_ts; +} + +asmlinkage long compat_sys_mq_timedsend(mqd_t mqdes, + const char __user *u_msg_ptr, + size_t msg_len, unsigned int msg_prio, + const struct compat_timespec __user *u_abs_timeout) +{ + struct timespec __user *u_ts; + + u_ts = compat_prepare_timeout(u_abs_timeout); + if (IS_ERR(u_ts)) + return -EFAULT; + + return sys_mq_timedsend(mqdes, u_msg_ptr, msg_len, + msg_prio, u_ts); +} + +asmlinkage ssize_t compat_sys_mq_timedreceive(mqd_t mqdes, + char __user *u_msg_ptr, + size_t msg_len, unsigned int __user *u_msg_prio, + const struct compat_timespec __user *u_abs_timeout) +{ + struct timespec *u_ts; + + u_ts = compat_prepare_timeout(u_abs_timeout); + if (IS_ERR(u_ts)) + return -EFAULT; + + return sys_mq_timedreceive(mqdes, u_msg_ptr, msg_len, + u_msg_prio, u_ts); +} + +static int get_compat_sigevent(struct sigevent *event, + const struct compat_sigevent __user *u_event) +{ + if (verify_area(VERIFY_READ, u_event, sizeof(*u_event))) + return -EFAULT; + + return __get_user(event->sigev_value.sival_int, + &u_event->sigev_value.sival_int) + | __get_user(event->sigev_signo, &u_event->sigev_signo) + | __get_user(event->sigev_notify, &u_event->sigev_notify) + | __get_user(event->sigev_notify_thread_id, + &u_event->sigev_notify_thread_id); +} + +asmlinkage long compat_sys_mq_notify(mqd_t mqdes, + const struct compat_sigevent __user *u_notification) +{ + mm_segment_t oldfs; + struct sigevent notification; + long ret; + + if (!u_notification) + return sys_mq_notify(mqdes, 0); + + if (get_compat_sigevent(¬ification, u_notification)) + return -EFAULT; + + oldfs = get_fs(); + set_fs(KERNEL_DS); + ret = sys_mq_notify(mqdes, ¬ification); + set_fs(oldfs); + + return ret; +} + +asmlinkage long compat_sys_mq_getsetattr(mqd_t mqdes, + const struct compat_mq_attr __user *u_mqstat, + struct compat_mq_attr __user *u_omqstat) +{ + struct mq_attr mqstat, omqstat; + struct mq_attr *p_mqstat = 0, *p_omqstat = 0; + mm_segment_t oldfs; + long ret; + + if (u_mqstat) { + p_mqstat = &mqstat; + if (get_compat_mq_attr(p_mqstat, u_mqstat)) + return -EFAULT; + } + + if (u_omqstat) + p_omqstat = &omqstat; + + oldfs = get_fs(); + set_fs(KERNEL_DS); + ret = sys_mq_getsetattr(mqdes, p_mqstat, p_omqstat); + set_fs(oldfs); + + if (ret) + return ret; + + return (u_omqstat) ? put_compat_mq_attr(&omqstat, u_omqstat) : 0; +} diff -puN ipc/Makefile~compat_mq ipc/Makefile --- 25/ipc/Makefile~compat_mq 2004-04-08 13:43:22.296091288 -0700 +++ 25-akpm/ipc/Makefile 2004-04-08 13:43:22.304090072 -0700 @@ -4,5 +4,6 @@ obj-$(CONFIG_SYSVIPC_COMPAT) += compat.o obj-$(CONFIG_SYSVIPC) += util.o msgutil.o msg.o sem.o shm.o -obj-$(CONFIG_POSIX_MQUEUE) += mqueue.o msgutil.o +obj_mq-$(CONFIG_COMPAT) += compat_mq.o +obj-$(CONFIG_POSIX_MQUEUE) += mqueue.o msgutil.o $(obj_mq-y) diff -puN kernel/sys.c~compat_mq kernel/sys.c --- 25/kernel/sys.c~compat_mq 2004-04-08 13:43:22.298090984 -0700 +++ 25-akpm/kernel/sys.c 2004-04-08 13:43:22.305089920 -0700 @@ -266,6 +266,11 @@ cond_syscall(sys_mq_timedsend) cond_syscall(sys_mq_timedreceive) cond_syscall(sys_mq_notify) cond_syscall(sys_mq_getsetattr) +cond_syscall(compat_sys_mq_open) +cond_syscall(compat_sys_mq_timedsend) +cond_syscall(compat_sys_mq_timedreceive) +cond_syscall(compat_sys_mq_notify) +cond_syscall(compat_sys_mq_getsetattr) /* arch-specific weak syscall entries */ cond_syscall(sys_pciconfig_read) _