Gerd Hoffmann wrote:
>> How married are people to this crazy 2x "32 bits" model?
>
> The tricky corner case are 32bit compat syscalls on 64bit kernel, where
> a noticable number of archs (IIRC at least ppc, mips and s390) can't
> pass unaligned 64bit values through the syscall abi. The 32/32 split
> was done because of that.
>
> I think using two unsigned longs instead of two u32 should work. And
> allows us to handle 128bit file offsets on 64bit. That I'll call
> future-proof interface! Nevertheless I'd like to have comments from the
> arch maintainers on the idea.
Patch attached for comments (builds, but otherwise not tested yet).
cheers,
Gerd
>From 83ba4d31b29e931c08da1570bb47a2defef353fe Mon Sep 17 00:00:00 2001
From: Gerd Hoffmann <kraxel@xxxxxxxxxx>
Date: Fri, 3 Apr 2009 11:08:00 +0200
Subject: [PATCH] preadv/pwritev: tweak interface.
Go from "u32" to "unsigned long" for the pos_{high,low} syscall
arguments. That way we avoid the 32/32 split for 64bit archs,
and we still side-step the ABI issue with unaligned 64bit args
on a number of 32bit archs. pos_high on 64bit is unused for
the time being. Some day we might be happy to have it though ;)
Signed-off-by: Gerd Hoffmann <kraxel@xxxxxxxxxx>
---
fs/compat.c | 6 +++---
fs/read_write.c | 12 ++++++++----
include/linux/compat.h | 2 +-
include/linux/syscalls.h | 4 ++--
4 files changed, 14 insertions(+), 10 deletions(-)
diff --git a/fs/compat.c b/fs/compat.c
index 1c859da..053f843 100644
--- a/fs/compat.c
+++ b/fs/compat.c
@@ -1236,9 +1236,9 @@ compat_sys_readv(unsigned long fd, const struct compat_iovec __user *vec,
asmlinkage ssize_t
compat_sys_preadv(unsigned long fd, const struct compat_iovec __user *vec,
- unsigned long vlen, u32 pos_high, u32 pos_low)
+ unsigned long vlen, unsigned long pos_high, unsigned long pos_low)
{
- loff_t pos = ((loff_t)pos_high << 32) | pos_low;
+ loff_t pos = ((loff_t)pos_high << BITS_PER_COMPAT_LONG) | pos_low;
struct file *file;
int fput_needed;
ssize_t ret;
@@ -1295,7 +1295,7 @@ asmlinkage ssize_t
compat_sys_pwritev(unsigned long fd, const struct compat_iovec __user *vec,
unsigned long vlen, u32 pos_high, u32 pos_low)
{
- loff_t pos = ((loff_t)pos_high << 32) | pos_low;
+ loff_t pos = ((loff_t)pos_high << BITS_PER_COMPAT_LONG) | pos_low;
struct file *file;
int fput_needed;
ssize_t ret;
diff --git a/fs/read_write.c b/fs/read_write.c
index 6d5d8ff..8201e60 100644
--- a/fs/read_write.c
+++ b/fs/read_write.c
@@ -731,10 +731,14 @@ SYSCALL_DEFINE3(writev, unsigned long, fd, const struct iovec __user *, vec,
return ret;
}
+#define HALF_LONG (BITS_PER_LONG / 2)
+
SYSCALL_DEFINE5(preadv, unsigned long, fd, const struct iovec __user *, vec,
- unsigned long, vlen, u32, pos_high, u32, pos_low)
+ unsigned long, vlen, unsigned long, pos_high, unsigned long, pos_low)
{
- loff_t pos = ((loff_t)pos_high << 32) | pos_low;
+ /* gcc optimizes away the shifts for
+ * sizeof(pos_high) == sizeof(pos), i.e. on 64bit archs */
+ loff_t pos = (((loff_t)pos_high << HALF_LONG) << HALF_LONG) | pos_low;
struct file *file;
ssize_t ret = -EBADF;
int fput_needed;
@@ -757,9 +761,9 @@ SYSCALL_DEFINE5(preadv, unsigned long, fd, const struct iovec __user *, vec,
}
SYSCALL_DEFINE5(pwritev, unsigned long, fd, const struct iovec __user *, vec,
- unsigned long, vlen, u32, pos_high, u32, pos_low)
+ unsigned long, vlen, unsigned long, pos_high, unsigned long, pos_low)
{
- loff_t pos = ((loff_t)pos_high << 32) | pos_low;
+ loff_t pos = (((loff_t)pos_high << HALF_LONG) << HALF_LONG) | pos_low;
struct file *file;
ssize_t ret = -EBADF;
int fput_needed;
diff --git a/include/linux/compat.h b/include/linux/compat.h
index 9723edd..065944e 100644
--- a/include/linux/compat.h
+++ b/include/linux/compat.h
@@ -193,7 +193,7 @@ asmlinkage ssize_t compat_sys_writev(unsigned long fd,
const struct compat_iovec __user *vec, unsigned long vlen);
asmlinkage ssize_t compat_sys_preadv(unsigned long fd,
const struct compat_iovec __user *vec,
- unsigned long vlen, u32 pos_high, u32 pos_low);
+ unsigned long vlen, unsigned long pos_high, unsigned long pos_low);
asmlinkage ssize_t compat_sys_pwritev(unsigned long fd,
const struct compat_iovec __user *vec,
unsigned long vlen, u32 pos_high, u32 pos_low);
diff --git a/include/linux/syscalls.h b/include/linux/syscalls.h
index b299a82..25e60b0 100644
--- a/include/linux/syscalls.h
+++ b/include/linux/syscalls.h
@@ -462,9 +462,9 @@ asmlinkage long sys_pread64(unsigned int fd, char __user *buf,
asmlinkage long sys_pwrite64(unsigned int fd, const char __user *buf,
size_t count, loff_t pos);
asmlinkage long sys_preadv(unsigned long fd, const struct iovec __user *vec,
- unsigned long vlen, u32 pos_high, u32 pos_low);
+ unsigned long vlen, unsigned long pos_high, unsigned long pos_low);
asmlinkage long sys_pwritev(unsigned long fd, const struct iovec __user *vec,
- unsigned long vlen, u32 pos_high, u32 pos_low);
+ unsigned long vlen, unsigned long pos_high, unsigned long pos_low);
asmlinkage long sys_getcwd(char __user *buf, unsigned long size);
asmlinkage long sys_mkdir(const char __user *pathname, int mode);
asmlinkage long sys_chdir(const char __user *filename);
--
1.6.1.3
