[gem5-dev] Change in gem5/gem5[develop]: tests: Add a test for KVM boot then switching cpus

2021-09-15 Thread Austin Harris (Gerrit) via gem5-dev
Austin Harris has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50230 )


Change subject: tests: Add a test for KVM boot then switching cpus
..

tests: Add a test for KVM boot then switching cpus

A simple test for the switchable processor to boot linux, switch cpus,
then simulate to completion. The boot script runs m5 exit twice, first
to signal the test to switch cpus and then to actually exit.

Jira Issue: https://gem5.atlassian.net/browse/GEM5-1086

Change-Id: I9a7a6539b94b7b3f6d789ddf879d321613aef87a
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/50230
Reviewed-by: Bobby R. Bruce 
Maintainer: Bobby R. Bruce 
Tested-by: kokoro 
---
A tests/gem5/configs/components-library/boot_kvm_switch_exit.py
1 file changed, 250 insertions(+), 0 deletions(-)

Approvals:
  Bobby R. Bruce: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass




diff --git a/tests/gem5/configs/components-library/boot_kvm_switch_exit.py  
b/tests/gem5/configs/components-library/boot_kvm_switch_exit.py

new file mode 100644
index 000..d508e04
--- /dev/null
+++ b/tests/gem5/configs/components-library/boot_kvm_switch_exit.py
@@ -0,0 +1,250 @@
+# Copyright (c) 2021 The Univerity of Texas at Austin
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met: redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer;
+# redistributions in binary form must reproduce the above copyright
+# notice, this list of conditions and the following disclaimer in the
+# documentation and/or other materials provided with the distribution;
+# neither the name of the copyright holders nor the names of its
+# contributors may be used to endorse or promote products derived from
+# this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+"""
+This script boots with KVM then switches processors and exits.
+"""
+
+import argparse
+import os
+import sys
+
+import m5
+from m5.objects import Root
+
+# This is a lame hack to get the imports working correctly.
+# TODO: This needs fixed.
+sys.path.append(
+os.path.join(
+os.path.dirname(os.path.abspath(__file__)),
+os.pardir,
+os.pardir,
+os.pardir,
+os.pardir,
+)
+)
+
+from components_library.boards.x86_board import X86Board
+from components_library.coherence_protocol import CoherenceProtocol
+from components_library.isas import ISA
+from components_library.memory.single_channel import SingleChannelDDR3_1600
+from components_library.processors.cpu_types import CPUTypes
+from components_library.processors.simple_switchable_processor import (
+SimpleSwitchableProcessor,
+)
+from components_library.resources.resource import Resource
+from components_library.runtime import (
+get_runtime_coherence_protocol, get_runtime_isa
+)
+from components_library.utils.requires import requires
+
+parser = argparse.ArgumentParser(
+description="A script to test switching cpus. This test boots"
+"the linux kernel with the KVM cpu, then switches cpus until exiting."
+)
+parser.add_argument(
+"-m",
+"--mem-system",
+type=str,
+choices=("classic", "mi_example", "mesi_two_level"),
+required=True,
+help="The memory system.",
+)
+parser.add_argument(
+"-n",
+"--num-cpus",
+type=int,
+choices=(1, 2, 4, 8),
+required=True,
+help="The number of CPUs.",
+)
+parser.add_argument(
+"-c",
+"--cpu",
+type=str,
+choices=("kvm", "atomic", "timing", "o3"),
+required=True,
+help="The CPU type.",
+)
+parser.add_argument(
+"-r",
+"--resource-directory",
+type=str,
+required=False,
+help="The directory in which resources will be downloaded or exist.",
+)
+parser.add_argument(
+"-o",
+"--override-download",
+action="store_true",
+help="Override a local resource if the hashes do not match.",
+)
+parser.add_argument(
+"-k",
+"--kernel-args",
+type=str,
+

[gem5-dev] Change in gem5/gem5[develop]: tests: Add a test for KVM boot then switching cpus

2021-09-10 Thread Austin Harris (Gerrit) via gem5-dev
Austin Harris has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50230 )



Change subject: tests: Add a test for KVM boot then switching cpus
..

tests: Add a test for KVM boot then switching cpus

A simple test for the switchable processor to boot linux, switch cpus,
then simulate to completion. The current boot disk requires a maximum
number of ticks must be specified to simulate after switching, however
this test will run to completion with a boot disk that reads in a script
to run with m5 readfile by running m5 exit twice.

Change-Id: I9a7a6539b94b7b3f6d789ddf879d321613aef87a
---
A tests/gem5/configs/components-library/boot_kvm_switch_exit.py
1 file changed, 255 insertions(+), 0 deletions(-)



diff --git a/tests/gem5/configs/components-library/boot_kvm_switch_exit.py  
b/tests/gem5/configs/components-library/boot_kvm_switch_exit.py

new file mode 100644
index 000..2de1bea
--- /dev/null
+++ b/tests/gem5/configs/components-library/boot_kvm_switch_exit.py
@@ -0,0 +1,255 @@
+# Copyright (c) 2021 The Regents of the University of California
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met: redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer;
+# redistributions in binary form must reproduce the above copyright
+# notice, this list of conditions and the following disclaimer in the
+# documentation and/or other materials provided with the distribution;
+# neither the name of the copyright holders nor the names of its
+# contributors may be used to endorse or promote products derived from
+# this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+"""
+This script boots with KVM then switches processors and exits.
+"""
+
+import m5
+from m5.objects import Root
+
+import sys
+import os
+
+# This is a lame hack to get the imports working correctly.
+# TODO: This needs fixed.
+sys.path.append(
+os.path.join(
+os.path.dirname(os.path.abspath(__file__)),
+os.pardir,
+os.pardir,
+os.pardir,
+os.pardir,
+)
+)
+
+from components_library.runtime import (
+get_runtime_coherence_protocol,
+get_runtime_isa,
+)
+from components_library.utils.requires import requires
+from components_library.boards.x86_board import X86Board
+from components_library.memory.single_channel import SingleChannelDDR3_1600
+from components_library.processors.simple_switchable_processor import (
+SimpleSwitchableProcessor,
+)
+from components_library.processors.cpu_types import CPUTypes
+from components_library.isas import ISA
+from components_library.coherence_protocol import CoherenceProtocol
+from components_library.resources.resource import Resource
+
+import os
+import argparse
+
+parser = argparse.ArgumentParser(
+description="A script to run the gem5 boot test. This test boots the "
+"linux kernel."
+)
+parser.add_argument(
+"-m",
+"--mem-system",
+type=str,
+choices=("classic", "mi_example", "mesi_two_level"),
+required=True,
+help="The memory system.",
+)
+parser.add_argument(
+"-n",
+"--num-cpus",
+type=int,
+choices=(1, 2, 4, 8),
+required=True,
+help="The number of CPUs.",
+)
+parser.add_argument(
+"-c",
+"--cpu",
+type=str,
+choices=("kvm", "atomic", "timing", "o3"),
+required=True,
+help="The CPU type.",
+)
+
+parser.add_argument(
+"-t",
+"--ticks-switched",
+type=int,
+required=False,
+help="The ticks to run after switching processors.",
+)
+
+parser.add_argument(
+"-r",
+"--resource-directory",
+type=str,
+required=False,
+help="The directory in which resources will be downloaded or exist.",
+)
+
+parser.add_argument(
+"-o",
+"--override-download",
+action="store_true",
+help="Override a local resource if the hashes do not match.",
+)
+
+args = parser.parse_args()
+
+coherence_protocol_required = None
+if args.mem_system == "mi_example":
+