* [PATCH 150/150] kernel/signal.c: use __ARCH_HAS_SA_RESTORER instead of SA_RESTORER
2013-03-26 15:18 [ 3.5.y.z extended stable ] Linux 3.5.7.9 stable review Luis Henriques
@ 2013-03-26 15:20 ` Luis Henriques
0 siblings, 0 replies; 3+ messages in thread
From: Luis Henriques @ 2013-03-26 15:20 UTC (permalink / raw)
To: linux-kernel, stable, kernel-team
Cc: Emese Revfy, PaX Team, Al Viro, Oleg Nesterov, Eric W. Biederman,
Serge Hallyn, Julien Tinnes, Andrew Morton, Linus Torvalds,
Luis Henriques
3.5.7.9 -stable review patch. If anyone has any objections, please let me know.
------------------
From: Andrew Morton <akpm@linux-foundation.org>
commit 522cff142d7d2f9230839c9e1f21a4d8bcc22a4a upstream.
__ARCH_HAS_SA_RESTORER is the preferred conditional for use in 3.9 and
later kernels, per Kees.
Cc: Emese Revfy <re.emese@gmail.com>
Cc: Emese Revfy <re.emese@gmail.com>
Cc: PaX Team <pageexec@freemail.hu>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Serge Hallyn <serge.hallyn@canonical.com>
Cc: Julien Tinnes <jln@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Luis Henriques <luis.henriques@canonical.com>
---
kernel/signal.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/signal.c b/kernel/signal.c
index f2d6a4f..0e9b729 100644
--- a/kernel/signal.c
+++ b/kernel/signal.c
@@ -483,7 +483,7 @@ flush_signal_handlers(struct task_struct *t, int force_default)
if (force_default || ka->sa.sa_handler != SIG_IGN)
ka->sa.sa_handler = SIG_DFL;
ka->sa.sa_flags = 0;
-#ifdef SA_RESTORER
+#ifdef __ARCH_HAS_SA_RESTORER
ka->sa.sa_restorer = NULL;
#endif
sigemptyset(&ka->sa.sa_mask);
--
1.8.1.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 150/150] kernel/signal.c: use __ARCH_HAS_SA_RESTORER instead of SA_RESTORER
@ 2013-03-27 14:53 PaX Team
2013-03-27 15:16 ` Luis Henriques
0 siblings, 1 reply; 3+ messages in thread
From: PaX Team @ 2013-03-27 14:53 UTC (permalink / raw)
To: linux-kernel, stable, kernel-team, Luis Henriques
Cc: Emese Revfy, Al Viro, Oleg Nesterov, Eric W. Biederman,
Serge Hallyn, Julien Tinnes, Andrew Morton, Linus Torvalds,
Luis Henriques
sorry if you got this twice, had smtpd problems...
On 26 Mar 2013 at 15:20, Luis Henriques wrote:
> 3.5.7.9 -stable review patch. If anyone has any objections, please let me know.
>
> ------------------
>
> From: Andrew Morton <akpm@linux-foundation.org>
>
> commit 522cff142d7d2f9230839c9e1f21a4d8bcc22a4a upstream.
>
> __ARCH_HAS_SA_RESTORER is the preferred conditional for use in 3.9 and
> later kernels, per Kees.
does __ARCH_HAS_SA_RESTORER exist in the 3.5 stable series at all? i thought it
was new to 3.9...
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 150/150] kernel/signal.c: use __ARCH_HAS_SA_RESTORER instead of SA_RESTORER
2013-03-27 14:53 [PATCH 150/150] kernel/signal.c: use __ARCH_HAS_SA_RESTORER instead of SA_RESTORER PaX Team
@ 2013-03-27 15:16 ` Luis Henriques
0 siblings, 0 replies; 3+ messages in thread
From: Luis Henriques @ 2013-03-27 15:16 UTC (permalink / raw)
To: PaX Team
Cc: linux-kernel, stable, kernel-team, Emese Revfy, Al Viro,
Oleg Nesterov, Eric W. Biederman, Serge Hallyn, Julien Tinnes,
Andrew Morton, Linus Torvalds
On Wed, Mar 27, 2013 at 03:53:25PM +0100, PaX Team wrote:
> sorry if you got this twice, had smtpd problems...
>
> On 26 Mar 2013 at 15:20, Luis Henriques wrote:
>
> > 3.5.7.9 -stable review patch. If anyone has any objections, please let me know.
> >
> > ------------------
> >
> > From: Andrew Morton <akpm@linux-foundation.org>
> >
> > commit 522cff142d7d2f9230839c9e1f21a4d8bcc22a4a upstream.
> >
> > __ARCH_HAS_SA_RESTORER is the preferred conditional for use in 3.9 and
> > later kernels, per Kees.
>
> does __ARCH_HAS_SA_RESTORER exist in the 3.5 stable series at all? i thought it
> was new to 3.9...
You're right, __ARCH_HAS_SA_RESTORER has been added on 3.9 only.
However, due to build failures for some archs (mips, ia64, ...) using
the upstream 2ca39528c01a933f6689cd6505ce65bd6d68a530 commit, Ben
Hutchings has cooked a patch that adds this definition to stable
series.
This patch is currently under review for the 3.5 kernel:
[PATCH 149/150] signal: Define __ARCH_HAS_SA_RESTORER so we know whether to clear sa_restorer
Cheers,
--
Luis
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-03-27 15:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-27 14:53 [PATCH 150/150] kernel/signal.c: use __ARCH_HAS_SA_RESTORER instead of SA_RESTORER PaX Team
2013-03-27 15:16 ` Luis Henriques
-- strict thread matches above, loose matches on Subject: below --
2013-03-26 15:18 [ 3.5.y.z extended stable ] Linux 3.5.7.9 stable review Luis Henriques
2013-03-26 15:20 ` [PATCH 150/150] kernel/signal.c: use __ARCH_HAS_SA_RESTORER instead of SA_RESTORER Luis Henriques
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).