Notice: We have migrated to GitLab launching 2024-05-01 see here: https://gitlab.rtems.org/

#2268 closed defect (fixed)

threadstartmultitasking.c has a FIXME on SMP path

Reported by: Joel Sherrill Owned by: Sebastian Huber
Priority: lowest Milestone: 4.11
Component: score Version: 4.11
Severity: critical Keywords:
Cc: Blocked By:
Blocking:

Description

If this comment is correct, then there should be a ticket to address this issue. If this problem still exists, then elaborate in this ticket and reference it in the code. The milestone may be wrong. Please adjust. I set it to 4.11 so we could evaluate it.

  • The _CPU_Context_Restart_self() implementations usually assume that self
  • context is executing. *
  • FIXME: We have a race condition here in case another thread already
  • performed scheduler operations and moved our heir thread to another
  • processor. The time frame for this is likely too small to be practically
  • relevant.

Change History (8)

comment:1 Changed on 02/13/15 at 20:57:44 by Sebastian Huber

Priority: normallowest
Severity: normalcritical

Yes, this problem still exists. In order to solve it we must update the is executing indicator similar to the code in _CPU_Context_switch().

comment:2 Changed on 02/13/15 at 21:00:03 by Joel Sherrill

OK. Can you update the comment to reference this ticket?

comment:3 Changed on 02/13/15 at 22:23:22 by Joel Sherrill

Component: GeneralSMP

comment:4 Changed on 02/14/15 at 13:40:43 by Sebastian Huber

Status: newassigned

comment:5 Changed on 02/17/15 at 08:41:55 by Sebastian Huber <sebastian.huber@…>

In [changeset:"c34f94f72df4e88e1c4ea7d7f97f14a9ec6296ef/rtems"]:

score: Add _CPU_SMP_Prepare_start_multitasking()

Update #2268.

comment:7 Changed on 10/10/17 at 06:27:10 by Sebastian Huber

Component: SMPscore

comment:8 Changed on 10/10/17 at 06:29:01 by Sebastian Huber

Component: scorecpukit
Note: See TracTickets for help on using tickets.