diff mbox

[2/2] arm: Wire up kcmp syscall

Message ID 20130219070703.GC20312@moon (mailing list archive)
State New, archived
Headers show

Commit Message

Cyrill Gorcunov Feb. 19, 2013, 7:07 a.m. UTC 
quilt eat original header, here is the former, sorry
for incovenience.
---
From: Alexander Kartashov <alekskartashov@parallels.com>
Subject: arm: Wire up kcmp syscall

Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com>
Cc: Russell King <linux@arm.linux.org.uk>
---
 arch/arm/include/uapi/asm/unistd.h |    2 +-
 arch/arm/kernel/calls.S            |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Cyrill Gorcunov Feb. 20, 2013, 7:56 a.m. UTC | #1 
On Tue, Feb 19, 2013 at 11:07:03AM +0400, Cyrill Gorcunov wrote:
> From: Alexander Kartashov <alekskartashov@parallels.com>
> Subject: arm: Wire up kcmp syscall
> 
> Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com>
> Cc: Russell King <linux@arm.linux.org.uk>

Ping? Guys is there any change to get it reviewed/acked/nacked?
Sorry for nagging but this syscall is quite important for
c/r functionality on arm machine.
Andrew Morton Feb. 20, 2013, 11:17 p.m. UTC | #2 
On Tue, 19 Feb 2013 11:07:03 +0400
Cyrill Gorcunov <gorcunov@openvz.org> wrote:

> From: Alexander Kartashov <alekskartashov@parallels.com>
> Subject: arm: Wire up kcmp syscall
> 
> Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com>
> Cc: Russell King <linux@arm.linux.org.uk>

This should have had signed-off-by:you, as you were on the patch's
delivery path.
Cyrill Gorcunov Feb. 21, 2013, 5:43 a.m. UTC | #3 
On Wed, Feb 20, 2013 at 03:17:23PM -0800, Andrew Morton wrote:
> On Tue, 19 Feb 2013 11:07:03 +0400
> Cyrill Gorcunov <gorcunov@openvz.org> wrote:
> 
> > From: Alexander Kartashov <alekskartashov@parallels.com>
> > Subject: arm: Wire up kcmp syscall
> > 
> > Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com>
> > Cc: Russell King <linux@arm.linux.org.uk>
> 
> This should have had signed-off-by:you, as you were on the patch's
> delivery path.

Ouch, sorry Andrew! Thanks!
Arnd Bergmann Feb. 23, 2013, 10:31 p.m. UTC | #4 
On Tuesday 19 February 2013, Cyrill Gorcunov wrote:
> From: Alexander Kartashov <alekskartashov@parallels.com>
> Subject: arm: Wire up kcmp syscall
> 
> Signed-off-by: Alexander Kartashov <alekskartashov@parallels.com>
> Cc: Russell King <linux@arm.linux.org.uk>

Acked-by: Arnd Bergmann <arnd@arndb.de>

Looks uncontroversial since the number is already reserved and
there is a cond_syscall() line, which means it does not
even depend on patch 1.

Russell, are you picking this one up?
diff mbox

Patch

Index: linux-2.6.git/arch/arm/include/uapi/asm/unistd.h
===================================================================
--- linux-2.6.git.orig/arch/arm/include/uapi/asm/unistd.h
+++ linux-2.6.git/arch/arm/include/uapi/asm/unistd.h
@@ -404,7 +404,7 @@ 
 #define __NR_setns			(__NR_SYSCALL_BASE+375)
 #define __NR_process_vm_readv		(__NR_SYSCALL_BASE+376)
 #define __NR_process_vm_writev		(__NR_SYSCALL_BASE+377)
-					/* 378 for kcmp */
+#define __NR_kcmp			(__NR_SYSCALL_BASE+378)
 #define __NR_finit_module		(__NR_SYSCALL_BASE+379)
 
 /*
Index: linux-2.6.git/arch/arm/kernel/calls.S
===================================================================
--- linux-2.6.git.orig/arch/arm/kernel/calls.S
+++ linux-2.6.git/arch/arm/kernel/calls.S
@@ -387,7 +387,7 @@ 
 /* 375 */	CALL(sys_setns)
 		CALL(sys_process_vm_readv)
 		CALL(sys_process_vm_writev)
-		CALL(sys_ni_syscall)	/* reserved for sys_kcmp */
+		CALL(sys_kcmp)
 		CALL(sys_finit_module)
 #ifndef syscalls_counted
 .equ syscalls_padding, ((NR_syscalls + 3) & ~3) - NR_syscalls