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, Russell King <rmk@arm.linux.org.uk>,
	Ian Molton <spyro@f2s.com>,
	dev-etrax@axis.com, Hirokazu Takata <takata@linux-m32r.org>,
	linux-mips@linux-mips.org, parisc-linux@parisc-linux.org,
	linuxppc-dev@ozlabs.org, linuxsh-dev@lists.sourceforge.net,
	linuxsh-shmedia-dev@lists.sourceforge.net,
	sparclinux@vger.kernel.org, ultralinux@vger.kernel.org,
	Chris Zankel <chris@zankel.net>,
	Akinobu Mita <mita@miraclelinux.com>
Subject: [patch 07/47] generic __{,test_and_}{set,clear,change}_bit() and test_bit()
Date: Tue, 14 Feb 2006 14:03:58 +0900	[thread overview]
Message-ID: <20060214050443.068971000@localhost.localdomain> (raw)
In-Reply-To: 20060214050351.252615000@localhost.localdomain

[-- Attachment #1: non-atomic-bitops.patch --]
[-- Type: text/plain, Size: 4243 bytes --]

This patch introduces the C-language equivalents of the functions below:

void __set_bit(int nr, volatile unsigned long *addr);
void __clear_bit(int nr, volatile unsigned long *addr);
void __change_bit(int nr, volatile unsigned long *addr);
int __test_and_set_bit(int nr, volatile unsigned long *addr);
int __test_and_clear_bit(int nr, volatile unsigned long *addr);
int __test_and_change_bit(int nr, volatile unsigned long *addr);
int test_bit(int nr, const volatile unsigned long *addr);

In include/asm-generic/bitops/non-atomic.h

This code largely copied from:
asm-powerpc/bitops.h

Signed-off-by: Akinobu Mita <mita@miraclelinux.com>
 include/asm-generic/bitops/non-atomic.h |  111 ++++++++++++++++++++++++++++++++
 1 files changed, 111 insertions(+)

Index: 2.6-rc/include/asm-generic/bitops/non-atomic.h
===================================================================
--- /dev/null
+++ 2.6-rc/include/asm-generic/bitops/non-atomic.h
@@ -0,0 +1,111 @@
+#ifndef _ASM_GENERIC_BITOPS_NON_ATOMIC_H_
+#define _ASM_GENERIC_BITOPS_NON_ATOMIC_H_
+
+#include <asm/types.h>
+
+#define BITOP_MASK(nr)		(1UL << ((nr) % BITS_PER_LONG))
+#define BITOP_WORD(nr)		((nr) / BITS_PER_LONG)
+
+/**
+ * __set_bit - Set a bit in memory
+ * @nr: the bit to set
+ * @addr: the address to start counting from
+ *
+ * Unlike set_bit(), this function is non-atomic and may be reordered.
+ * If it's called on the same region of memory simultaneously, the effect
+ * may be that only one operation succeeds.
+ */
+static inline void __set_bit(int nr, volatile unsigned long *addr)
+{
+	unsigned long mask = BITOP_MASK(nr);
+	unsigned long *p = ((unsigned long *)addr) + BITOP_WORD(nr);
+
+	*p  |= mask;
+}
+
+static inline void __clear_bit(int nr, volatile unsigned long *addr)
+{
+	unsigned long mask = BITOP_MASK(nr);
+	unsigned long *p = ((unsigned long *)addr) + BITOP_WORD(nr);
+
+	*p &= ~mask;
+}
+
+/**
+ * __change_bit - Toggle a bit in memory
+ * @nr: the bit to change
+ * @addr: the address to start counting from
+ *
+ * Unlike change_bit(), this function is non-atomic and may be reordered.
+ * If it's called on the same region of memory simultaneously, the effect
+ * may be that only one operation succeeds.
+ */
+static inline void __change_bit(int nr, volatile unsigned long *addr)
+{
+	unsigned long mask = BITOP_MASK(nr);
+	unsigned long *p = ((unsigned long *)addr) + BITOP_WORD(nr);
+
+	*p ^= mask;
+}
+
+/**
+ * __test_and_set_bit - Set a bit and return its old value
+ * @nr: Bit to set
+ * @addr: Address to count from
+ *
+ * This operation is non-atomic and can be reordered.  
+ * If two examples of this operation race, one can appear to succeed
+ * but actually fail.  You must protect multiple accesses with a lock.
+ */
+static inline int __test_and_set_bit(int nr, volatile unsigned long *addr)
+{
+	unsigned long mask = BITOP_MASK(nr);
+	unsigned long *p = ((unsigned long *)addr) + BITOP_WORD(nr);
+	unsigned long old = *p;
+
+	*p = old | mask;
+	return (old & mask) != 0;
+}
+
+/**
+ * __test_and_clear_bit - Clear a bit and return its old value
+ * @nr: Bit to clear
+ * @addr: Address to count from
+ *
+ * This operation is non-atomic and can be reordered.  
+ * If two examples of this operation race, one can appear to succeed
+ * but actually fail.  You must protect multiple accesses with a lock.
+ */
+static inline int __test_and_clear_bit(int nr, volatile unsigned long *addr)
+{
+	unsigned long mask = BITOP_MASK(nr);
+	unsigned long *p = ((unsigned long *)addr) + BITOP_WORD(nr);
+	unsigned long old = *p;
+
+	*p = old & ~mask;
+	return (old & mask) != 0;
+}
+
+/* WARNING: non atomic and it can be reordered! */
+static inline int __test_and_change_bit(int nr,
+					    volatile unsigned long *addr)
+{
+	unsigned long mask = BITOP_MASK(nr);
+	unsigned long *p = ((unsigned long *)addr) + BITOP_WORD(nr);
+	unsigned long old = *p;
+
+	*p = old ^ mask;
+	return (old & mask) != 0;
+}
+
+/**
+ * test_bit - Determine whether a bit is set
+ * @nr: bit number to test
+ * @addr: Address to start counting from
+ */
+static inline int test_bit(int nr, const volatile unsigned long *addr)
+{
+	return 1UL & (addr[BITOP_WORD(nr)] >> (nr & (BITS_PER_LONG-1)));
+}
+
+#endif /* _ASM_GENERIC_BITOPS_NON_ATOMIC_H_ */

--

  parent reply	other threads:[~2006-02-14  5:22 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 ` Akinobu Mita [this message]
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 ` [patch 19/47] alpha: use generic bitops Akinobu Mita
2006-02-14  5:04 ` [patch 20/47] arm: " 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=20060214050443.068971000@localhost.localdomain \
    --to=mita@miraclelinux.com \
    --cc=akpm@osdl.org \
    --cc=chris@zankel.net \
    --cc=dev-etrax@axis.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@linux-mips.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=linuxsh-dev@lists.sourceforge.net \
    --cc=linuxsh-shmedia-dev@lists.sourceforge.net \
    --cc=parisc-linux@parisc-linux.org \
    --cc=rmk@arm.linux.org.uk \
    --cc=sparclinux@vger.kernel.org \
    --cc=spyro@f2s.com \
    --cc=takata@linux-m32r.org \
    --cc=ultralinux@vger.kernel.org \
    /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