diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2018-11-24 11:51:28 +0100 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2018-12-07 14:22:01 +0100 |
commit | 3899bc1a4b3294306ae2fd3f8ff0ee10365d9f4b (patch) | |
tree | 3912fc89bbde8a64a20111e02c618e85eb7bb000 /cpukit/score/src/objectallocate.c | |
parent | score: Rename Objects_Information::allocation_size (diff) | |
download | rtems-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/objectallocate.c')
-rw-r--r-- | cpukit/score/src/objectallocate.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/cpukit/score/src/objectallocate.c b/cpukit/score/src/objectallocate.c index 81e322757c..2f991477da 100644 --- a/cpukit/score/src/objectallocate.c +++ b/cpukit/score/src/objectallocate.c @@ -62,16 +62,15 @@ Objects_Control *_Objects_Allocate_unprotected( * extend information base. */ - if ( !the_object ) { + if ( the_object == NULL ) { _Objects_Extend_information( information ); the_object = _Objects_Get_inactive( information ); } - if ( the_object ) { - uint32_t block; + if ( the_object != NULL ) { + Objects_Maximum block; - block = (uint32_t) _Objects_Get_index( the_object->id ) - - _Objects_Get_index( information->minimum_id ); + block = _Objects_Get_index( the_object->id ) - OBJECTS_INDEX_MINIMUM; block /= information->objects_per_block; information->inactive_per_block[ block ]--; |