diff -r df3f34bd5d47 -r 8a9733ade4a4 src/arch/arm/isa/insts/misc.isa --- a/src/arch/arm/isa/insts/misc.isa Fri Aug 13 12:00:18 2010 -0500 +++ b/src/arch/arm/isa/insts/misc.isa Fri Aug 13 12:00:58 2010 -0500 @@ -667,7 +667,7 @@ exec_output += PredOpExecute.subst(setendIop) clrexCode = ''' - unsigned memAccessFlags = ArmISA::TLB::Clrex|3|Request::LLSC; + unsigned memAccessFlags = Request::CLREX|3|Request::LLSC; fault = xc->read(0, (uint32_t&)Mem, memAccessFlags); ''' clrexIop = InstObjParams("clrex", "Clrex","PredOp", diff -r df3f34bd5d47 -r 8a9733ade4a4 src/arch/arm/isa/templates/misc.isa --- a/src/arch/arm/isa/templates/misc.isa Fri Aug 13 12:00:18 2010 -0500 +++ b/src/arch/arm/isa/templates/misc.isa Fri Aug 13 12:00:58 2010 -0500 @@ -367,7 +367,7 @@ if (%(predicate_test)s) { if (fault == NoFault) { - unsigned memAccessFlags = ArmISA::TLB::Clrex|3|Request::LLSC; + unsigned memAccessFlags = Request::CLREX|3|Request::LLSC; fault = xc->read(0, (uint32_t&)Mem, memAccessFlags); } } else { diff -r df3f34bd5d47 -r 8a9733ade4a4 src/arch/arm/tlb.hh --- a/src/arch/arm/tlb.hh Fri Aug 13 12:00:18 2010 -0500 +++ b/src/arch/arm/tlb.hh Fri Aug 13 12:00:58 2010 -0500 @@ -78,8 +78,7 @@ // Because zero otherwise looks like a valid setting and may be used // accidentally, this bit must be non-zero to show it was used on // purpose. - MustBeOne = 0x20, - Clrex = 0x40 + MustBeOne = 0x20 }; protected: typedef std::multimap PageTable; diff -r df3f34bd5d47 -r 8a9733ade4a4 src/arch/arm/tlb.cc --- a/src/arch/arm/tlb.cc Fri Aug 13 12:00:18 2010 -0500 +++ b/src/arch/arm/tlb.cc Fri Aug 13 12:00:58 2010 -0500 @@ -358,9 +358,10 @@ //If this is a clrex instruction, provide a PA of 0 with no fault //This will force the monitor to set the tracked address to 0 //a bit of a hack but this effectively clrears this processors monitor - if (flags & Clrex){ + if (flags & Request::CLREX){ req->setPaddr(0); req->setFlags(Request::UNCACHEABLE); + req->setFlags(Request::CLREX); return NoFault; } if ((req->isInstFetch() && (!sctlr.i)) || diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/cache_impl.hh --- a/src/mem/cache/cache_impl.hh Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/cache_impl.hh Fri Aug 13 12:00:58 2010 -0500 @@ -273,12 +273,14 @@ Cache::access(PacketPtr pkt, BlkType *&blk, int &lat, PacketList &writebacks) { - int id = pkt->req->hasContextId() ? pkt->req->contextId() : -1; - blk = tags->accessBlock(pkt->getAddr(), lat, id); - - if (pkt->req->isUncacheable()) { - if (blk != NULL) { - tags->invalidateBlk(blk); + if (pkt->req->isUncacheable()) { + if (pkt->req->isClrex()) { + tags->clearLocks(); + } else { + blk = tags->findBlock(pkt->getAddr()); + if (blk != NULL) { + tags->invalidateBlk(blk); + } } blk = NULL; @@ -286,6 +288,8 @@ return false; } + int id = pkt->req->hasContextId() ? pkt->req->contextId() : -1; + blk = tags->accessBlock(pkt->getAddr(), lat, id); DPRINTF(Cache, "%s%s %x %s\n", pkt->cmdString(), pkt->req->isInstFetch() ? " (ifetch)" : "", @@ -410,11 +414,13 @@ } if (pkt->req->isUncacheable()) { - int lat = hitLatency; - int id = pkt->req->hasContextId() ? pkt->req->contextId() : -1; - BlkType *blk = tags->accessBlock(pkt->getAddr(), lat, id); - if (blk != NULL) { - tags->invalidateBlk(blk); + if (pkt->req->isClrex()) { + tags->clearLocks(); + } else { + BlkType *blk = tags->findBlock(pkt->getAddr()); + if (blk != NULL) { + tags->invalidateBlk(blk); + } } // writes go in write buffer, reads use MSHR diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/tags/base.hh --- a/src/mem/cache/tags/base.hh Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/tags/base.hh Fri Aug 13 12:00:58 2010 -0500 @@ -140,6 +140,12 @@ * exits. */ virtual void cleanupRefs() {} + + /** + *iterated through all blocks and clear all locks + *Needed to clear all lock tracking at once + */ + virtual void clearLocks() {} }; class BaseTagsCallback : public Callback diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/tags/fa_lru.hh --- a/src/mem/cache/tags/fa_lru.hh Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/tags/fa_lru.hh Fri Aug 13 12:00:58 2010 -0500 @@ -280,6 +280,12 @@ { return (tag); } + + /** + *iterated through all blocks and clear all locks + *Needed to clear all lock tracking at once + */ + virtual void clearLocks(); }; #endif // __MEM_CACHE_TAGS_FA_LRU_HH__ diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/tags/fa_lru.cc --- a/src/mem/cache/tags/fa_lru.cc Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/tags/fa_lru.cc Fri Aug 13 12:00:58 2010 -0500 @@ -286,3 +286,11 @@ } return true; } + +void +FALRU::clearLocks() +{ + for (int i = 0; i < numBlocks; i++){ + blks[i].clearLoadLocks(); + } +} diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/tags/iic.hh --- a/src/mem/cache/tags/iic.hh Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/tags/iic.hh Fri Aug 13 12:00:58 2010 -0500 @@ -439,6 +439,11 @@ IICTag* findVictim(Addr addr, PacketList &writebacks); void insertBlock(Addr addr, BlkType *blk, int context_src); + /** + *iterated through all blocks and clear all locks + *Needed to clear all lock tracking at once + */ + virtual void clearLocks(); /** * Called at end of simulation to complete average block reference stats. @@ -497,6 +502,7 @@ * @param data_ptr The data block to free. */ void freeDataBlock(unsigned long data_ptr); + }; #endif // __IIC_HH__ diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/tags/iic.cc --- a/src/mem/cache/tags/iic.cc Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/tags/iic.cc Fri Aug 13 12:00:58 2010 -0500 @@ -632,6 +632,14 @@ } void +IIC::clearLocks() +{ + for (int i = 0; i < numTags; i++){ + tagStore[i].clearLoadLocks(); + } +} + +void IIC::cleanupRefs() { for (unsigned i = 0; i < numTags; ++i) { diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/tags/lru.hh --- a/src/mem/cache/tags/lru.hh Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/tags/lru.hh Fri Aug 13 12:00:58 2010 -0500 @@ -224,6 +224,11 @@ { return hitLatency; } + /** + *iterated through all blocks and clear all locks + *Needed to clear all lock tracking at once + */ + virtual void clearLocks(); /** * Called at end of simulation to complete average block reference stats. diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/cache/tags/lru.cc --- a/src/mem/cache/tags/lru.cc Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/cache/tags/lru.cc Fri Aug 13 12:00:58 2010 -0500 @@ -217,6 +217,14 @@ } void +LRU::clearLocks() +{ + for (int i = 0; i < numBlocks; i++){ + blks[i].clearLoadLocks(); + } +} + +void LRU::cleanupRefs() { for (unsigned i = 0; i < numSets*assoc; ++i) { diff -r df3f34bd5d47 -r 8a9733ade4a4 src/mem/request.hh --- a/src/mem/request.hh Fri Aug 13 12:00:18 2010 -0500 +++ b/src/mem/request.hh Fri Aug 13 12:00:58 2010 -0500 @@ -71,6 +71,8 @@ static const FlagsType UNCACHEABLE = 0x00001000; /** This request is to a memory mapped register. */ static const FlagsType MMAPED_IPR = 0x00002000; + /** This request is a clear exclusive. */ + static const FlagsType CLREX = 0x00004000; /** The request should ignore unaligned access faults */ static const FlagsType NO_ALIGN_FAULT = 0x00020000; @@ -456,6 +458,7 @@ bool isSwap() const { return _flags.isSet(MEM_SWAP|MEM_SWAP_COND); } bool isCondSwap() const { return _flags.isSet(MEM_SWAP_COND); } bool isMmapedIpr() const { return _flags.isSet(MMAPED_IPR); } + bool isClrex() const { return _flags.isSet(CLREX); } bool isMisaligned() const