public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Akinobu Mita <mita@miraclelinux.com>
To: linux-kernel@vger.kernel.org
Cc: akpm@osdl.org, Richard Henderson <rth@twiddle.net>,
	Ivan Kokshaysky <ink@jurassic.park.msu.ru>,
	Akinobu Mita <mita@miraclelinux.com>
Subject: [patch 19/47] alpha: use generic bitops
Date: Tue, 14 Feb 2006 14:04:10 +0900	[thread overview]
Message-ID: <20060214050445.411720000@localhost.localdomain> (raw)
In-Reply-To: 20060214050351.252615000@localhost.localdomain

[-- Attachment #1: alpha-2.patch --]
[-- Type: text/plain, Size: 5196 bytes --]

- unless defined(__alpha_cix__) and defined(__alpha_fix__)

  - remove generic_fls()
  - remove generic_hweight{64,32,16,8}()

- remove generic_fls64()
- remove find_{next,first}{,_zero}_bit()
- remove ext2_{set,clear,test,find_first_zero,find_next_zero}_bit()
- remove minix_{test,set,test_and_clear,test,find_first_zero}_bit()

Signed-off-by: Akinobu Mita <mita@miraclelinux.com>
 arch/alpha/Kconfig         |    8 ++
 include/asm-alpha/bitops.h |  123 ++-------------------------------------------
 2 files changed, 15 insertions(+), 116 deletions(-)

Index: 2.6-rc/include/asm-alpha/bitops.h
===================================================================
--- 2.6-rc.orig/include/asm-alpha/bitops.h
+++ 2.6-rc/include/asm-alpha/bitops.h
@@ -319,9 +319,9 @@ static inline int fls(int word)
 	return 64 - __kernel_ctlz(word & 0xffffffff);
 }
 #else
-#define fls	generic_fls
+#include <asm-generic/bitops/fls.h>
 #endif
-#define fls64   generic_fls64
+#include <asm-generic/bitops/fls64.h>
 
 /* Compute powers of two for the given integer.  */
 static inline long floor_log2(unsigned long word)
@@ -358,112 +358,12 @@ static inline unsigned long hweight64(un
 #define hweight16(x)	(unsigned int) hweight64((x) & 0xfffful)
 #define hweight8(x)	(unsigned int) hweight64((x) & 0xfful)
 #else
-static inline unsigned long hweight64(unsigned long w)
-{
-	unsigned long result;
-	for (result = 0; w ; w >>= 1)
-		result += (w & 1);
-	return result;
-}
-
-#define hweight32(x) generic_hweight32(x)
-#define hweight16(x) generic_hweight16(x)
-#define hweight8(x)  generic_hweight8(x)
+#include <asm-generic/bitops/hweight.h>
 #endif
 
 #endif /* __KERNEL__ */
 
-/*
- * Find next zero bit in a bitmap reasonably efficiently..
- */
-static inline unsigned long
-find_next_zero_bit(const void *addr, unsigned long size, unsigned long offset)
-{
-	const unsigned long *p = addr;
-	unsigned long result = offset & ~63UL;
-	unsigned long tmp;
-
-	p += offset >> 6;
-	if (offset >= size)
-		return size;
-	size -= result;
-	offset &= 63UL;
-	if (offset) {
-		tmp = *(p++);
-		tmp |= ~0UL >> (64-offset);
-		if (size < 64)
-			goto found_first;
-		if (~tmp)
-			goto found_middle;
-		size -= 64;
-		result += 64;
-	}
-	while (size & ~63UL) {
-		if (~(tmp = *(p++)))
-			goto found_middle;
-		result += 64;
-		size -= 64;
-	}
-	if (!size)
-		return result;
-	tmp = *p;
- found_first:
-	tmp |= ~0UL << size;
-	if (tmp == ~0UL)        /* Are any bits zero? */
-		return result + size; /* Nope. */
- found_middle:
-	return result + ffz(tmp);
-}
-
-/*
- * Find next one bit in a bitmap reasonably efficiently.
- */
-static inline unsigned long
-find_next_bit(const void * addr, unsigned long size, unsigned long offset)
-{
-	const unsigned long *p = addr;
-	unsigned long result = offset & ~63UL;
-	unsigned long tmp;
-
-	p += offset >> 6;
-	if (offset >= size)
-		return size;
-	size -= result;
-	offset &= 63UL;
-	if (offset) {
-		tmp = *(p++);
-		tmp &= ~0UL << offset;
-		if (size < 64)
-			goto found_first;
-		if (tmp)
-			goto found_middle;
-		size -= 64;
-		result += 64;
-	}
-	while (size & ~63UL) {
-		if ((tmp = *(p++)))
-			goto found_middle;
-		result += 64;
-		size -= 64;
-	}
-	if (!size)
-		return result;
-	tmp = *p;
- found_first:
-	tmp &= ~0UL >> (64 - size);
-	if (!tmp)
-		return result + size;
- found_middle:
-	return result + __ffs(tmp);
-}
-
-/*
- * The optimizer actually does good code for this case.
- */
-#define find_first_zero_bit(addr, size) \
-	find_next_zero_bit((addr), (size), 0)
-#define find_first_bit(addr, size) \
-	find_next_bit((addr), (size), 0)
+#include <asm-generic/bitops/find.h>
 
 #ifdef __KERNEL__
 
@@ -487,21 +387,12 @@ sched_find_first_bit(unsigned long b[3])
 	return __ffs(b0) + ofs;
 }
 
