Re: [PATCH 17/22] 2.6.22-rc3 perfmon2 : modified powerpc files

[David Rientjes <rientjes@google.com>]

On Tue, 5 Jun 2007, Stephane Eranian wrote:

> > > diff --exclude=.git -urp linux-2.6.22.base/include/asm-powerpc/unistd.h linux-2.6.22/include/asm-powerpc/unistd.h
> > > --- linux-2.6.22.base/include/asm-powerpc/unistd.h	2007-05-29 03:17:57.000000000 -0700
> > > +++ linux-2.6.22/include/asm-powerpc/unistd.h	2007-05-29 03:24:14.000000000 -0700
> > > @@ -330,10 +330,22 @@
> > >  #define __NR_signalfd		305
> > >  #define __NR_timerfd		306
> > >  #define __NR_eventfd		307
> > > +#define __NR_pfm_create_context	308
> > > +#define __NR_pfm_write_pmcs	(__NR_pfm_create_context+1)
> > > +#define __NR_pfm_write_pmds	(__NR_pfm_create_context+2)
> > > +#define __NR_pfm_read_pmds	(__NR_pfm_create_context+3)
> > > +#define __NR_pfm_load_context	(__NR_pfm_create_context+4)
> > > +#define __NR_pfm_start		(__NR_pfm_create_context+5)
> > > +#define __NR_pfm_stop		(__NR_pfm_create_context+6)
> > > +#define __NR_pfm_restart	(__NR_pfm_create_context+7)
> > > +#define __NR_pfm_create_evtsets	(__NR_pfm_create_context+8)
> > > +#define __NR_pfm_getinfo_evtsets (__NR_pfm_create_context+9)
> > > +#define __NR_pfm_delete_evtsets (__NR_pfm_create_context+10)
> > > +#define __NR_pfm_unload_context	(__NR_pfm_create_context+11)
> > >  
> > >  #ifdef __KERNEL__
> > >  
> > > -#define __NR_syscalls		308
> > > +#define __NR_syscalls		319
> > >  
> > 
> > The highest numbered syscall, __NR_pfm_create_context+11, is 319 and 
> > __NR_syscalls is set to the same, which is wrong.
> > 
> You mean, it is off by one, right? Should be 320, then.
> 

Yes, and the same off by one problem exists in your i386 patch that adds 
the new syscalls as well.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/