diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2013-06-05 11:48:57 +0200 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2013-06-07 17:06:43 +0200 |
| commit | 2d2352bab92c51c2fd857b9555242545bd08c95e (patch) | |
| tree | de11a05e5b361a161e93c98866aa704ed24ed3ae /cpukit/rtems/src/taskdelete.c | |
| parent | score: Add _Objects_Put_for_get_isr_disable() (diff) | |
| download | rtems-2d2352bab92c51c2fd857b9555242545bd08c95e.tar.bz2 | |
score: Add and use _Objects_Put()
Add and use _Objects_Put_without_thread_dispatch(). These two functions
pair with the _Objects_Get() function. This helps to introduce object
specific SMP locks to avoid lock contention.
Diffstat (limited to 'cpukit/rtems/src/taskdelete.c')
| -rw-r--r-- | cpukit/rtems/src/taskdelete.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/cpukit/rtems/src/taskdelete.c b/cpukit/rtems/src/taskdelete.c index 6ab579821f..492f3e73f4 100644 --- a/cpukit/rtems/src/taskdelete.c +++ b/cpukit/rtems/src/taskdelete.c @@ -54,11 +54,11 @@ rtems_status_code rtems_task_delete( the_information = _Objects_Get_information_id( the_thread->Object.id ); #if defined(RTEMS_DEBUG) - if ( !the_information ) { - _Thread_Enable_dispatch(); - return RTEMS_INVALID_ID; - /* This should never happen if _Thread_Get() works right */ - } + if ( !the_information ) { + _Objects_Put( &the_thread->Object ); + return RTEMS_INVALID_ID; + /* This should never happen if _Thread_Get() works right */ + } #endif #if defined(RTEMS_MULTIPROCESSING) @@ -76,8 +76,9 @@ rtems_status_code rtems_task_delete( _RTEMS_tasks_Free( the_thread ); + /* FIXME: Lock order reversal */ _RTEMS_Unlock_allocator(); - _Thread_Enable_dispatch(); + _Objects_Put( &the_thread->Object ); return RTEMS_SUCCESSFUL; #if defined(RTEMS_MULTIPROCESSING) |