diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-06-29 15:33:26 +0200 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-06-30 07:57:44 +0200 |
commit | 029877282edb8aa7a2095702742ce95c8246729e (patch) | |
tree | b1fbd380bac4071d05712957f9d03c6cc7521299 /cpukit/rtems/src/eventseize.c | |
parent | score: Fix thread lock on SMP configurations (diff) | |
download | rtems-029877282edb8aa7a2095702742ce95c8246729e.tar.bz2 |
score: Avoid atomic fences for thread wait flags
The use of atomic fences is brittle and may break due to changes in
different areas which is hard to manage.
Diffstat (limited to 'cpukit/rtems/src/eventseize.c')
-rw-r--r-- | cpukit/rtems/src/eventseize.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/cpukit/rtems/src/eventseize.c b/cpukit/rtems/src/eventseize.c index c91d308438..7f5903df89 100644 --- a/cpukit/rtems/src/eventseize.c +++ b/cpukit/rtems/src/eventseize.c @@ -91,13 +91,7 @@ rtems_status_code _Event_Seize( _Thread_Set_state( executing, block_state ); - /* - * See _Event_Surrender() and _Thread_Timeout(), corresponding atomic - * variable is Thread_Control::Wait::flags. - */ - _Atomic_Fence( ATOMIC_ORDER_ACQUIRE ); - - success = _Thread_Wait_flags_try_change( + success = _Thread_Wait_flags_try_change_acquire( executing, intend_to_block, wait_class | THREAD_WAIT_STATE_BLOCKED |