All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/2] sigaltstack: SS_AUTODISARM fixes
@ 2017-02-05 10:12 Stas Sergeev
       [not found] ` <20170205101213.8163-2-stsp@list.ru>
  0 siblings, 1 reply; 5+ messages in thread
From: Stas Sergeev @ 2017-02-05 10:12 UTC (permalink / raw)
  Cc: Stas Sergeev, deller, linux-kernel, Andy Lutomirski

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT
       [not found] ` <20170205101213.8163-2-stsp@list.ru>
@ 2017-02-05 18:31   ` Andy Lutomirski
  2017-02-06  9:30     ` Stas Sergeev
  0 siblings, 1 reply; 5+ messages in thread
From: Andy Lutomirski @ 2017-02-05 18:31 UTC (permalink / raw)
  To: Stas Sergeev
  Cc: Stas Sergeev, Andrew Morton, Milosz Tanski, Andy Lutomirski,
	Al Viro, Arnd Bergmann, Thomas Gleixner, Ingo Molnar,
	Oleg Nesterov, Nicolas Pitre, Waiman Long, Dave Hansen,
	Dmitry Safonov, Wang Xiaoqiang, linux-kernel, stable

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT
  2017-02-05 18:31   ` [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT Andy Lutomirski
@ 2017-02-06  9:30     ` Stas Sergeev
  2017-02-07  0:04       ` Andrew Morton
  0 siblings, 1 reply; 5+ messages in thread
From: Stas Sergeev @ 2017-02-06  9:30 UTC (permalink / raw)
  To: Andy Lutomirski; +Cc: Helge Deller, David Miller, Linux kernel

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?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT
  2017-02-06  9:30     ` Stas Sergeev
@ 2017-02-07  0:04       ` Andrew Morton
  2017-02-07  8:55         ` [PATCH 2/2] tests: improve output of sigaltstack testcase Stas Sergeev
  0 siblings, 1 reply; 5+ messages in thread
From: Andrew Morton @ 2017-02-07  0:04 UTC (permalink / raw)
  To: Stas Sergeev
  Cc: Andy Lutomirski, Helge Deller, David Miller, Linux kernel, Oleg Nesterov

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.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH 2/2] tests: improve output of sigaltstack testcase
  2017-02-07  0:04       ` Andrew Morton
@ 2017-02-07  8:55         ` Stas Sergeev
  0 siblings, 0 replies; 5+ messages in thread
From: Stas Sergeev @ 2017-02-07  8:55 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Andy Lutomirski, Helge Deller, David Miller, Linux kernel,
	Oleg Nesterov, David Miller

[-- Attachment #1: Type: text/plain, Size: 1246 bytes --]

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.

[-- Attachment #2: 0002-tests-improve-output-of-sigaltstack-testcase.patch --]
[-- Type: text/x-patch, Size: 1902 bytes --]

>From d4dc2ac235b5a18f5bdf916152509f8b0e11276e Mon Sep 17 00:00:00 2001
From: Stas Sergeev <stsp@list.ru>
Date: Sat, 4 Feb 2017 22:11:49 +0300
Subject: [PATCH 2/2] tests: improve output of sigaltstack testcase

Currently it uses %i for bitmasks, which makes it difficult
to properly decode the values. Use %x instead.

Signed-off-by: Stas Sergeev <stsp@users.sourceforge.net>
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
---
 tools/testing/selftests/sigaltstack/sas.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/tools/testing/selftests/sigaltstack/sas.c b/tools/testing/selftests/sigaltstack/sas.c
index 1bb0125..ccd0734 100644
--- a/tools/testing/selftests/sigaltstack/sas.c
+++ b/tools/testing/selftests/sigaltstack/sas.c
@@ -57,7 +57,7 @@ void my_usr1(int sig, siginfo_t *si, void *u)
 		exit(EXIT_FAILURE);
 	}
 	if (stk.ss_flags != SS_DISABLE)
-		printf("[FAIL]\tss_flags=%i, should be SS_DISABLE\n",
+		printf("[FAIL]\tss_flags=%x, should be SS_DISABLE\n",
 				stk.ss_flags);
 	else
 		printf("[OK]\tsigaltstack is disabled in sighandler\n");
@@ -122,7 +122,8 @@ int main(void)
 	if (stk.ss_flags == SS_DISABLE) {
 		printf("[OK]\tInitial sigaltstack state was SS_DISABLE\n");
 	} else {
-		printf("[FAIL]\tInitial sigaltstack state was %i; should have been SS_DISABLE\n", stk.ss_flags);
+		printf("[FAIL]\tInitial sigaltstack state was %x; "
+		       "should have been SS_DISABLE\n", stk.ss_flags);
 		return EXIT_FAILURE;
 	}
 
@@ -165,7 +166,7 @@ int main(void)
 		exit(EXIT_FAILURE);
 	}
 	if (stk.ss_flags != SS_AUTODISARM) {
-		printf("[FAIL]\tss_flags=%i, should be SS_AUTODISARM\n",
+		printf("[FAIL]\tss_flags=%x, should be SS_AUTODISARM\n",
 				stk.ss_flags);
 		exit(EXIT_FAILURE);
 	}
-- 
2.9.3


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-02-07  9:18 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-05 10:12 [PATCH v2 0/2] sigaltstack: SS_AUTODISARM fixes Stas Sergeev
     [not found] ` <20170205101213.8163-2-stsp@list.ru>
2017-02-05 18:31   ` [PATCH 1/2] sigaltstack: support SS_AUTODISARM for CONFIG_COMPAT Andy Lutomirski
2017-02-06  9:30     ` Stas Sergeev
2017-02-07  0:04       ` Andrew Morton
2017-02-07  8:55         ` [PATCH 2/2] tests: improve output of sigaltstack testcase Stas Sergeev

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.