From: "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>
To: "Alejandro Colomar (man-pages)" <alx.manpages@gmail.com>
Cc: mtk.manpages@gmail.com, linux-man <linux-man@vger.kernel.org>
Subject: Re: sigprocmask.2: s/kernel_sigset_t/sigset_t/
Date: Sat, 13 Feb 2021 17:05:15 +0100 [thread overview]
Message-ID: <89b4b88d-a5e1-345e-b2c8-3d9479177e22@gmail.com> (raw)
In-Reply-To: <cd449862-373b-c058-6abf-fd4b4efc8acf@gmail.com>
Hi Alex,
On 2/12/21 4:28 PM, Alejandro Colomar (man-pages) wrote:
> Hi Michael,
>
> AFAICT, the syscalls [rt_]sigprocmask use '[old_]sigset_t' (there's the
> '__user' modifier too), so 'kernel_' should be removed from the types,
> shouldn't it?
Looks like you're right. Will you send a patch?
Thanks,
Michael
> [[
> SYNOPSIS
> #include <signal.h>
>
> /* Prototype for the glibc wrapper function */
> int sigprocmask(int how, const sigset_t *restrict set,
> sigset_t *restrict oldset);
>
> /* Prototype for the underlying system call */
> int rt_sigprocmask(int how, const kernel_sigset_t *set,
> kernel_sigset_t *oldset, size_t sigsetsize);
>
> /* Prototype for the legacy system call (deprecated) */
> int sigprocmask(int how, const old_kernel_sigset_t *set,
> old_kernel_sigset_t *oldset);
>
> Feature Test Macro Requirements for glibc (see fea‐
> ture_test_macros(7)):
>
> sigprocmask():
> _POSIX_C_SOURCE
>
> ]]
>
> .../linux$ grep_syscall rt_sigprocmask
> kernel/signal.c:3050:
> SYSCALL_DEFINE4(rt_sigprocmask, int, how, sigset_t __user *, nset,
> sigset_t __user *, oset, size_t, sigsetsize)
> kernel/signal.c:3081:
> COMPAT_SYSCALL_DEFINE4(rt_sigprocmask, int, how, compat_sigset_t __user
> *, nset,
> compat_sigset_t __user *, oset, compat_size_t, sigsetsize)
> include/linux/compat.h:698:
> asmlinkage long compat_sys_rt_sigprocmask(int how, compat_sigset_t
> __user *set,
> compat_sigset_t __user *oset,
> compat_size_t sigsetsize);
> include/linux/syscalls.h:696:
> asmlinkage long sys_rt_sigprocmask(int how, sigset_t __user *set,
> sigset_t __user *oset, size_t sigsetsize);
> .../linux$ grep_syscall sigprocmask
> kernel/compat.c:40:
> COMPAT_SYSCALL_DEFINE3(sigprocmask, int, how,
> compat_old_sigset_t __user *, nset,
> compat_old_sigset_t __user *, oset)
> kernel/signal.c:4236:
> SYSCALL_DEFINE3(sigprocmask, int, how, old_sigset_t __user *, nset,
> old_sigset_t __user *, oset)
> include/linux/compat.h:882:
> asmlinkage long compat_sys_sigprocmask(int how, compat_old_sigset_t
> __user *nset,
> compat_old_sigset_t __user *oset);
> include/linux/syscalls.h:1136:
> asmlinkage long sys_sigprocmask(int how, old_sigset_t __user *set,
> old_sigset_t __user *oset);
> .../linux$
>
>
>
--
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
next prev parent reply other threads:[~2021-02-13 16:06 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-12 15:28 sigprocmask.2: s/kernel_sigset_t/sigset_t/ Alejandro Colomar (man-pages)
2021-02-13 16:05 ` Michael Kerrisk (man-pages) [this message]
2021-02-13 16:41 ` Alejandro Colomar (man-pages)
2021-02-13 23:30 ` Alejandro Colomar (man-pages)
2021-02-14 8:42 ` Michael Kerrisk (man-pages)
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=89b4b88d-a5e1-345e-b2c8-3d9479177e22@gmail.com \
--to=mtk.manpages@gmail.com \
--cc=alx.manpages@gmail.com \
--cc=linux-man@vger.kernel.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