Gabe Black has uploaded a new patch set (#2). ( https://gem5-review.googlesource.com/5242 )

Change subject: base: Build caching into the AddrRangeMap class.
......................................................................

base: Build caching into the AddrRangeMap class.

Rather than have each consumer of the AddrRangeMap implement caching lookups
on their own, this change adds a centralized mechanism to the AddrRangeMap
class itself.

Some benefits of this approach are that the cache handles deleted entries
correctly/automatically, the cache is maintained by adding/removing entries
from a linked list rather than moving elements in an array and checking valid bits, and it's easy to enable in places which might otherwise not bother with
caching. The amount of caching is tunable to balance overhead with improved
lookup performance.

This version also fixes a bug found in the existing version and now exposed by
the AddrRangeMap unit test.

Change-Id: Ic25997e23de4eea501e47f039bb52ed0502c58d2
---
M src/base/addr_range_map.hh
1 file changed, 116 insertions(+), 83 deletions(-)


--
To view, visit https://gem5-review.googlesource.com/5242
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic25997e23de4eea501e47f039bb52ed0502c58d2
Gerrit-Change-Number: 5242
Gerrit-PatchSet: 2
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Assignee: Andreas Hansson <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-CC: Andreas Hansson <[email protected]>
Gerrit-CC: Nikos Nikoleris <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to