[rtems commit] score: Remove _Thread_queue_Do_nothing_extract()

Sebastian Huber sebh at rtems.org
Fri Oct 1 11:14:31 UTC 2021


Module:    rtems
Branch:    master
Commit:    18c8a270c296addff87f96b8c248f27eba31c24f
Changeset: http://git.rtems.org/rtems/commit/?id=18c8a270c296addff87f96b8c248f27eba31c24f

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Thu Sep 30 13:48:31 2021 +0200

score: Remove _Thread_queue_Do_nothing_extract()

This function was unused.  It was a relict of the thread queue rework done
during the SMP support development.  In an early stage, the extract operation
was called with a NULL thread queue.  However, this is no longer the case.  The
extract operation is only called if we have a non-NULL thread queue.

---

 cpukit/score/src/threadqops.c | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

diff --git a/cpukit/score/src/threadqops.c b/cpukit/score/src/threadqops.c
index a42876c..972af21 100644
--- a/cpukit/score/src/threadqops.c
+++ b/cpukit/score/src/threadqops.c
@@ -59,17 +59,6 @@ void _Thread_queue_Do_nothing_priority_actions(
   _Priority_Actions_initialize_empty( priority_actions );
 }
 
-static void _Thread_queue_Do_nothing_extract(
-  Thread_queue_Queue   *queue,
-  Thread_Control       *the_thread,
-  Thread_queue_Context *queue_context
-)
-{
-  (void) queue;
-  (void) the_thread;
-  (void) queue_context;
-}
-
 static void _Thread_queue_Queue_enqueue(
   Thread_queue_Queue   *queue,
   Thread_Control       *the_thread,
@@ -1461,12 +1450,11 @@ static Thread_Control *_Thread_queue_Priority_inherit_surrender(
 }
 
 const Thread_queue_Operations _Thread_queue_Operations_default = {
-  .priority_actions = _Thread_queue_Do_nothing_priority_actions,
-  .extract = _Thread_queue_Do_nothing_extract
+  .priority_actions = _Thread_queue_Do_nothing_priority_actions
   /*
    * The default operations are only used in _Thread_Priority_apply() and
    * _Thread_Continue() and do not have a thread queue associated with them, so
-   * the enqueue and first operations are superfluous.
+   * the enqueue, extract, surrender, and first operations are superfluous.
    */
 };
 



More information about the vc mailing list