From b2669a7875526be1036cfb5c451d210213758c22 Mon Sep 17 00:00:00 2001 From: Lena Olson Date: Sun, 5 Feb 2017 17:20:34 -0600 Subject: [PATCH] ruby: fix MOESI_hammer directory to work with > 3GB memory 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 Maintainer: Jason Lowe-Power --- src/mem/ruby/structures/DirectoryMemory.cc | 13 ++++++++++++- src/mem/ruby/structures/DirectoryMemory.py | 1 + 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/src/mem/ruby/structures/DirectoryMemory.cc b/src/mem/ruby/structures/DirectoryMemory.cc index f79382c26..ee77931d7 100644 --- a/src/mem/ruby/structures/DirectoryMemory.cc +++ b/src/mem/ruby/structures/DirectoryMemory.cc @@ -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; diff --git a/src/mem/ruby/structures/DirectoryMemory.py b/src/mem/ruby/structures/DirectoryMemory.py index bb9765bdb..2518380b2 100644 --- a/src/mem/ruby/structures/DirectoryMemory.py +++ b/src/mem/ruby/structures/DirectoryMemory.py @@ -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") -- 2.30.2