diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-11-24 06:13:11 +0100 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-11-24 08:46:20 +0100 |
| commit | 620b23ece54a0bb43189bb9c0f14adf447cbafe6 (patch) | |
| tree | 10aa72eb67ac01ab85f932ac25c09ba3e5d97da8 /cpukit/score/src/futex.c | |
| parent | score: Fix interrupt profiling (diff) | |
| download | rtems-620b23ece54a0bb43189bb9c0f14adf447cbafe6.tar.bz2 | |
score: Optimize _Thread_queue_Enqueue()
Move thread state for _Thread_queue_Enqueue() to the thread queue
context. This reduces the parameter count of _Thread_queue_Enqueue()
from five to four (ARM for example has only four function parameter
registers). Since the thread state is used after several function calls
inside _Thread_queue_Enqueue() this parameter was saved on the stack
previously.
Diffstat (limited to 'cpukit/score/src/futex.c')
| -rw-r--r-- | cpukit/score/src/futex.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/cpukit/score/src/futex.c b/cpukit/score/src/futex.c index efac831df5..f200895562 100644 --- a/cpukit/score/src/futex.c +++ b/cpukit/score/src/futex.c @@ -92,6 +92,10 @@ int _Futex_Wait( struct _Futex_Control *_futex, int *uaddr, int val ) executing = _Futex_Queue_acquire_critical( futex, &queue_context ); if ( *uaddr == val ) { + _Thread_queue_Context_set_thread_state( + &queue_context, + STATES_WAITING_FOR_SYS_LOCK_FUTEX + ); _Thread_queue_Context_set_do_nothing_enqueue_callout( &queue_context ); _Thread_queue_Context_set_no_timeout( &queue_context ); _Thread_queue_Context_set_ISR_level( &queue_context, level ); @@ -99,7 +103,6 @@ int _Futex_Wait( struct _Futex_Control *_futex, int *uaddr, int val ) &futex->Queue.Queue, FUTEX_TQ_OPERATIONS, executing, - STATES_WAITING_FOR_SYS_LOCK_FUTEX, &queue_context ); eno = 0; |