public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
* [Linux-ia64] bogus __clear_bit macro
@ 2003-03-11 19:01 Chen, Kenneth W
  2003-03-11 22:37 ` Peter Chubb
  0 siblings, 1 reply; 2+ messages in thread
From: Chen, Kenneth W @ 2003-03-11 19:01 UTC (permalink / raw)
  To: linux-ia64

David,

It looks like when __clear_bit() was implemented, the deletion of the macro falls through the crack.  This macro prohibit the actual __clear_bit to be invoked.

--- include/asm-ia64/bitops.h-	Tue Mar  4 19:29:35 2003
+++ include/asm-ia64/bitops.h	Tue Mar 11 10:49:36 2003
@@ -450,8 +450,6 @@
 
 #ifdef __KERNEL__
 
-#define __clear_bit(nr, addr)        clear_bit(nr, addr)
-
 #define ext2_set_bit                 test_and_set_bit
 #define ext2_clear_bit               test_and_clear_bit
 #define ext2_test_bit                test_bit


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

* [Linux-ia64] bogus __clear_bit macro
  2003-03-11 19:01 [Linux-ia64] bogus __clear_bit macro Chen, Kenneth W
@ 2003-03-11 22:37 ` Peter Chubb
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Chubb @ 2003-03-11 22:37 UTC (permalink / raw)
  To: linux-ia64

>>>>> "Kenneth" = Kenneth W Chen <Chen> writes:

Kenneth> David,
Kenneth> It looks like when __clear_bit() was implemented, the deletion of the macro falls through the crack.  This macro prohibit the actual __clear_bit to be invoked.


While you're fixing bitops.h, making test_bit take a const qualified
arg will kill some warnings in reiserfs...

--- /tmp/geta24522      2003-03-12 09:19:31.000000000 +1100
+++ linux/include/asm-ia64/bitops.h    2003-03-12 09:17:53.000000000 +1100
@@ -275,7 +275,7 @@
 }

 static __inline__ int
-test_bit (int nr, volatile void *addr)
+test_bit (int nr, const volatile void *addr)
 {
	return 1 & (((const volatile __u32 *) addr)[nr >> 5] >> (nr & 31));
 }


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

end of thread, other threads:[~2003-03-11 22:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-03-11 19:01 [Linux-ia64] bogus __clear_bit macro Chen, Kenneth W
2003-03-11 22:37 ` Peter Chubb

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