Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

bitops: merge little and big endian definisions in asm-generic/bitops/le.h

This patch series introduces little-endian bit operations in asm/bitops.h
for all architectures and converts all ext2 non-atomic and minix bit
operations to use little-endian bit operations. It enables us to remove
ext2 non-atomic and minix bit operations from asm/bitops.h. The reason
they should be removed from asm/bitops.h is as follows:

For ext2 non-atomic bit operations, they are used for little-endian byte
order bitmap access by some filesystems and modules. But using ext2_*()
functions on a module other than ext2 filesystem makes some feel strange.

For minix bit operations, they are only used by minix filesystem and are
useless by other modules. Because byte order of inode and block bitmap is

This patch:

In order to make the forthcoming changes smaller, this merges macro
definisions in asm-generic/bitops/le.h for big-endian and little-endian as
much as possible.

This also removes unused BITOP_WORD macro.

Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

authored by

Akinobu Mita and committed by
Linus Torvalds
63ab595f 12ce2242

+20 -26
+20 -26
include/asm-generic/bitops/le.h
··· 4 4 #include <asm/types.h> 5 5 #include <asm/byteorder.h> 6 6 7 - #define BITOP_WORD(nr) ((nr) / BITS_PER_LONG) 8 - #define BITOP_LE_SWIZZLE ((BITS_PER_LONG-1) & ~0x7) 9 - 10 7 #if defined(__LITTLE_ENDIAN) 11 8 12 - #define generic_test_le_bit(nr, addr) test_bit(nr, addr) 13 - #define generic___set_le_bit(nr, addr) __set_bit(nr, addr) 14 - #define generic___clear_le_bit(nr, addr) __clear_bit(nr, addr) 9 + #define BITOP_LE_SWIZZLE 0 15 10 16 - #define generic_test_and_set_le_bit(nr, addr) test_and_set_bit(nr, addr) 17 - #define generic_test_and_clear_le_bit(nr, addr) test_and_clear_bit(nr, addr) 18 - 19 - #define generic___test_and_set_le_bit(nr, addr) __test_and_set_bit(nr, addr) 20 - #define generic___test_and_clear_le_bit(nr, addr) __test_and_clear_bit(nr, addr) 21 - 22 - #define generic_find_next_zero_le_bit(addr, size, offset) find_next_zero_bit(addr, size, offset) 11 + #define generic_find_next_zero_le_bit(addr, size, offset) \ 12 + find_next_zero_bit(addr, size, offset) 23 13 #define generic_find_next_le_bit(addr, size, offset) \ 24 - find_next_bit(addr, size, offset) 14 + find_next_bit(addr, size, offset) 15 + #define generic_find_first_zero_le_bit(addr, size) \ 16 + find_first_zero_bit(addr, size) 25 17 26 18 #elif defined(__BIG_ENDIAN) 19 + 20 + #define BITOP_LE_SWIZZLE ((BITS_PER_LONG-1) & ~0x7) 21 + 22 + extern unsigned long generic_find_next_zero_le_bit(const unsigned long *addr, 23 + unsigned long size, unsigned long offset); 24 + extern unsigned long generic_find_next_le_bit(const unsigned long *addr, 25 + unsigned long size, unsigned long offset); 26 + 27 + #define generic_find_first_zero_le_bit(addr, size) \ 28 + generic_find_next_zero_le_bit((addr), (size), 0) 29 + 30 + #else 31 + #error "Please fix <asm/byteorder.h>" 32 + #endif 27 33 28 34 #define generic_test_le_bit(nr, addr) \ 29 35 test_bit((nr) ^ BITOP_LE_SWIZZLE, (addr)) ··· 47 41 __test_and_set_bit((nr) ^ BITOP_LE_SWIZZLE, (addr)) 48 42 #define generic___test_and_clear_le_bit(nr, addr) \ 49 43 __test_and_clear_bit((nr) ^ BITOP_LE_SWIZZLE, (addr)) 50 - 51 - extern unsigned long generic_find_next_zero_le_bit(const unsigned long *addr, 52 - unsigned long size, unsigned long offset); 53 - extern unsigned long generic_find_next_le_bit(const unsigned long *addr, 54 - unsigned long size, unsigned long offset); 55 - 56 - #else 57 - #error "Please fix <asm/byteorder.h>" 58 - #endif 59 - 60 - #define generic_find_first_zero_le_bit(addr, size) \ 61 - generic_find_next_zero_le_bit((addr), (size), 0) 62 44 63 45 #endif /* _ASM_GENERIC_BITOPS_LE_H_ */