[gem5-dev] Change in gem5/gem5[develop]: mem: Add Units to mem stats

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39276 ) Change subject: mem: Add Units to mem stats .. mem: Add Units to mem stats Add units to mem stats except mem-ruby stats

[gem5-dev] Change in gem5/gem5[develop]: base-stats: Add Units to Stats

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/38855 ) Change subject: base-stats: Add Units to Stats .. base-stats: Add Units to Stats This commit adds an option to add SI and

[gem5-dev] Change in gem5/gem5[develop]: arch-arm,arch-riscv,arch-x86: Add units to stats

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39416 ) Change subject: arch-arm,arch-riscv,arch-x86: Add units to stats .. arch-arm,arch-riscv,arch-x86: Add units to stats

[gem5-dev] Change in gem5/gem5[develop]: dev,dev-arm: Add units to stats in /src/dev

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39415 ) Change subject: dev,dev-arm: Add units to stats in /src/dev .. dev,dev-arm: Add units to stats in /src/dev Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: cpu,mem: Converting stats to supported units

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/41013 ) Change subject: cpu,mem: Converting stats to supported units .. cpu,mem: Converting stats to supported units There are

[gem5-dev] Change in gem5/gem5[develop]: sim: Add units to src/sim

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39296 ) Change subject: sim: Add units to src/sim .. sim: Add units to src/sim Change-Id: I5fa147aa1319d62be1790bbd74fd097ac566f808

[gem5-dev] Change in gem5/gem5[develop]: mem: Fix/Improve stats in src/mem

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39975 ) Change subject: mem: Fix/Improve stats in src/mem .. mem: Fix/Improve stats in src/mem * Improved/Fixed the stat

[gem5-dev] Change in gem5/gem5[develop]: mem: Change warmupCycle stat to warmupTick

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40575 ) Change subject: mem: Change warmupCycle stat to warmupTick .. mem: Change warmupCycle stat to warmupTick Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: learning-gem5: Add units to stats

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39417 ) Change subject: learning-gem5: Add units to stats .. learning-gem5: Add units to stats Change-Id:

[gem5-dev] Re: removing partial linking hurts simulator performance, locality suspected

2021-02-10 Thread Jason Lowe-Power via gem5-dev
This is interesting! My takeaway: we need bigger i-caches to run gem5 ;). Is the binary about the same size for the two conditions? Could it be simply that the instruction working set is bigger when not using partial linking? That said, a 3% performance difference isn't a big problem, in my

[gem5-dev] Change in gem5/gem5[develop]: scons: Pull Configure() to earlier in SConstruct so we can use it more.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40860 ) Change subject: scons: Pull Configure() to earlier in SConstruct so we can use it more. .. scons: Pull Configure() to

[gem5-dev] Change in gem5/gem5[develop]: cpu: Remove units from stats description

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40735 ) Change subject: cpu: Remove units from stats description .. cpu: Remove units from stats description A recent change

[gem5-dev] Change in gem5/gem5[develop]: dev-arm: Add VRAM to VExpress_GEM5_Base

2021-02-10 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40973 ) Change subject: dev-arm: Add VRAM to VExpress_GEM5_Base .. dev-arm: Add VRAM to VExpress_GEM5_Base Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: dev,dev-arm: Remove units from stats description

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40675 ) Change subject: dev,dev-arm: Remove units from stats description .. dev,dev-arm: Remove units from stats description A

[gem5-dev] Change in gem5/gem5[develop]: cpu: Delete the empty, default off "nocpu".

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40871 ) Change subject: cpu: Delete the empty, default off "nocpu". .. cpu: Delete the empty, default off "nocpu". This very nearly

[gem5-dev] Change in gem5/gem5[develop]: base-stats: Print Units in Stats dump

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40622 ) Change subject: base-stats: Print Units in Stats dump .. base-stats: Print Units in Stats dump Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: mem: Remove units from stats description

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40655 ) Change subject: mem: Remove units from stats description .. mem: Remove units from stats description The change

[gem5-dev] Change in gem5/gem5[develop]: arch-riscv: Fixed the style of stats variable names in TlbStats

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40695 ) Change subject: arch-riscv: Fixed the style of stats variable names in TlbStats .. arch-riscv: Fixed the style of stats

[gem5-dev] Change in gem5/gem5[develop]: mem: Initialize pendingWrites stat of NVMStats

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41073 ) Change subject: mem: Initialize pendingWrites stat of NVMStats .. mem: Initialize pendingWrites stat of NVMStats

[gem5-dev] Change in gem5/gem5[develop]: scons: Add support for debug info compression.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40715 ) Change subject: scons: Add support for debug info compression. .. scons: Add support for debug info compression. If

[gem5-dev] Change in gem5/gem5[develop]: scons: Remove partial linking.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40795 ) Change subject: scons: Remove partial linking. .. scons: Remove partial linking. This feature didn't actually provide any

[gem5-dev] Change in gem5/gem5[develop]: scons: Update the Check(Cxx|Link)Flag checks to set their vars.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40855 ) Change subject: scons: Update the Check(Cxx|Link)Flag checks to set their vars. .. scons: Update the Check(Cxx|Link)Flag

