Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1573871 > unrolled thread

[PATCH v2 0/2] sigaltstack: SS_AUTODISARM fixes

Back to article view | Back to linux.kernel

Contents

  [PATCH v2 0/2] sigaltstack: SS_AUTODISARM fixes Stas Sergeev <stsp@list.ru> - 2017-02-05 12:20 +0100
    Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT Andy Lutomirski <luto@amacapital.net> - 2017-02-05 19:40 +0100
      Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT Stas Sergeev <stsp@list.ru> - 2017-02-06 10:40 +0100
        Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for  CONFIG_COMPAT Andrew Morton <akpm@linux-foundation.org> - 2017-02-07 01:10 +0100
          [PATCH 2/2] tests: improve output of sigaltstack testcase Stas Sergeev <stsp@list.ru> - 2017-02-07 10:20 +0100

#1573871 — [PATCH v2 0/2] sigaltstack: SS_AUTODISARM fixes

This patch set adds the missing SS_AUTODISARM handling for
compatibility mode (CONFIG_COMPAT). It is needed for dosemu
to not crash when built with -m32.

CC: deller@gmx.de
CC: linux-kernel@vger.kernel.org
CC: Andy Lutomirski <luto@kernel.org>

Stas Sergeev (2):
  sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT
  tests: improve output of sigaltstack testcase

 include/linux/compat.h                    |  4 +++-
 kernel/signal.c                           | 11 +++++++++--
 tools/testing/selftests/sigaltstack/sas.c |  7 ++++---
 3 files changed, 16 insertions(+), 6 deletions(-)

-- 
2.9.3

[toc] | [next] | [standalone]

#1573947 — Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT

On Sun, Feb 5, 2017 at 2:12 AM, Stas Sergeev <stsp@list.ru> wrote:
> Currently SS_AUTODISARM is not supported in compatibility
> mode, but does not return -EINVAL either. This makes dosemu
> built with -m32 on x86_64 to crash. Also the kernel's sigaltstack
> selftest fails if compiled with -m32.
>
> This patch adds the needed support.
>
> Suggesting for -stable.

This looks correct.  I find it odd that we used to check the old sp to
figure out whether to put SA_ONSTACK into the signal frame, but I do
think the new behavior is better.  In any event, this changes the
compat case to work like the native case, and no one noticed when the
native behavior changed, so it's fine by me.

--Andy

[toc] | [prev] | [next] | [standalone]

#1574502 — Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT

05.02.2017 21:31, Andy Lutomirski пишет:
> On Sun, Feb 5, 2017 at 2:12 AM, Stas Sergeev <stsp@list.ru> wrote:
>> Currently SS_AUTODISARM is not supported in compatibility
>> mode, but does not return -EINVAL either. This makes dosemu
>> built with -m32 on x86_64 to crash. Also the kernel's sigaltstack
>> selftest fails if compiled with -m32.
>>
>> This patch adds the needed support.
>>
>> Suggesting for -stable.
> This looks correct.
So it seems my patches haven't made it into LKML
and I don't know if they made it into stable@ because
the archive link:
http://dir.gmane.org/gmane.linux.kernel.stable
doesn't work.
There is definitely something wrong here.

So did they pass into a stable ML?
If not, Andy can I use your Acked-by when/if resending?
Are there any known problems with MLs and the archives now?

[toc] | [prev] | [next] | [standalone]

#1575270 — Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT

On Mon, 6 Feb 2017 12:30:38 +0300 Stas Sergeev <stsp@list.ru> wrote:

> 05.02.2017 21:31, Andy Lutomirski __________:
> > On Sun, Feb 5, 2017 at 2:12 AM, Stas Sergeev <stsp@list.ru> wrote:
> >> Currently SS_AUTODISARM is not supported in compatibility
> >> mode, but does not return -EINVAL either. This makes dosemu
> >> built with -m32 on x86_64 to crash. Also the kernel's sigaltstack
> >> selftest fails if compiled with -m32.
> >>
> >> This patch adds the needed support.
> >>
> >> Suggesting for -stable.
> > This looks correct.
> So it seems my patches haven't made it into LKML
> and I don't know if they made it into stable@ because
> the archive link:
> http://dir.gmane.org/gmane.linux.kernel.stable
> doesn't work.
> There is definitely something wrong here.
> 
> So did they pass into a stable ML?
> If not, Andy can I use your Acked-by when/if resending?
> Are there any known problems with MLs and the archives now?

I have a copy of [1/2] because I was on cc.  I cannot locate a copy of
[2/2].  Please cc Oleg on signal-related changes.

[toc] | [prev] | [next] | [standalone]

#1575501 — [PATCH 2/2] tests: improve output of sigaltstack testcase

[Multipart message — attachments visible in raw view] — view raw

07.02.2017 03:04, Andrew Morton пишет:
> On Mon, 6 Feb 2017 12:30:38 +0300 Stas Sergeev <stsp@list.ru> wrote:
>
>> So it seems my patches haven't made it into LKML
>> and I don't know if they made it into stable@ because
>> the archive link:
>> http://dir.gmane.org/gmane.linux.kernel.stable
>> doesn't work.
>> There is definitely something wrong here.
>>
>> So did they pass into a stable ML?
>> If not, Andy can I use your Acked-by when/if resending?
>> Are there any known problems with MLs and the archives now?
> I have a copy of [1/2] because I was on cc.  I cannot locate a copy of
> [2/2].  Please cc Oleg on signal-related changes.
Thanks for taking these!
2/2 had the CCs to

CC: Shuah Khan<shuah@kernel.org>
CC: Ingo Molnar<mingo@kernel.org>
CC: Andy Lutomirski<luto@kernel.org>
CC:linux-kselftest@vger.kernel.org
CC:linux-kernel@vger.kernel.org
CC:trivial@kernel.org

as it only changed the self-test.
Obviously I am getting spam-blocked somewhere.
I generate the CC list with get_maintainers.pl. Is this
the right thing to do?
Oleg was CCed on [1/2] as get_maintainers.pl mentioned
him.

I am attaching the [2/2] to this e-mail.
If this doesn't work, I can try to re-send it with more
CCs, but it will likely be blocked again.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web