[MIPS] Fix BUG(), BUG_ON() handling
authorAtsushi Nemoto <anemo@mba.ocn.ne.jp>
Thu, 12 Apr 2007 11:02:54 +0000 (20:02 +0900)
committerRalf Baechle <ralf@linux-mips.org>
Fri, 20 Apr 2007 13:58:37 +0000 (14:58 +0100)
With commit 63dc68a8cf60cb110b147dab1704d990808b39e2, kernel can not
handle BUG() and BUG_ON() properly since get_user() returns false for
kernel code.  Use __get_user() to skip unnecessary access_ok().  This
patch also make BRK_BUG code encoded in the TNE instruction.

Signed-off-by: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
arch/mips/kernel/traps.c
include/asm-mips/bug.h

index 6b356f62289e64eaf4bbc144c679c21a098ad7ad..493cb29b8a4284ed13ecd8e22a8abe650cf468b4 100644 (file)
@@ -650,7 +650,7 @@ asmlinkage void do_bp(struct pt_regs *regs)
        unsigned int opcode, bcode;
        siginfo_t info;
 
-       if (get_user(opcode, (unsigned int __user *) exception_epc(regs)))
+       if (__get_user(opcode, (unsigned int __user *) exception_epc(regs)))
                goto out_sigsegv;
 
        /*
@@ -700,7 +700,7 @@ asmlinkage void do_tr(struct pt_regs *regs)
        unsigned int opcode, tcode = 0;
        siginfo_t info;
 
-       if (get_user(opcode, (unsigned int __user *) exception_epc(regs)))
+       if (__get_user(opcode, (unsigned int __user *) exception_epc(regs)))
                goto out_sigsegv;
 
        /* Immediate versions don't provide a code.  */
index 4d560a533940603c6955c753b0ba34daec45f905..7eb63de808bc153cf3f14ed5b2f180365f0f168b 100644 (file)
@@ -18,7 +18,8 @@ do {                                                                  \
 
 #define BUG_ON(condition)                                              \
 do {                                                                   \
-       __asm__ __volatile__("tne $0, %0" : : "r" (condition));         \
+       __asm__ __volatile__("tne $0, %0, %1"                           \
+                            : : "r" (condition), "i" (BRK_BUG));       \
 } while (0)
 
 #define HAVE_ARCH_BUG_ON