qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 5/9] linux-user: mips: Update syscall numbers to kernel 5.5 l


From: Aleksandar Markovic
Subject: Re: [PATCH 5/9] linux-user: mips: Update syscall numbers to kernel 5.5 level
Date: Thu, 13 Feb 2020 02:26:32 +0100

On Wed, Feb 12, 2020 at 10:10 PM Laurent Vivier <address@hidden> wrote:
>
> Le 04/02/2020 à 13:07, Aleksandar Markovic a écrit :
> > From: Aleksandar Markovic <address@hidden>
> >
> > Update mips syscall numbers based on Linux kernel tag v5.5.
> >
> > CC: Aurelien Jarno <address@hidden>
> > CC: Aleksandar Rikalo <address@hidden>
> > Signed-off-by: Aleksandar Markovic <address@hidden>
> > ---
> >  linux-user/mips/cpu_loop.c     | 78 
> > +++++++++++++++++++++++++++++++++++++++++-
> >  linux-user/mips/syscall_nr.h   | 45 ++++++++++++++++++++++++
> >  linux-user/mips64/syscall_nr.h | 13 +++++++
> >  3 files changed, 135 insertions(+), 1 deletion(-)
> >
> > diff --git a/linux-user/mips/cpu_loop.c b/linux-user/mips/cpu_loop.c
> > index 39915b3..b81479b 100644
> > --- a/linux-user/mips/cpu_loop.c
> > +++ b/linux-user/mips/cpu_loop.c
> > @@ -25,8 +25,9 @@
> >  #include "internal.h"
> >
> >  # ifdef TARGET_ABI_MIPSO32
> > +#  define MIPS_SYSCALL_NUMBER_UNUSED -1
> >  #  define MIPS_SYS(name, args) args,
> > -static const uint8_t mips_syscall_args[] = {
> > +static const int8_t mips_syscall_args[] = {
> >          MIPS_SYS(sys_syscall    , 8)    /* 4000 */
> >          MIPS_SYS(sys_exit       , 1)
> >          MIPS_SYS(sys_fork       , 0)
> > @@ -390,6 +391,75 @@ static const uint8_t mips_syscall_args[] = {
> >          MIPS_SYS(sys_copy_file_range, 6) /* 360 */
> >          MIPS_SYS(sys_preadv2, 6)
> >          MIPS_SYS(sys_pwritev2, 6)
> > +        MIPS_SYS(sys_pkey_mprotect, 4)
> > +        MIPS_SYS(sys_pkey_alloc, 2)
> > +        MIPS_SYS(sys_pkey_free, 1)                 /* 365 */
> > +        MIPS_SYS(sys_statx, 5)
> > +        MIPS_SYS(sys_rseq, 4)
> > +        MIPS_SYS(sys_io_pgetevents, 6)
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,                /* 370 */
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,                /* 375 */
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,                /* 380 */
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,                /* 385 */
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,                /* 390 */
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYSCALL_NUMBER_UNUSED,
> > +        MIPS_SYS(sys_semget, 3)
> > +        MIPS_SYS(sys_semctl, 4)
> > +        MIPS_SYS(sys_shmget, 3)                    /* 395 */
> > +        MIPS_SYS(sys_shmctl, 3)
> > +        MIPS_SYS(sys_shmat, 3)
> > +        MIPS_SYS(sys_shmdt, 1)
> > +        MIPS_SYS(sys_msgget, 2)
> > +        MIPS_SYS(sys_msgsnd, 4)                    /* 400 */
> > +        MIPS_SYS(sys_msgrcv, 5)
> > +        MIPS_SYS(sys_msgctl, 3)
> > +        MIPS_SYS(sys_timer_gettime64, 2)
> > +        MIPS_SYS(sys_timer_settime64, 4)
> > +        MIPS_SYS(sys_timerfd_gettime64, 2)         /* 410 */
> > +        MIPS_SYS(sys_timerfd_settime64, 4)
>
> These have different names in the kernel:
>
> clock_gettime64                  403
> clock_settime64                  404
> clock_adjtime64                  405
> clock_getres_time64              406
> clock_nanosleep_time64           407
>

In fact, the whole range 403 - 407 was missing, names are good for
408-411, and all following them.

> > +        MIPS_SYS(sys_utimensat_time64, 4)
> > +        MIPS_SYS(sys_pselect6_time64, 6)
> > +        MIPS_SYS(sys_ppoll_time64, 5)
> > +        MIPS_SYSCALL_NUMBER_UNUSED,                /* 415 */
> > +        MIPS_SYS(sys_io_pgetevents_time64, 6)
> > +        MIPS_SYS(sys_recvmmsg_time64, 5)
> > +        MIPS_SYS(sys_mq_timedsend_time64, 5)
> > +        MIPS_SYS(sys_mq_timedreceive_time64, 5)
> > +        MIPS_SYS(sys_semtimedop_time64, 4)         /* 420 */
> > +        MIPS_SYS(sys_rt_sigtimedwait_time64, 4)
> > +        MIPS_SYS(sys_futex_time64, 6)
> > +        MIPS_SYS(sys_sched_rr_get_interval_time64, 2)
> > +        MIPS_SYS(sys_pidfd_send_signal, 4)
> > +        MIPS_SYS(sys_io_uring_setup, 2)            /* 425 */
> > +        MIPS_SYS(sys_io_uring_enter, 6)
> > +        MIPS_SYS(sys_io_uring_register, 4)
> > +        MIPS_SYS(sys_open_tree, 3)
> > +        MIPS_SYS(sys_move_mount, 5)
> > +        MIPS_SYS(sys_fsopen, 2)                    /* 430 */
> > +        MIPS_SYS(sys_fsconfig, 5)
> > +        MIPS_SYS(sys_fsmount, 3)
> > +        MIPS_SYS(sys_fspick, 3)
> > +        MIPS_SYS(sys_pidfd_open, 2)
> > +        MIPS_SYS(sys_clone3, 2)                    /* 435 */
>
> This comes from arch/mips/kernel/syscalls/syscall_o32.tbl for the
> syscall number.
>
> Where do you find the number of the arguments in the kernel?
>

>From strace repository:

https://github.com/strace/strace/blob/master/linux/syscallent-common.h
https://github.com/strace/strace/blob/master/linux/syscallent-common-32.h

> > +
> >  };
> >  #  undef MIPS_SYS
> >  # endif /* O32 */
> > @@ -447,8 +517,14 @@ void cpu_loop(CPUMIPSState *env)
> >  # ifdef TARGET_ABI_MIPSO32
> >              syscall_num = env->active_tc.gpr[2] - 4000;
> >              if (syscall_num >= sizeof(mips_syscall_args)) {
> > +                /* syscall_num is larger that any defined for MIPS O32 */
> > +                ret = -TARGET_ENOSYS;
> > +            } else if (mips_syscall_args[syscall_num] ==
> > +                       MIPS_SYSCALL_NUMBER_UNUSED) {
> > +                /* syscall_num belongs to the range not defined for MIPS 
> > O32 */
> >                  ret = -TARGET_ENOSYS;
> >              } else {
> > +                /* syscall_num is valid */
> >                  int nb_args;
> >                  abi_ulong sp_reg;
> >                  abi_ulong arg5 = 0, arg6 = 0, arg7 = 0, arg8 = 0;
> > diff --git a/linux-user/mips/syscall_nr.h b/linux-user/mips/syscall_nr.h
> > index 7fa7fa5..0be3af1 100644
> > --- a/linux-user/mips/syscall_nr.h
> > +++ b/linux-user/mips/syscall_nr.h
> > @@ -376,5 +376,50 @@
> >  #define TARGET_NR_statx                 (TARGET_NR_Linux + 366)
> >  #define TARGET_NR_rseq                  (TARGET_NR_Linux + 367)
> >  #define TARGET_NR_io_pgetevents         (TARGET_NR_Linux + 368)
> > +/* room for arch specific calls */
> > +#define TARGET_NR_semget                (TARGET_NR_Linux + 393)
> > +#define TARGET_NR_semctl                (TARGET_NR_Linux + 394)
> > +#define TARGET_NR_shmget                (TARGET_NR_Linux + 395)
> > +#define TARGET_NR_shmctl                (TARGET_NR_Linux + 396)
> > +#define TARGET_NR_shmat                 (TARGET_NR_Linux + 397)
> > +#define TARGET_NR_shmdt                 (TARGET_NR_Linux + 398)
> > +#define TARGET_NR_msgget                (TARGET_NR_Linux + 399)
> > +#define TARGET_NR_msgsnd                (TARGET_NR_Linux + 400)
> > +#define TARGET_NR_msgrcv                (TARGET_NR_Linux + 401)
> > +#define TARGET_NR_msgctl                (TARGET_NR_Linux + 402)
> > +/* 403-423 common for 32-bit archs */
> > +#define TARGET_NR_clock_gettime64              (TARGET_NR_Linux + 403)
> > +#define TARGET_NR_clock_settime64              (TARGET_NR_Linux + 404)
> > +#define TARGET_NR_clock_adjtime64              (TARGET_NR_Linux + 405)
> > +#define TARGET_NR_clock_getres_time64          (TARGET_NR_Linux + 406)
> > +#define TARGET_NR_clock_nanosleep_time64       (TARGET_NR_Linux + 407)
> > +#define TARGET_NR_timer_gettime64              (TARGET_NR_Linux + 408)
> > +#define TARGET_NR_timer_settime64              (TARGET_NR_Linux + 409)
> > +#define TARGET_NR_timerfd_gettime64            (TARGET_NR_Linux + 410)
> > +#define TARGET_NR_timerfd_settime64            (TARGET_NR_Linux + 411)
> > +#define TARGET_NR_utimensat_time64             (TARGET_NR_Linux + 412)
> > +#define TARGET_NR_pselect6_time64              (TARGET_NR_Linux + 413)
> > +#define TARGET_NR_ppoll_time64                 (TARGET_NR_Linux + 414)
> > +#define TARGET_NR_io_pgetevents_time64         (TARGET_NR_Linux + 416)
> > +#define TARGET_NR_recvmmsg_time64              (TARGET_NR_Linux + 417)
> > +#define TARGET_NR_mq_timedsend_time64          (TARGET_NR_Linux + 418)
> > +#define TARGET_NR_mq_timedreceive_time64       (TARGET_NR_Linux + 419)
> > +#define TARGET_NR_semtimedop_time64            (TARGET_NR_Linux + 420)
> > +#define TARGET_NR_rt_sigtimedwait_time64       (TARGET_NR_Linux + 421)
> > +#define TARGET_NR_futex_time64                 (TARGET_NR_Linux + 422)
> > +#define TARGET_NR_sched_rr_get_interval_time64 (TARGET_NR_Linux + 423)
> > +/* 424 onwards common for all archs */
> > +#define TARGET_NR_pidfd_send_signal            (TARGET_NR_Linux + 424)
> > +#define TARGET_NR_io_uring_setup               (TARGET_NR_Linux + 425)
> > +#define TARGET_NR_io_uring_enter               (TARGET_NR_Linux + 426)
> > +#define TARGET_NR_io_uring_register            (TARGET_NR_Linux + 427)
> > +#define TARGET_NR_open_tree                    (TARGET_NR_Linux + 428)
> > +#define TARGET_NR_move_mount                   (TARGET_NR_Linux + 429)
> > +#define TARGET_NR_fsopen                       (TARGET_NR_Linux + 430)
> > +#define TARGET_NR_fsconfig                     (TARGET_NR_Linux + 431)
> > +#define TARGET_NR_fsmount                      (TARGET_NR_Linux + 432)
> > +#define TARGET_NR_fspick                       (TARGET_NR_Linux + 433)
> > +#define TARGET_NR_pidfd_open                   (TARGET_NR_Linux + 434)
> > +#define TARGET_NR_clone3                       (TARGET_NR_Linux + 435)
>
> These values come from linux/arch/mips/kernel/syscalls/syscall_o32.tbl
>
> >  #endif
> > diff --git a/linux-user/mips64/syscall_nr.h b/linux-user/mips64/syscall_nr.h
> > index db40f69..c86943a 100644
> > --- a/linux-user/mips64/syscall_nr.h
> > +++ b/linux-user/mips64/syscall_nr.h
> > @@ -674,6 +674,19 @@
> >  #define TARGET_NR_statx                 (TARGET_NR_Linux + 326)
> >  #define TARGET_NR_rseq                  (TARGET_NR_Linux + 327)
> >  #define TARGET_NR_io_pgetevents         (TARGET_NR_Linux + 328)
> > +/* 329 through 423 are reserved to sync up with other architectures */
> > +#define TARGET_NR_pidfd_send_signal     (TARGET_NR_Linux + 424)
> > +#define TARGET_NR_io_uring_setup        (TARGET_NR_Linux + 425)
> > +#define TARGET_NR_io_uring_enter        (TARGET_NR_Linux + 426)
> > +#define TARGET_NR_io_uring_register     (TARGET_NR_Linux + 427)
> > +#define TARGET_NR_open_tree             (TARGET_NR_Linux + 428)
> > +#define TARGET_NR_move_mount            (TARGET_NR_Linux + 429)
> > +#define TARGET_NR_fsopen                (TARGET_NR_Linux + 430)
> > +#define TARGET_NR_fsconfig              (TARGET_NR_Linux + 431)
> > +#define TARGET_NR_fsmount               (TARGET_NR_Linux + 432)
> > +#define TARGET_NR_fspick                (TARGET_NR_Linux + 433)
> > +#define TARGET_NR_pidfd_open            (TARGET_NR_Linux + 434)
> > +#define TARGET_NR_clone3                (TARGET_NR_Linux + 435)
> >  #endif
>
> These values come from linux/arch/mips/kernel/syscalls/syscall_n64.tbl
>
> So why don't you update the value for
> arch/mips/kernel/syscalls/syscall_n32.tbl (also in mips64/syscall_nr.h,
> enclosed in TARGET_ABI32)?
>
> clock_gettime64                  403
> clock_settime64                  404
> clock_adjtime64                  405
> clock_getres_time64              406
> clock_nanosleep_time64           407
> timer_gettime64                  408
> timer_settime64                  409
> timerfd_gettime64                410
> timerfd_settime64                411
> utimensat_time64                 412
> pselect6_time64                  413
> ppoll_time64                     414
> io_pgetevents_time64             416
> recvmmsg_time64                  417
> mq_timedsend_time64              418
> mq_timedreceive_time64           419
> semtimedop_time64                420
> rt_sigtimedwait_time64           421
> futex_time64                     422
> sched_rr_get_interval_time64     423
> pidfd_send_signal                424
> io_uring_setup                   425
> io_uring_enter                   426
> io_uring_register                427
> open_tree                        428
> move_mount                       429
> fsopen                           430
> fsconfig                         431
> fsmount                          432
> fspick                           433
> pidfd_open                       434
> clone3                           435
>

I am going to add all this.

I'll send v2 shortly.

Thanks,
Aleksandar

> Thanks,
> Laurent
>



reply via email to

[Prev in Thread] Current Thread [Next in Thread]