All of lore.kernel.org
 help / color / mirror / Atom feed
From: Viresh Kumar <viresh.kumar@linaro.org>
To: ltp@lists.linux.it
Subject: [LTP] [PATCH v2 1/1] lap/syscalls: Update syscall numbers
Date: Thu, 27 Feb 2020 08:55:24 +0530	[thread overview]
Message-ID: <20200227032524.yldx3diebom3wqod@vireshk-i7> (raw)
In-Reply-To: <20200226232649.8736-1-pvorel@suse.cz>

On 27-02-20, 00:26, Petr Vorel wrote:
> From: Viresh Kumar <viresh.kumar@linaro.org>
> 
> to sync with the kernel v5.6-rc3.

This doesn't look right, you missed some text here ?

> As this was performed in some sort of automated way, few extra changes
> may be noticed while converting from kernel files to LTP, like a
> different format of defining syscalls or removal of obsolete syscalls.
> 
> The diff generated against sh.in file is enormous and looks like someone
> did a mistake while adding the syscall numbers there as they look to be
> mostly incorrect from 220 to 393. They are fixed as a consequence of
> this exercise.
> 
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> [ pvorel: various fixes for 32bit syscalls in 64bit arch and vice versa ]

Sorry about making these mistakes :(

> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> Changes v1->v2:
> * remove wrong umount in ia64
> * fixed typo sched_rr_get_interval_ti (=> sched_rr_get_interval_time64)
> * various fixes for 32bit syscalls in 64bit arch and vice versa
> 
> TODO: not sure about spu and nospu ABI in powerpc (whether they go to
> 32 bit or 64 bit or is ignored)
> 
> === DIFF from v1 ===
> diff --git include/lapi/syscalls/s390.in include/lapi/syscalls/s390.in
>  pkey_free 386
> -semtimedop 392

Why is this removed ?

> diff --git include/lapi/syscalls/s390x.in include/lapi/syscalls/s390x.in
> index dfb7ee811..88cc9b86b 100644
> --- include/lapi/syscalls/s390x.in
> +++ include/lapi/syscalls/s390x.in
> @@ -10,17 +10,12 @@ link 9
>  unlink 10
>  execve 11
>  chdir 12
> -time 13

And the ones like these..

Because they are only for 32 bit arch and not 64 ?

> diff --git include/lapi/syscalls/sparc.in include/lapi/syscalls/sparc.in
> -sched_rr_get_interval_ti423
> +sched_rr_get_interval_time64 423

Really bad that I made these mistakes with my vim tricks.

-- 
viresh

  reply	other threads:[~2020-02-27  3:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-26 23:26 [LTP] [PATCH v2 1/1] lap/syscalls: Update syscall numbers Petr Vorel
2020-02-27  3:25 ` Viresh Kumar [this message]
2020-02-27  6:44   ` Petr Vorel
2020-02-27  8:03     ` Viresh Kumar
2020-02-28  6:49 ` Petr Vorel
2020-02-28  6:52 ` Petr Vorel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200227032524.yldx3diebom3wqod@vireshk-i7 \
    --to=viresh.kumar@linaro.org \
    --cc=ltp@lists.linux.it \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.