[PATCH] make powerpc BUG_ON() OK with pointers and bitwise

Subsystems: the rest

STALE6950d

2 messages, 2 authors, 2007-07-26 · open the first message on its own page

[PATCH] make powerpc BUG_ON() OK with pointers and bitwise

From: Al Viro <hidden>
Date: 2007-07-26 16:41:30

Since powerpc insists on printing the _value_ of condition
and on casting it to long...  At least let's make it a force-cast.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
---
 include/asm-powerpc/bug.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/include/asm-powerpc/bug.h b/include/asm-powerpc/bug.h
index f6fa394..a248b8b 100644
--- a/include/asm-powerpc/bug.h
+++ b/include/asm-powerpc/bug.h
@@ -79,7 +79,7 @@
 		_EMIT_BUG_ENTRY					\
 		: : "i" (__FILE__), "i" (__LINE__), "i" (0),	\
 		  "i" (sizeof(struct bug_entry)),		\
-		  "r" ((long)(x)));				\
+		  "r" ((__force long)(x)));			\
 	}							\
 } while (0)
 
-- 
1.5.3.GIT

Re: [PATCH] make powerpc BUG_ON() OK with pointers and bitwise

From: Paul Mackerras <hidden>
Date: 2007-07-26 23:30:55

Al Viro writes:
Since powerpc insists on printing the _value_ of condition
We don't _print_ the value of the condition, we give it to a
conditional-trap instruction which will trap if it is non-zero.
and on casting it to long...  At least let's make it a force-cast.
The long cast is to make sure it's sign-extended to the full register
width.  Adding __force looks fine.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Acked-by: Paul Mackerras <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help