public inbox for linux-man@vger.kernel.org
 help / color / mirror / Atom feed
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/

  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