When turning asserts into if's don't forget to invert.
authorRon Dreslinski <rdreslin@umich.edu>
Wed, 11 Oct 2006 04:19:31 +0000 (00:19 -0400)
committerRon Dreslinski <rdreslin@umich.edu>
Wed, 11 Oct 2006 04:19:31 +0000 (00:19 -0400)
src/mem/cache/base_cache.cc:
    When turning asserts into if's don't forget to invert.
    Must be too sleepy.

--HG--
extra : convert_revision : ea38d5a4b4ddde7b5266b3b2c83bbc256218af9a

src/mem/cache/base_cache.cc

index 8b724209e2b9c34a82a4a125ee4d750fc0227497..328e1c7ccce0c8229ffa3e6639ae3c1911f3149d 100644 (file)
@@ -135,7 +135,7 @@ BaseCache::CachePort::recvRetry()
     else if (!isCpuSide)
     {
         DPRINTF(CachePort, "%s attempting to send a retry for MSHR\n", name());
-        if (cache->doMasterRequest()) {
+        if (!cache->doMasterRequest()) {
             //This can happen if I am the owner of a block and see an upgrade
             //while the block was in my WB Buffers.  I just remove the
             //wb and de-assert the masterRequest
@@ -243,7 +243,7 @@ BaseCache::CacheEvent::process()
         else if (!cachePort->isCpuSide)
         {            //MSHR
             DPRINTF(CachePort, "%s trying to send a MSHR request\n", cachePort->name());
-            if (cachePort->cache->doMasterRequest()) {
+            if (!cachePort->cache->doMasterRequest()) {
                 //This can happen if I am the owner of a block and see an upgrade
                 //while the block was in my WB Buffers.  I just remove the
                 //wb and de-assert the masterRequest