+#include <asm-generic/bitops/ext2-non-atomic.h>
 
-#define ext2_set_bit                 __test_and_set_bit
 #define ext2_set_bit_atomic(l,n,a)   test_and_set_bit(n,a)
-#define ext2_clear_bit               __test_and_clear_bit
 #define ext2_clear_bit_atomic(l,n,a) test_and_clear_bit(n,a)
-#define ext2_test_bit                test_bit
-#define ext2_find_first_zero_bit     find_first_zero_bit
-#define ext2_find_next_zero_bit      find_next_zero_bit
-
-/* Bitmap functions for the minix filesystem.  */
-#define minix_test_and_set_bit(nr,addr) __test_and_set_bit(nr,addr)
-#define minix_set_bit(nr,addr) __set_bit(nr,addr)
-#define minix_test_and_clear_bit(nr,addr) __test_and_clear_bit(nr,addr)
-#define minix_test_bit(nr,addr) test_bit(nr,addr)
-#define minix_find_first_zero_bit(addr,size) find_first_zero_bit(addr,size)
+
+#include <asm-generic/bitops/minix.h>
 
 #endif /* __KERNEL__ */
 
Index: 2.6-rc/arch/alpha/Kconfig
===================================================================
--- 2.6-rc.orig/arch/alpha/Kconfig
+++ 2.6-rc/arch/alpha/Kconfig
@@ -25,6 +25,10 @@ config RWSEM_XCHGADD_ALGORITHM
 	bool
 	default y
 
+config GENERIC_FIND_NEXT_BIT
+	bool
+	default y
+
 config GENERIC_CALIBRATE_DELAY
 	bool
 	default y
@@ -447,6 +451,10 @@ config ALPHA_IRONGATE
 	depends on ALPHA_NAUTILUS
 	default y
 
+config GENERIC_HWEIGHT
+	bool
+	default y if !ALPHA_EV6 && !ALPHA_EV67
+
 config ALPHA_AVANTI
 	bool
 	depends on ALPHA_XL || ALPHA_AVANTI_CH

