lkml.org 
[lkml]   [2009]   [Feb]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Subject[PATCH] x86: remove unneeded endless loop in BUG()
From
Date
Since __builtin_trap() will always generate an illegal instruction, we
can replace the explicit asm("ud2") with it.

This way gcc will understand that the function never returns, plus it
won't emit any extra instructions.

Signed-off-by: Petr Tesarik <ptesarik@suse.cz>

diff --git a/arch/x86/include/asm/bug.h b/arch/x86/include/asm/bug.h
index d9cf1cd..9b7c50a 100644
--- a/arch/x86/include/asm/bug.h
+++ b/arch/x86/include/asm/bug.h
@@ -4,6 +4,13 @@
#ifdef CONFIG_BUG
#define HAVE_ARCH_BUG

+#define DO_BUG() \
+do { \
+ __builtin_trap(); \
+ printk("BUG: failure at %s:%d/%s()!\n", __FILE__, __LINE__, __func__); \
+ panic("BUG!"); \
+} while(0)
+
#ifdef CONFIG_DEBUG_BUGVERBOSE

#ifdef CONFIG_X86_32
@@ -14,7 +21,7 @@

#define BUG() \
do { \
- asm volatile("1:\tud2\n" \
+ asm volatile("1:\n" \
".pushsection __bug_table,\"a\"\n" \
__BUG_C0 \
"\t.word %c1, 0\n" \
@@ -22,15 +29,11 @@ do { \
".popsection" \
: : "i" (__FILE__), "i" (__LINE__), \
"i" (sizeof(struct bug_entry))); \
- for (;;) ; \
+ DO_BUG(); \
} while (0)

#else
-#define BUG() \
-do { \
- asm volatile("ud2"); \
- for (;;) ; \
-} while (0)
+#define BUG DO_BUG
#endif

#endif /* !CONFIG_BUG */



\
 
 \ /
  Last update: 2009-02-19 19:41    [W:0.064 / U:0.432 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site