source: rtems/cpukit/score/src/scheduleredfunblock.c @ dafa5d88

5
Last change on this file since dafa5d88 was b8a5abf, checked in by Sebastian Huber <sebastian.huber@…>, on 02/26/15 at 09:33:36

score: Update _Thread_Heir only if necessary

Previously, the _Thread_Heir was updated unconditionally in case a new
heir was determined. The _Thread_Dispatch_necessary was only updated in
case the executing thread was preemptible or an internal thread was
unblocked. Change this to update the _Thread_Heir and
_Thread_Dispatch_necessary only in case the currently selected heir
thread is preemptible or a dispatch is forced. Move the schedule
decision into the change priority operation and use the schedule
operation only in rtems_task_mode() in case preemption is enabled or an
ASR dispatch is necessary. This is a behaviour change. Previously, the
RTEMS_NO_PREEMPT also prevented signal delivery in certain cases (not
always). Now, signal delivery is no longer influenced by
RTEMS_NO_PREEMPT. Since the currently selected heir thread is used to
determine if a new heir is chosen, non-preemptible heir threads
currently not executing now prevent a new heir. This may have an
application impact, see change test tm04. Document this change in sp04.

Update #2273.

  • Property mode set to 100644
File size: 1.4 KB
Line 
1/**
2 * @file
3 *
4 * @brief Scheduler EDF Unblock
5 * @ingroup ScoreScheduler
6 */
7
8/*
9 *  Copyright (C) 2011 Petr Benes.
10 *  Copyright (C) 2011 On-Line Applications Research Corporation (OAR).
11 *
12 *  The license and distribution terms for this file may be
13 *  found in the file LICENSE in this distribution or at
14 *  http://www.rtems.org/license/LICENSE.
15 */
16
17#if HAVE_CONFIG_H
18#include "config.h"
19#endif
20
21#include <rtems/score/scheduleredfimpl.h>
22#include <rtems/score/schedulerimpl.h>
23#include <rtems/score/thread.h>
24
25Scheduler_Void_or_thread _Scheduler_EDF_Unblock(
26  const Scheduler_Control *scheduler,
27  Thread_Control          *the_thread
28)
29{
30  _Scheduler_EDF_Enqueue( scheduler, the_thread );
31  /* TODO: flash critical section? */
32
33  /*
34   *  If the thread that was unblocked is more important than the heir,
35   *  then we have a new heir.  This may or may not result in a
36   *  context switch.
37   *
38   *  Normal case:
39   *    If the current thread is preemptible, then we need to do
40   *    a context switch.
41   *  Pseudo-ISR case:
42   *    Even if the thread isn't preemptible, if the new heir is
43   *    a pseudo-ISR system task, we need to do a context switch.
44   */
45  if ( _Scheduler_Is_priority_lower_than(
46         scheduler,
47         _Thread_Heir->current_priority,
48         the_thread->current_priority )) {
49    _Scheduler_Update_heir(
50      the_thread,
51      the_thread->current_priority == PRIORITY_PSEUDO_ISR
52    );
53  }
54
55  SCHEDULER_RETURN_VOID_OR_NULL;
56}
Note: See TracBrowser for help on using the repository browser.