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

2016-10-13 Thread Cron Daemon
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/simple-timing-ruby: FAILED! * build/ALPHA/tests/opt/quick/se/00.hello/alpha/tru64/simple-timing-ruby: FAILED! * build/ALPHA/tests/opt/quick/se/50.memtest/alpha/linux/memtest-ruby: FAILED! * build/ALPHA/tests/opt/quick/se/60.ru

[gem5-dev] changeset in gem5: ruby: Fix regressions and make Ruby configs P...

2016-10-13 Thread Andreas Hansson
changeset 6ce719503eae in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=6ce719503eae description: ruby: Fix regressions and make Ruby configs Python packages This patch moves the addition of network options into the Ruby module to avoid the regressions

Re: [gem5-dev] Review Request 3595: dev: Add m5 op to toggle synchronization for dist-gem5.

2016-10-13 Thread Gabor Dozsa
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3595/#review8830 --- Ship it! Ship it! Thanks for the fixes! - Gabor Dozsa On Oct. 12, 201

Re: [gem5-dev] Review Request 3624: arch: [Patch 1/5] Added RISC-V base instruction set RV64I

2016-10-13 Thread Andreas Hansson
On Oct. 12, 2016, 8:31 a.m., Alec Roelke wrote: > > Some minor questions and cosmetic issues. > > > > Could you please also confirm that the files that are copied (and then > > modified) started out as hg copy (as previously discussed). > > > > Besides that I think this is looking really good.

[gem5-dev] Review Request 3663: config: Make configs/common a Python package

2016-10-13 Thread Andreas Hansson
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3663/ --- Review request for Default. Repository: gem5 Description --- Changeset 11671

Re: [gem5-dev] Review Request 3663: config: Make configs/common a Python package

2016-10-13 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3663/#review8832 --- Ship it! I noticed one small thing, no need to repost. Thanks for doing

Re: [gem5-dev] Review Request 3663: config: Make configs/common a Python package

2016-10-13 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3663/#review8833 --- Ship it! Ship It! - Tony Gutierrez On Oct. 13, 2016, 7:10 a.m., Andre

Re: [gem5-dev] Review Request 3624: arch: [Patch 1/5] Added RISC-V base instruction set RV64I

2016-10-13 Thread Alec Roelke
On Oct. 12, 2016, 8:31 a.m., Alec Roelke wrote: > > Some minor questions and cosmetic issues. > > > > Could you please also confirm that the files that are copied (and then > > modified) started out as hg copy (as previously discussed). > > > > Besides that I think this is looking really good.

Re: [gem5-dev] Review Request 3624: arch: [Patch 1/5] Added RISC-V base instruction set RV64I

2016-10-13 Thread Alec Roelke
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3624/ --- (Updated Oct. 13, 2016, 4:48 p.m.) Review request for Default. Repository: gem5

Re: [gem5-dev] Review Request 3630: riscv: [Patch 5/5] Added missing support for timing CPU models

2016-10-13 Thread Alec Roelke
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3630/ --- (Updated Oct. 13, 2016, 4:50 p.m.) Review request for Default. Repository: gem5

Re: [gem5-dev] Review Request 3624: arch: [Patch 1/5] Added RISC-V base instruction set RV64I

2016-10-13 Thread Andreas Hansson
On Oct. 12, 2016, 8:31 a.m., Alec Roelke wrote: > > Some minor questions and cosmetic issues. > > > > Could you please also confirm that the files that are copied (and then > > modified) started out as hg copy (as previously discussed). > > > > Besides that I think this is looking really good.

Re: [gem5-dev] Review Request 3624: arch: [Patch 1/5] Added RISC-V base instruction set RV64I

2016-10-13 Thread Andreas Hansson
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3624/#review8836 --- Ship it! Looks good to me. It would be good if someone else could review

Re: [gem5-dev] Review Request 3630: riscv: [Patch 5/5] Added missing support for timing CPU models

2016-10-13 Thread Alec Roelke
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3630/ --- (Updated Oct. 13, 2016, 5:10 p.m.) Review request for Default. Repository: gem5

Re: [gem5-dev] Review Request 3663: config: Make configs/common a Python package

2016-10-13 Thread Matthias Jung
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3663/#review8837 --- Ship it! Ship It! - Matthias Jung On Okt. 13, 2016, 2:10 nachm., Andr

[gem5-dev] Review Request 3664: ruby: make a RequestDesc class instead of std::pair

2016-10-13 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3664/ --- Review request for Default. Repository: gem5 Description --- Changeset 11855

Re: [gem5-dev] Review Request 3664: ruby: make a RequestDesc class instead of std::pair

