[3/3] linux-user: correct timerfd_create syscall numbers

Message ID	1455544812-14403-3-git-send-email-riku.voipio@linaro.org (mailing list archive)
State	New, archived
Headers	show Return-Path: <qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org> From: riku.voipio@linaro.org To: qemu-devel@nongnu.org Date: Mon, 15 Feb 2016 16:00:12 +0200 Message-Id: <1455544812-14403-3-git-send-email-riku.voipio@linaro.org> Cc: Riku Voipio <riku.voipio@linaro.org> Subject: [Qemu-devel] [PATCH 3/3] linux-user: correct timerfd_create syscall numbers Precedence: list Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Sender: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org

Message ID

1455544812-14403-3-git-send-email-riku.voipio@linaro.org (mailing list archive)

State

New, archived

Headers

From: riku.voipio@linaro.org
To: qemu-devel@nongnu.org
Date: Mon, 15 Feb 2016 16:00:12 +0200
Message-Id: <1455544812-14403-3-git-send-email-riku.voipio@linaro.org>
Cc: Riku Voipio <riku.voipio@linaro.org>
Subject: [Qemu-devel] [PATCH 3/3] linux-user: correct timerfd_create syscall
	numbers
Precedence: list
Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org
Sender: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org

Commit Message

Riku Voipio Feb. 15, 2016, 2 p.m. UTC

From: Riku Voipio <riku.voipio@linaro.org>

x86, m68k, ppc, sh4 and sparc failed to enable timerfd, because they
didn't have timerfd_create system call defined. Instead Qemu
defined timerfd syscall. Checking with kernel sources, it appears
kernel developers reused timerfd syscall number with timerfd_create,
presumably since no userspace called the old syscall number.

Reportd-by: Laurent Vivier <laurent@vivier.eu>
Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
---
 linux-user/i386/syscall_nr.h   | 2 +-
 linux-user/m68k/syscall_nr.h   | 2 +-
 linux-user/ppc/syscall_nr.h    | 2 +-
 linux-user/sh4/syscall_nr.h    | 2 +-
 linux-user/sparc/syscall_nr.h  | 2 +-
 linux-user/x86_64/syscall_nr.h | 2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

Comments

Laurent Vivier Feb. 15, 2016, 2:35 p.m. UTC | #1

Le 15/02/2016 15:00, riku.voipio@linaro.org a écrit :
> From: Riku Voipio <riku.voipio@linaro.org>
> 
> x86, m68k, ppc, sh4 and sparc failed to enable timerfd, because they
> didn't have timerfd_create system call defined. Instead Qemu
> defined timerfd syscall. Checking with kernel sources, it appears
> kernel developers reused timerfd syscall number with timerfd_create,
> presumably since no userspace called the old syscall number.
> 
> Reportd-by: Laurent Vivier <laurent@vivier.eu>
> Signed-off-by: Riku Voipio <riku.voipio@linaro.org>

Reviewed-by: Laurent Vivier <laurent@vivier.eu>

