flags: Change naming of functions to be clearer
authorNathan Binkert <nate@binkert.org>
Sat, 6 Dec 2008 22:18:18 +0000 (14:18 -0800)
committerNathan Binkert <nate@binkert.org>
Sat, 6 Dec 2008 22:18:18 +0000 (14:18 -0800)
src/arch/alpha/faults.cc
src/base/flags.hh
src/mem/packet.hh
src/mem/packet_access.hh
src/mem/request.hh

index e89cf5c6405ee3b4331db2aedd8887246b2dbf5d..e93e1671124a83ca5a157de131d1cfbfe23b9276 100644 (file)
@@ -144,7 +144,7 @@ DtbFault::invoke(ThreadContext *tc)
     // read, like the EV5).  The EV6 approach is cleaner and seems to
     // work with EV5 PAL code, but not the other way around.
     if (!tc->misspeculating() &&
-        reqFlags.none(Request::VPTE|Request::NO_FAULT)) {
+        reqFlags.noneSet(Request::VPTE|Request::NO_FAULT)) {
         // set VA register with faulting address
         tc->setMiscRegNoEffect(IPR_VA, vaddr);
 
index 712b65319282379f110f794e57bb1b43927f47ec..2e05063916247a4d02f2ed2cefbfb13ede4729c3 100644 (file)
@@ -59,12 +59,12 @@ class Flags
         return *this;
     }
     
-    bool any() const { return _flags; }
-    bool any(Type flags) const { return (_flags & flags); }
-    bool all() const { return !(~_flags); }
-    bool all(Type flags) const { return (_flags & flags) == flags; }
-    bool none() const { return _flags == 0; }
-    bool none(Type flags) const { return (_flags & flags) == 0; }
+    bool isSet() const { return _flags; }
+    bool isSet(Type flags) const { return (_flags & flags); }
+    bool allSet() const { return !(~_flags); }
+    bool allSet(Type flags) const { return (_flags & flags) == flags; }
+    bool noneSet() const { return _flags == 0; }
+    bool noneSet(Type flags) const { return (_flags & flags) == 0; }
     void clear() { _flags = 0; }
     void clear(Type flags) { _flags &= ~flags; }
     void set(Type flags) { _flags |= flags; }
index b75490168e81c00def4f134cbfae1ef2b2c41cb5..41f599fa0701258cfd8bfecf33f8995726f7cd2c 100644 (file)
@@ -407,15 +407,15 @@ class Packet : public FastAlloc, public Printable
 
     // Snoop flags
     void assertMemInhibit()     { flags.set(MEM_INHIBIT); }
-    bool memInhibitAsserted()   { return flags.any(MEM_INHIBIT); }
+    bool memInhibitAsserted()   { return flags.isSet(MEM_INHIBIT); }
     void assertShared()         { flags.set(SHARED); }
-    bool sharedAsserted()       { return flags.any(SHARED); }
+    bool sharedAsserted()       { return flags.isSet(SHARED); }
 
     // Special control flags
     void setExpressSnoop()      { flags.set(EXPRESS_SNOOP); }
-    bool isExpressSnoop()       { return flags.any(EXPRESS_SNOOP); }
+    bool isExpressSnoop()       { return flags.isSet(EXPRESS_SNOOP); }
     void setSupplyExclusive()   { flags.set(SUPPLY_EXCLUSIVE); }
-    bool isSupplyExclusive()    { return flags.any(SUPPLY_EXCLUSIVE); }
+    bool isSupplyExclusive()    { return flags.isSet(SUPPLY_EXCLUSIVE); }
 
     // Network error conditions... encapsulate them as methods since
     // their encoding keeps changing (from result field to command
@@ -439,19 +439,19 @@ class Packet : public FastAlloc, public Printable
     void copyError(Packet *pkt) { assert(pkt->isError()); cmd = pkt->cmd; }
 
     /// Accessor function to get the source index of the packet.
-    NodeID getSrc() const    { assert(flags.any(VALID_SRC)); return src; }
+    NodeID getSrc() const    { assert(flags.isSet(VALID_SRC)); return src; }
     /// Accessor function to set the source index of the packet.
     void setSrc(NodeID _src) { src = _src; flags.set(VALID_SRC); }
     /// Reset source field, e.g. to retransmit packet on different bus.
     void clearSrc() { flags.clear(VALID_SRC); }
 
     /// Accessor function for the destination index of the packet.
