summaryrefslogtreecommitdiffstats
path: root/cpukit/score/src/objectgetlocal.c
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2018-11-24 11:51:28 +0100
committerSebastian Huber <sebastian.huber@embedded-brains.de>2018-12-07 14:22:01 +0100
commit3899bc1a4b3294306ae2fd3f8ff0ee10365d9f4b (patch)
tree3912fc89bbde8a64a20111e02c618e85eb7bb000 /cpukit/score/src/objectgetlocal.c
parentscore: Rename Objects_Information::allocation_size (diff)
downloadrtems-3899bc1a4b3294306ae2fd3f8ff0ee10365d9f4b.tar.bz2
score: Optimize object lookup
Use the maximum ID for the ID to object translation. Using the maximum ID gets rid of an additional load from the object information in _Objects_Get(). In addition, object lookups fail for every ID in case the object information is cleared to zero. This makes it a bit more robust during system startup (see new tests in spconfig02). The local table no longer needs a NULL pointer entry at array index zero. Adjust all the object iteration loops accordingly. Remove Objects_Information::minimum_id since it contains only redundant information. Add _Objects_Get_minimum_id() to get the minimum ID. Update #3621.
Diffstat (limited to 'cpukit/score/src/objectgetlocal.c')
-rw-r--r--cpukit/score/src/objectgetlocal.c21
1 files changed, 14 insertions, 7 deletions
diff --git a/cpukit/score/src/objectgetlocal.c b/cpukit/score/src/objectgetlocal.c
index a1a81db436..e8a109f457 100644
--- a/cpukit/score/src/objectgetlocal.c
+++ b/cpukit/score/src/objectgetlocal.c
@@ -31,22 +31,29 @@ Objects_Control *_Objects_Get(
const Objects_Information *information
)
{
- uint32_t index;
+ Objects_Id delta;
+ Objects_Id maximum_id;
+ Objects_Id end;
- index = id - information->minimum_id + 1;
+ maximum_id = information->maximum_id;
+ delta = maximum_id - id;
+ end = _Objects_Get_index( maximum_id );
- if ( information->maximum >= index ) {
+ if ( RTEMS_PREDICT_TRUE( delta < end ) ) {
+ ISR_Level level;
Objects_Control *the_object;
- _ISR_lock_ISR_disable( lock_context );
+ _ISR_Local_disable( level );
+ _ISR_lock_Context_set_level( lock_context, level );
- the_object = information->local_table[ index ];
- if ( the_object != NULL ) {
+ the_object =
+ information->local_table[ end - OBJECTS_INDEX_MINIMUM - delta ];
+ if ( RTEMS_PREDICT_TRUE( the_object != NULL ) ) {
/* ISR disabled on behalf of caller */
return the_object;
}
- _ISR_lock_ISR_enable( lock_context );
+ _ISR_Local_enable( level );
}
return NULL;