Jason Lowe-Power has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/34035 )

Change subject: cpu: Set ContextId on request from trace CPU
......................................................................

cpu: Set ContextId on request from trace CPU

Adds a contextId to the trace CPU in one more case that was missing.
Without this a panic is triggered in the cache.

Change-Id: I78bd70ad1e3657c9a6a1d56c234c007c2e2b586c
Signed-off-by: Jason Lowe-Power <ja...@lowepower.com>
---
M src/cpu/trace/trace_cpu.cc
1 file changed, 3 insertions(+), 0 deletions(-)



diff --git a/src/cpu/trace/trace_cpu.cc b/src/cpu/trace/trace_cpu.cc
index dd91257..13f194c 100644
--- a/src/cpu/trace/trace_cpu.cc
+++ b/src/cpu/trace/trace_cpu.cc
@@ -653,6 +653,9 @@
         node_ptr->physAddr, node_ptr->size, node_ptr->flags, masterID);
     req->setReqInstSeqNum(node_ptr->seqNum);

+ // If this is not done it triggers assert in L1 cache for invalid contextId
+    req->setContext(ContextID(0));
+
     req->setPC(node_ptr->pc);
     // If virtual address is valid, set the virtual address field
     // of the request.

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/34035
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I78bd70ad1e3657c9a6a1d56c234c007c2e2b586c
Gerrit-Change-Number: 34035
Gerrit-PatchSet: 1
Gerrit-Owner: Jason Lowe-Power <power...@gmail.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to