summaryrefslogtreecommitdiffstats
path: root/cpukit/score/src/schedulerpriorityaffinitysmp.c
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2017-10-10 11:22:21 +0200
committerSebastian Huber <sebastian.huber@embedded-brains.de>2017-10-11 07:37:54 +0200
commita3ad4afba883e56447aa40f92c1ad08d835d2564 (patch)
tree1736c0390a1d4fbb4f86ab22f7811a5a099fdbaa /cpukit/score/src/schedulerpriorityaffinitysmp.c
parentscore: Add _Thread_queue_Dispatch_disable() (diff)
downloadrtems-a3ad4afba883e56447aa40f92c1ad08d835d2564.tar.bz2
posix: Validate affinity sets by the scheduler
Update #2514.
Diffstat (limited to 'cpukit/score/src/schedulerpriorityaffinitysmp.c')
-rw-r--r--cpukit/score/src/schedulerpriorityaffinitysmp.c19
1 files changed, 8 insertions, 11 deletions
diff --git a/cpukit/score/src/schedulerpriorityaffinitysmp.c b/cpukit/score/src/schedulerpriorityaffinitysmp.c
index c35883f142..e8301226f3 100644
--- a/cpukit/score/src/schedulerpriorityaffinitysmp.c
+++ b/cpukit/score/src/schedulerpriorityaffinitysmp.c
@@ -23,9 +23,6 @@
#include <rtems/score/schedulerpriorityimpl.h>
#include <rtems/score/schedulersmpimpl.h>
#include <rtems/score/schedulerprioritysmpimpl.h>
-#include <rtems/score/wkspace.h>
-#include <rtems/score/cpusetimpl.h>
-
#include <rtems/score/priority.h>
/*
@@ -606,27 +603,27 @@ bool _Scheduler_priority_affinity_SMP_Set_affinity(
const Processor_mask *affinity
)
{
+ Scheduler_Context *context;
Scheduler_priority_affinity_SMP_Node *node;
States_Control current_state;
+ Processor_mask my_affinity;
cpu_set_t cpuset;
- size_t cpusetsize;
- cpusetsize = sizeof( cpuset );
- _Processor_mask_To_cpu_set_t( affinity, cpusetsize, &cpuset );
- /*
- * Validate that the cpset meets basic requirements.
- */
- if ( !_CPU_set_Is_valid( &cpuset, cpusetsize ) ) {
+ context = _Scheduler_Get_context( scheduler );
+ _Processor_mask_And( &my_affinity, &context->Processors, affinity );
+
+ if ( _Processor_mask_Count( &my_affinity ) == 0 ) {
return false;
}
+ _Processor_mask_To_cpu_set_t( &my_affinity, sizeof( cpuset ), &cpuset );
node = _Scheduler_priority_affinity_SMP_Node_downcast( node_base );
/*
* The old and new set are the same, there is no point in
* doing anything.
*/
- if ( CPU_EQUAL_S( cpusetsize, &cpuset, node->Affinity.set ) )
+ if ( CPU_EQUAL( &cpuset, node->Affinity.set ) )
return true;
current_state = thread->current_state;