Mirrors_Framework/rtems
1
0
Fork 0
mirror of https://gitlab.rtems.org/rtems/rtos/rtems.git synced 2026-09-28 01:08:52 +08:00
Code Issues Packages Projects Releases Wiki Activity

mpci: Avoid Giant lock

Browse Source 
The object creation/deletion is protected by the object allocator lock.

Update #2555.
This commit is contained in:
Sebastian Huber
2016-03-18 15:36:58 +01:00
parent 19e855d745
commit be88976440
1 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cpukit/score/src/objectmp.c
+11 -7
View File
@@ -91,7 +91,9 @@ void _Objects_MP_Open (
the_global_object->Object.id = the_id; the_global_object->Object.id = the_id;
the_global_object->name = the_name; the_global_object->name = the_name;
_Chain_Prepend( _Assert( _Objects_Allocator_is_owner() );
_Chain_Prepend_unprotected(
&information->global_table[ _Objects_Get_node( the_id ) ], &information->global_table[ _Objects_Get_node( the_id ) ],
&the_global_object->Object.Node &the_global_object->Object.Node
); );
@@ -135,6 +137,8 @@ void _Objects_MP_Close (
Chain_Node *the_node; Chain_Node *the_node;
Objects_MP_Control *the_object; Objects_MP_Control *the_object;
_Assert( _Objects_Allocator_is_owner() );
the_chain = &information->global_table[ _Objects_Get_node( the_id ) ]; the_chain = &information->global_table[ _Objects_Get_node( the_id ) ];
for ( the_node = _Chain_First( the_chain ) ; for ( the_node = _Chain_First( the_chain ) ;
@@ -145,7 +149,7 @@ void _Objects_MP_Close (
if ( _Objects_Are_ids_equal( the_object->Object.id, the_id ) ) { if ( _Objects_Are_ids_equal( the_object->Object.id, the_id ) ) {
_Chain_Extract( the_node ); _Chain_Extract_unprotected( the_node );
_Objects_MP_Free_global_object( the_object ); _Objects_MP_Free_global_object( the_object );
return; return;
} }
@@ -191,7 +195,7 @@ Objects_Name_or_id_lookup_errors _Objects_MP_Global_name_search (
high_node = nodes_to_search; high_node = nodes_to_search;
} }
_Thread_Disable_dispatch(); _Objects_Allocator_lock();
for ( node_index = low_node ; node_index <= high_node ; node_index++ ) { for ( node_index = low_node ; node_index <= high_node ; node_index++ ) {
@@ -211,14 +215,14 @@ Objects_Name_or_id_lookup_errors _Objects_MP_Global_name_search (
if ( the_object->name == name_to_use ) { if ( the_object->name == name_to_use ) {
*the_id = the_object->Object.id; *the_id = the_object->Object.id;
_Thread_Enable_dispatch(); _Objects_Allocator_unlock();
return OBJECTS_NAME_OR_ID_LOOKUP_SUCCESSFUL; return OBJECTS_NAME_OR_ID_LOOKUP_SUCCESSFUL;
} }
} }
} }
} }
_Thread_Enable_dispatch(); _Objects_Allocator_unlock();
return OBJECTS_INVALID_NAME; return OBJECTS_INVALID_NAME;
} }
@@ -254,7 +258,7 @@ void _Objects_MP_Is_remote (
return; return;
} }
_Thread_Disable_dispatch(); _Objects_Allocator_lock();
the_chain = &information->global_table[ node ]; the_chain = &information->global_table[ node ];
@@ -272,7 +276,7 @@ void _Objects_MP_Is_remote (
} }
} }
_Thread_Enable_dispatch(); _Objects_Allocator_unlock();
*location = OBJECTS_ERROR; *location = OBJECTS_ERROR;
*the_object = NULL; *the_object = NULL;