Re: [gem5-dev] Verbose test output when running from scons/regress.py

2016-05-31 Thread Steve Reinhardt
I like just printing the summary too. The current model (which is really 'print everything when you actually run the test, but only print the result if the test was not re-run') has been a pain ever since scons initialization got really slow. Steve On Tue, May 31, 2016 at 9:39 AM Jason Lowe-Pow

[gem5-dev] Review Request 3493: dev: Redefine 'length' in EthPacketData

2016-05-31 Thread Michael LeBeane
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3493/ --- Review request for Default. Repository: gem5 Description --- Changeset 11479

Re: [gem5-dev] Review Request 3489: mem: Add snoop traffic statistic

2016-05-31 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3489/#review8391 --- Ship it! Ship It! - Jason Lowe-Power On May 31, 2016, 10:43 a.m., Cur

Re: [gem5-dev] Verbose test output when running from scons/regress.py

2016-05-31 Thread Jason Lowe-Power
Hi Andreas, I like the idea to always print a summary. This is the information that's most important. The current (old?) way of dumping the stats differences was hard for a human to parse anyway. Adding a note in the scons output as to how to see more information whenever a test doesn't pass would

[gem5-dev] Verbose test output when running from scons/regress.py

2016-05-31 Thread Andreas Sandberg
Hi Everyone, Andreas (the other one) just pointed out that the regress script used to print a summary of the test results the second time you run it. As of earlier today, this is no longer the case. Instead of printing a summary, it now prints the same results as the first time the test was run.

[gem5-dev] changeset in gem5: stats: update and fix e273e86a873d

2016-05-31 Thread Curtis Dunham
changeset be6065c1d8d2 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=be6065c1d8d2 description: stats: update and fix e273e86a873d diffstat: tests/long/fs/10.linux-boot/ref/arm/linux/realview-minor-dual/stats.txt |62 +- tests/long/fs/10.linux-b

Re: [gem5-dev] Review Request 3378: scons: make build better on FreeBSD

2016-05-31 Thread Bjoern A. Zeeb
> On April 13, 2016, 4:03 p.m., Andreas Hansson wrote: > > SConstruct, line 639 > > > > > > Surely this is included by default?! > > Bjoern A. Zeeb wrote: > clang -cc1 -v /dev/null > clang -cc1 version 3.4.1 based u

Re: [gem5-dev] Review Request 3378: scons: make build better on FreeBSD

2016-05-31 Thread Andreas Hansson
> On April 13, 2016, 4:03 p.m., Andreas Hansson wrote: > > SConstruct, line 639 > > > > > > Surely this is included by default?! > > Bjoern A. Zeeb wrote: > clang -cc1 -v /dev/null > clang -cc1 version 3.4.1 based u

Re: [gem5-dev] Review Request 3378: scons: make build better on FreeBSD

2016-05-31 Thread Bjoern A. Zeeb
> On April 13, 2016, 4:03 p.m., Andreas Hansson wrote: > > SConstruct, line 639 > > > > > > Surely this is included by default?! clang -cc1 -v /dev/null clang -cc1 version 3.4.1 based upon LLVM 3.4.1 default target x86_64-

[gem5-dev] changeset in gem5: arm: Correctly check translation mode (aarch6...

2016-05-31 Thread Andreas Sandberg
changeset 55256a05d9e9 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=55256a05d9e9 description: arm: Correctly check translation mode (aarch64/aarch32) According to the ARM ARM (see AArch32.TranslateAddress in the pseudocode library), the TLB should b

[gem5-dev] changeset in gem5: arm: Enable LPAE support by default

2016-05-31 Thread Andreas Sandberg
changeset e7c9353aa537 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=e7c9353aa537 description: arm: Enable LPAE support by default LPAE has been tested with Linux 4.4 and seems to work just fine. Let's enable it by default. Change-Id: Id88c6

[gem5-dev] Review Request 3490: base: Add total() to Vector2D stat

2016-05-31 Thread Curtis Dunham
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3490/ --- Review request for Default. Repository: gem5 Description --- base: Add total

[gem5-dev] Review Request 3489: mem: Add snoop traffic statistic

2016-05-31 Thread Curtis Dunham
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3489/ --- Review request for Default. Repository: gem5 Description --- mem: Add snoop

[gem5-dev] changeset in gem5: scons: Use the new test framework from scons

2016-05-31 Thread Andreas Sandberg
changeset f3184dd8257f in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=f3184dd8257f description: scons: Use the new test framework from scons Rewrite the SCons script responsible for launching tests to use the new test framework. The biggest visible ch

[gem5-dev] changeset in gem5: tests: Fix incorrect stat.txt ignore when upd...

2016-05-31 Thread Andreas Sandberg
changeset e9b095599766 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=e9b095599766 description: tests: Fix incorrect stat.txt ignore when updating refs ClassicTest was incorrectly ignoring stats.txt when updating reference statistics. This was caused

[gem5-dev] changeset in gem5: stats: update for snoop filter tweak

2016-05-31 Thread Curtis Dunham
changeset e273e86a873d in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=e273e86a873d description: stats: update for snoop filter tweak diffstat: tests/long/fs/10.linux-boot/ref/alpha/linux/tsunami-minor/stats.txt |10 +- tests/long/fs/10.lin

[gem5-dev] changeset in gem5: mem, config: Selective use of snoop filter

2016-05-31 Thread Stephan Diestelhorst
changeset 9345c4320477 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=9345c4320477 description: mem, config: Selective use of snoop filter Disable the default snoop filter in the SystemXBar so that the typical membus does not have a snoop filter by de

Re: [gem5-dev] Review Request 3132: misc: Separate stats file is generated for SystemC-gem5 co-simulation

2016-05-31 Thread Andrew Bardsley
> On Nov. 16, 2015, 1:38 p.m., Andreas Hansson wrote: > > It is not clear to me why this is not just calling the "normal" stats dump. > > It seems like an unfortunate amount of code duplication. > > Abdul Mutaal Ahmad wrote: > It is using the same API as used in util/cxx_config. Even the >

[gem5-dev] Cron /z/m5/regression/do-regression quick

2016-05-31 Thread Cron Daemon
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/minor-timing passed. * build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/o3-timing passed. * build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/simple-atomic passed. * build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/simple