diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2015-02-17 10:00:43 +0100 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2015-02-17 13:42:53 +0100 |
commit | 8d6e6eebedc1b70d8ae6c1dac13393b9e9c1a3b1 (patch) | |
tree | 3dc2d588f6bf8e0ea9c2725692e54f19f91f965a /cpukit/score/src/threadhandler.c | |
parent | score: Add _CPU_SMP_Prepare_start_multitasking() (diff) | |
download | rtems-8d6e6eebedc1b70d8ae6c1dac13393b9e9c1a3b1.tar.bz2 |
score: Fix FP context restore via _Thread_Handler
After a context switch we end up in the second part of
_Thread_Dispatch() or in _Thread_Handler() in case of new threads. Use
the same function _Thread_Restore_fp() to restore the floating-point
context. It makes no sense to do this in _Thread_Start_multitasking().
This fixes also a race condition in SMP configurations.
Update #2268.
Diffstat (limited to 'cpukit/score/src/threadhandler.c')
-rw-r--r-- | cpukit/score/src/threadhandler.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/cpukit/score/src/threadhandler.c b/cpukit/score/src/threadhandler.c index f8a9a62429..db7302807c 100644 --- a/cpukit/score/src/threadhandler.c +++ b/cpukit/score/src/threadhandler.c @@ -51,16 +51,7 @@ void _Thread_Handler( void ) * through _Thread_Dispatch on our first invocation. So the normal * code path for performing the FP context switch is not hit. */ - #if ( CPU_HARDWARE_FP == TRUE ) || ( CPU_SOFTWARE_FP == TRUE ) - #if ( CPU_USE_DEFERRED_FP_SWITCH == TRUE ) - if ( (executing->fp_context != NULL) && - !_Thread_Is_allocated_fp( executing ) ) { - if ( _Thread_Allocated_fp != NULL ) - _Context_Save_fp( &_Thread_Allocated_fp->fp_context ); - _Thread_Allocated_fp = executing; - } - #endif - #endif + _Thread_Restore_fp( executing ); /* * Take care that 'begin' extensions get to complete before |