From: Cyrill Gorcunov <gorcunov@gmail.com>
To: Denys Vlasenko <vda.linux@googlemail.com>
Cc: Andy Lutomirski <luto@amacapital.net>,
Pavel Emelyanov <xemul@parallels.com>,
Oleg Nesterov <oleg@redhat.com>, Andrey Wagin <avagin@gmail.com>,
Andy Lutomirski <luto@kernel.org>, Ingo Molnar <mingo@kernel.org>,
Andi Kleen <andi@firstfloor.org>,
"H. Peter Anvin" <hpa@zytor.com>,
Al Viro <viro@zeniv.linux.org.uk>, X86 ML <x86@kernel.org>,
LKML <linux-kernel@vger.kernel.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
Borislav Petkov <bp@alien8.de>,
Pavel Emelyanov <xemul@openvz.org>
Subject: Re: [PATCH v3 1/2] x86_64,signal: Fix SS handling for signals delivered to 64-bit programs
Date: Fri, 20 Mar 2015 15:04:19 +0300 [thread overview]
Message-ID: <20150320120419.GS27066@moon> (raw)
In-Reply-To: <20150320115622.GR27066@moon>
On Fri, Mar 20, 2015 at 02:56:22PM +0300, Cyrill Gorcunov wrote:
> > >
> > > fwiw currently we're passing zero in this __pad0 (replying to your
> > > previous email, so we can workaround in the kernel assuming zero
> > > as a special case, not that good but better than nothing).
> >
> > Special-casing zero sounds not that bad to me.
> > It can be removed after a few years - just don't forget
> > to document it in a good comment: why we have special
> > case? What software required it?
> > In which version of that software the need to have this hack
> > was eliminated?
>
> To be fair, such special case would be ideal for us, so that
> if noone object against such hack, i would cook a patch.
Denys, note though that Andy pointed a downside for such approach as well:
| The only real down side I can see to special casing zero is that it
| really is possible to end up with zero in there. For example, the
| SIGSEGV you get do to the failed sigreturn probably has sigcontext->ss
| == 0 :)
which I don't know how to resolve.
next prev parent reply other threads:[~2015-03-20 12:04 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-12 20:57 [PATCH v3 0/2] x86_64: Sigcontext improvements Andy Lutomirski
2015-03-12 20:57 ` [PATCH v3 1/2] x86_64,signal: Fix SS handling for signals delivered to 64-bit programs Andy Lutomirski
2015-03-13 16:13 ` Borislav Petkov
2015-03-16 8:11 ` Ingo Molnar
2015-03-16 12:08 ` [tip:x86/asm] x86/signal/64: " tip-bot for Andy Lutomirski
2015-03-17 8:44 ` tip-bot for Andy Lutomirski
2015-03-18 17:19 ` [PATCH v3 1/2] x86_64,signal: " Andrey Wagin
2015-03-18 17:48 ` Oleg Nesterov
2015-03-18 18:06 ` Andy Lutomirski
2015-03-18 18:17 ` Cyrill Gorcunov
2015-03-18 18:20 ` Andy Lutomirski
2015-03-18 18:45 ` Cyrill Gorcunov
2015-03-18 18:25 ` Oleg Nesterov
2015-03-18 18:32 ` Andy Lutomirski
2015-03-18 19:13 ` Cyrill Gorcunov
2015-03-18 18:13 ` Cyrill Gorcunov
2015-03-18 18:31 ` Oleg Nesterov
2015-03-18 18:50 ` Cyrill Gorcunov
2015-03-18 19:52 ` Andrey Wagin
2015-03-18 20:02 ` Oleg Nesterov
2015-03-18 21:26 ` Andy Lutomirski
2015-03-18 21:34 ` Pavel Emelyanov
2015-03-18 22:03 ` Andy Lutomirski
2015-03-19 7:35 ` Cyrill Gorcunov
2015-03-19 16:08 ` Andy Lutomirski
2015-03-19 16:19 ` Cyrill Gorcunov
2015-03-20 11:43 ` Denys Vlasenko
2015-03-20 11:56 ` Cyrill Gorcunov
2015-03-20 12:04 ` Cyrill Gorcunov [this message]
2015-03-20 14:07 ` Oleg Nesterov
2015-03-20 14:47 ` Cyrill Gorcunov
2015-04-10 21:59 ` Andy Lutomirski
2015-04-10 22:11 ` Cyrill Gorcunov
2015-04-10 22:16 ` Andy Lutomirski
2015-04-10 22:20 ` Cyrill Gorcunov
2015-03-12 20:57 ` [PATCH v3 2/2] x86_64,signal: Remove 'fs' and 'gs' from sigcontext Andy Lutomirski
2015-03-16 12:08 ` [tip:x86/asm] x86/signal/64: " tip-bot for Andy Lutomirski
2015-03-17 8:44 ` tip-bot for Andy Lutomirski
2015-03-13 15:31 ` [PATCH v3 0/2] x86_64: Sigcontext improvements Oleg Nesterov
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=20150320120419.GS27066@moon \
--to=gorcunov@gmail.com \
--cc=andi@firstfloor.org \
--cc=avagin@gmail.com \
--cc=bp@alien8.de \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@amacapital.net \
--cc=luto@kernel.org \
--cc=mingo@kernel.org \
--cc=oleg@redhat.com \
--cc=torvalds@linux-foundation.org \
--cc=vda.linux@googlemail.com \
--cc=viro@zeniv.linux.org.uk \
--cc=x86@kernel.org \
--cc=xemul@openvz.org \
--cc=xemul@parallels.com \
/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.