On Mon, Oct 1, 2018 at 4:18 PM Firoz Khan <firoz.khan@xxxxxxxxxx> wrote:
>
> Hi Arnd,
>
> On Fri, 28 Sep 2018 at 17:41, Arnd Bergmann <arnd@xxxxxxxx> wrote:
> >
> > On Thu, Sep 27, 2018 at 4:16 PM Firoz Khan <firoz.khan@xxxxxxxxxx> wrote:
> > >
> > > Add __NR_old_getpagesize in order to not break old user space
> > > as it is reserved for backwards compatibility with old __NR_
> > > getpagesize.
> >
> > That description doesn't really make sense here.
>
> Sure I'll some more details. But rest of the patch looks good?
> This is some I was bit confused! In header there is no entry, but
> in system call table has the entry for the system call.
I think adding some macro for it makes sense, and the
__NR_old_getpagesize name you picked is fine for that.
I think generally speaking we want the unistd.h macros to
match up with the system call entry points, and there are
currently multiple ways they diverge:
- System calls have been removed from the kernels, and the
macros in unistd.h are still there on some architectures but not
others. This is something we should clean up /after/ you are
done.
- System calls have a number but are not implemented on all
architectures because they don't make sense (e.g. pk pkey_add).
These should probably have an entry in kernel/sys_ni.c if they
don't already have one.
- System calls have an entry but no macro, like the ia64
old_getpagesize. I think if you encounter one of these,
we need to fix them up before your series so we can
complete the conversion to generated tables.
Arnd