-    NodeID getDest() const     { assert(flags.any(VALID_DST)); return dest; }
+    NodeID getDest() const     { assert(flags.isSet(VALID_DST)); return dest; }
     /// Accessor function to set the destination index of the packet.
     void setDest(NodeID _dest) { dest = _dest; flags.set(VALID_DST); }
 
-    Addr getAddr() const { assert(flags.all(VALID_ADDR)); return addr; }
-    int getSize() const  { assert(flags.all(VALID_SIZE)); return size; }
+    Addr getAddr() const { assert(flags.isSet(VALID_ADDR)); return addr; }
+    int getSize() const  { assert(flags.isSet(VALID_SIZE)); return size; }
     Addr getOffset(int blkSize) const { return getAddr() & (Addr)(blkSize - 1); }
 
     /**
@@ -465,7 +465,7 @@ class Packet : public FastAlloc, public Printable
            addr(_req->paddr), size(_req->size), dest(_dest), time(curTick),
            senderState(NULL)
     {
-        if (req->flags.any(Request::VALID_PADDR))
+        if (req->flags.isSet(Request::VALID_PADDR))
             flags.set(VALID_ADDR|VALID_SIZE);
     }
 
@@ -479,7 +479,7 @@ class Packet : public FastAlloc, public Printable
            addr(_req->paddr & ~(_blkSize - 1)), size(_blkSize), dest(_dest),
            time(curTick), senderState(NULL)
     {
-        if (req->flags.any(Request::VALID_PADDR))
+        if (req->flags.isSet(Request::VALID_PADDR))
             flags.set(VALID_ADDR|VALID_SIZE);
     }
 
@@ -492,7 +492,7 @@ class Packet : public FastAlloc, public Printable
      */
     Packet(Packet *pkt, bool clearFlags = false)
         :  cmd(pkt->cmd), req(pkt->req),
