diff options
author | Michael Ellerman <michael@ellerman.id.au> | 2005-10-21 02:01:34 -0400 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2005-10-21 08:47:52 -0400 |
commit | 05d84681abcb33fe34accb5982c413daeb4208c4 (patch) | |
tree | 713c1ac65109ab1e2b0bd30a29bad13f2caee38e | |
parent | 90eac727c6d7afbe707ce408edf97c33385fa08c (diff) |
[PATCH] powerpc: Fix mmap returning 64 bit addresses
The merge of syscalls.c & sys_ppc32.c (30286ef6e044bc3d9019c3d8b900572e3fa05e65)
broke mmap, if the mmap returned a 64 bit address.
do_mmap2 was taking the return value from do_mmap_pgoff (an unsigned long), and
storing it in an int, before returning it to sys_mmap as an unsigned long. So
we were losing the high bits of the address.
You would have thought the compiler could catch this for us ...
Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
-rw-r--r-- | arch/powerpc/kernel/syscalls.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/kernel/syscalls.c b/arch/powerpc/kernel/syscalls.c index 7a23721365a6..f72ced11212d 100644 --- a/arch/powerpc/kernel/syscalls.c +++ b/arch/powerpc/kernel/syscalls.c | |||
@@ -162,7 +162,7 @@ static inline unsigned long do_mmap2(unsigned long addr, size_t len, | |||
162 | unsigned long fd, unsigned long off, int shift) | 162 | unsigned long fd, unsigned long off, int shift) |
163 | { | 163 | { |
164 | struct file * file = NULL; | 164 | struct file * file = NULL; |
165 | int ret = -EINVAL; | 165 | unsigned long ret = -EINVAL; |
166 | 166 | ||
167 | if (shift) { | 167 | if (shift) { |
168 | if (off & ((1 << shift) - 1)) | 168 | if (off & ((1 << shift) - 1)) |