diff options
author | Brandon Potter <brandon.potter@amd.com> | 2019-10-18 14:43:14 -0400 |
---|---|---|
committer | Bobby R. Bruce <bbruce@ucdavis.edu> | 2019-11-20 18:39:09 +0000 |
commit | b4c9996d894118be04cdf4ed793b35a1d5001942 (patch) | |
tree | cd3b53dd0d0f6c7cd10c71c72592d6859a33ff1f /src/sim | |
parent | 1c4d64fb104245e7e655113155be02a5209e46c0 (diff) | |
download | gem5-b4c9996d894118be04cdf4ed793b35a1d5001942.tar.xz |
base,tests: Expanded GTests for addr_range.hh
These tests assume the "end address" is not included in the range. This
exposed some bugs in addr_range.hh which have been fixed. Where
appropriate code comments in addr_range.hh have been extended to improve
understanding of the class's behavior.
Hard-coded AddrRange values in the project have been updated to take
into account that end address is now exclusive. The python params.py
interface has been updated to conform to this new standard.
Change-Id: Idd1e75d5771d198c4b8142b28de0f3a6e9007a52
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/22427
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
Diffstat (limited to 'src/sim')
-rw-r--r-- | src/sim/system.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/sim/system.cc b/src/sim/system.cc index f2bbd8cbc..0e7db5964 100644 --- a/src/sim/system.cc +++ b/src/sim/system.cc @@ -408,7 +408,7 @@ System::allocPhysPages(int npages) Addr next_return_addr = pagePtr << PageShift; - AddrRange m5opRange(0xffff0000, 0xffffffff); + AddrRange m5opRange(0xffff0000, 0x100000000); if (m5opRange.contains(next_return_addr)) { warn("Reached m5ops MMIO region\n"); return_addr = 0xffffffff; |