-           data(pkt->flags.any(STATIC_DATA) ? pkt->data : NULL),
+           data(pkt->flags.isSet(STATIC_DATA) ? pkt->data : NULL),
            addr(pkt->addr), size(pkt->size), src(pkt->src), dest(pkt->dest),
            time(curTick), senderState(pkt->senderState)
     {
@@ -526,7 +526,7 @@ class Packet : public FastAlloc, public Printable
     void
     reinitFromRequest()
     {
-        assert(req->flags.any(Request::VALID_PADDR));
+        assert(req->flags.isSet(Request::VALID_PADDR));
         flags = 0;
         addr = req->paddr;
         size = req->size;
@@ -551,7 +551,7 @@ class Packet : public FastAlloc, public Printable
         cmd = cmd.responseCommand();
 
         dest = src;
-        flags.set(VALID_DST, flags.any(VALID_SRC));
+        flags.set(VALID_DST, flags.isSet(VALID_SRC));
         flags.clear(VALID_SRC);
     }
 
@@ -590,7 +590,7 @@ class Packet : public FastAlloc, public Printable
     void
     dataStatic(T *p)
     {
-        assert(flags.none(STATIC_DATA|DYNAMIC_DATA|ARRAY_DATA));
+        assert(flags.noneSet(STATIC_DATA|DYNAMIC_DATA|ARRAY_DATA));
         data = (PacketDataPtr)p;
         flags.set(STATIC_DATA);
     }
@@ -603,7 +603,7 @@ class Packet : public FastAlloc, public Printable
     void
     dataDynamicArray(T *p)
     {
-        assert(flags.none(STATIC_DATA|DYNAMIC_DATA|ARRAY_DATA));
+        assert(flags.noneSet(STATIC_DATA|DYNAMIC_DATA|ARRAY_DATA));
         data = (PacketDataPtr)p;
         flags.set(DYNAMIC_DATA|ARRAY_DATA);
     }
@@ -616,7 +616,7 @@ class Packet : public FastAlloc, public Printable
     void
     dataDynamic(T *p)
     {
-        assert(flags.none(STATIC_DATA|DYNAMIC_DATA|ARRAY_DATA));
+        assert(flags.noneSet(STATIC_DATA|DYNAMIC_DATA|ARRAY_DATA));
         data = (PacketDataPtr)p;
         flags.set(DYNAMIC_DATA);
     }
@@ -628,7 +628,7 @@ class Packet : public FastAlloc, public Printable
     T*
     getPtr()
     {
-        assert(flags.any(STATIC_DATA|DYNAMIC_DATA));
+        assert(flags.isSet(STATIC_DATA|DYNAMIC_DATA));
         return (T*)data;
     }
 
@@ -689,9 +689,9 @@ class Packet : public FastAlloc, public Printable
     void
     deleteData()
     {
-        if (flags.any(ARRAY_DATA))
+        if (flags.isSet(ARRAY_DATA))
             delete [] data;
-        else if (flags.any(DYNAMIC_DATA))
+        else if (flags.isSet(DYNAMIC_DATA))
             delete data;
 
         flags.clear(STATIC_DATA|DYNAMIC_DATA|ARRAY_DATA);
@@ -703,11 +703,11 @@ class Packet : public FastAlloc, public Printable
     allocate()
     {
         if (data) {
-            assert(flags.any(STATIC_DATA|DYNAMIC_DATA));
+            assert(flags.isSet(STATIC_DATA|DYNAMIC_DATA));
             return;
         }
 
-        assert(flags.none(STATIC_DATA|DYNAMIC_DATA));
+        assert(flags.noneSet(STATIC_DATA|DYNAMIC_DATA));
         flags.set(DYNAMIC_DATA|ARRAY_DATA);
         data = new uint8_t[getSize()];
     }
index 1e233ee576654a507e7802d86f96757dd2c4c2a4..f70d508b21997237bbf1840180012b028e786918 100644 (file)
@@ -46,7 +46,7 @@ template <typename T>
 inline T
 Packet::get()
 {
-    assert(flags.any(STATIC_DATA|DYNAMIC_DATA));
+    assert(flags.isSet(STATIC_DATA|DYNAMIC_DATA));
     assert(sizeof(T) <= size);
     return TheISA::gtoh(*(T*)data);
 }
@@ -56,7 +56,7 @@ template <typename T>
 inline void
 Packet::set(T v)
 {
-    assert(flags.any(STATIC_DATA|DYNAMIC_DATA));
+    assert(flags.isSet(STATIC_DATA|DYNAMIC_DATA));
     assert(sizeof(T) <= size);
     *(T*)data = TheISA::htog(v);
 }
index c7cadaa035d5168994ffe6381acf1f27e5ecb2f5..f60eb17facf7139f8ea43de4bd15a375060a3d4c 100644 (file)
@@ -230,7 +230,7 @@ class Request : public FastAlloc
     void
     setPaddr(Addr _paddr)
     {
-        assert(flags.any(VALID_VADDR));
+        assert(flags.isSet(VALID_VADDR));
         paddr = _paddr;
         flags.set(VALID_PADDR);
     }
@@ -241,8 +241,8 @@ class Request : public FastAlloc
      */
     void splitOnVaddr(Addr split_addr, RequestPtr &req1, RequestPtr &req2)
     {
-        assert(flags.any(VALID_VADDR));
-        assert(flags.none(VALID_PADDR));
+        assert(flags.isSet(VALID_VADDR));
+        assert(flags.noneSet(VALID_PADDR));
         assert(split_addr > vaddr && split_addr < vaddr + size);
         req1 = new Request;
         *req1 = *this;
@@ -259,7 +259,7 @@ class Request : public FastAlloc
     Addr
     getPaddr()
     {
-        assert(flags.any(VALID_PADDR));
+        assert(flags.isSet(VALID_PADDR));
         return paddr;
     }
 
@@ -269,7 +269,7 @@ class Request : public FastAlloc
     int
     getSize()
     {
-        assert(flags.any(VALID_SIZE));
+        assert(flags.isSet(VALID_SIZE));
         return size;
     }
 
@@ -277,64 +277,62 @@ class Request : public FastAlloc
     Tick
     getTime()
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
         return time;
     }
 
     void
     setTime(Tick when)
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
         time = when;
     }
 
-    void resetTime() { setTime(curTick); }
-
     /** Accessor for flags. */
     Flags
     getFlags()
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
         return flags & PUBLIC_FLAGS;
     }
 
     Flags
     anyFlags(Flags _flags)
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
-        assert(_flags.none(~PUBLIC_FLAGS));
-        return flags.any(_flags);
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
+        assert(_flags.noneSet(~PUBLIC_FLAGS));
+        return flags.isSet(_flags);
     }
 
     Flags
     allFlags(Flags _flags)
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
-        assert(_flags.none(~PUBLIC_FLAGS));
-        return flags.all(_flags);
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
+        assert(_flags.noneSet(~PUBLIC_FLAGS));
+        return flags.allSet(_flags);
     }
 
     /** Accessor for flags. */
     void
     setFlags(Flags _flags)
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
-        assert(_flags.none(~PUBLIC_FLAGS));
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
+        assert(_flags.noneSet(~PUBLIC_FLAGS));
         flags.set(_flags);
     }
 
     void
     clearFlags(Flags _flags)
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
-        assert(_flags.none(~PUBLIC_FLAGS));
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
+        assert(_flags.noneSet(~PUBLIC_FLAGS));
         flags.clear(_flags);
     }
 
     void
     clearFlags()
     {
-        assert(flags.any(VALID_PADDR|VALID_VADDR));
+        assert(flags.isSet(VALID_PADDR|VALID_VADDR));
         flags.clear(PUBLIC_FLAGS);
     }
 
