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

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

[Arnaud Patard]

[-- Attachment #1: Type: text/plain, Size: 224 bytes --]


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.


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: fcntl_linuxspecific.patch --]
[-- Type: text/x-diff, Size: 1138 bytes --]

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 */

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

[Riku Voipio]

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 */