Willy Wolff has submitted this change and it was merged. (
https://gem5-review.googlesource.com/c/public/gem5/+/17748 )
Change subject: config, arm: memoryMode test
..
config, arm: memoryMode test
Commit 9a13acaa367769c38859342de9bc35aac59a6710 doesn't comply with the
same behaviour. The conditional test the memory mode on the same cluster,
while it should test the other cluster.
Change-Id: If3a7863f0905e66a548001d8e74689f5dd07179c
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/17748
Tested-by: kokoro
Reviewed-by: Nikos Nikoleris
Reviewed-by: Jason Lowe-Power
Reviewed-by: Giacomo Travaglini
Maintainer: Jason Lowe-Power
---
M configs/example/arm/fs_bigLITTLE.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
Nikos Nikoleris: Looks good to me, approved
Giacomo Travaglini: Looks good to me, approved
kokoro: Regressions pass
diff --git a/configs/example/arm/fs_bigLITTLE.py
b/configs/example/arm/fs_bigLITTLE.py
index 678b038..539d1e7 100644
--- a/configs/example/arm/fs_bigLITTLE.py
+++ b/configs/example/arm/fs_bigLITTLE.py
@@ -242,7 +242,7 @@
# Figure out the memory mode
if options.big_cpus > 0 and options.little_cpus > 0 and \
- system.littleCluster.memoryMode() !=
system.littleCluster.memoryMode():
+ system.bigCluster.memoryMode() != system.littleCluster.memoryMode():
m5.util.panic("Memory mode missmatch among CPU clusters")
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/17748
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: If3a7863f0905e66a548001d8e74689f5dd07179c
Gerrit-Change-Number: 17748
Gerrit-PatchSet: 3
Gerrit-Owner: Willy Wolff
Gerrit-Reviewer: Giacomo Travaglini
Gerrit-Reviewer: Jason Lowe-Power
Gerrit-Reviewer: Nikos Nikoleris
Gerrit-Reviewer: Willy Wolff
Gerrit-Reviewer: kokoro
Gerrit-MessageType: merged
___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev
