[gem5-dev] Change in public/gem5[master]: tests: Resurrect initest input file(s).

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/6041 Change subject: tests: Resurrect initest input file(s). .. tests: Resurrect initest input file(s). Delete the current version of

[gem5-dev] Change in public/gem5[master]: arch-riscv: Move compressed ops out of ISA

Alec Roelke has uploaded this change for review. ( https://gem5-review.googlesource.com/6026 Change subject: arch-riscv: Move compressed ops out of ISA .. arch-riscv: Move compressed ops out of ISA This patch moves static

[gem5-dev] Change in public/gem5[master]: arch-riscv: Move static_inst into a directory

Alec Roelke has uploaded this change for review. ( https://gem5-review.googlesource.com/6021 Change subject: arch-riscv: Move static_inst into a directory .. arch-riscv: Move static_inst into a directory This patch creates

[gem5-dev] Change in public/gem5[master]: arch-riscv: Move unknown out of ISA description

Alec Roelke has uploaded this change for review. ( https://gem5-review.googlesource.com/6023 Change subject: arch-riscv: Move unknown out of ISA description .. arch-riscv: Move unknown out of ISA description This patch

[gem5-dev] Change in public/gem5[master]: arch-riscv: Remove static parts of AMOs out of ISA

Alec Roelke has uploaded this change for review. ( https://gem5-review.googlesource.com/6025 Change subject: arch-riscv: Remove static parts of AMOs out of ISA .. arch-riscv: Remove static parts of AMOs out of ISA This

[gem5-dev] Change in public/gem5[master]: arch-riscv: Move standard ops out of ISA

Alec Roelke has uploaded this change for review. ( https://gem5-review.googlesource.com/6022 Change subject: arch-riscv: Move standard ops out of ISA .. arch-riscv: Move standard ops out of ISA This patch removes static

[gem5-dev] Change in public/gem5[master]: arch-riscv: Move parts of mem insts out of ISA

Alec Roelke has uploaded this change for review. ( https://gem5-review.googlesource.com/6024 Change subject: arch-riscv: Move parts of mem insts out of ISA .. arch-riscv: Move parts of mem insts out of ISA This patch moves

[gem5-dev] Change in public/gem5[master]: arch-riscv: Add missing system calls

Alec Roelke has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5322 ) Change subject: arch-riscv: Add missing system calls .. arch-riscv: Add missing system calls This patch adds all system

[gem5-dev] Change in public/gem5[master]: sim-se: Add default to SyscallDesc constructor

Alec Roelke has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5501 ) Change subject: sim-se: Add default to SyscallDesc constructor .. sim-se: Add default to SyscallDesc constructor This

Re: [gem5-dev] gem5.opt => opt/gem5?

I thought about this a bit more, and one thing that would be worse under this scheme would be that generated source files would be generated multiple times when building multiple builds since they'd no longer be in a common location. I think it's pretty rare for people to build multiple builds,

[gem5-dev] Change in public/gem5[master]: tests: Fix the stats unit test.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/6001 Change subject: tests: Fix the stats unit test. .. tests: Fix the stats unit test. This has been broken since February. The

[gem5-dev] Change in public/gem5[master]: sparc: Move integer StaticInst base classes out of the ISA desc.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5483 ) Change subject: sparc: Move integer StaticInst base classes out of the ISA desc. .. sparc: Move integer StaticInst base

[gem5-dev] Change in public/gem5[master]: sparc: Move the mem base classes out of the ISA description.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5482 ) Change subject: sparc: Move the mem base classes out of the ISA description. .. sparc: Move the mem base classes out of the

[gem5-dev] Change in public/gem5[master]: sparc: Move the microop/macroop base classes out of the ISA desc.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5481 ) Change subject: sparc: Move the microop/macroop base classes out of the ISA desc. .. sparc: Move the microop/macroop

[gem5-dev] Change in public/gem5[master]: sparc: Return debug faults from unimplemented instructions.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5465 ) Change subject: sparc: Return debug faults from unimplemented instructions. .. sparc: Return debug faults from

[gem5-dev] Change in public/gem5[master]: sparc: Pull the unimplemented formats out of the ISA description.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5464 ) Change subject: sparc: Pull the unimplemented formats out of the ISA description. .. sparc: Pull the unimplemented

[gem5-dev] Change in public/gem5[master]: sparc: Pull the "Uknown" StaticInst class out of the ISA description.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5463 ) Change subject: sparc: Pull the "Uknown" StaticInst class out of the ISA description. .. sparc: Pull the "Uknown"

[gem5-dev] Change in public/gem5[master]: sparc: Pull most of the Nop format out of the ISA description.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5462 ) Change subject: sparc: Pull most of the Nop format out of the ISA description. .. sparc: Pull most of the Nop format out

[gem5-dev] Change in public/gem5[master]: sparc: Pull more StaticInst base classes out of the ISA desc.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5461 ) Change subject: sparc: Pull more StaticInst base classes out of the ISA desc. .. sparc: Pull more StaticInst base

[gem5-dev] Change in public/gem5[master]: sparc: Pull flat static instruction classes out of the ISA.

Gabe Black has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5422 ) Change subject: sparc: Pull flat static instruction classes out of the ISA. .. sparc: Pull flat static instruction classes

[gem5-dev] Change in public/gem5[master]: scons: Move some compiler flag setting code to the SConstruct.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/5981 Change subject: scons: Move some compiler flag setting code to the SConstruct. .. scons: Move some compiler flag setting code to

[gem5-dev] Change in public/gem5[master]: scons: Minor cleanup of how partial linking is handled in makeEnv.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/5985 Change subject: scons: Minor cleanup of how partial linking is handled in makeEnv. .. scons: Minor cleanup of how partial linking

