* [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