Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/52663 )

Change subject: tests: Fix boot_kvm_fork_run.py
......................................................................

tests: Fix boot_kvm_fork_run.py

The patch
https://gem5-review.googlesource.com/c/public/gem5/+/51949
changed the way in which kernel arguments were passed to the
'set_kernel_disk_workload' function. The 'kernel_args' parameter
overrides the kernel arguments, not amends them as before. This test
script was not updated to take into account this functionality, and, as
such, the test failed. This patch fixes this.

Change-Id: I737c59329acde3a064f933bc4d31e20cf6ca55ae
---
M tests/gem5/configs/boot_kvm_fork_run.py
1 file changed, 20 insertions(+), 1 deletion(-)



diff --git a/tests/gem5/configs/boot_kvm_fork_run.py b/tests/gem5/configs/boot_kvm_fork_run.py
index 507acee..edbd26e 100644
--- a/tests/gem5/configs/boot_kvm_fork_run.py
+++ b/tests/gem5/configs/boot_kvm_fork_run.py
@@ -203,6 +203,8 @@

 motherboard.connect_things()

+kernal_args = motherboard.get_default_kernel_args() + [args.kernel_args]
+
 # Set the Full System workload.
 motherboard.set_kernel_disk_workload(
     kernel=Resource(
@@ -221,7 +223,7 @@
         m5 exit # exit in children and parent
         """
     ),
-    kernel_args=[args.kernel_args]
+    kernel_args=kernal_args
 )



--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/52663
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: I737c59329acde3a064f933bc4d31e20cf6ca55ae
Gerrit-Change-Number: 52663
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby R. Bruce <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
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