--

  parent reply	other threads:[~2006-02-14  5:17 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-14  5:03 [patch 00/47] generic bitops Akinobu Mita
2006-02-14  5:03 ` [patch 01/47] alpha: use config options instead of __alpha_fix__ and __alpha_cix__ Akinobu Mita
2006-02-14  5:03 ` [patch 02/47] ia64: use cpu_set() instead of __set_bit() Akinobu Mita
2006-02-14  5:03 ` [patch 03/47] parisc: add ()-pair in __ffz() macro Akinobu Mita
2006-02-14  5:03 ` [patch 04/47] cris: remove unnecessary local_irq_restore() Akinobu Mita
2006-02-14  5:03 ` [patch 05/47] use non atomic operations for minix_*_bit() and ext2_*_bit() Akinobu Mita
2006-02-14  5:03 ` [patch 06/47] generic {,test_and_}{set,clear,change}_bit() Akinobu Mita
2006-02-14  5:03 ` [patch 07/47] generic __{,test_and_}{set,clear,change}_bit() and test_bit() Akinobu Mita
2006-02-14  5:03 ` [patch 08/47] generic __ffs() Akinobu Mita
2006-02-14  5:04 ` [patch 09/47] generic ffz() Akinobu Mita
2006-02-14  5:04 ` [patch 10/47] generic fls() Akinobu Mita
2006-02-14  5:04 ` [patch 11/47] generic fls64() Akinobu Mita
2006-02-14  5:04 ` [patch 12/47] generic find_{next,first}{,_zero}_bit() Akinobu Mita
2006-02-14  5:04 ` [patch 13/47] generic sched_find_first_bit() Akinobu Mita
2006-02-14  5:04 ` [patch 14/47] generic ffs() Akinobu Mita
2006-02-14  5:04 ` [patch 15/47] generic hweight{64,32,16,8}() Akinobu Mita
2006-02-14  5:04 ` [patch 16/47] generic ext2_{set,clear,test,find_first_zero,find_next_zero}_bit() Akinobu Mita
2006-02-14  5:04 ` [patch 17/47] generic ext2_{set,clear}_bit_atomic() Akinobu Mita
2006-02-14  5:04 ` [patch 18/47] generic minix_{test,set,test_and_clear,test,find_first_zero}_bit() Akinobu Mita
2006-02-14  5:04 ` Akinobu Mita [this message]
2006-02-14  5:04 ` [patch 20/47] arm: use generic bitops Akinobu Mita
2006-02-14  5:04 ` [patch 21/47] arm26: " Akinobu Mita
2006-02-14  5:04 ` [patch 22/47] cris: " Akinobu Mita
2006-02-14  5:04 ` [patch 23/47] frv: " Akinobu Mita
2006-02-14  5:04 ` [patch 24/47] h8300: " Akinobu Mita
2006-02-14  5:04 ` [patch 25/47] i386: " Akinobu Mita
2006-02-14  5:04 ` [patch 26/47] ia64: " Akinobu Mita
2006-02-14  5:04 ` [patch 27/47] m32r: " Akinobu Mita
2006-02-14  5:04 ` [patch 28/47] m68k: " Akinobu Mita
2006-02-14  5:04 ` [patch 29/47] m68knommu: " Akinobu Mita
2006-02-14  5:04 ` [patch 30/47] mips: " Akinobu Mita
2006-02-14  5:04 ` [patch 31/47] parisc: " Akinobu Mita
2006-02-14  5:04 ` [patch 32/47] powerpc: " Akinobu Mita
2006-02-14  5:04 ` [patch 33/47] s390: " Akinobu Mita
2006-02-14  5:04 ` [patch 34/47] sh: " Akinobu Mita
2006-02-14  5:04 ` [patch 35/47] sh64: " Akinobu Mita
2006-02-14  5:04 ` [patch 36/47] sparc: " Akinobu Mita
2006-02-14  5:04 ` [patch 37/47] sparc64: " Akinobu Mita
2006-02-14  5:04 ` [patch 38/47] v850: " Akinobu Mita
2006-02-14  5:04 ` [patch 39/47] x86_64: " Akinobu Mita
2006-02-14  5:04 ` [patch 40/47] xtensa: " Akinobu Mita
2006-02-14  5:04 ` [patch 41/47] update include/asm-generic/bitops.h Akinobu Mita
2006-02-14  5:04 ` [patch 42/47] make thread_info.flags an unsigned long Akinobu Mita
2006-02-14  5:04 ` [patch 43/47] ia64: make partial_page.bitmap " Akinobu Mita
2006-02-14  5:04 ` [patch 44/47] ntfs: remove generic_ffs() Akinobu Mita
2006-02-14  5:04 ` [patch 45/47] remove unused generic bitops in include/linux/bitops.h Akinobu Mita
2006-02-14  5:04 ` [patch 46/47] hweight() related cleanup Akinobu Mita
2006-02-14  5:04 ` [patch 47/47] hweight() speedup Akinobu Mita

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060214050445.411720000@localhost.localdomain \
    --to=mita@miraclelinux.com \
    --cc=akpm@osdl.org \
    --cc=ink@jurassic.park.msu.ru \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rth@twiddle.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox