Changeset 042072b in rtems

Timestamp:

06/14/16 08:12:34 (7 years ago)

Author:

Sebastian Huber <sebastian.huber@…>

Branches:

5, master

Children:

b61d5cac

Parents:

3eda7e5

git-author:

Sebastian Huber <sebastian.huber@…> (06/14/16 08:12:34)

git-committer:

Sebastian Huber <sebastian.huber@…> (06/14/16 09:57:47)

Message:

score: _Scheduler_priority_Ready_queue_initialize

Use priority maximum of scheduler instance. This avoids a potential
memory corruption on SMP configurations.

Location:

cpukit/score

Files:

• include/rtems/score/schedulerpriorityimpl.h (modified) (1 diff)
• src/schedulerpriority.c (modified) (1 diff)
• src/schedulerprioritysmp.c (modified) (1 diff)
• src/schedulerstrongapa.c (modified) (1 diff)

cpukit/score/include/rtems/score/schedulerpriorityimpl.h

@@ -55 +55 @@
  */
 RTEMS_INLINE_ROUTINE void _Scheduler_priority_Ready_queue_initialize(
-  Chain_Control *ready_queues
+  Chain_Control    *ready_queues,
+  Priority_Control  maximum_priority
 )
 {
   size_t index;
 
-  /* initialize ready queue structures */
-  for( index=0; index <= PRIORITY_MAXIMUM; index++)
-    _Chain_Initialize_empty( &ready_queues[index] );
+  for ( index = 0 ; index <= (size_t) maximum_priority ; ++index ) {
+    _Chain_Initialize_empty( &ready_queues[ index ] );
+  }
 }
 

cpukit/score/src/schedulerpriority.c

@@ -28 +28 @@
 
   _Priority_bit_map_Initialize( &context->Bit_map );
-  _Scheduler_priority_Ready_queue_initialize( &context->Ready[ 0 ] );
+  _Scheduler_priority_Ready_queue_initialize(
+    &context->Ready[ 0 ],
+    scheduler->maximum_priority
+  );
 }

cpukit/score/src/schedulerprioritysmp.c

@@ -40 +40 @@
   _Scheduler_SMP_Initialize( &self->Base );
   _Priority_bit_map_Initialize( &self->Bit_map );
-  _Scheduler_priority_Ready_queue_initialize( &self->Ready[ 0 ] );
+  _Scheduler_priority_Ready_queue_initialize(
+    &self->Ready[ 0 ],
+    scheduler->maximum_priority
+  );
 }
 

cpukit/score/src/schedulerstrongapa.c

@@ -166 +166 @@
   _Scheduler_SMP_Initialize( &self->Base );
   _Priority_bit_map_Initialize( &self->Bit_map );
-  _Scheduler_priority_Ready_queue_initialize( &self->Ready[ 0 ] );
+  _Scheduler_priority_Ready_queue_initialize(
+    &self->Ready[ 0 ],
+    scheduler->maximum_priority
+  );
 }