diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-08-04 10:20:29 +0200 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-08-04 10:20:29 +0200 |
| commit | ee0e41355d1faa5f74085effc4f57c3a7fc1f884 (patch) | |
| tree | c845395523e084218ca6715977648b807d30605d /cpukit/score/src/schedulercbsreleasejob.c | |
| parent | score: Fix _Thread_queue_Path_release() (diff) | |
| download | rtems-ee0e41355d1faa5f74085effc4f57c3a7fc1f884.tar.bz2 | |
score: Fix a release/cancel job race condition
Split up the potential thread priority change in the scheduler
release/cancel job operation. Protect the rate monotonic period state
with a dedicated SMP lock. This avoids a race condition during
_Rate_monotonic_Timeout() while _Rate_monotonic_Cancel() is called on
another processor.
Diffstat (limited to 'cpukit/score/src/schedulercbsreleasejob.c')
| -rw-r--r-- | cpukit/score/src/schedulercbsreleasejob.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/cpukit/score/src/schedulercbsreleasejob.c b/cpukit/score/src/schedulercbsreleasejob.c index 124c02bac0..d2169af899 100644 --- a/cpukit/score/src/schedulercbsreleasejob.c +++ b/cpukit/score/src/schedulercbsreleasejob.c @@ -21,7 +21,7 @@ #include <rtems/score/schedulercbsimpl.h> -void _Scheduler_CBS_Release_job( +Thread_Control *_Scheduler_CBS_Release_job( const Scheduler_Control *scheduler, Thread_Control *the_thread, uint64_t deadline @@ -30,8 +30,6 @@ void _Scheduler_CBS_Release_job( Scheduler_CBS_Node *node; Scheduler_CBS_Server *serv_info; - _Scheduler_EDF_Release_job( scheduler, the_thread, deadline ); - node = _Scheduler_CBS_Thread_get_node( the_thread ); serv_info = node->cbs_server; @@ -39,4 +37,6 @@ void _Scheduler_CBS_Release_job( if ( serv_info != NULL ) { the_thread->cpu_time_budget = serv_info->parameters.budget; } + + return _Scheduler_EDF_Release_job( scheduler, the_thread, deadline ); } |