diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-08-02 11:26:56 +0200 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-08-03 13:57:30 +0200 |
| commit | ff2e6c647d166fa54769f3c300855ef7f8020668 (patch) | |
| tree | 2fe5ea9069fc561d1344e54e0524950aefd86e21 /cpukit/score/src/threadinitialize.c | |
| parent | posix: Fix for RTEMS_DEBUG (diff) | |
| download | rtems-ff2e6c647d166fa54769f3c300855ef7f8020668.tar.bz2 | |
score: Fix and simplify thread wait locks
There was a subtile race condition in _Thread_queue_Do_extract_locked().
It must first update the thread wait flags and then restore the default
thread wait state. In the previous implementation this could lead under
rare timing conditions to an ineffective _Thread_Wait_tranquilize()
resulting to a corrupt system state.
Update #2556.
Diffstat (limited to 'cpukit/score/src/threadinitialize.c')
| -rw-r--r-- | cpukit/score/src/threadinitialize.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cpukit/score/src/threadinitialize.c b/cpukit/score/src/threadinitialize.c index 209be9ebb1..b5fef590fc 100644 --- a/cpukit/score/src/threadinitialize.c +++ b/cpukit/score/src/threadinitialize.c @@ -186,7 +186,7 @@ bool _Thread_Initialize( &the_thread->Wait.Lock.Default, "Thread Wait Default Lock" ); - _Chain_Initialize_empty( &the_thread->Wait.Lock.Pending_requests ); + _Thread_queue_Gate_open( &the_thread->Wait.Lock.Tranquilizer ); _SMP_lock_Stats_initialize( &the_thread->Potpourri_stats, "Thread Potpourri" ); #endif |