Re: [Qemu-devel] [PATCH] linux-user/syscall.c: add missing linux-specific fnctl commands

[Riku Voipio <riku.voipio@iki.fi>]

On Thu, Jun 18, 2009 at 01:38:03PM +0200, Arnaud Patard wrote:
> This patch is adding missing linux-specific fcntl command. I've tested
> with ltp test SETLEASE and GETLEASE but there was no tests for F_NOTIFY
> and F_DUPFD_CLOEXEC.

Looks fine, picked up for next batch of patches to send.

> Signed-off-by: Arnaud Patard <arnaud.patard@rtp-net.org>

> Index: qemu/linux-user/syscall.c
> ===================================================================
> --- qemu.orig/linux-user/syscall.c
> +++ qemu/linux-user/syscall.c
> @@ -3654,6 +3654,11 @@ static int target_to_host_fcntl_cmd(int 
>  	case TARGET_F_SETLKW64:
>  	    return F_SETLKW64;
>  #endif
> +	case TARGET_F_SETLEASE:
> +	case TARGET_F_GETLEASE:
> +	case TARGET_F_NOTIFY:
> +	case TARGET_F_DUPFD_CLOEXEC:
> +	    return cmd;
>  	default:
>              return -TARGET_EINVAL;
>      }
> Index: qemu/linux-user/syscall_defs.h
> ===================================================================
> --- qemu.orig/linux-user/syscall_defs.h
> +++ qemu/linux-user/syscall_defs.h
> @@ -1732,6 +1732,11 @@ struct target_statfs64 {
>  };
>  #endif
>  
> +#define TARGET_F_LINUX_SPECIFIC_BASE 1024
> +#define TARGET_F_SETLEASE      (TARGET_F_LINUX_SPECIFIC_BASE + 0)
> +#define TARGET_F_GETLEASE      (TARGET_F_LINUX_SPECIFIC_BASE + 1)
> +#define TARGET_F_NOTIFY        (TARGET_F_LINUX_SPECIFIC_BASE + 2)
> +#define TARGET_F_DUPFD_CLOEXEC (TARGET_F_LINUX_SPECIFIC_BASE + 6)
>  
>  #define TARGET_F_DUPFD         0       /* dup */
>  #define TARGET_F_GETFD         1       /* get close_on_exec */