diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2017-07-09 13:47:50 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-07-09 13:47:50 -0400 |
commit | c8b2ba83fb01336f094226895087b644df0ec397 (patch) | |
tree | 2f3daeaf155b12bf50137efbdc5c764eb314a716 | |
parent | 7cb328c30a71a450278031f932d2134c11165f4c (diff) | |
parent | 5671360f29c68d9079914438f6a0109ef62f82a8 (diff) |
Merge branch 'locking-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
Pull locking fixes from Thomas Gleixner:
- Fix the EINTR logic in rwsem-spinlock to avoid double locking by a
writer and a reader
- Add a missing include to qspinlocks
* 'locking-urgent-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip:
locking/qspinlock: Explicitly include asm/prefetch.h
locking/rwsem-spinlock: Fix EINTR branch in __down_write_common()
-rw-r--r-- | kernel/locking/qspinlock.c | 1 | ||||
-rw-r--r-- | kernel/locking/rwsem-spinlock.c | 4 |
2 files changed, 3 insertions, 2 deletions
diff --git a/kernel/locking/qspinlock.c b/kernel/locking/qspinlock.c index b2caec7315af..fd24153e8a48 100644 --- a/kernel/locking/qspinlock.c +++ b/kernel/locking/qspinlock.c | |||
@@ -28,6 +28,7 @@ | |||
28 | #include <linux/percpu.h> | 28 | #include <linux/percpu.h> |
29 | #include <linux/hardirq.h> | 29 | #include <linux/hardirq.h> |
30 | #include <linux/mutex.h> | 30 | #include <linux/mutex.h> |
31 | #include <linux/prefetch.h> | ||
31 | #include <asm/byteorder.h> | 32 | #include <asm/byteorder.h> |
32 | #include <asm/qspinlock.h> | 33 | #include <asm/qspinlock.h> |
33 | 34 | ||
diff --git a/kernel/locking/rwsem-spinlock.c b/kernel/locking/rwsem-spinlock.c index c65f7989f850..20819df98125 100644 --- a/kernel/locking/rwsem-spinlock.c +++ b/kernel/locking/rwsem-spinlock.c | |||
@@ -231,8 +231,8 @@ int __sched __down_write_common(struct rw_semaphore *sem, int state) | |||
231 | 231 | ||
232 | out_nolock: | 232 | out_nolock: |
233 | list_del(&waiter.list); | 233 | list_del(&waiter.list); |
234 | if (!list_empty(&sem->wait_list)) | 234 | if (!list_empty(&sem->wait_list) && sem->count >= 0) |
235 | __rwsem_do_wake(sem, 1); | 235 | __rwsem_do_wake(sem, 0); |
236 | raw_spin_unlock_irqrestore(&sem->wait_lock, flags); | 236 | raw_spin_unlock_irqrestore(&sem->wait_lock, flags); |
237 | 237 | ||
238 | return -EINTR; | 238 | return -EINTR; |