[gem5-dev] Re: multi-ISA gem5 proof of concept

Looks good to me. Perhaps this is explained elsewhere, but what's the logic in coupling the CPU Type (Timing, Atomic, o3) with the ISA? Why do we interact with this like `--cpu-type=X86AtomicSimpleCPU` and not `--cpu-type AtomicSimpleCPU --isa=X86`? Current CPU types are switchable during a

[gem5-dev] Re: multi-ISA gem5 proof of concept

This looks quite promising! Uploading a branch to gerrit is a good idea. A couple of notes: 1. We are going to have to be very careful with when/how we merge this and its effect on users. For instance, changing the CPU from "AtomicSimpleCPU" to "ArmAtomicSimpleCPU" will break many users' use

[gem5-dev] Re: failing kvm tests

Yip, I accidentally added some X86 tests to the quick/Kokoro tests. Thanks for pointing that out. The fix can be found here: https://gem5-review.googlesource.com/c/public/gem5/+/52563 -- Dr. Bobby R. Bruce Room 3050, Kemper Hall, UC Davis Davis, CA, 95616 web: https://www.bobbybruce.net On

[gem5-dev] Change in gem5/gem5[develop]: tests: Move MI_Example tests to Long/Nightly

Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/52563 ) Change subject: tests: Move MI_Example tests to Long/Nightly .. tests: Move MI_Example tests to Long/Nightly

[gem5-dev] Re: multi-ISA gem5 proof of concept

On Mon, Nov 8, 2021 at 8:40 AM Jason Lowe-Power wrote: > This looks quite promising! Uploading a branch to gerrit is a good idea. > Ok, I'll do that. All but 4 of the changes in this composite branch are uploaded for review, and these few extras are because they depend on all of the switching

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Fix resource downloader download to cwd upon failure

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52423 ) Change subject: stdlib: Fix resource downloader download to cwd upon failure .. stdlib: Fix resource downloader download

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Remove SimpleBoard as a superclass

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/51790 ) Change subject: stdlib: Remove SimpleBoard as a superclass .. stdlib: Remove SimpleBoard as a superclass Previously

[gem5-dev] Change in gem5/gem5[develop]: tests: Update RISCV boot tests to use Ubuntu resource

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52089 ) ( 2 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: tests: Update RISCV boot tests to use Ubuntu

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Add missing @overrides to RISCVBoard

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52185 ) Change subject: stdlib: Add missing @overrides to RISCVBoard .. stdlib: Add missing @overrides to RISCVBoard Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Create 'KernelDiskWorkload' abstract class

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/51949 ) Change subject: stdlib: Create 'KernelDiskWorkload' abstract class .. stdlib: Create 'KernelDiskWorkload' abstract class

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Automatically set disk root partition in set_workload

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/51948 ) Change subject: stdlib: Automatically set disk root partition in set_workload .. stdlib: Automatically set disk root

[gem5-dev] Change in gem5/gem5[develop]: stdlib,configs: Remove setting resource_directory in examples

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/51950 ) ( 12 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: stdlib,configs: Remove setting

[gem5-dev] Change in gem5/gem5[develop]: stdlib,tests,configs: Replace Resource override download

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52086 ) Change subject: stdlib,tests,configs: Replace Resource override download .. stdlib,tests,configs: Replace Resource

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Move 'connect_things' to AbstractBoard constructor

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52184 ) Change subject: stdlib: Move 'connect_things' to AbstractBoard constructor .. stdlib: Move 'connect_things' to

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Move 'connect_things' to the AbstractBoard

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52183 ) ( 12 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: stdlib: Move 'connect_things' to the

[gem5-dev] Change in gem5/gem5[develop]: stdlib,tests,configs: Create 'SEBinaryWorkload' class

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52186 ) ( 10 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: stdlib,tests,configs: Create

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Remove unneeded imports in the gem5 Python Library

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52203 ) ( 8 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: stdlib: Remove unneeded imports in the gem5

[gem5-dev] Change in gem5/gem5[develop]: stdlib: Remove 'exit_on_work_items' from boards' constructor

Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/52223 ) ( 6 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: stdlib: Remove 'exit_on_work_items' from

[gem5-dev] Re: multi-ISA gem5 proof of concept

This is discussed here: https://gem5.atlassian.net/browse/GEM5-1058 The CPUs are ISA specific as used, and this makes that more explicit. It also makes it clear whether a CPU is supported with a particular ISA, since if it isn't, it's not possible to put it into a config without adding the

[gem5-dev] Change in gem5/gem5[develop]: tests: Add a weekly test for SST integration.

Hoa Nguyen has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/52564 ) Change subject: tests: Add a weekly test for SST integration. .. tests: Add a weekly test for SST integration.

[gem5-dev] Re: multi-ISA gem5 proof of concept

On Mon, Nov 8, 2021 at 2:13 PM Gabe Black wrote: > > > On Mon, Nov 8, 2021 at 8:40 AM Jason Lowe-Power > wrote: > >> This looks quite promising! Uploading a branch to gerrit is a good idea. >> > > Ok, I'll do that. All but 4 of the changes in this composite branch are > uploaded for review, and

[gem5-dev] Change in gem5/gem5[develop]: Fix for 11.16 fastmodel

Earl Ou has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/52543 ) Change subject: Fix for 11.16 fastmodel .. Fix for 11.16 fastmodel Change-Id:

[gem5-dev] Jenkins build is back to normal : nightly #36

See ___ gem5-dev mailing list -- gem5-dev@gem5.org To unsubscribe send an email to gem5-dev-le...@gem5.org %(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Re: failing kvm tests

Just a quick note, I noticed when running quick regressions now that it builds both GC_X86 and X86. I'm assuming that's because the KVM tests are still set up to use X86? It would be nice to fix that to avoid building for an additional target. Gabe On Wed, Nov 3, 2021 at 10:55 AM Bobby Bruce