> ---
>  linux-user/i386/syscall_nr.h   | 2 +-
>  linux-user/m68k/syscall_nr.h   | 2 +-
>  linux-user/ppc/syscall_nr.h    | 2 +-
>  linux-user/sh4/syscall_nr.h    | 2 +-
>  linux-user/sparc/syscall_nr.h  | 2 +-
>  linux-user/x86_64/syscall_nr.h | 2 +-
>  6 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/linux-user/i386/syscall_nr.h b/linux-user/i386/syscall_nr.h
> index fa3f0b4..bc1bc23 100644
> --- a/linux-user/i386/syscall_nr.h
> +++ b/linux-user/i386/syscall_nr.h
> @@ -324,7 +324,7 @@
>  #define TARGET_NR_epoll_pwait	319
>  #define TARGET_NR_utimensat		320
>  #define TARGET_NR_signalfd		321
> -#define TARGET_NR_timerfd		322
> +#define TARGET_NR_timerfd_create	322
>  #define TARGET_NR_eventfd		323
>  #define TARGET_NR_fallocate		324
>  #define TARGET_NR_timerfd_settime	325
> diff --git a/linux-user/m68k/syscall_nr.h b/linux-user/m68k/syscall_nr.h
> index a2daba0..4b50fb2 100644
> --- a/linux-user/m68k/syscall_nr.h
> +++ b/linux-user/m68k/syscall_nr.h
> @@ -317,7 +317,7 @@
>  #define TARGET_NR_epoll_pwait	315
>  #define TARGET_NR_utimensat		316
>  #define TARGET_NR_signalfd		317
> -#define TARGET_NR_timerfd		318
> +#define TARGET_NR_timerfd_create	318
>  #define TARGET_NR_eventfd		319
>  #define TARGET_NR_fallocate		320
>  #define TARGET_NR_timerfd_settime	321
> diff --git a/linux-user/ppc/syscall_nr.h b/linux-user/ppc/syscall_nr.h
> index 0a5fd54..46ed8a6 100644
> --- a/linux-user/ppc/syscall_nr.h
> +++ b/linux-user/ppc/syscall_nr.h
> @@ -319,7 +319,7 @@
>  #define TARGET_NR_epoll_pwait	303
>  #define TARGET_NR_utimensat		304
>  #define TARGET_NR_signalfd		305
> -#define TARGET_NR_timerfd		306
> +#define TARGET_NR_timerfd_create	306
>  #define TARGET_NR_eventfd		307
>  #define TARGET_NR_sync_file_range2	308
>  #define TARGET_NR_fallocate		309
> diff --git a/linux-user/sh4/syscall_nr.h b/linux-user/sh4/syscall_nr.h
> index bdf8742..5009984 100644
> --- a/linux-user/sh4/syscall_nr.h
> +++ b/linux-user/sh4/syscall_nr.h
> @@ -323,7 +323,7 @@
>  #define TARGET_NR_epoll_pwait	319
>  #define TARGET_NR_utimensat		320
>  #define TARGET_NR_signalfd		321
> -#define TARGET_NR_timerfd		322
> +#define TARGET_NR_timerfd_create	322
>  #define TARGET_NR_eventfd		323
>  #define TARGET_NR_fallocate		324
>  #define TARGET_NR_timerfd_settime	325
> diff --git a/linux-user/sparc/syscall_nr.h b/linux-user/sparc/syscall_nr.h
> index 5b582a5..732b105 100644
> --- a/linux-user/sparc/syscall_nr.h
> +++ b/linux-user/sparc/syscall_nr.h
> @@ -278,7 +278,7 @@
>  #define TARGET_NR_epoll_pwait	309
>  #define TARGET_NR_utimensat		310
>  #define TARGET_NR_signalfd		311
> -#define TARGET_NR_timerfd		312
> +#define TARGET_NR_timerfd_create	312
>  #define TARGET_NR_eventfd		313
>  #define TARGET_NR_fallocate		314
>  #define TARGET_NR_timerfd_settime	315
> diff --git a/linux-user/x86_64/syscall_nr.h b/linux-user/x86_64/syscall_nr.h
> index f00fa2b..16397b3 100644
> --- a/linux-user/x86_64/syscall_nr.h
> +++ b/linux-user/x86_64/syscall_nr.h
> @@ -281,7 +281,7 @@
>  #define TARGET_NR_utimensat		280
>  #define TARGET_NR_epoll_pwait	281
>  #define TARGET_NR_signalfd		282
> -#define TARGET_NR_timerfd		283
> +#define TARGET_NR_timerfd_create	283
>  #define TARGET_NR_eventfd		284
>  #define TARGET_NR_fallocate		285
>  #define TARGET_NR_timerfd_settime	286
>

diff --git a/linux-user/i386/syscall_nr.h b/linux-user/i386/syscall_nr.h
index fa3f0b4..bc1bc23 100644
--- a/linux-user/i386/syscall_nr.h
+++ b/linux-user/i386/syscall_nr.h
@@ -324,7 +324,7 @@ 
 #define TARGET_NR_epoll_pwait	319
 #define TARGET_NR_utimensat		320
 #define TARGET_NR_signalfd		321
