[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH 04/12] linux-user: Make ipc syscall's
From: |
Peter Maydell |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH 04/12] linux-user: Make ipc syscall's third argument an abi_long |
Date: |
Mon, 4 Aug 2014 18:09:00 +0100 |
On 4 August 2014 17:45, Tom Musta <address@hidden> wrote:
> For those target ABIs that use the ipc system call (e.g. POWER),
> the third argument is used in the shmat path as a pointer. It
> therefore must be declared as an abi_long (versus int) so that
> the address bits are not lost in truncation. The "int-ness" of
> this argument is retained for all other system calls via an
> explicit cast.
>
> Signed-off-by: Tom Musta <address@hidden>
>
> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index fb03e96..bf6dd1e 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -3122,7 +3122,7 @@ static inline abi_long do_shmdt(abi_ulong shmaddr)
> /* ??? This only works with linear mappings. */
> /* do_ipc() must return target values and target errnos. */
> static abi_long do_ipc(unsigned int call, int first,
> - int second, int third,
> + int second, abi_long third,
> abi_long ptr, abi_long fifth)
Given where and how this is called, I would suggest
that all its arguments (except 'call') hould be abi_long,
because it's just out-of-line handling of one particular case
from do_syscall(), and the arguments there are all abi_long.
> {
> int version;
> @@ -3137,7 +3137,7 @@ static abi_long do_ipc(unsigned int call, int first,
> break;
>
> case IPCOP_semget:
> - ret = get_errno(semget(first, second, third));
> + ret = get_errno(semget(first, second, (int)third));
> break;
This cast isn't needed because semget()'s third argument is
int type anyway. I suspect most or perhaps even all the other
casts below are also unneeded.
thanks
-- PMM
- [Qemu-ppc] [PATCH 00/12] target-ppc: Linux-User Mode Bug Fixes for Power, Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 01/12] linux-user: PPC64 semid_ds Doesnt Include _unused1 and _unused2, Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 02/12] linux-user: Dereference Pointer Argument to ipc/semctl Sys Call, Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 03/12] linux-user: Properly Handle semun Structure In Cross-Endian Situations, Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 04/12] linux-user: Make ipc syscall's third argument an abi_long, Tom Musta, 2014/08/04
- Re: [Qemu-ppc] [Qemu-devel] [PATCH 04/12] linux-user: Make ipc syscall's third argument an abi_long,
Peter Maydell <=
- [Qemu-ppc] [PATCH 05/12] linux-user: Conditionally Pass Attribute Pointer to mq_open(), Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 06/12] linux-user: Detect Negative Message Sizes in msgsnd System Call, Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 07/12] linux-user: Handle NULL argument to sched_{get, set}param, Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 08/12] linux-user: Detect fault in sched_rr_get_interval, Tom Musta, 2014/08/04
- [Qemu-ppc] [PATCH 09/12] linux-user: Minimum Sig Handler Stack Size for PPC64 ELF V2, Tom Musta, 2014/08/04