[rtems commit] score: Simplify _Thread_Start()
Sebastian Huber
sebh at rtems.org
Mon Jan 11 07:47:32 UTC 2016
Module: rtems
Branch: master
Commit: 1506658c07ad05dd58e2781af4d3df67c90b9f70
Changeset: http://git.rtems.org/rtems/commit/?id=1506658c07ad05dd58e2781af4d3df67c90b9f70
Author: Sebastian Huber <sebastian.huber at embedded-brains.de>
Date: Fri Jan 8 12:11:03 2016 +0100
score: Simplify _Thread_Start()
---
cpukit/posix/src/pthreadcreate.c | 2 +-
cpukit/rtems/src/taskstart.c | 2 +-
cpukit/score/include/rtems/score/threadimpl.h | 6 +-----
cpukit/score/src/threadcreateidle.c | 31 +++++++++++++++++----------
cpukit/score/src/threadstart.c | 17 ++-------------
5 files changed, 25 insertions(+), 33 deletions(-)
diff --git a/cpukit/posix/src/pthreadcreate.c b/cpukit/posix/src/pthreadcreate.c
index 611477e..3185ab6 100644
--- a/cpukit/posix/src/pthreadcreate.c
+++ b/cpukit/posix/src/pthreadcreate.c
@@ -228,7 +228,7 @@ int pthread_create(
/*
* POSIX threads are allocated and started in one operation.
*/
- status = _Thread_Start( the_thread, &entry, NULL );
+ status = _Thread_Start( the_thread, &entry );
#if defined(RTEMS_DEBUG)
/*
diff --git a/cpukit/rtems/src/taskstart.c b/cpukit/rtems/src/taskstart.c
index 39b3e9c..416c2df 100644
--- a/cpukit/rtems/src/taskstart.c
+++ b/cpukit/rtems/src/taskstart.c
@@ -64,7 +64,7 @@ rtems_status_code rtems_task_start(
switch ( location ) {
case OBJECTS_LOCAL:
- successfully_started = _Thread_Start( the_thread, &entry, NULL );
+ successfully_started = _Thread_Start( the_thread, &entry );
_Objects_Put( &the_thread->Object );
diff --git a/cpukit/score/include/rtems/score/threadimpl.h b/cpukit/score/include/rtems/score/threadimpl.h
index f212e23..4177a97 100644
--- a/cpukit/score/include/rtems/score/threadimpl.h
+++ b/cpukit/score/include/rtems/score/threadimpl.h
@@ -187,14 +187,10 @@ bool _Thread_Initialize(
*
* @param the_thread The thread to be started.
* @param entry The thread entry information.
- * @param[in,out] cpu The processor if used to start an idle thread
- * during system initialization. Must be set to @c NULL to start a normal
- * thread.
*/
bool _Thread_Start(
Thread_Control *the_thread,
- const Thread_Entry_information *entry,
- Per_CPU_Control *cpu
+ const Thread_Entry_information *entry
);
bool _Thread_Restart(
diff --git a/cpukit/score/src/threadcreateidle.c b/cpukit/score/src/threadcreateidle.c
index e9d12dc..3430881 100644
--- a/cpukit/score/src/threadcreateidle.c
+++ b/cpukit/score/src/threadcreateidle.c
@@ -21,20 +21,14 @@
#include <rtems/score/threadimpl.h>
#include <rtems/score/schedulerimpl.h>
#include <rtems/score/stackimpl.h>
+#include <rtems/score/userextimpl.h>
#include <rtems/config.h>
static void _Thread_Create_idle_for_cpu( Per_CPU_Control *cpu )
{
- Thread_Entry_information entry = {
- .adaptor = _Thread_Entry_adaptor_idle,
- .Kinds = {
- .Idle = {
- .entry = rtems_configuration_get_idle_task()
- }
- }
- };
- Objects_Name name;
- Thread_Control *idle;
+ Objects_Name name;
+ Thread_Control *idle;
+ const Scheduler_Control *scheduler;
name.name_u32 = _Objects_Build_name( 'I', 'D', 'L', 'E' );
@@ -67,7 +61,22 @@ static void _Thread_Create_idle_for_cpu( Per_CPU_Control *cpu )
cpu->heir =
cpu->executing = idle;
- _Thread_Start( idle, &entry, cpu );
+ idle->Start.Entry.adaptor = _Thread_Entry_adaptor_idle;
+ idle->Start.Entry.Kinds.Idle.entry = rtems_configuration_get_idle_task();
+
+ _Thread_Load_environment( idle );
+
+ scheduler = _Scheduler_Get_by_CPU( cpu );
+
+#if defined(RTEMS_SMP)
+ if (scheduler == NULL) {
+ return;
+ }
+#endif
+
+ idle->current_state = STATES_READY;
+ _Scheduler_Start_idle( scheduler, idle, cpu );
+ _User_extensions_Thread_start( idle );
}
void _Thread_Create_idle( void )
diff --git a/cpukit/score/src/threadstart.c b/cpukit/score/src/threadstart.c
index 97399ba..0cdf2bc 100644
--- a/cpukit/score/src/threadstart.c
+++ b/cpukit/score/src/threadstart.c
@@ -21,30 +21,17 @@
#include <rtems/score/threadimpl.h>
#include <rtems/score/isrlevel.h>
-#include <rtems/score/schedulerimpl.h>
#include <rtems/score/userextimpl.h>
bool _Thread_Start(
Thread_Control *the_thread,
- const Thread_Entry_information *entry,
- Per_CPU_Control *cpu
+ const Thread_Entry_information *entry
)
{
if ( _States_Is_dormant( the_thread->current_state ) ) {
the_thread->Start.Entry = *entry;
_Thread_Load_environment( the_thread );
-
- if ( cpu == NULL ) {
- _Thread_Ready( the_thread );
- } else {
- const Scheduler_Control *scheduler = _Scheduler_Get_by_CPU( cpu );
-
- if ( scheduler != NULL ) {
- the_thread->current_state = STATES_READY;
- _Scheduler_Start_idle( scheduler, the_thread, cpu );
- }
- }
-
+ _Thread_Ready( the_thread );
_User_extensions_Thread_start( the_thread );
return true;
More information about the vc
mailing list