public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] [x86-64] Add getcpu and epoll_pwait system calls.
@ 2007-05-08 18:37 Chris Wedgwood
  2007-05-08 21:28 ` Chris Wedgwood
  2007-05-08 22:13 ` Andi Kleen
  0 siblings, 2 replies; 7+ messages in thread
From: Chris Wedgwood @ 2007-05-08 18:37 UTC (permalink / raw)
  To: linux-kernel

Signed-off-by: Chris Wedgwood <cw@f00f.org>
---
 include/asm-x86_64/unistd.h |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/include/asm-x86_64/unistd.h b/include/asm-x86_64/unistd.h
index 26e23e0..aa7d4bf 100644
--- a/include/asm-x86_64/unistd.h
+++ b/include/asm-x86_64/unistd.h
@@ -619,6 +619,10 @@ __SYSCALL(__NR_sync_file_range, sys_sync_file_range)
 __SYSCALL(__NR_vmsplice, sys_vmsplice)
 #define __NR_move_pages		279
 __SYSCALL(__NR_move_pages, sys_move_pages)
+#define __NR_getcpu		280
+__SYSCALL(__NR_getcpu, sys_getcpu)
+#define __NR_epoll_pwait	281
+__SYSCALL(__NR_epoll_pwait, sys_epoll_pwait)
 
 #ifndef __NO_STUBS
 #define __ARCH_WANT_OLD_READDIR
-- 
1.5.1.3


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] [x86-64] Add getcpu and epoll_pwait system calls.
  2007-05-08 18:37 [PATCH] [x86-64] Add getcpu and epoll_pwait system calls Chris Wedgwood
@ 2007-05-08 21:28 ` Chris Wedgwood
  2007-05-08 22:13 ` Andi Kleen
  1 sibling, 0 replies; 7+ messages in thread
From: Chris Wedgwood @ 2007-05-08 21:28 UTC (permalink / raw)
  To: linux-kernel; +Cc: Andi Kleen

On Tue, May 08, 2007 at 11:37:26AM -0700, Chris Wedgwood wrote:

> +#define __NR_getcpu		280

I see something was merged just now that uses 280.  Should I resubmit
this using 281 & 282?

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] [x86-64] Add getcpu and epoll_pwait system calls.
  2007-05-08 18:37 [PATCH] [x86-64] Add getcpu and epoll_pwait system calls Chris Wedgwood
  2007-05-08 21:28 ` Chris Wedgwood
@ 2007-05-08 22:13 ` Andi Kleen
  2007-05-08 22:16   ` Chris Wedgwood
  1 sibling, 1 reply; 7+ messages in thread
From: Andi Kleen @ 2007-05-08 22:13 UTC (permalink / raw)
  To: Chris Wedgwood; +Cc: linux-kernel

On Tue, May 08, 2007 at 11:37:26AM -0700, Chris Wedgwood wrote:
> Signed-off-by: Chris Wedgwood <cw@f00f.org>
> ---
>  include/asm-x86_64/unistd.h |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> diff --git a/include/asm-x86_64/unistd.h b/include/asm-x86_64/unistd.h
> index 26e23e0..aa7d4bf 100644
> --- a/include/asm-x86_64/unistd.h
> +++ b/include/asm-x86_64/unistd.h
> @@ -619,6 +619,10 @@ __SYSCALL(__NR_sync_file_range, sys_sync_file_range)
>  __SYSCALL(__NR_vmsplice, sys_vmsplice)
>  #define __NR_move_pages		279
>  __SYSCALL(__NR_move_pages, sys_move_pages)
> +#define __NR_getcpu		280
> +__SYSCALL(__NR_getcpu, sys_getcpu)

Nope.  There already is a getcpu vsyscall. This is not needed.

> +#define __NR_epoll_pwait	281
> +__SYSCALL(__NR_epoll_pwait, sys_epoll_pwait)


Will add thanks

-Andi

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] [x86-64] Add getcpu and epoll_pwait system calls.
  2007-05-08 22:13 ` Andi Kleen
