Re: [Qemu-devel] [PATCH] linux-user: Fix compilation when F_SETPIPE_SZ isn't defined

[Laurent Vivier]

Le 30/06/2016 à 18:33, Peter Maydell a écrit :
> Older kernels don't have F_SETPIPE_SZ and F_GETPIPE_SZ (in
> particular RHEL6's system headers don't define these). Add
> ifdefs so that we can gracefully fall back to not supporting
> those guest ioctls rather than failing to build.
> 
> Signed-off-by: Peter Maydell <address@hidden>

Reviewed-by: Laurent Vivier <address@hidden>

> ---
> Reported by an anonymous IRC user...
> 
>  linux-user/syscall.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/linux-user/syscall.c b/linux-user/syscall.c
> index 6ad8239..165fd06 100644
> --- a/linux-user/syscall.c
> +++ b/linux-user/syscall.c
> @@ -5598,10 +5598,12 @@ static int target_to_host_fcntl_cmd(int cmd)
>       case TARGET_F_SETOWN_EX:
>           return F_SETOWN_EX;
>  #endif
> +#ifdef F_SETPIPE_SZ
>          case TARGET_F_SETPIPE_SZ:
>              return F_SETPIPE_SZ;
>          case TARGET_F_GETPIPE_SZ:
>              return F_GETPIPE_SZ;
> +#endif
>       default:
>              return -TARGET_EINVAL;
>      }
>