From: "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>
To: Stas Sergeev <stsp@list.ru>
Cc: mtk.manpages@gmail.com, linux-man <linux-man@vger.kernel.org>,
Andy Lutomirski <luto@amacapital.net>,
Oleg Nesterov <oleg@redhat.com>,
lkml <linux-kernel@vger.kernel.org>
Subject: Re: Documenting sigaltstack SS_AUTODISRM
Date: Mon, 30 Oct 2017 11:04:58 +0100 [thread overview]
Message-ID: <a5565692-7f23-d236-3b01-142088310741@gmail.com> (raw)
In-Reply-To: <3a4f9f3e-fc33-cf98-2322-27087664813f@list.ru>
[So, things fell on the floor, a while back.]
On 05/25/2017 11:17 AM, Stas Sergeev wrote:
> 24.05.2017 14:09, Michael Kerrisk (man-pages) пишет:
>> One could do this I suppose, but I read POSIX differently from
>> you and, more importantly, SS_ONSTACK breaks portability on
>> numerous other systems and is a no-op on Linux. So, the Linux man
>> page really should warn against its use in the strongest terms.
> So how about instead of the strongest terms towards
> the code's author, just explain that SS_ONSTACK is a
> bit-value on some/many OSes, and as such, 0 is a
> valid value to enable sas on them, plus all the other
> values would give EINVAL?
> No strongest terms will help w/o an explanation,
> because people will keep looking for something that
> suits as a missing SS_ENABLE.
Fair enough. I've removed the statement in the manual page
about "confusion". By now the page says:
BUGS
In the lead up to the release of the Linux 2.4 kernel, a change
was made to allow sigaltstack() to accept SS_ONSTACK in
ss.ss_flags, which results in behavior that is the same as when
ss_flags is 0 (i.e., the inclusion of SS_ONSTACK in ss.ss_flags is
a no-op). On other implementations, and according to POSIX.1,
SS_ONSTACK appears only as a reported flag in old_ss.ss_flags. On
Linux, there is no need ever to specify this flag in ss.ss_flags,
and indeed doing so should be avoided on portability grounds: var‐
ious other systems give an error if SS_ONSTACK is specified in
ss.ss_flags.
Cheers,
Michael
--
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:[~2017-10-30 10:05 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-22 20:38 Michael Kerrisk (man-pages)
2017-05-22 23:36 ` Stas Sergeev
2017-05-23 10:35 ` Michael Kerrisk (man-pages)
2017-05-23 11:03 ` Stas Sergeev
2017-05-23 12:26 ` Michael Kerrisk (man-pages)
2017-05-23 23:01 ` Stas Sergeev
2017-05-24 11:09 ` Michael Kerrisk (man-pages)
2017-05-24 20:12 ` Stas Sergeev
2017-10-30 12:38 ` Michael Kerrisk (man-pages)
2017-11-06 22:26 ` Michael Kerrisk (man-pages)
2017-11-06 22:28 ` Stas Sergeev
2017-11-08 7:41 ` Michael Kerrisk (man-pages)
2017-05-25 9:17 ` Stas Sergeev
2017-10-30 10:04 ` Michael Kerrisk (man-pages) [this message]
2017-10-30 10:21 ` walter harms
2017-10-30 10:50 ` Michael Kerrisk (man-pages)
2017-10-30 10:58 ` Stas Sergeev
2017-10-30 12:54 ` walter harms
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=a5565692-7f23-d236-3b01-142088310741@gmail.com \
--to=mtk.manpages@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-man@vger.kernel.org \
--cc=luto@amacapital.net \
--cc=oleg@redhat.com \
--cc=stsp@list.ru \
/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
all inboxes | Powered by JetHome®