-#define TARGET_NR_timerfd		322
+#define TARGET_NR_timerfd_create	322
 #define TARGET_NR_eventfd		323
 #define TARGET_NR_fallocate		324
 #define TARGET_NR_timerfd_settime	325
diff --git a/linux-user/m68k/syscall_nr.h b/linux-user/m68k/syscall_nr.h
index a2daba0..4b50fb2 100644
--- a/linux-user/m68k/syscall_nr.h
+++ b/linux-user/m68k/syscall_nr.h
@@ -317,7 +317,7 @@ 
 #define TARGET_NR_epoll_pwait	315
 #define TARGET_NR_utimensat		316
 #define TARGET_NR_signalfd		317
-#define TARGET_NR_timerfd		318
+#define TARGET_NR_timerfd_create	318
 #define TARGET_NR_eventfd		319
 #define TARGET_NR_fallocate		320
 #define TARGET_NR_timerfd_settime	321
diff --git a/linux-user/ppc/syscall_nr.h b/linux-user/ppc/syscall_nr.h
index 0a5fd54..46ed8a6 100644
--- a/linux-user/ppc/syscall_nr.h
+++ b/linux-user/ppc/syscall_nr.h
@@ -319,7 +319,7 @@ 
 #define TARGET_NR_epoll_pwait	303
 #define TARGET_NR_utimensat		304
 #define TARGET_NR_signalfd		305
-#define TARGET_NR_timerfd		306
+#define TARGET_NR_timerfd_create	306
 #define TARGET_NR_eventfd		307
 #define TARGET_NR_sync_file_range2	308
 #define TARGET_NR_fallocate		309
diff --git a/linux-user/sh4/syscall_nr.h b/linux-user/sh4/syscall_nr.h
index bdf8742..5009984 100644
--- a/linux-user/sh4/syscall_nr.h
+++ b/linux-user/sh4/syscall_nr.h
@@ -323,7 +323,7 @@ 
 #define TARGET_NR_epoll_pwait	319
 #define TARGET_NR_utimensat		320
 #define TARGET_NR_signalfd		321
-#define TARGET_NR_timerfd		322
+#define TARGET_NR_timerfd_create	322
 #define TARGET_NR_eventfd		323
 #define TARGET_NR_fallocate		324
 #define TARGET_NR_timerfd_settime	325
diff --git a/linux-user/sparc/syscall_nr.h b/linux-user/sparc/syscall_nr.h
index 5b582a5..732b105 100644
--- a/linux-user/sparc/syscall_nr.h
+++ b/linux-user/sparc/syscall_nr.h
@@ -278,7 +278,7 @@ 
 #define TARGET_NR_epoll_pwait	309
 #define TARGET_NR_utimensat		310
 #define TARGET_NR_signalfd		311
-#define TARGET_NR_timerfd		312
+#define TARGET_NR_timerfd_create	312
 #define TARGET_NR_eventfd		313
 #define TARGET_NR_fallocate		314
 #define TARGET_NR_timerfd_settime	315
diff --git a/linux-user/x86_64/syscall_nr.h b/linux-user/x86_64/syscall_nr.h
index f00fa2b..16397b3 100644
--- a/linux-user/x86_64/syscall_nr.h
+++ b/linux-user/x86_64/syscall_nr.h
@@ -281,7 +281,7 @@ 
 #define TARGET_NR_utimensat		280
 #define TARGET_NR_epoll_pwait	281
 #define TARGET_NR_signalfd		282
-#define TARGET_NR_timerfd		283
+#define TARGET_NR_timerfd_create	283
 #define TARGET_NR_eventfd		284
 #define TARGET_NR_fallocate		285
 #define TARGET_NR_timerfd_settime	286

[3/3] linux-user: correct timerfd_create syscall numbers

Commit Message

Comments

Patch