Giacomo Travaglini has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/57291 )
Change subject: cpu: Handle external TLBI Sync requests in TimingCPU
......................................................................
cpu: Handle external TLBI Sync requests in TimingCPU
JIRA: https://gem5.atlassian.net/browse/GEM5-1097
Change-Id: I4e92f7886a296f119720b8bcda6bea722df76153
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/57291
Reviewed-by: Giacomo Travaglini <giacomo.travagl...@arm.com>
Maintainer: Giacomo Travaglini <giacomo.travagl...@arm.com>
Reviewed-by: Andreas Sandberg <andreas.sandb...@arm.com>
Tested-by: kokoro <noreply+kok...@google.com>
---
M src/cpu/simple/timing.cc
1 file changed, 36 insertions(+), 3 deletions(-)
Approvals:
Andreas Sandberg: Looks good to me, approved
Giacomo Travaglini: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/cpu/simple/timing.cc b/src/cpu/simple/timing.cc
index 9ce7140..188611f 100644
--- a/src/cpu/simple/timing.cc
+++ b/src/cpu/simple/timing.cc
@@ -1097,14 +1097,31 @@
}
// Making it uniform across all CPUs:
- // The CPUs need to be woken up only on an invalidation packet (when
using caches)
- // or on an incoming write packet (when not using caches)
- // It is not necessary to wake up the processor on all incoming packets
+ // The CPUs need to be woken up only on an invalidation packet
+ // (when using caches) or on an incoming write packet (when not
+ // using caches) It is not necessary to wake up the processor on
+ // all incoming packets
if (pkt->isInvalidate() || pkt->isWrite()) {
for (auto &t_info : cpu->threadInfo) {
t_info->thread->getIsaPtr()->handleLockedSnoop(pkt,
cacheBlockMask);
}
+ } else if (pkt->req && pkt->req->isTlbiExtSync()) {
+ // We received a TLBI_EXT_SYNC request.
+ // In a detailed sim we would wait for memory ops to complete,
+ // but in our simple case we just respond immediately
+ auto reply_req = Request::createMemManagement(
+ Request::TLBI_EXT_SYNC_COMP,
+ cpu->dataRequestorId());
+
+ // Extra Data = the transaction ID of the Sync we're completing
+ reply_req->setExtraData(pkt->req->getExtraData());
+ PacketPtr reply_pkt = Packet::createRead(reply_req);
+
+ // TODO - reserve some credit for these responses?
+ if (!sendTimingReq(reply_pkt)) {
+ panic("Couldn't send TLBI_EXT_SYNC_COMP message");
+ }
}
}
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/57291
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: I4e92f7886a296f119720b8bcda6bea722df76153
Gerrit-Change-Number: 57291
Gerrit-PatchSet: 11
Gerrit-Owner: Giacomo Travaglini <giacomo.travagl...@arm.com>
Gerrit-Reviewer: Andreas Sandberg <andreas.sandb...@arm.com>
Gerrit-Reviewer: Gabe Black <gabe.bl...@gmail.com>
Gerrit-Reviewer: Giacomo Travaglini <giacomo.travagl...@arm.com>
Gerrit-Reviewer: Jason Lowe-Power <ja...@lowepower.com>
Gerrit-Reviewer: Jason Lowe-Power <power...@gmail.com>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
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