diff options
| author |   Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-06 14:42:57 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.osdl.org> | 2006-12-06 14:42:57 -0800 |
| commit | f9e9dcb38f5106fa8cdac04a9e967d5487f1cd20 (patch) | |
| tree | 8d1cbf17b9b54eacde5a7bf7adf4d5ba6cb119dc /include/asm-i386/atomic.h | |
| parent | [PATCH] Remove 'volatile' from spinlock_types (diff) | |
| download | linux-dev-f9e9dcb38f5106fa8cdac04a9e967d5487f1cd20.tar.xz linux-dev-f9e9dcb38f5106fa8cdac04a9e967d5487f1cd20.zip | |
x86[-64]:Remove 'volatile' from atomic_t
Any code that relies on the volatile would be a bug waiting to happen
anyway.
Don't encourage people to think that putting 'volatile' on data
structures somehow fixes problems. We should always use proper locking
(and other serialization) techniques.
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'include/asm-i386/atomic.h')
| -rw-r--r-- | include/asm-i386/atomic.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/asm-i386/atomic.h b/include/asm-i386/atomic.h index 51a166242522..a6c024e2506f 100644 --- a/include/asm-i386/atomic.h +++ b/include/asm-i386/atomic.h @@ -14,7 +14,7 @@ * on us. We need to use _exactly_ the address the user gave us, * not some alias that contains the same information. */ -typedef struct { volatile int counter; } atomic_t; +typedef struct { int counter; } atomic_t; #define ATOMIC_INIT(i) { (i) } |