[gem5-dev] Change in gem5/gem5[develop]: scons: Extract the gem5 specific Configure call to its own file.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40858 ) Change subject: scons: Extract the gem5 specific Configure call to its own file. .. scons: Extract the gem5 specific

[gem5-dev] Change in gem5/gem5[develop]: scons: Trim down a check for mac OS and arch setting flags.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40857 ) Change subject: scons: Trim down a check for mac OS and arch setting flags. .. scons: Trim down a check for mac OS and arch

[gem5-dev] Change in gem5/gem5[develop]: scons: Remove "TIMEOUT" variable and checks.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40856 ) Change subject: scons: Remove "TIMEOUT" variable and checks. .. scons: Remove "TIMEOUT" variable and checks. This was used

[gem5-dev] Change in gem5/gem5[develop]: base: Fix copyright of base/stats/SConscript

2021-02-10 Thread Daniel Carvalho (Gerrit) via gem5-dev
Daniel Carvalho has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40755 ) Change subject: base: Fix copyright of base/stats/SConscript .. base: Fix copyright of base/stats/SConscript The

[gem5-dev] Change in gem5/gem5[develop]: mem: Initialize pendingWrites stat of NVMStats

2021-02-10 Thread Hoa Nguyen (Gerrit) via gem5-dev
Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/41073 ) Change subject: mem: Initialize pendingWrites stat of NVMStats .. mem: Initialize pendingWrites stat of NVMStats The

[gem5-dev] Re: test library error

2021-02-10 Thread Bobby Bruce via gem5-dev
Hey Gabe, This is weird, though I cannot replicate it. I'd also guess it's weird timeout. Have you seen it elsewhere? Kind regards, Bobby -- Dr. Bobby R. Bruce Room 2235, Kemper Hall, UC Davis Davis, CA, 95616 web: https://www.bobbybruce.net On Mon, Feb 8, 2021 at 9:31 PM Gabe Black via

[gem5-dev] Re: test library error

2021-02-10 Thread Gabe Black via gem5-dev
I don't think so, I just wanted to point it out since it would otherwise probably never get seen. I do occasionally see weird errors like this, but not all that often. I haven't paid close enough attention to them to notice if they follow a pattern. Gabe On Wed, Feb 10, 2021 at 11:30 AM Bobby

[gem5-dev] Change in gem5/gem5[develop]: python: more readable Ruby dot topology

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41093 ) Change subject: python: more readable Ruby dot topology .. python: more readable Ruby dot topology Controllers

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: add TBEStorage structure

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41157 ) Change subject: mem-ruby: add TBEStorage structure .. mem-ruby: add TBEStorage structure The TBEStorage is used

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: extended transaction profiling

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41158 ) Change subject: mem-ruby: extended transaction profiling .. mem-ruby: extended transaction profiling Adds

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: add andMask to WriteMask

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41154 ) Change subject: mem-ruby: add andMask to WriteMask .. mem-ruby: add andMask to WriteMask Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: fixes for masked writes

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41134 ) Change subject: mem-ruby: fixes for masked writes .. mem-ruby: fixes for masked writes This adds

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: warns on masked functional writes

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41133 ) Change subject: mem-ruby: warns on masked functional writes .. mem-ruby: warns on masked functional writes It's

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: fix Sequencer latency reporting

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41114 ) Change subject: mem-ruby: fix Sequencer latency reporting .. mem-ruby: fix Sequencer latency reporting Most

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: fix functional reads in abstract ctrl

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41153 ) Change subject: mem-ruby: fix functional reads in abstract ctrl .. mem-ruby: fix functional reads in abstract

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: removes data ptr from RubyRequest

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41113 ) Change subject: mem-ruby: removes data ptr from RubyRequest .. mem-ruby: removes data ptr from RubyRequest It's

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: add wakeup_port statement

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41155 ) Change subject: mem-ruby: add wakeup_port statement .. mem-ruby: add wakeup_port statement While the

[gem5-dev] Change in gem5/gem5[develop]: mem-ruby: intToTick helper

2021-02-10 Thread Gerrit
Tiago Mück has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41156 ) Change subject: mem-ruby: intToTick helper .. mem-ruby: intToTick helper Change-Id:

[gem5-dev] Re: removing partial linking hurts simulator performance, locality suspected

2021-02-10 Thread Gabe Black via gem5-dev
Yeah, that wouldn't hurt :-). The size difference of the .text section, what I assume is the most relevant for icache misses, is only a 27712 byte *decrease*, from 0x16ab445 to 0x16a4805, so the size without partial linking actually went down slightly. I think what may actually be happening is

[gem5-dev] weird issue using gcr.io/gem5-test/clang-version-9 docker image?

2021-02-10 Thread Gabe Black via gem5-dev
Hi folks. I was trying to debug a problem in one of my scons cleanups which was failing on kokoro when trying to build under clang. I think I fixed it, but after doing so it started to fail because it didn't like the version of python it was finding, and sure enough that image has python-config

[gem5-dev] Change in gem5/gem5[develop]: scons: In Check(Cxx|Link)Flag, only install the flag if it exists.

2021-02-10 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/41159 ) Change subject: scons: In Check(Cxx|Link)Flag, only install the flag if it exists. .. scons: In