[PATCH 3/8] score: Optimize thread queue enqueue
Sebastian Huber
sebastian.huber at embedded-brains.de
Tue Sep 6 12:40:23 UTC 2016
Optimize the enqueue to empty thread queue case.
---
cpukit/score/src/threadqops.c | 47 +++++++++++++++++++++++++++----------------
1 file changed, 30 insertions(+), 17 deletions(-)
diff --git a/cpukit/score/src/threadqops.c b/cpukit/score/src/threadqops.c
index 41506bf..1545f29 100644
--- a/cpukit/score/src/threadqops.c
+++ b/cpukit/score/src/threadqops.c
@@ -45,27 +45,30 @@ static void _Thread_queue_Do_nothing_extract(
static Thread_queue_Heads *_Thread_queue_Queue_enqueue(
Thread_queue_Queue *queue,
Thread_Control *the_thread,
- void ( *initialize )( Thread_queue_Heads * ),
+ void ( *initialize )( Thread_queue_Heads *, Thread_Control * ),
void ( *enqueue )( Thread_queue_Heads *, Thread_Control * )
)
{
- Thread_queue_Heads *heads = queue->heads;
- Thread_queue_Heads *spare_heads = the_thread->Wait.spare_heads;
+ Thread_queue_Heads *heads;
+ Thread_queue_Heads *spare_heads;
+ heads = queue->heads;
+ spare_heads = the_thread->Wait.spare_heads;
the_thread->Wait.spare_heads = NULL;
if ( heads == NULL ) {
_Assert( spare_heads != NULL );
_Assert( _Chain_Is_empty( &spare_heads->Free_chain ) );
+
heads = spare_heads;
queue->heads = heads;
- ( *initialize )( heads );
+ _Chain_Prepend_unprotected( &heads->Free_chain, &spare_heads->Free_node );
+ ( *initialize )( heads, the_thread );
+ } else {
+ _Chain_Prepend_unprotected( &heads->Free_chain, &spare_heads->Free_node );
+ ( *enqueue )( heads, the_thread );
}
- _Chain_Prepend_unprotected( &heads->Free_chain, &spare_heads->Free_node );
-
- ( *enqueue )( heads, the_thread );
-
return heads;
}
@@ -92,10 +95,11 @@ static void _Thread_queue_Queue_extract(
}
static void _Thread_queue_FIFO_do_initialize(
- Thread_queue_Heads *heads
+ Thread_queue_Heads *heads,
+ Thread_Control *the_thread
)
{
- _Chain_Initialize_empty( &heads->Heads.Fifo );
+ _Chain_Initialize_one( &heads->Heads.Fifo, &the_thread->Wait.Node.Chain );
}
static void _Thread_queue_FIFO_do_enqueue(
@@ -235,14 +239,22 @@ static void _Thread_queue_Priority_priority_change(
}
static void _Thread_queue_Priority_do_initialize(
- Thread_queue_Heads *heads
+ Thread_queue_Heads *heads,
+ Thread_Control *the_thread
)
{
+ Thread_queue_Priority_queue *priority_queue;
+
+ priority_queue = _Thread_queue_Priority_queue( heads, the_thread );
+
#if defined(RTEMS_SMP)
- _Chain_Initialize_empty( &heads->Heads.Fifo );
-#else
- _RBTree_Initialize_empty( &heads->Heads.Priority.Queue );
+ _Chain_Initialize_one( &heads->Heads.Fifo, &priority_queue->Node );
#endif
+
+ _RBTree_Initialize_one(
+ &priority_queue->Queue,
+ &the_thread->Wait.Node.RBTree
+ );
}
static void _Thread_queue_Priority_do_enqueue(
@@ -250,9 +262,10 @@ static void _Thread_queue_Priority_do_enqueue(
Thread_Control *the_thread
)
{
- Thread_queue_Priority_queue *priority_queue =
- _Thread_queue_Priority_queue( heads, the_thread );
- Priority_Control current_priority;
+ Thread_queue_Priority_queue *priority_queue;
+ Priority_Control current_priority;
+
+ priority_queue = _Thread_queue_Priority_queue( heads, the_thread );
#if defined(RTEMS_SMP)
if ( _RBTree_Is_empty( &priority_queue->Queue ) ) {
--
1.8.4.5
More information about the devel
mailing list