mem: Change some default values in the Request class.
authorGabe Black <gabeblack@google.com>
Thu, 5 Mar 2020 00:00:02 +0000 (16:00 -0800)
committerGabe Black <gabeblack@google.com>
Tue, 10 Mar 2020 01:10:44 +0000 (01:10 +0000)
These values are more abnormal than the 0s they replace, and so it
would be more obvious when something is accidentally left
uninitialized.

Change-Id: Ie7f14abe9e22f9df1ff238f29d4a783c890f4a20
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/26237
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>

src/mem/request.hh

index 823e6bb7cb7a8bfe1cd190977cd85d5945644898..01252bf66850a78ee3fbf3b53ddd0be62f925bec 100644 (file)
@@ -316,7 +316,7 @@ class Request
      * latencies. This field is set to curTick() any time paddr or vaddr
      * is written.
      */
-    Tick _time = 0;
+    Tick _time = MaxTick;
 
     /**
      * The task id associated with this request
@@ -339,7 +339,7 @@ class Request
     uint32_t _substreamId = 0;
 
     /** The virtual address of the request. */
-    Addr _vaddr = 0;
+    Addr _vaddr = MaxAddr;
 
     /**
      * Extra data for the request, such as the return value of
@@ -347,10 +347,10 @@ class Request
     uint64_t _extraData = 0;
 
     /** The context ID (for statistics, locks, and wakeups). */
-    ContextID _contextId = 0;
+    ContextID _contextId = InvalidContextID;
 
     /** program counter of initiating access; for tracing/debugging */
-    Addr _pc = 0;
+    Addr _pc = MaxAddr;
 
     /** Sequence number of the instruction that creates the request */
     InstSeqNum _reqInstSeqNum = 0;