diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2015-02-14 13:54:28 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-02-14 13:54:28 -0500 |
commit | c833e17e276bd5d5f174aa924c4f102754ebc2be (patch) | |
tree | 90fb3d7c9b6ab66fa5d651675f054d5a7aa8a01a /arch/x86/include | |
parent | fee5429e028c414d80d036198db30454cfd91b7a (diff) | |
parent | c5b19946eb76c67566aae6a84bf2b10ad59295ea (diff) |
Merge tag 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/borntraeger/linux
Pull ACCESS_ONCE() rule tightening from Christian Borntraeger:
"Tighten rules for ACCESS_ONCE
This series tightens the rules for ACCESS_ONCE to only work on scalar
types. It also contains the necessary fixups as indicated by build
bots of linux-next. Now everything is in place to prevent new
non-scalar users of ACCESS_ONCE and we can continue to convert code to
READ_ONCE/WRITE_ONCE"
* tag 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/borntraeger/linux:
kernel: Fix sparse warning for ACCESS_ONCE
next: sh: Fix compile error
kernel: tighten rules for ACCESS ONCE
mm/gup: Replace ACCESS_ONCE with READ_ONCE
x86/spinlock: Leftover conversion ACCESS_ONCE->READ_ONCE
x86/xen/p2m: Replace ACCESS_ONCE with READ_ONCE
ppc/hugetlbfs: Replace ACCESS_ONCE with READ_ONCE
ppc/kvm: Replace ACCESS_ONCE with READ_ONCE
Diffstat (limited to 'arch/x86/include')
-rw-r--r-- | arch/x86/include/asm/spinlock.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86/include/asm/spinlock.h b/arch/x86/include/asm/spinlock.h index 625660f8a2fc..7050d864f520 100644 --- a/arch/x86/include/asm/spinlock.h +++ b/arch/x86/include/asm/spinlock.h | |||
@@ -183,10 +183,10 @@ static __always_inline void arch_spin_lock_flags(arch_spinlock_t *lock, | |||
183 | 183 | ||
184 | static inline void arch_spin_unlock_wait(arch_spinlock_t *lock) | 184 | static inline void arch_spin_unlock_wait(arch_spinlock_t *lock) |
185 | { | 185 | { |
186 | __ticket_t head = ACCESS_ONCE(lock->tickets.head); | 186 | __ticket_t head = READ_ONCE(lock->tickets.head); |
187 | 187 | ||
188 | for (;;) { | 188 | for (;;) { |
189 | struct __raw_tickets tmp = ACCESS_ONCE(lock->tickets); | 189 | struct __raw_tickets tmp = READ_ONCE(lock->tickets); |
190 | /* | 190 | /* |
191 | * We need to check "unlocked" in a loop, tmp.head == head | 191 | * We need to check "unlocked" in a loop, tmp.head == head |
192 | * can be false positive because of overflow. | 192 | * can be false positive because of overflow. |