[gem5-dev] Change in public/gem5[master]: scons: Switch from "guards" to "tags" on source files.

Hello Jason Lowe-Power, Andreas Sandberg, I'd like you to reexamine a change. Please visit https://gem5-review.googlesource.com/5822 to look at the new patch set (#3). Change subject: scons: Switch from "guards" to "tags" on source files.

[gem5-dev] Change in public/gem5[master]: scons: Remove the extra_deps option from the helper function make_obj.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/5983 Change subject: scons: Remove the extra_deps option from the helper function make_obj. .. scons: Remove the extra_deps option

[gem5-dev] Change in public/gem5[master]: scons: Break make_obj into make_static and make_shared functions.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/5984 Change subject: scons: Break make_obj into make_static and make_shared functions. .. scons: Break make_obj into make_static and

[gem5-dev] Change in public/gem5[master]: scons: Get rid of a flag which makes Werror optional.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/5982 Change subject: scons: Get rid of a flag which makes Werror optional. .. scons: Get rid of a flag which makes Werror optional. This

Re: [gem5-dev] SPARC cls and statute of limitation on reviews

Ok, sounds good. I was thinking that sort of thing would make sense, but didn't want to go that way unilaterally. Gabe On Tue, Nov 21, 2017 at 6:05 AM, Andreas Sandberg wrote: > Hi Gabe, > > I won't have time to review them anytime soon. Since only Jason has > offered

[gem5-dev] Change in public/gem5[master]: arch-arm: ArmPMU refactor

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5764 ) Change subject: arch-arm: ArmPMU refactor .. arch-arm: ArmPMU refactor Change the definition of PMU events in order

[gem5-dev] Change in public/gem5[master]: cpu, cpu, sim: move Cycle probe update

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5762 ) Change subject: cpu, cpu, sim: move Cycle probe update .. cpu, cpu, sim: move Cycle probe update Move the code

[gem5-dev] Change in public/gem5[master]: arch-arm: Do not increment PMU cycle event in WFI/WFE

Andreas Sandberg has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5763 ) Change subject: arch-arm: Do not increment PMU cycle event in WFI/WFE .. arch-arm: Do not increment PMU cycle event

[gem5-dev] Change in public/gem5[master]: sim: Fix need to save address space info during serialization.

Austin Harris has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5901 ) Change subject: sim: Fix need to save address space info during serialization. .. sim: Fix need to save address space

Re: [gem5-dev] gem5.opt => opt/gem5?

Hey Gabe, I don't have any strong feelings on this, just a couple of points. 1) I think we should support "backwards compatibility" if we do this by having symlinks from the old names to the new names. This way we don't break all of the scripts people have been using. 2) How much effort is this

[gem5-dev] Change in public/gem5[master]: scons: Build error introduced by site_tools/default.py

Giacomo Travaglini has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5941 ) Change subject: scons: Build error introduced by site_tools/default.py .. scons: Build error introduced by

[gem5-dev] Change in public/gem5[master]: arch-arm: Fix MSR/MRS disassemble

Giacomo Travaglini has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5861 ) Change subject: arch-arm: Fix MSR/MRS disassemble .. arch-arm: Fix MSR/MRS disassemble This patch is fixing the

[gem5-dev] Change in public/gem5[master]: arch-arm: Fix MCR/MRC disassemble

Giacomo Travaglini has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5862 ) Change subject: arch-arm: Fix MCR/MRC disassemble .. arch-arm: Fix MCR/MRC disassemble This patch is fixing the

Re: [gem5-dev] SPARC cls and statute of limitation on reviews

Hi Gabe, I won't have time to review them anytime soon. Since only Jason has offered to review some of them, I'd suggest that we go for a more relaxed review policy for SPARC. I would be happy for you to self approve these changes since you are the SPARC maintainer and there are no other obvious

Re: [gem5-dev] gem5.opt => opt/gem5?

Hi Gabe, I can definitely see the reasoning behind this. I think revisiting this is a good idea. However, I'm a bit reluctant since it affects a lot of existing build scripts and test infrastructure. Another benefit that you didn't mention is that this avoid some annoying "issues" when the

[gem5-dev] Change in public/gem5[master]: cpu-o3: Prevent cpu from suspending if it is already draining

Nikos Nikoleris has submitted this change and it was merged. ( https://gem5-review.googlesource.com/5881 ) Change subject: cpu-o3: Prevent cpu from suspending if it is already draining .. cpu-o3: Prevent cpu from

[gem5-dev] Change in public/gem5[master]: scons: Build error introduced by site_tools/default.py

Giacomo Travaglini has uploaded this change for review. ( https://gem5-review.googlesource.com/5941 Change subject: scons: Build error introduced by site_tools/default.py .. scons: Build error introduced by

[gem5-dev] Change in public/gem5[master]: arch-arm: HVC instruction undefined in secure EL1

Hello Andreas Sandberg, I'd like you to do a code review. Please visit https://gem5-review.googlesource.com/5921 to review the following change. Change subject: arch-arm: HVC instruction undefined in secure EL1 ..

[gem5-dev] Cron <m5test@zizzer> /z/m5/regression/do-regression quick

* build/RISCV/tests/opt/quick/se/00.hello/riscv/linux/simple-atomic: FAILED! * build/RISCV/tests/opt/quick/se/02.insttest/riscv/linux-rv64c/simple-timing: FAILED! * build/RISCV/tests/opt/quick/se/00.hello/riscv/linux/simple-timing-ruby: FAILED! *