Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2026-06-06 00:14:22 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

sched/sched/remove_readytorun.c: Fix CPU affinity issues in SMP

Browse Source 
THOR-208

In "nxsched_remove_readytorun":
the logical may choose the head of g_readytorun(the greatest priority)
task as the next while ignoring the cpu affinity of it.

Signed-off-by: chenhonglin <chenhonglin@xiaomi.com>
Change-Id: I819dd6f3fc0feff4e378cfda2a6c581814065cb3
This commit is contained in:
chenhonglin
2021-09-24 10:19:02 +08:00
parent 7f3456de09
commit 1621fcb09e
1 changed files with 4 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
sched/sched/sched_removereadytorun.c
+4 -8
View File
@@ -209,20 +209,16 @@ bool nxsched_remove_readytorun(FAR struct tcb_s *rtcb)
if (rtrtcb != NULL && rtrtcb->sched_priority >= nxttcb->sched_priority)
{
FAR struct tcb_s *tmptcb;
/* The TCB at the head of the ready to run list has the higher
* priority. Remove that task from the head of the g_readytorun
* list and add to the head of the g_assignedtasks[cpu] list.
*/
tmptcb = (FAR struct tcb_s *)
dq_remfirst((FAR dq_queue_t *)&g_readytorun);
dq_rem((FAR dq_entry_t *)rtrtcb, (FAR dq_queue_t *)&g_readytorun);
dq_addfirst((FAR dq_entry_t *)rtrtcb, tasklist);
dq_addfirst((FAR dq_entry_t *)tmptcb, tasklist);
tmptcb->cpu = cpu;
nxttcb = tmptcb;
rtrtcb->cpu = cpu;
nxttcb = rtrtcb;
}
/* Will pre-emption be disabled after the switch? If the lockcount is