Ciro Santilli has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/33015 )

Change subject: tests: ignore 32-bit arm dual linux boot tests
......................................................................

tests: ignore 32-bit arm dual linux boot tests

As mentioned on the JIRA issue, uncacheable requests done after
cacheable requests had been done to the address make the cache writeback
and write trash data to memory.

We believe that the kernel must be doing earlier invalidation by set and
way earlier on to prevent this, but that is not implemented in gem5 yet.

The problem can be worked around by booting in atomic without caches and
checkpointing after init, because uncacheable accesses are only done on
early stages of CPU bringup, which is the more common use case anyways.

The aarch64 Linux kernel developers have stated that set and way
invalidates are not going to be used in aarch64, which further reduces the
importance of implementing this immediatly

JIRA: https://gem5.atlassian.net/browse/GEM5-640
Change-Id: Ieba31e707dcc09693d7a87ed9d51c3d1ffa3abe0
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/33015
Reviewed-by: Jason Lowe-Power <[email protected]>
Reviewed-by: Giacomo Travaglini <[email protected]>
Maintainer: Jason Lowe-Power <[email protected]>
Maintainer: Giacomo Travaglini <[email protected]>
Tested-by: kokoro <[email protected]>
---
M tests/gem5/fs/linux/arm/test.py
1 file changed, 7 insertions(+), 5 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  Giacomo Travaglini: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/tests/gem5/fs/linux/arm/test.py b/tests/gem5/fs/linux/arm/test.py
index 39cb011..d68e434 100644
--- a/tests/gem5/fs/linux/arm/test.py
+++ b/tests/gem5/fs/linux/arm/test.py
@@ -53,17 +53,13 @@

 arm_fs_long_tests = [
     'realview-simple-atomic',
-    'realview-simple-atomic-dual',
     'realview-simple-atomic-checkpoint',
     'realview-simple-timing',
-    'realview-simple-timing-dual',
     'realview-switcheroo-atomic',
     'realview-switcheroo-timing',
     'realview-o3',
     'realview-o3-checker',
-    'realview-o3-dual',
     'realview-minor',
-    'realview-minor-dual',
     'realview-switcheroo-noncaching-timing',
     'realview-switcheroo-o3',
     'realview-switcheroo-full',
@@ -75,9 +71,15 @@
     'realview64-switcheroo-o3',
     'realview64-switcheroo-full',
     'realview-simple-timing-ruby',
-    'realview-simple-timing-dual-ruby',
     'realview64-simple-timing-ruby',
     'realview64-simple-timing-dual-ruby',
+
+    # https://gem5.atlassian.net/browse/GEM5-640
+    #'realview-simple-atomic-dual',
+    #'realview-simple-timing-dual',
+    #'realview-o3-dual',
+    #'realview-minor-dual',
+    #'realview-simple-timing-dual-ruby',
 ]

 tarball = 'aarch-system-201901106.tar.bz2'

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/33015
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: Ieba31e707dcc09693d7a87ed9d51c3d1ffa3abe0
Gerrit-Change-Number: 33015
Gerrit-PatchSet: 4
Gerrit-Owner: Ciro Santilli <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Ciro Santilli <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Richard Cooper <[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

Reply via email to