ruby: fix MOESI_hammer directory to work with > 3GB memory
authorLena Olson <leolson@google.com>
Sun, 5 Feb 2017 23:20:34 +0000 (17:20 -0600)
committerJason Lowe-Power <jason@lowepower.com>
Thu, 9 Mar 2017 17:20:05 +0000 (11:20 -0600)
The MOESI_hammer directory assumes a contiguous address space, but X86
has an IO gap from 3-4GB. This patch allows the directory to work with
more than 3GB of memory on X86.

Assumptions: the physical address space (range of possible physical
addresses) is 0-XGB when X <= 3GB, and 0-(X+1)GB when X > 3GB. If there
is no IO gap this patch should still work.

Change-Id: I5453a09e953643cada2c096a91d339a3676f55ee
Reviewed-on: https://gem5-review.googlesource.com/2169
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

src/mem/ruby/structures/DirectoryMemory.cc
src/mem/ruby/structures/DirectoryMemory.py

index f79382c268cd3299b62c9acb412bc208c866c39d..ee77931d7ef3e42f445432b1828dcb7cec053b86 100644 (file)
@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 1999-2008 Mark D. Hill and David A. Wood
+ * Copyright (c) 2017 Google Inc.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -24,6 +25,8 @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: Lena Olson
  */
 
 #include "mem/ruby/structures/DirectoryMemory.hh"
@@ -33,6 +36,7 @@
 #include "debug/RubyStats.hh"
 #include "mem/ruby/slicc_interface/RubySlicc_Util.hh"
 #include "mem/ruby/system/RubySystem.hh"
+#include "sim/system.hh"
 
 using namespace std;
 
@@ -44,7 +48,14 @@ DirectoryMemory::DirectoryMemory(const Params *p)
     : SimObject(p)
 {
     m_version = p->version;
-    m_size_bytes = p->size;
+    // In X86, there is an IO gap in the 3-4GB range.
+    if (p->system->getArch() == Arch::X86ISA && p->size > 0xc0000000){
+        // We need to add 1GB to the size for the gap
+        m_size_bytes = p->size + 0x40000000;
+    }
+    else {
+        m_size_bytes = p->size;
+    }
     m_size_bits = floorLog2(m_size_bytes);
     m_num_entries = 0;
     m_numa_high_bit = p->numa_high_bit;
index bb9765bdbbd1041caafaf05b355554edb63c1b2b..2518380b2830fa8f0e88133978836c372c4473dc 100644 (file)
@@ -40,3 +40,4 @@ class RubyDirectoryMemory(SimObject):
     # the default value of the numa high bit is specified in the command line
     # option and must be passed into the directory memory sim object
     numa_high_bit = Param.Int("numa high bit")
+    system = Param.System(Parent.any, "system object")