@@ -342,7 +340,7 @@ class Request : public FastAlloc
     Addr
     getVaddr()
     {
-        assert(flags.any(VALID_VADDR));
+        assert(flags.isSet(VALID_VADDR));
         return vaddr;
     }
 
@@ -350,7 +348,7 @@ class Request : public FastAlloc
     int
     getAsid()
     {
-        assert(flags.any(VALID_VADDR));
+        assert(flags.isSet(VALID_VADDR));
         return asid;
     }
 
@@ -358,7 +356,7 @@ class Request : public FastAlloc
     uint8_t
     getAsi()
     {
-        assert(flags.any(VALID_VADDR));
+        assert(flags.isSet(VALID_VADDR));
         return flags & ASI_BITS;
     }
 
@@ -366,7 +364,7 @@ class Request : public FastAlloc
     void
     setAsi(uint8_t a)
     {
-        assert(flags.any(VALID_VADDR));
+        assert(flags.isSet(VALID_VADDR));
         flags.update(a, ASI_BITS);
     }
 
@@ -374,8 +372,8 @@ class Request : public FastAlloc
     bool
     isMmapedIpr()
     {
-        assert(flags.any(VALID_PADDR));
-        return flags.any(MMAPED_IPR);
+        assert(flags.isSet(VALID_PADDR));
+        return flags.isSet(MMAPED_IPR);
     }
 
     /** Accessor function for asi.*/
@@ -390,14 +388,14 @@ class Request : public FastAlloc
     bool
     extraDataValid()
     {
-        return flags.any(VALID_EXTRA_DATA);
+        return flags.isSet(VALID_EXTRA_DATA);
     }
 
     /** Accessor function for store conditional return value.*/
     uint64_t
     getExtraData() const
     {
-        assert(flags.any(VALID_EXTRA_DATA));
+        assert(flags.isSet(VALID_EXTRA_DATA));
         return extraData;
     }
 
@@ -413,7 +411,7 @@ class Request : public FastAlloc
     int
     contextId() const
     {
-        assert(flags.any(VALID_CONTEXT_ID));
+        assert(flags.isSet(VALID_CONTEXT_ID));
         return _contextId;
     }
 
@@ -421,7 +419,7 @@ class Request : public FastAlloc
     int
     threadId() const
     {
-        assert(flags.any(VALID_THREAD_ID));
+        assert(flags.isSet(VALID_THREAD_ID));
         return _threadId;
     }
 
@@ -429,27 +427,27 @@ class Request : public FastAlloc
     Addr
     getPC() const
     {
-        assert(flags.any(VALID_PC));
+        assert(flags.isSet(VALID_PC));
         return pc;
     }
 
     /** Accessor Function to Check Cacheability. */
-    bool isUncacheable() const { return flags.any(UNCACHEABLE); }
-    bool isInstRead() const { return flags.any(INST_READ); }
-    bool isLocked() const { return flags.any(LOCKED); }
-    bool isSwap() const { return flags.any(MEM_SWAP|MEM_SWAP_COND); }
-    bool isCondSwap() const { return flags.any(MEM_SWAP_COND); }
+    bool isUncacheable() const { return flags.isSet(UNCACHEABLE); }
+    bool isInstRead() const { return flags.isSet(INST_READ); }
+    bool isLocked() const { return flags.isSet(LOCKED); }
+    bool isSwap() const { return flags.isSet(MEM_SWAP|MEM_SWAP_COND); }
+    bool isCondSwap() const { return flags.isSet(MEM_SWAP_COND); }
 
     bool
     isMisaligned() const
     {
-        if (flags.any(NO_ALIGN_FAULT))
+        if (flags.isSet(NO_ALIGN_FAULT))
             return false;
 
         if ((vaddr & 0x1))
             return true;
 
-        if (flags.any(NO_HALF_WORD_ALIGN_FAULT))
+        if (flags.isSet(NO_HALF_WORD_ALIGN_FAULT))
             return false;
 
         if ((vaddr & 0x2))