Skip to content

Commit d12b802

Browse files
Peter ZijlstraKAGA-KOKO
authored andcommitted
locking/rtmutex: Fix misleading comment
Going through the RCU-boost and rtmutex code, I ran into this utterly confusing comment. Fix it to avoid confusing future readers. [ tglx: Wordsmithed the comment ] Signed-off-by: Peter Zijlstra (Intel) <[email protected]> Signed-off-by: Thomas Gleixner <[email protected]> Reviewed-by: Sebastian Andrzej Siewior <[email protected]> Link: https://lore.kernel.org/all/[email protected]
1 parent 77abd3b commit d12b802

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

kernel/locking/rtmutex_api.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -175,10 +175,10 @@ bool __sched __rt_mutex_futex_unlock(struct rt_mutex_base *lock,
175175
}
176176

177177
/*
178-
* We've already deboosted, mark_wakeup_next_waiter() will
179-
* retain preempt_disabled when we drop the wait_lock, to
180-
* avoid inversion prior to the wakeup. preempt_disable()
181-
* therein pairs with rt_mutex_postunlock().
178+
* mark_wakeup_next_waiter() deboosts and retains preemption
179+
* disabled when dropping the wait_lock, to avoid inversion prior
180+
* to the wakeup. preempt_disable() therein pairs with the
181+
* preempt_enable() in rt_mutex_postunlock().
182182
*/
183183
mark_wakeup_next_waiter(wqh, lock);
184184

0 commit comments

Comments
 (0)