@ 2007-05-08 22:16   ` Chris Wedgwood
  2007-05-08 22:24     ` Andi Kleen
  0 siblings, 1 reply; 7+ messages in thread
From: Chris Wedgwood @ 2007-05-08 22:16 UTC (permalink / raw)
  To: Andi Kleen; +Cc: linux-kernel

On Wed, May 09, 2007 at 12:13:48AM +0200, Andi Kleen wrote:

> Nope.  There already is a getcpu vsyscall. This is not needed.

The kbuild magic that checks for missing syscalls needs to be taught
about this then I take it?

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] [x86-64] Add getcpu and epoll_pwait system calls.
  2007-05-08 22:16   ` Chris Wedgwood
@ 2007-05-08 22:24     ` Andi Kleen
  2007-05-08 22:43       ` Chris Wedgwood
  0 siblings, 1 reply; 7+ messages in thread
From: Andi Kleen @ 2007-05-08 22:24 UTC (permalink / raw)
  To: Chris Wedgwood; +Cc: Andi Kleen, linux-kernel

On Tue, May 08, 2007 at 03:16:56PM -0700, Chris Wedgwood wrote:
> On Wed, May 09, 2007 at 12:13:48AM +0200, Andi Kleen wrote:
> 
> > Nope.  There already is a getcpu vsyscall. This is not needed.
> 
> The kbuild magic that checks for missing syscalls needs to be taught
> about this then I take it?
> 

Somehow yes.  But i'm not going to add a useless syscall just to shut it up.

-Andi

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] [x86-64] Add getcpu and epoll_pwait system calls.
  2007-05-08 22:24     ` Andi Kleen
@ 2007-05-08 22:43       ` Chris Wedgwood
  2007-05-09  8:43         ` Russell King
  0 siblings, 1 reply; 7+ messages in thread
From: Chris Wedgwood @ 2007-05-08 22:43 UTC (permalink / raw)
  To: Andi Kleen; +Cc: linux-kernel

On Wed, May 09, 2007 at 12:24:32AM +0200, Andi Kleen wrote:

> Somehow yes.  But i'm not going to add a useless syscall just to
> shut it up.

It turns out this has come up in other places.  Sam has a suggestion
on how to silence this per-arch so I'll post a patch once that change
is in.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] [x86-64] Add getcpu and epoll_pwait system calls.
  2007-05-08 22:43       ` Chris Wedgwood
@ 2007-05-09  8:43         ` Russell King
  0 siblings, 0 replies; 7+ messages in thread
From: Russell King @ 2007-05-09  8:43 UTC (permalink / raw)
  To: Chris Wedgwood; +Cc: Andi Kleen, linux-kernel

On Tue, May 08, 2007 at 03:43:22PM -0700, Chris Wedgwood wrote:
> On Wed, May 09, 2007 at 12:24:32AM +0200, Andi Kleen wrote:
> 
> > Somehow yes.  But i'm not going to add a useless syscall just to
> > shut it up.
> 
> It turns out this has come up in other places.  Sam has a suggestion
> on how to silence this per-arch so I'll post a patch once that change
> is in.

We knew about this when David and myself designed the detection
method, and a method of ignoring those calls which were never going
to be implemented on a particular architecture.

Just add:

#define __IGNORE_getcpu

to asm/unistd.h, preferably in a __KERNEL__ protected area.

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2007-05-09  8:44 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-05-08 18:37 [PATCH] [x86-64] Add getcpu and epoll_pwait system calls Chris Wedgwood
2007-05-08 21:28 ` Chris Wedgwood
2007-05-08 22:13 ` Andi Kleen
2007-05-08 22:16   ` Chris Wedgwood
2007-05-08 22:24     ` Andi Kleen
2007-05-08 22:43       ` Chris Wedgwood
2007-05-09  8:43         ` Russell King

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox