Re: [PATCH 2/7] wire up sys_time_change_notify() on ARM

From: Alexander Shishkin
Date: Tue Aug 31 2010 - 08:42:57 EST


On Tue, Aug 31, 2010 at 02:37:47 +0200, Mikael Pettersson wrote:
> Alexander Shishkin writes:
> > sys_time_change_notify() is a new syscall with number and types of
> > parameters such that no ARM-specific processing is needed.
> >
> > Tested with 2.6.36-rc3 using Documentation/time-change-notify-example.c.
> >
> > Signed-off-by: Alexander Shishkin <virtuoso@xxxxxxxxx>
> > CC: Russell King <linux@xxxxxxxxxxxxxxxx>
> > CC: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> > CC: David Howells <dhowells@xxxxxxxxxx>
> > CC: Christoph Hellwig <hch@xxxxxx>
> > CC: "H. Peter Anvin" <hpa@xxxxxxxxx>
> > CC: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> > CC: linux-kernel@xxxxxxxxxxxxxxx
> > ---
> > arch/arm/include/asm/unistd.h | 1 +
> > arch/arm/kernel/calls.S | 1 +
> > 2 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/include/asm/unistd.h b/arch/arm/include/asm/unistd.h
> > index d02cfb6..ce38a6f 100644
> > --- a/arch/arm/include/asm/unistd.h
> > +++ b/arch/arm/include/asm/unistd.h
> > @@ -393,6 +393,7 @@
> > #define __NR_perf_event_open (__NR_SYSCALL_BASE+364)
> > #define __NR_recvmmsg (__NR_SYSCALL_BASE+365)
> > #define __NR_accept4 (__NR_SYSCALL_BASE+366)
> > +#define __NR_time_change_notify (__NR_SYSCALL_BASE+367)
> >
> > /*
> > * The following SWIs are ARM private.
> > diff --git a/arch/arm/kernel/calls.S b/arch/arm/kernel/calls.S
> > index afeb71f..f1e64ba 100644
> > --- a/arch/arm/kernel/calls.S
> > +++ b/arch/arm/kernel/calls.S
> > @@ -376,6 +376,7 @@
> > CALL(sys_perf_event_open)
> > /* 365 */ CALL(sys_recvmmsg)
> > CALL(sys_accept4)
> > + CALL(sys_time_change_notify)
> > #ifndef syscalls_counted
> > .equ syscalls_padding, ((NR_syscalls + 3) & ~3) - NR_syscalls
> > #define syscalls_counted
>
> No, you need to use 370 as the syscall number on ARM as 367-369
> have been assigned to the fanotify and prlimit64 syscalls; see
>
> http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=6343/1

I'm aware of that, but I don't know how to synchronize this with Russell's
devel branch. This patchset applies against 2.6.36-rc3. I guess, when it
comes to merging it, I'll update it against Russell's tree.

Regards,
--
Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/