Bobby Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/66614?usp=email )

Change subject: tests: Add missing `_pre_instantiate()`
......................................................................

tests: Add missing `_pre_instantiate()`

As of this change:
https://gem5-review.googlesource.com/c/public/gem5/+/65051, the
`_pre_instantiate` function must be called prior to `m5.instantiate`
when using the stdlib without the Simulator module. The
"test/gem5/replacement-policies/run_replacement_policy_test.py" lacked
this and was causing errors when running replacement policy tests.

Change-Id: Id4364c63a466c19318ac2d423c2ab90aba02d7c8
---
M tests/gem5/replacement-policies/run_replacement_policy_test.py
1 file changed, 17 insertions(+), 0 deletions(-)



diff --git a/tests/gem5/replacement-policies/run_replacement_policy_test.py b/tests/gem5/replacement-policies/run_replacement_policy_test.py
index 1006109..31076c6 100644
--- a/tests/gem5/replacement-policies/run_replacement_policy_test.py
+++ b/tests/gem5/replacement-policies/run_replacement_policy_test.py
@@ -85,6 +85,7 @@
 )
 root = Root(full_system=False, system=motherboard)

+motherboard._pre_instantiate()
 m5.instantiate()

 generator.start_traffic()

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/66614?usp=email 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: Id4364c63a466c19318ac2d423c2ab90aba02d7c8
Gerrit-Change-Number: 66614
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce <bbr...@ucdavis.edu>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org

Reply via email to