From ddfa96cf455ba4a287930942514cdf0f7f2afa77 Mon Sep 17 00:00:00 2001 From: Andreas Hansson Date: Fri, 21 Aug 2015 07:03:23 -0400 Subject: mem: Add explicit Cache subclass and make BaseCache abstract Open up for other subclasses to BaseCache and transition to using the explicit Cache subclass. --HG-- rename : src/mem/cache/BaseCache.py => src/mem/cache/Cache.py --- src/mem/cache/BaseCache.py | 83 ------------------------------------------ src/mem/cache/Cache.py | 89 ++++++++++++++++++++++++++++++++++++++++++++++ src/mem/cache/SConscript | 2 +- src/mem/cache/base.cc | 12 ++----- src/mem/cache/base.hh | 3 +- src/mem/cache/cache.cc | 11 ++++-- src/mem/cache/cache.hh | 3 +- 7 files changed, 104 insertions(+), 99 deletions(-) delete mode 100644 src/mem/cache/BaseCache.py create mode 100644 src/mem/cache/Cache.py (limited to 'src/mem') diff --git a/src/mem/cache/BaseCache.py b/src/mem/cache/BaseCache.py deleted file mode 100644 index d908430e5..000000000 --- a/src/mem/cache/BaseCache.py +++ /dev/null @@ -1,83 +0,0 @@ -# Copyright (c) 2012-2013, 2015 ARM Limited -# All rights reserved. -# -# The license below extends only to copyright in the software and shall -# not be construed as granting a license to any other intellectual -# property including but not limited to intellectual property relating -# to a hardware implementation of the functionality of the software -# licensed hereunder. You may use the software subject to the license -# terms below provided that you ensure that this notice is replicated -# unmodified and in its entirety in all distributions of the software, -# modified or unmodified, in source code or in binary form. -# -# Copyright (c) 2005-2007 The Regents of The University of Michigan -# All rights reserved. -# -# Redistribution and use in source and binary forms, with or without -# modification, are permitted provided that the following conditions are -# met: redistributions of source code must retain the above copyright -# notice, this list of conditions and the following disclaimer; -# redistributions in binary form must reproduce the above copyright -# notice, this list of conditions and the following disclaimer in the -# documentation and/or other materials provided with the distribution; -# neither the name of the copyright holders nor the names of its -# contributors may be used to endorse or promote products derived from -# this software without specific prior written permission. -# -# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS -# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT -# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR -# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT -# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, -# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT -# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, -# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY -# 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: Nathan Binkert - -from m5.params import * -from m5.proxy import * -from MemObject import MemObject -from Prefetcher import BasePrefetcher -from Tags import * - -class BaseCache(MemObject): - type = 'BaseCache' - cxx_header = "mem/cache/base.hh" - - size = Param.MemorySize("Capacity") - assoc = Param.Unsigned("Associativity") - - hit_latency = Param.Cycles("Hit latency") - response_latency = Param.Cycles("Latency for the return path on a miss"); - - max_miss_count = Param.Counter(0, - "Number of misses to handle before calling exit") - - mshrs = Param.Unsigned("Number of MSHRs (max outstanding requests)") - demand_mshr_reserve = Param.Unsigned(1, "MSHRs reserved for demand access") - tgts_per_mshr = Param.Unsigned("Max number of accesses per MSHR") - write_buffers = Param.Unsigned(8, "Number of write buffers") - - forward_snoops = Param.Bool(True, - "Forward snoops from mem side to cpu side") - is_read_only = Param.Bool(False, "Is this cache read only (e.g. inst)") - - prefetcher = Param.BasePrefetcher(NULL,"Prefetcher attached to cache") - prefetch_on_access = Param.Bool(False, - "Notify the hardware prefetcher on every access (not just misses)") - - tags = Param.BaseTags(LRU(), "Tag store (replacement policy)") - sequential_access = Param.Bool(False, - "Whether to access tags and data sequentially") - - cpu_side = SlavePort("Upstream port closer to the CPU and/or device") - mem_side = MasterPort("Downstream port closer to memory") - - addr_ranges = VectorParam.AddrRange([AllMemory], - "Address range for the CPU-side port (to allow striping)") - - system = Param.System(Parent.any, "System we belong to") diff --git a/src/mem/cache/Cache.py b/src/mem/cache/Cache.py new file mode 100644 index 000000000..8ad1177e7 --- /dev/null +++ b/src/mem/cache/Cache.py @@ -0,0 +1,89 @@ +# Copyright (c) 2012-2013, 2015 ARM Limited +# All rights reserved. +# +# The license below extends only to copyright in the software and shall +# not be construed as granting a license to any other intellectual +# property including but not limited to intellectual property relating +# to a hardware implementation of the functionality of the software +# licensed hereunder. You may use the software subject to the license +# terms below provided that you ensure that this notice is replicated +# unmodified and in its entirety in all distributions of the software, +# modified or unmodified, in source code or in binary form. +# +# Copyright (c) 2005-2007 The Regents of The University of Michigan +# All rights reserved. +# +# Redistribution and use in source and binary forms, with or without +# modification, are permitted provided that the following conditions are +# met: redistributions of source code must retain the above copyright +# notice, this list of conditions and the following disclaimer; +# redistributions in binary form must reproduce the above copyright +# notice, this list of conditions and the following disclaimer in the +# documentation and/or other materials provided with the distribution; +# neither the name of the copyright holders nor the names of its +# contributors may be used to endorse or promote products derived from +# this software without specific prior written permission. +# +# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS +# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT +# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR +# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT +# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, +# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT +# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, +# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY +# 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: Nathan Binkert +# Andreas Hansson + +from m5.params import * +from m5.proxy import * +from MemObject import MemObject +from Prefetcher import BasePrefetcher +from Tags import * + +class BaseCache(MemObject): + type = 'BaseCache' + abstract = True + cxx_header = "mem/cache/base.hh" + + size = Param.MemorySize("Capacity") + assoc = Param.Unsigned("Associativity") + + hit_latency = Param.Cycles("Hit latency") + response_latency = Param.Cycles("Latency for the return path on a miss"); + + max_miss_count = Param.Counter(0, + "Number of misses to handle before calling exit") + + mshrs = Param.Unsigned("Number of MSHRs (max outstanding requests)") + demand_mshr_reserve = Param.Unsigned(1, "MSHRs reserved for demand access") + tgts_per_mshr = Param.Unsigned("Max number of accesses per MSHR") + write_buffers = Param.Unsigned(8, "Number of write buffers") + + forward_snoops = Param.Bool(True, + "Forward snoops from mem side to cpu side") + is_read_only = Param.Bool(False, "Is this cache read only (e.g. inst)") + + prefetcher = Param.BasePrefetcher(NULL,"Prefetcher attached to cache") + prefetch_on_access = Param.Bool(False, + "Notify the hardware prefetcher on every access (not just misses)") + + tags = Param.BaseTags(LRU(), "Tag store (replacement policy)") + sequential_access = Param.Bool(False, + "Whether to access tags and data sequentially") + + cpu_side = SlavePort("Upstream port closer to the CPU and/or device") + mem_side = MasterPort("Downstream port closer to memory") + + addr_ranges = VectorParam.AddrRange([AllMemory], + "Address range for the CPU-side port (to allow striping)") + + system = Param.System(Parent.any, "System we belong to") + +class Cache(BaseCache): + type = 'Cache' + cxx_header = 'mem/cache/cache.hh' diff --git a/src/mem/cache/SConscript b/src/mem/cache/SConscript index a4fbe04c0..1c356ef6a 100644 --- a/src/mem/cache/SConscript +++ b/src/mem/cache/SConscript @@ -30,7 +30,7 @@ Import('*') -SimObject('BaseCache.py') +SimObject('Cache.py') Source('base.cc') Source('cache.cc') diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc index c270d5b65..41b6f38aa 100644 --- a/src/mem/cache/base.cc +++ b/src/mem/cache/base.cc @@ -65,13 +65,13 @@ BaseCache::CacheSlavePort::CacheSlavePort(const std::string &_name, { } -BaseCache::BaseCache(const Params *p) +BaseCache::BaseCache(const BaseCacheParams *p, unsigned blk_size) : MemObject(p), cpuSidePort(nullptr), memSidePort(nullptr), mshrQueue("MSHRs", p->mshrs, 4, p->demand_mshr_reserve, MSHRQueue_MSHRs), writeBuffer("write buffer", p->write_buffers, p->mshrs+1000, 0, MSHRQueue_WriteBuffer), - blkSize(p->system->cacheLineSize()), + blkSize(blk_size), lookupLatency(p->hit_latency), forwardLatency(p->hit_latency), fillLatency(p->response_latency), @@ -774,11 +774,3 @@ BaseCache::regStats() ; } - -BaseCache * -BaseCacheParams::create() -{ - assert(tags); - - return new Cache(this); -} diff --git a/src/mem/cache/base.hh b/src/mem/cache/base.hh index 3e6f5cab2..3baec36d9 100644 --- a/src/mem/cache/base.hh +++ b/src/mem/cache/base.hh @@ -473,8 +473,7 @@ class BaseCache : public MemObject virtual void regStats(); public: - typedef BaseCacheParams Params; - BaseCache(const Params *p); + BaseCache(const BaseCacheParams *p, unsigned blk_size); ~BaseCache() {} virtual void init(); diff --git a/src/mem/cache/cache.cc b/src/mem/cache/cache.cc index 2426a0636..911785479 100644 --- a/src/mem/cache/cache.cc +++ b/src/mem/cache/cache.cc @@ -63,8 +63,8 @@ #include "mem/cache/prefetch/base.hh" #include "sim/sim_exit.hh" -Cache::Cache(const Params *p) - : BaseCache(p), +Cache::Cache(const CacheParams *p) + : BaseCache(p, p->system->cacheLineSize()), tags(p->tags), prefetcher(p->prefetcher), doFastWrites(true), @@ -2382,6 +2382,13 @@ CpuSidePort::CpuSidePort(const std::string &_name, Cache *_cache, { } +Cache* +CacheParams::create() +{ + assert(tags); + + return new Cache(this); +} /////////////// // // MemSidePort diff --git a/src/mem/cache/cache.hh b/src/mem/cache/cache.hh index 06d78a272..447a55229 100644 --- a/src/mem/cache/cache.hh +++ b/src/mem/cache/cache.hh @@ -57,6 +57,7 @@ #include "mem/cache/blk.hh" #include "mem/cache/mshr.hh" #include "mem/cache/tags/base.hh" +#include "params/Cache.hh" #include "sim/eventq.hh" //Forward decleration @@ -419,7 +420,7 @@ class Cache : public BaseCache public: /** Instantiates a basic cache object. */ - Cache(const Params *p); + Cache(const CacheParams *p); /** Non-default destructor is needed to deallocate memory. */ virtual ~Cache(); -- cgit v1.2.3