mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] Fix typos from signal_32/64.h merge
@ 2008-07-14 20:40 Herton Ronaldo Krzesinski
  2008-07-14 20:49 ` Luiz Fernando N. Capitulino
  2008-07-18 15:58 ` Ingo Molnar
  0 siblings, 2 replies; 3+ messages in thread
From: Herton Ronaldo Krzesinski @ 2008-07-14 20:40 UTC (permalink / raw)
  To: LKML
  Cc: Luiz Fernando N. Capitulino, Ingo Molnar, Thomas Gleixner,
	Roland McGrath, stable

Fallout from commit 33185c504f8e521b398536b5a8d415779a24593c ("x86:
merge signal_32/64.h")

Thanks to Dick Streefland who provided an useful testcase on
http://lkml.org/lkml/2008/3/17/205 (only applicable to 2.6.24.x), that
helped a lot as a deterministic way to bisect an issue that leaded to
this fix.

Signed-off-by: Herton Ronaldo Krzesinski <herton@mandriva.com.br>
Signed-off-by: Luiz Fernando N. Capitulino <lcapitulino@mandriva.com.br>
---
 include/asm-x86/signal.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/asm-x86/signal.h b/include/asm-x86/signal.h
index f15186d..6dac493 100644
--- a/include/asm-x86/signal.h
+++ b/include/asm-x86/signal.h
@@ -181,12 +181,12 @@ typedef struct sigaltstack {
 #ifdef __KERNEL__
 #include <asm/sigcontext.h>
 
-#ifdef __386__
+#ifdef __i386__
 
 #define __HAVE_ARCH_SIG_BITOPS
 
 #define sigaddset(set,sig)		    \
-	(__builtin_constantp(sig)	    \
+	(__builtin_constant_p(sig)	    \
 	 ? __const_sigaddset((set), (sig))  \
 	 : __gen_sigaddset((set), (sig)))
 
-- 
1.5.6



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

* Re: [PATCH] Fix typos from signal_32/64.h merge
  2008-07-14 20:40 [PATCH] Fix typos from signal_32/64.h merge Herton Ronaldo Krzesinski
@ 2008-07-14 20:49 ` Luiz Fernando N. Capitulino
  2008-07-18 15:58 ` Ingo Molnar
  1 sibling, 0 replies; 3+ messages in thread
From: Luiz Fernando N. Capitulino @ 2008-07-14 20:49 UTC (permalink / raw)
  To: Herton Ronaldo Krzesinski
  Cc: LKML, Ingo Molnar, Thomas Gleixner, Roland McGrath, stable

Em Mon, 14 Jul 2008 17:40:23 -0300
Herton Ronaldo Krzesinski <herton@mandriva.com.br> escreveu:

| Fallout from commit 33185c504f8e521b398536b5a8d415779a24593c ("x86:
| merge signal_32/64.h")
| 
| Thanks to Dick Streefland who provided an useful testcase on
| http://lkml.org/lkml/2008/3/17/205 (only applicable to 2.6.24.x), that
| helped a lot as a deterministic way to bisect an issue that leaded to
| this fix.

 Just a small note. On 2.6.24.x this causes gdb to get unexpected
SIGTRAPs, which (for some reason) is easy to reproduce when debugging
C++/QT programs.

 And I have no idea on what kind of bug this is causing in latest
2.6.26 kernel.

-- 
Luiz Fernando N. Capitulino

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

* Re: [PATCH] Fix typos from signal_32/64.h merge
  2008-07-14 20:40 [PATCH] Fix typos from signal_32/64.h merge Herton Ronaldo Krzesinski
  2008-07-14 20:49 ` Luiz Fernando N. Capitulino
@ 2008-07-18 15:58 ` Ingo Molnar
  1 sibling, 0 replies; 3+ messages in thread
From: Ingo Molnar @ 2008-07-18 15:58 UTC (permalink / raw)
  To: Herton Ronaldo Krzesinski
  Cc: LKML, Luiz Fernando N. Capitulino, Thomas Gleixner,
	Roland McGrath, stable


* Herton Ronaldo Krzesinski <herton@mandriva.com.br> wrote:

> Fallout from commit 33185c504f8e521b398536b5a8d415779a24593c ("x86: 
> merge signal_32/64.h")
> 
> Thanks to Dick Streefland who provided an useful testcase on 
> http://lkml.org/lkml/2008/3/17/205 (only applicable to 2.6.24.x), that 
> helped a lot as a deterministic way to bisect an issue that leaded to 
> this fix.

applied to tip/x86/urgent, thanks Herton!

	Ingo

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

end of thread, other threads:[~2008-07-18 15:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-07-14 20:40 [PATCH] Fix typos from signal_32/64.h merge Herton Ronaldo Krzesinski
2008-07-14 20:49 ` Luiz Fernando N. Capitulino
2008-07-18 15:58 ` Ingo Molnar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®