Nathanael Premillieu has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/48683 )
Change subject: mem-cache: print VA and PA in the prefetch queues
......................................................................
mem-cache: print VA and PA in the prefetch queues
As prefetcher can use VA and need translation, it is
interesting to see both VA and PA when printing the queues.
PA is printed as 0 if translation has not happened yet.
Also fix a bug when the pkt is not yet created.
Change-Id: I7cd225379c2930a8d6a7882efdb3dc7bc49fb8a3
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/48683
Tested-by: kokoro <[email protected]>
Reviewed-by: Daniel Carvalho <[email protected]>
Maintainer: Daniel Carvalho <[email protected]>
---
M src/mem/cache/prefetch/queued.cc
1 file changed, 5 insertions(+), 2 deletions(-)
Approvals:
Daniel Carvalho: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/src/mem/cache/prefetch/queued.cc
b/src/mem/cache/prefetch/queued.cc
index ae4913d..c54f442 100644
--- a/src/mem/cache/prefetch/queued.cc
+++ b/src/mem/cache/prefetch/queued.cc
@@ -131,8 +131,11 @@
for (const_iterator it = queue.cbegin(); it != queue.cend();
it++, pos++) {
- DPRINTF(HWPrefetchQueue, "%s[%d]: Prefetch Req Addr: %#x
prio: %3d\n",
- queue_name, pos, it->pkt->getAddr(), it->priority);
+ Addr vaddr = it->pfInfo.getAddr();
+ /* Set paddr to 0 if not yet translated */
+ Addr paddr = it->pkt ? it->pkt->getAddr() : 0;
+ DPRINTF(HWPrefetchQueue, "%s[%d]: Prefetch Req VA: %#x PA: %#x "
+ "prio: %3d\n", queue_name, pos, vaddr, paddr,
it->priority);
}
}
2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the
submitted one.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/48683
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: I7cd225379c2930a8d6a7882efdb3dc7bc49fb8a3
Gerrit-Change-Number: 48683
Gerrit-PatchSet: 4
Gerrit-Owner: Nathanael Premillieu <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Nathanael Premillieu <[email protected]>
Gerrit-Reviewer: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
