[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/6 v4] linux-user/syscall.c: Switch all macros
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 6/6 v4] linux-user/syscall.c: Switch all macros which are not defined in tilegx |
Date: |
Fri, 27 Feb 2015 19:24:31 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
Am 22.02.2015 um 14:37 schrieb Chen Gang S:
> For tilegx, several syscall macros are not supported, so switch them to
> avoid building break.
>
> Signed-off-by: Chen Gang <address@hidden>
> ---
> linux-user/syscall.c | 50 +++++++++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 49 insertions(+), 1 deletion(-)
LGTM,
Reviewed-by: Andreas Färber <address@hidden>
However I suggest "conditionalize syscalls" or "disable syscalls" over
"switch ... macros/them".
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)
- [Qemu-devel] [PATCH 0/6 v4] tilegx: Can load elf64 tilegx binary successfully for linux-user, Chen Gang S, 2015/02/22
- [Qemu-devel] [PATCH 1/6 v4] target-tilegx: Firstly add to qemu with minimized features, Chen Gang S, 2015/02/22
- [Qemu-devel] [PATCH 3/6 v4] linux-user: tilegx: Add target features support within qemu, Chen Gang S, 2015/02/22
- [Qemu-devel] [PATCH 4/6 v4] linux-user: Support tilegx architecture in syscall, Chen Gang S, 2015/02/22
- [Qemu-devel] [PATCH 5/6 v4] linux-user: Support tilegx architecture in linux-user, Chen Gang S, 2015/02/22
- [Qemu-devel] [PATCH 6/6 v4] linux-user/syscall.c: Switch all macros which are not defined in tilegx, Chen Gang S, 2015/02/22
- Re: [Qemu-devel] [PATCH 6/6 v4] linux-user/syscall.c: Switch all macros which are not defined in tilegx,
Andreas Färber <=
- Re: [Qemu-devel] [PATCH 0/6 v4] tilegx: Can load elf64 tilegx binary successfully for linux-user, Richard Henderson, 2015/02/22
- Re: [Qemu-devel] [PATCH 0/6 v4] tilegx: Can load elf64 tilegx binary successfully for linux-user, Bastian Koppelmann, 2015/02/22