2016-10-13 Thread Joe Gross
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3664/#review8838 --- Ship it! Nice fix, this is much clearer and seems like generally safer c

Re: [gem5-dev] Review Request 3664: ruby: make a RequestDesc class instead of std::pair

2016-10-13 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3664/#review8839 --- Ship it! src/mem/ruby/system/GPUCoalescer.cc (line 877)

Re: [gem5-dev] Review Request 3664: ruby: make a RequestDesc class instead of std::pair

2016-10-13 Thread Tony Gutierrez
> On Oct. 13, 2016, 2:50 p.m., Jason Lowe-Power wrote: > > src/mem/ruby/system/GPUCoalescer.cc, line 878 > > > > > > IIRC, you can do `reqCoalescer[line_addr].emplace_back(pkt, > > primary_type, secondary_type);` to avoid a

[gem5-dev] changeset in gem5: mem: update DDR3 die revision

2016-10-13 Thread Omar Naji
changeset b4015b449dd3 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=b4015b449dd3 description: mem: update DDR3 die revision Change-Id: I8992ddc1664c3ed4b2d36d8a34e4ce8be113b9de Reviewed-by: Radhika Jagtap diffstat: src/mem/DRAMCtrl.py | 16

[gem5-dev] changeset in gem5: mem: Modify drain to ensure banks and power a...

2016-10-13 Thread Wendy Elsasser
changeset 8a882e297eb2 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=8a882e297eb2 description: mem: Modify drain to ensure banks and power are idled Add constraint that all ranks have to be in PWR_IDLE before signaling drain complete This wi

[gem5-dev] changeset in gem5: mem: Sort memory commands and update DRAMPower

2016-10-13 Thread Wendy Elsasser
changeset 60d18201148d in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=60d18201148d description: mem: Sort memory commands and update DRAMPower Add local variable to stores commands to be issued. These commands are in order within a single bank but wil

[gem5-dev] changeset in gem5: stats: update references

2016-10-13 Thread Curtis Dunham
changeset b4d943429dc6 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=b4d943429dc6 description: stats: update references diffstat: tests/long/fs/10.linux-boot/ref/alpha/linux/tsunami-minor/config.ini | 48 +- tests/long/fs/10.linux-boot/re

[gem5-dev] changeset in gem5: mem: add DRAM powerdown timing

2016-10-13 Thread Omar Naji
changeset 9f3ccf96bb5a in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=9f3ccf96bb5a description: mem: add DRAM powerdown timing diffstat: src/mem/DRAMCtrl.py | 30 ++ src/mem/dram_ctrl.cc | 3 ++- src/mem/dram_ctrl.hh | 2 ++ 3 file

[gem5-dev] changeset in gem5: mem: Add DRAM low-power functionality

2016-10-13 Thread Wendy Elsasser
changeset 8c6991a00515 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=8c6991a00515 description: mem: Add DRAM low-power functionality Added power-down state transitions to the DRAM controller model. Added per rank parameter, outstandingEvents, which

[gem5-dev] changeset in gem5: mem: add DRAM powerdown current

2016-10-13 Thread Omar Naji
changeset 4aa51b4a2f24 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=4aa51b4a2f24 description: mem: add DRAM powerdown current Change-Id: I763cffe0c69f5ebbbf6a6eb12bec5c13d5d0161d Reviewed-by: Andreas Hansson Reviewed-by: Radhika Jagtap di

[gem5-dev] changeset in gem5: mem: Add callback to compute stats prior to d...

2016-10-13 Thread Wendy Elsasser
changeset beaf1afe2f83 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=beaf1afe2f83 description: mem: Add callback to compute stats prior to dump event The per rank statistics are periodically updated based on state transition and refresh events.

[gem5-dev] changeset in gem5: mem: make DDR4 x16

2016-10-13 Thread Omar Naji
changeset 55276af429ed in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=55276af429ed description: mem: make DDR4 x16 diffstat: src/mem/DRAMCtrl.py | 56 ++-- 1 files changed, 28 insertions(+), 28 deletions(-) diffs (1

[gem5-dev] changeset in gem5: isa, arm: Add missing AArch32 FP instructions

2016-10-13 Thread Mitch Hayenga
changeset 520509f3e66c in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=520509f3e66c description: isa,arm: Add missing AArch32 FP instructions This commit adds missing non-predicated, scalar floating point instructions. Specifically VRINT* floating poi

Re: [gem5-dev] Review Request 3659: stats: Add more information to uninitialized error

2016-10-13 Thread Joe Gross
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3659/#review8841 --- Ship it! Ship It! - Joe Gross On Oct. 11, 2016, 9:45 a.m., Jason Lowe