summaryrefslogtreecommitdiffstats
path: root/cpukit
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2016-06-14 17:23:03 +0200
committerSebastian Huber <sebastian.huber@embedded-brains.de>2016-06-22 14:00:28 +0200
commit6bab009a61ae9ca7f7ac76140f8f13fbaab784c6 (patch)
tree7f10aaf87cf912151429fb1cfbde65071e804d87 /cpukit
parentposix: Delete POSIX_API_Control::schedpolicy (diff)
downloadrtems-6bab009a61ae9ca7f7ac76140f8f13fbaab784c6.tar.bz2
posix: Delete POSIX_API_Control::schedparam
This field was redundant.
Diffstat (limited to 'cpukit')
-rw-r--r--cpukit/posix/include/rtems/posix/threadsup.h4
-rw-r--r--cpukit/posix/src/pthread.c14
-rw-r--r--cpukit/posix/src/pthreadcreate.c3
-rw-r--r--cpukit/posix/src/pthreadgetschedparam.c2
-rw-r--r--cpukit/posix/src/pthreadsetschedparam.c5
-rw-r--r--cpukit/posix/src/pthreadsetschedprio.c1
-rw-r--r--cpukit/score/include/rtems/score/thread.h1
7 files changed, 16 insertions, 14 deletions
diff --git a/cpukit/posix/include/rtems/posix/threadsup.h b/cpukit/posix/include/rtems/posix/threadsup.h
index a9f36ef1ad..e18a42995a 100644
--- a/cpukit/posix/include/rtems/posix/threadsup.h
+++ b/cpukit/posix/include/rtems/posix/threadsup.h
@@ -45,10 +45,10 @@ extern "C" {
typedef struct {
/** Back pointer to thread of this POSIX API control. */
Thread_Control *thread;
+
/** This is the POSIX threads attribute set. */
pthread_attr_t Attributes;
- /** This is the thread's current set of scheduling parameters. */
- struct sched_param schedparam;
+
/**
* This is the timer which controls when the thread executes at
* high and low priority when using the sporadic scheduler.
diff --git a/cpukit/posix/src/pthread.c b/cpukit/posix/src/pthread.c
index 33c2830aef..ead24cf4f7 100644
--- a/cpukit/posix/src/pthread.c
+++ b/cpukit/posix/src/pthread.c
@@ -116,7 +116,9 @@ void _POSIX_Threads_Sporadic_budget_TSR( Watchdog_Control *watchdog )
_Watchdog_Per_CPU_remove_relative( &api->Sporadic_timer );
_POSIX_Threads_Sporadic_timer_insert( the_thread, api );
- new_priority = _POSIX_Priority_To_core( api->schedparam.sched_priority );
+ new_priority = _POSIX_Priority_To_core(
+ api->Attributes.schedparam.sched_priority
+ );
_Thread_State_release( the_thread, &lock_context );
@@ -167,7 +169,9 @@ void _POSIX_Threads_Sporadic_budget_callout(
_Thread_Change_priority(
the_thread,
- _POSIX_Priority_To_core( api->schedparam.sched_ss_low_priority ),
+ _POSIX_Priority_To_core(
+ api->Attributes.schedparam.sched_ss_low_priority
+ ),
NULL,
_POSIX_Threads_Sporadic_budget_callout_filter,
true
@@ -193,9 +197,9 @@ static bool _POSIX_Threads_Create_extension(
/* XXX check all fields are touched */
api->thread = created;
_POSIX_Threads_Initialize_attributes( &api->Attributes );
- api->schedparam = _POSIX_Threads_Default_attributes.schedparam;
- api->schedparam.sched_priority =
- _POSIX_Priority_From_core( created->current_priority );
+ api->Attributes.schedparam.sched_priority = _POSIX_Priority_From_core(
+ created->current_priority
+ );
/*
* If the thread is not a posix thread, then all posix signals are blocked
diff --git a/cpukit/posix/src/pthreadcreate.c b/cpukit/posix/src/pthreadcreate.c
index 019ccc86a8..c7c233857a 100644
--- a/cpukit/posix/src/pthreadcreate.c
+++ b/cpukit/posix/src/pthreadcreate.c
@@ -111,7 +111,7 @@ int pthread_create(
case PTHREAD_INHERIT_SCHED:
api = executing->API_Extensions[ THREAD_API_POSIX ];
schedpolicy = api->Attributes.schedpolicy;
- schedparam = api->schedparam;
+ schedparam = api->Attributes.schedparam;
break;
case PTHREAD_EXPLICIT_SCHED:
@@ -226,7 +226,6 @@ int pthread_create(
api = the_thread->API_Extensions[ THREAD_API_POSIX ];
_POSIX_Threads_Copy_attributes( &api->Attributes, the_attr );
- api->schedparam = schedparam;
if ( schedpolicy == SCHED_SPORADIC ) {
_ISR_lock_ISR_disable( &lock_context );
diff --git a/cpukit/posix/src/pthreadgetschedparam.c b/cpukit/posix/src/pthreadgetschedparam.c
index d03a8a8113..ed149f2e2b 100644
--- a/cpukit/posix/src/pthreadgetschedparam.c
+++ b/cpukit/posix/src/pthreadgetschedparam.c
@@ -52,7 +52,7 @@ int pthread_getschedparam(
api = the_thread->API_Extensions[ THREAD_API_POSIX ];
*policy = api->Attributes.schedpolicy;
- *param = api->schedparam;
+ *param = api->Attributes.schedparam;
param->sched_priority = _POSIX_Priority_From_core(
the_thread->real_priority
);
diff --git a/cpukit/posix/src/pthreadsetschedparam.c b/cpukit/posix/src/pthreadsetschedparam.c
index 30ddd0347c..a2e13a0f17 100644
--- a/cpukit/posix/src/pthreadsetschedparam.c
+++ b/cpukit/posix/src/pthreadsetschedparam.c
@@ -84,7 +84,6 @@ int pthread_setschedparam(
_Watchdog_Per_CPU_remove_relative( &api->Sporadic_timer );
}
- api->schedparam = *param;
api->Attributes.schedpolicy = policy;
api->Attributes.schedparam = *param;
@@ -97,7 +96,9 @@ int pthread_setschedparam(
case SCHED_RR:
the_thread->cpu_time_budget =
rtems_configuration_get_ticks_per_timeslice();
- new_priority = _POSIX_Priority_To_core( api->schedparam.sched_priority );
+ new_priority = _POSIX_Priority_To_core(
+ api->Attributes.schedparam.sched_priority
+ );
break;
}
diff --git a/cpukit/posix/src/pthreadsetschedprio.c b/cpukit/posix/src/pthreadsetschedprio.c
index 856e49df04..b7166bc8c8 100644
--- a/cpukit/posix/src/pthreadsetschedprio.c
+++ b/cpukit/posix/src/pthreadsetschedprio.c
@@ -43,7 +43,6 @@ int pthread_setschedprio( pthread_t thread, int prio )
cpu_self = _Thread_Dispatch_disable_critical( &lock_context );
_Thread_State_acquire_critical( the_thread, &lock_context );
- api->schedparam.sched_priority = prio;
api->Attributes.schedparam.sched_priority = prio;
_Thread_State_release( the_thread, &lock_context );
diff --git a/cpukit/score/include/rtems/score/thread.h b/cpukit/score/include/rtems/score/thread.h
index d98658cb9d..7c5f0793c3 100644
--- a/cpukit/score/include/rtems/score/thread.h
+++ b/cpukit/score/include/rtems/score/thread.h
@@ -711,7 +711,6 @@ struct _Thread_Control {
* the following fields
*
* - POSIX_API_Control::Attributes,
- * - POSIX_API_Control::schedparam,
* - RTEMS_API_Control::Signal,
* - Thread_Control::budget_algorithm,
* - Thread_Control::budget_callout,