svn commit: r313816 - head/sys/kern
Ryan Stone
rstone at FreeBSD.org
Thu Feb 16 20:06:22 UTC 2017
Author: rstone
Date: Thu Feb 16 20:06:21 2017
New Revision: 313816
URL: https://svnweb.freebsd.org/changeset/base/313816
Log:
Fix a typo in my previous commit
Somehow in the late stages of testing my sched_ule patch, a character was
accidentally deleted from the file. Correct this.
While I'm committing anyway, the previous commit message requires some
clarification: in the normal case of unlending priority after releasing
a mutex, the thread that was doing the lending will be woken up and
immediately become the highest-priority thread, and in that case no
priority inversion would take place. However, if that thread is pinned
to a different CPU, then the currently running thread that just had its
priority lowered will not be preempted and then priority inversion can
occur.
Reported by: O. Hartmann (typo), jhb (scheduler clarification)
MFC after: 1 month
Pointy hat to: rstone
Modified:
head/sys/kern/sched_ule.c
Modified: head/sys/kern/sched_ule.c
==============================================================================
--- head/sys/kern/sched_ule.c Thu Feb 16 19:58:02 2017 (r313815)
+++ head/sys/kern/sched_ule.c Thu Feb 16 20:06:21 2017 (r313816)
@@ -1047,7 +1047,7 @@ tdq_notify(struct tdq *tdq, int pri)
if (tdq->tdq_ipipending)
return;
- cpu = TD_ID(tdq);
+ cpu = TDQ_ID(tdq);
ctd = pcpu_find(cpu)->pc_curthread;
if (!sched_shouldpreempt(pri, ctd->td_priority, 1))
return;
More information about the svn-src-all
mailing list