All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stas Sergeev <stsp@list.ru>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Andy Lutomirski <luto@amacapital.net>,
	Helge Deller <deller@gmx.de>, David Miller <davem@davemloft.net>,
	Linux kernel <linux-kernel@vger.kernel.org>,
	Oleg Nesterov <oleg@redhat.com>,
	David Miller <davem@davemloft.net>
Subject: [PATCH 2/2] tests: improve output of sigaltstack testcase
Date: Tue, 7 Feb 2017 11:55:48 +0300	[thread overview]
Message-ID: <b7b4c45d-2f21-de6c-d1c8-16c8386da27c@list.ru> (raw)
In-Reply-To: <20170206160443.55c0f1daac793f1464053cd6@linux-foundation.org>

[-- 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


      reply	other threads:[~2017-02-07  9:18 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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         ` Stas Sergeev [this message]

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=b7b4c45d-2f21-de6c-d1c8-16c8386da27c@list.ru \
    --to=stsp@list.ru \
    --cc=akpm@linux-foundation.org \
    --cc=davem@davemloft.net \
    --cc=deller@gmx.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luto@amacapital.net \
    --cc=oleg@redhat.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.