From: Yury Norov <ynorov@caviumnetworks.com>
To: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Cc: <libc-alpha@sourceware.org>
Subject: Re: [PATCH v2 01/16] Add __ASSUME_SYSVIPC_SYSCALL for Linux
Date: Thu, 03 Nov 2016 17:07:00 -0000 [thread overview]
Message-ID: <20161103170641.GA31761@yury-N73SV> (raw)
In-Reply-To: <1478114813-3526-2-git-send-email-adhemerval.zanella@linaro.org>
On Wed, Nov 02, 2016 at 05:26:38PM -0200, Adhemerval Zanella wrote:
> Changes from previous version:
>
> - Use __ASSUME_SYSVIPC_SYSCALL instead of __NR_syscall to issue the
> wired syscall or the ipc one.
>
> --
>
> On current minimum supported kernels, the SysV IPC on Linux is provided
> by either the ipc syscalls or correspondent wire syscalls. Also, for
> architectures that supports wire syscalls all syscalls are supported
> in a set (msgct, msgrcv, msgsnd, msgget, semctl, semget, semop, semtimedop,
> shmctl, shmat, shmget, shmdt).
>
> The architectures that only supports ipc syscall are:
>
> - i386, m68k, microblaze, mips32, powerpc (powerpc32, powerpc64, and
> powerpc64le), s390 (32 and 64 bits), sh, sparc32, and sparc64.
>
> And the architectures that only supports wired syscalls are:
>
> - aarch64, alpha, hppa, ia64, mips64, mips64n32, nios2, tile
> (tilepro, tilegx, and tilegx64), and x86_64
>
> Also arm is the only one that supports both wire syscalls and the
> ipc, although the ipc one is deprecated.
>
> This patch adds a new define, __ASSUME_SYSVIPC_SYSCALL, that wired
> syscalls are supported on the system and the general idea is to use
> it where possible.
>
> I also checked the syscall table for all architectures on Linux 4.9
> and there is no change on described support for Linux 2.6.32/3.2.
>
> * sysdeps/unix/sysv/linux/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): New define.
> * sysdeps/unix/sysv/linux/i386/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): Undef.
> * sysdeps/unix/sysv/linux/m68k/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): Likewise.
> * sysdeps/unix/sysv/linux/mips/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): Likewise.
> * sysdeps/unix/sysv/linux/powerpc/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): Likewise.
> * sysdeps/unix/sysv/linux/s390/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): Likewise.
> * sysdeps/unix/sysv/linux/sh/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): Likewise.
> * sysdeps/unix/sysv/linux/sparc/kernel-features.h
> (__ASSUME_SYSVIPC_SYSCALL): Likewise.
> ---
> ChangeLog | 19 +++++++++++++++++++
> sysdeps/unix/sysv/linux/i386/kernel-features.h | 3 +++
> sysdeps/unix/sysv/linux/kernel-features.h | 4 ++++
> sysdeps/unix/sysv/linux/m68k/kernel-features.h | 3 +++
> sysdeps/unix/sysv/linux/mips/kernel-features.h | 2 ++
> sysdeps/unix/sysv/linux/powerpc/kernel-features.h | 3 +++
> sysdeps/unix/sysv/linux/s390/kernel-features.h | 3 +++
> sysdeps/unix/sysv/linux/sh/kernel-features.h | 3 +++
> sysdeps/unix/sysv/linux/sparc/kernel-features.h | 3 +++
> 9 files changed, 43 insertions(+)
>
> diff --git a/sysdeps/unix/sysv/linux/i386/kernel-features.h b/sysdeps/unix/sysv/linux/i386/kernel-features.h
> index 148963c..3da13d4 100644
> --- a/sysdeps/unix/sysv/linux/i386/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/i386/kernel-features.h
> @@ -52,3 +52,6 @@
> # undef __ASSUME_SENDMSG_SYSCALL
> # undef __ASSUME_RECVMSG_SYSCALL
> #endif
> +
> +/* i686 only supports ipc syscall. */
> +#undef __ASSUME_SYSVIPC_SYSCALL
> diff --git a/sysdeps/unix/sysv/linux/kernel-features.h b/sysdeps/unix/sysv/linux/kernel-features.h
> index 1d3b554..cb0bc28 100644
> --- a/sysdeps/unix/sysv/linux/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/kernel-features.h
> @@ -147,3 +147,7 @@
> separate syscalls were only added later. */
> #define __ASSUME_SENDMSG_SYSCALL 1
> #define __ASSUME_RECVMSG_SYSCALL 1
> +
> +/* Support for SysV IPC through wired syscalls. All supported architectures
> + either support ipc syscall and/or all the ipc correspondent syscalls. */
> +#define __ASSUME_SYSVIPC_SYSCALL 1
> diff --git a/sysdeps/unix/sysv/linux/m68k/kernel-features.h b/sysdeps/unix/sysv/linux/m68k/kernel-features.h
> index 46ec601..f0fd7ce 100644
> --- a/sysdeps/unix/sysv/linux/m68k/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/m68k/kernel-features.h
> @@ -54,3 +54,6 @@
> # undef __ASSUME_REQUEUE_PI
> # undef __ASSUME_SET_ROBUST_LIST
> #endif
> +
> +/* m68k only supports ipc syscall. */
> +#undef __ASSUME_SYSVIPC_SYSCALL
> diff --git a/sysdeps/unix/sysv/linux/mips/kernel-features.h b/sysdeps/unix/sysv/linux/mips/kernel-features.h
> index b486d90..5e88c8e 100644
> --- a/sysdeps/unix/sysv/linux/mips/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/mips/kernel-features.h
> @@ -32,6 +32,8 @@
> pairs to start with an even-number register. */
> #if _MIPS_SIM == _ABIO32
> # define __ASSUME_ALIGNED_REGISTER_PAIRS 1
> +/* mips32 only supports ipc syscall. */
> +# undef __ASSUME_SYSVIPC_SYSCALL
> #endif
>
> /* Define that mips64-n32 is a ILP32 ABI to set the correct interface to
> diff --git a/sysdeps/unix/sysv/linux/powerpc/kernel-features.h b/sysdeps/unix/sysv/linux/powerpc/kernel-features.h
> index 42a53f2..a81caf3 100644
> --- a/sysdeps/unix/sysv/linux/powerpc/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/powerpc/kernel-features.h
> @@ -52,3 +52,6 @@
> #endif
>
> #include_next <kernel-features.h>
> +
> +/* powerpc only supports ipc syscall. */
> +#undef __ASSUME_SYSVIPC_SYSCALL
> diff --git a/sysdeps/unix/sysv/linux/s390/kernel-features.h b/sysdeps/unix/sysv/linux/s390/kernel-features.h
> index b3edee4..8fd7853 100644
> --- a/sysdeps/unix/sysv/linux/s390/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/s390/kernel-features.h
> @@ -48,3 +48,6 @@
> # undef __ASSUME_SENDMSG_SYSCALL
> # undef __ASSUME_RECVMSG_SYSCALL
> #endif
> +
Trailing whitespace
> +/* s390 only supports ipc syscall. */
> +#undef __ASSUME_SYSVIPC_SYSCALL
> diff --git a/sysdeps/unix/sysv/linux/sh/kernel-features.h b/sysdeps/unix/sysv/linux/sh/kernel-features.h
> index c5240fa..cb09603 100644
> --- a/sysdeps/unix/sysv/linux/sh/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/sh/kernel-features.h
> @@ -49,3 +49,6 @@
> the kernel interface for p{read,write}64 adds a dummy long argument
> before the offset. */
> #define __ASSUME_PRW_DUMMY_ARG 1
> +
> +/* sh only supports ipc syscall. */
> +#undef __ASSUME_SYSVIPC_SYSCALL
> diff --git a/sysdeps/unix/sysv/linux/sparc/kernel-features.h b/sysdeps/unix/sysv/linux/sparc/kernel-features.h
> index 69c9c7c..20578bb 100644
> --- a/sysdeps/unix/sysv/linux/sparc/kernel-features.h
> +++ b/sysdeps/unix/sysv/linux/sparc/kernel-features.h
> @@ -37,3 +37,6 @@
> # undef __ASSUME_REQUEUE_PI
> # undef __ASSUME_SET_ROBUST_LIST
> #endif
> +
> +/* sparc only supports ipc syscall. */
> +#undef __ASSUME_SYSVIPC_SYSCALL
> --
> 2.7.4
next prev parent reply other threads:[~2016-11-03 17:07 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-02 19:27 [PATCH v2 00/16] Consolidate Linux sysvipc implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 16/16] Add SYSV shared memory test Adhemerval Zanella
2016-11-03 17:14 ` Yury Norov
2016-11-02 19:27 ` [PATCH v2 14/16] Use shmdt syscall for linux implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 03/16] Consolidate Linux msgrcv implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 07/16] Consolidate Linux semctl implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 08/16] Use semget syscall for Linux implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 06/16] Add SYSV message queue test Adhemerval Zanella
2016-11-03 17:09 ` Yury Norov
2016-11-03 20:12 ` Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 13/16] Consolidate Linux shmctl implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 12/16] Use shmat syscall for Linux implementation Adhemerval Zanella
2016-11-07 11:02 ` Arnd Bergmann
2016-11-07 13:10 ` Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 04/16] Use msgsnd " Adhemerval Zanella
2016-11-03 14:15 ` Yury Norov
2016-11-03 20:27 ` Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 01/16] Add __ASSUME_SYSVIPC_SYSCALL for Linux Adhemerval Zanella
2016-11-02 21:10 ` Joseph Myers
2016-11-02 22:27 ` Adhemerval Zanella
2016-11-02 22:34 ` Adhemerval Zanella
2016-11-03 17:07 ` Yury Norov [this message]
2016-11-03 20:13 ` Adhemerval Zanella
2016-11-07 11:28 ` Arnd Bergmann
2016-11-07 13:17 ` Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 02/16] Consolidate Linux msgctl implementation Adhemerval Zanella
2016-11-04 15:33 ` Yury Norov
2016-11-04 17:03 ` Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 05/16] Use msgget syscall for Linux implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 11/16] Add SYSV semaphore test Adhemerval Zanella
2016-11-03 17:10 ` Yury Norov
2016-11-03 20:11 ` Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 09/16] Use semop syscall for Linux implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 10/16] Consolidate Linux semtimedop implementation Adhemerval Zanella
2016-11-02 19:27 ` [PATCH v2 15/16] Use shmget syscall for linux implementation Adhemerval Zanella
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=20161103170641.GA31761@yury-N73SV \
--to=ynorov@caviumnetworks.com \
--cc=adhemerval.zanella@linaro.org \
--cc=libc-alpha@sourceware.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).