diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-07-25 16:35:37 +0200 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-07-27 10:55:30 +0200 |
| commit | 1fcac5adc5ed38fb88ce4c6d24b2ca2e27e3cd10 (patch) | |
| tree | 7d2fed265befa680902ff146acb3305a360f9116 /cpukit/rtems/src/eventsurrender.c | |
| parent | score: Priority inherit thread queue operations (diff) | |
| download | rtems-1fcac5adc5ed38fb88ce4c6d24b2ca2e27e3cd10.tar.bz2 | |
score: Turn thread lock into thread wait lock
The _Thread_Lock_acquire() function had a potentially infinite run-time
due to the lack of fairness at atomic operations level.
Update #2412.
Update #2556.
Update #2765.
Diffstat (limited to 'cpukit/rtems/src/eventsurrender.c')
| -rw-r--r-- | cpukit/rtems/src/eventsurrender.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/cpukit/rtems/src/eventsurrender.c b/cpukit/rtems/src/eventsurrender.c index a9bef5916c..c805b0e89d 100644 --- a/cpukit/rtems/src/eventsurrender.c +++ b/cpukit/rtems/src/eventsurrender.c @@ -77,7 +77,7 @@ rtems_status_code _Event_Surrender( rtems_event_set seized_events; bool unblock; - _Thread_Lock_acquire_default_critical( the_thread, lock_context ); + _Thread_Wait_acquire_default_critical( the_thread, lock_context ); _Event_sets_Post( event_in, &event->pending_events ); pending_events = event->pending_events; @@ -116,14 +116,14 @@ rtems_status_code _Event_Surrender( Per_CPU_Control *cpu_self; cpu_self = _Thread_Dispatch_disable_critical( lock_context ); - _Thread_Lock_release_default( the_thread, lock_context ); + _Thread_Wait_release_default( the_thread, lock_context ); _Thread_Timer_remove( the_thread ); _Thread_Unblock( the_thread ); _Thread_Dispatch_enable( cpu_self ); } else { - _Thread_Lock_release_default( the_thread, lock_context ); + _Thread_Wait_release_default( the_thread, lock_context ); } return RTEMS_SUCCESSFUL; |