diff options
Diffstat (limited to 'kernel/locking')
| -rw-r--r-- | kernel/locking/mutex.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/kernel/locking/mutex.c b/kernel/locking/mutex.c index 16b2d3cc88b0..4cccea6b8934 100644 --- a/kernel/locking/mutex.c +++ b/kernel/locking/mutex.c | |||
| @@ -224,20 +224,14 @@ ww_mutex_set_context_slowpath(struct ww_mutex *lock, | |||
| 224 | static noinline | 224 | static noinline |
| 225 | bool mutex_spin_on_owner(struct mutex *lock, struct task_struct *owner) | 225 | bool mutex_spin_on_owner(struct mutex *lock, struct task_struct *owner) |
| 226 | { | 226 | { |
| 227 | bool ret; | 227 | bool ret = true; |
| 228 | 228 | ||
| 229 | rcu_read_lock(); | 229 | rcu_read_lock(); |
| 230 | while (true) { | 230 | while (lock->owner == owner) { |
| 231 | /* Return success when the lock owner changed */ | ||
| 232 | if (lock->owner != owner) { | ||
| 233 | ret = true; | ||
| 234 | break; | ||
| 235 | } | ||
| 236 | |||
| 237 | /* | 231 | /* |
| 238 | * Ensure we emit the owner->on_cpu, dereference _after_ | 232 | * Ensure we emit the owner->on_cpu, dereference _after_ |
| 239 | * checking lock->owner still matches owner, if that fails, | 233 | * checking lock->owner still matches owner. If that fails, |
| 240 | * owner might point to free()d memory, if it still matches, | 234 | * owner might point to freed memory. If it still matches, |
| 241 | * the rcu_read_lock() ensures the memory stays valid. | 235 | * the rcu_read_lock() ensures the memory stays valid. |
| 242 | */ | 236 | */ |
| 243 | barrier(); | 237 | barrier(); |
