MIME-Version: 1.0
In-Reply-To: <55CD2F9A.9040300@list.ru>
References: <55CA90B4.2010205@list.ru>
	<CA+55aFw5DzEn+jYmY0KwbkytBzowo45o3Czh1cQZ_aaR1+WZYA@mail.gmail.com>
	<55CCD921.4040301@list.ru>
	<CA+55aFycGSrN27gP4kh6eLZdkSxFvVqS8cZn1YJairD3+z3Zpg@mail.gmail.com>
	<CALCETrUDYw3oobs27QNraCQo8z3eR-3xoBmQE-fQkGvh=Ax0sA@mail.gmail.com>
	<CA+55aFz_KF5p0LcxroCVJ5zPrUmthR6X0Q1YWKMepDYKn=J-sA@mail.gmail.com>
	<20150813200823.GS2059@uranus>
	<CA+55aFyYNgPoh0VxqcGUxB-_z7oK+zB18YhJQQrozdb_gLAp5g@mail.gmail.com>
	<55CD0F29.4070604@gmail.com>
	<CA+55aFyqb29r+1YP2iYStFOij9X0QDWqjgmee=JcTkB-q8iEdA@mail.gmail.com>
	<55CD13F3.1070904@list.ru>
	<CALCETrVX-Op1U-RED9bkb3R9v7jcu5O1ur1Yt08ppZ0VFyZx7A@mail.gmail.com>
	<55CD1968.7070002@list.ru>
	<CALCETrUJS9kgNgggbnYJrLr93ARtocZBAGDVuudWahM7JhQuwQ@mail.gmail.com>
	<55CD1F79.2010508@list.ru>
	<CALCETrWf=rRZLJ=RWS_s2NDN5GNZT87DKTgt7eS81ncrXxi5hA@mail.gmail.com>
	<55CD2F9A.9040300@list.ru>
Date: Thu, 13 Aug 2015 17:08:21 -0700
Message-ID: <CA+55aFzrKQm1vxLgnxvHtmmmVQ2L7E15bVkCRsr1FZ-jrdfnXg@mail.gmail.com>
Subject: Re: [regression] x86/signal/64: Fix SS handling for signals delivered
 to 64-bit programs breaks dosemu
From: Linus Torvalds <torvalds@linux-foundation.org>
To: Stas Sergeev <stsp@list.ru>
Cc: Andy Lutomirski <luto@amacapital.net>,
        Raymond Jennings <shentino@gmail.com>,
        Cyrill Gorcunov <gorcunov@gmail.com>,
        Pavel Emelyanov <xemul@parallels.com>,
        Linux kernel <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1489
Lines: 36

On Thu, Aug 13, 2015 at 5:00 PM, Stas Sergeev <stsp@list.ru> wrote:
> 14.08.2015 02:00, Andy Lutomirski пишет:
>>
>> DOSEMU, when you set that flag, WRFSBASE gets enabled, and glibc's
>> threading library starts using WRFSBASE instead of arch_prctl.
>
> Hmm, how about the following:
>
> prctl(ARCH_SET_SIGNAL_FS, my_tls)
> If my_tls==NULL - use current fsbase (including one of WRFSBASE).
> If my_tls==(void)-1 - don't restore.
>
> Can this work?

I'm really inclined to wonder whether we need the change and such a flag at all.

Basically, no _normal_ application will ever play with segments at all
on x86-64. So our current behavior of not touching any segments at all
for signal handling would seem to be the right thing to do - because
it handles all the sane cases optimally.

And applications that *do* play with segments very much know they do
so, and we already put the onus on *them* to save/restore segments.
That's how dosemu clearly works today.

So why not just keep to that policy?  It has worked fairly well so
far. Only when we tried to change that policy did we hit these
problems, because existing applications obviously already live with
what we do (or rather, what we _don't_ do) right now...

                 Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/