From 5902e29e84037b6f0dd470c09a73f5590368d755 Mon Sep 17 00:00:00 2001 From: Andreas Hansson Date: Thu, 30 Jul 2015 03:41:39 -0400 Subject: mem: Tidy up CacheBlk class This patch modernises and tidies up the CacheBlk, removing dead code. --- src/mem/cache/blk.hh | 71 +++++++++++++++++----------------------------------- 1 file changed, 23 insertions(+), 48 deletions(-) (limited to 'src') diff --git a/src/mem/cache/blk.hh b/src/mem/cache/blk.hh index 09599b694..f082a9ef0 100644 --- a/src/mem/cache/blk.hh +++ b/src/mem/cache/blk.hh @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012-2014 ARM Limited + * Copyright (c) 2012-2015 ARM Limited * All rights reserved. * * The license below extends only to copyright in the software and shall @@ -45,20 +45,19 @@ * Definitions of a simple cache block class. */ -#ifndef __CACHE_BLK_HH__ -#define __CACHE_BLK_HH__ +#ifndef __MEM_CACHE_BLK_HH__ +#define __MEM_CACHE_BLK_HH__ #include #include "base/printable.hh" #include "mem/packet.hh" #include "mem/request.hh" -#include "sim/core.hh" // for Tick /** * Cache block status bit assignments */ -enum CacheBlkStatusBits { +enum CacheBlkStatusBits : unsigned { /** valid, readable */ BlkValid = 0x01, /** write permission */ @@ -67,8 +66,6 @@ enum CacheBlkStatusBits { BlkReadable = 0x04, /** dirty (modified) */ BlkDirty = 0x08, - /** block was referenced */ - BlkReferenced = 0x10, /** block was a hardware prefetch yet unaccessed*/ BlkHWPrefetched = 0x20, /** block holds data from the secure memory space */ @@ -98,7 +95,7 @@ class CacheBlk */ uint8_t *data; /** the number of bytes stored in this block. */ - int size; + unsigned size; /** block state: OR of CacheBlkStatusBit */ typedef unsigned State; @@ -119,7 +116,7 @@ class CacheBlk bool isTouched; /** Number of references to this block since it was brought in. */ - int refCount; + unsigned refCount; /** holds the source requestor ID for this block. */ int srcMasterId; @@ -138,7 +135,7 @@ class CacheBlk Addr highAddr; // high address of lock range // check for matching execution context - bool matchesContext(Request *req) + bool matchesContext(const RequestPtr req) const { Addr req_low = req->getPaddr(); Addr req_high = req_low + req->getSize() -1; @@ -146,7 +143,7 @@ class CacheBlk (req_low >= lowAddr) && (req_high <= highAddr); } - bool overlapping(Request *req) + bool overlapping(const RequestPtr req) const { Addr req_low = req->getPaddr(); Addr req_high = req_low + req->getSize() - 1; @@ -154,7 +151,7 @@ class CacheBlk return (req_low <= highAddr) && (req_high >= lowAddr); } - Lock(Request *req) + Lock(const RequestPtr req) : contextId(req->contextId()), lowAddr(req->getPaddr()), highAddr(lowAddr + req->getSize() - 1) @@ -176,24 +173,8 @@ class CacheBlk tickInserted(0) {} - /** - * Copy the state of the given block into this one. - * @param rhs The block to copy. - * @return a const reference to this block. - */ - const CacheBlk& operator=(const CacheBlk& rhs) - { - asid = rhs.asid; - tag = rhs.tag; - data = rhs.data; - size = rhs.size; - status = rhs.status; - whenReady = rhs.whenReady; - set = rhs.set; - refCount = rhs.refCount; - task_id = rhs.task_id; - return *this; - } + CacheBlk(const CacheBlk&) = delete; + CacheBlk& operator=(const CacheBlk&) = delete; /** * Checks the write permissions of this block. @@ -245,15 +226,6 @@ class CacheBlk return (status & BlkDirty) != 0; } - /** - * Check if this block has been referenced. - * @return True if the block has been referenced. - */ - bool isReferenced() const - { - return (status & BlkReferenced) != 0; - } - /** * Check if this block was the result of a hardware prefetch, yet to * be touched. @@ -282,21 +254,21 @@ class CacheBlk void trackLoadLocked(PacketPtr pkt) { assert(pkt->isLLSC()); - lockList.push_front(Lock(pkt->req)); + lockList.emplace_front(pkt->req); } /** * Clear the list of valid load locks. Should be called whenever * block is written to or invalidated. */ - void clearLoadLocks(Request *req = NULL) + void clearLoadLocks(RequestPtr req = nullptr) { if (!req) { // No request, invaldate all locks to this line lockList.clear(); } else { // Only invalidate locks that overlap with this request - std::list::iterator lock_itr = lockList.begin(); + auto lock_itr = lockList.begin(); while (lock_itr != lockList.end()) { if (lock_itr->overlapping(req)) { lock_itr = lockList.erase(lock_itr); @@ -350,16 +322,19 @@ class CacheBlk */ bool checkWrite(PacketPtr pkt) { - Request *req = pkt->req; + // common case + if (!pkt->isLLSC() && lockList.empty()) + return true; + + RequestPtr req = pkt->req; + if (pkt->isLLSC()) { // it's a store conditional... have to check for matching // load locked. bool success = false; - for (std::list::iterator i = lockList.begin(); - i != lockList.end(); ++i) - { - if (i->matchesContext(req)) { + for (const auto& l : lockList) { + if (l.matchesContext(req)) { // it's a store conditional, and as far as the memory // system can tell, the requesting context's lock is // still valid. @@ -412,4 +387,4 @@ class CacheBlkVisitor virtual bool operator()(CacheBlk &blk) = 0; }; -#endif //__CACHE_BLK_HH__ +#endif //__MEM_CACHE_BLK_HH__ -- cgit v1.2.3