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

2016-09-28 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/si

[gem5-dev] Review Request 3639: Fix for ARM's Streamline conversion script.

2016-09-28 Thread Shawn Rosti
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3639/ --- Review request for Default. Repository: gem5 Description --- With help from

Re: [gem5-dev] Review Request 3639: Fix for ARM's Streamline conversion script.

2016-09-28 Thread Dam Sunwoo
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3639/#review8744 --- Ship it! Ship It! - Dam Sunwoo On Sept. 28, 2016, 3:26 p.m., Shawn Ro

Re: [gem5-dev] Review Request 3602: mem: Add DRAM low-power functionality

2016-09-28 Thread Wendy Elsasser
> On Sept. 19, 2016, 11:03 p.m., Éder F. Zulian wrote: > > The data members readEntries, writeEntries and outstandingEvents seem to be > > redundant information that could be extracted from queues (readQueue, > > writeQueue and respQueue, the last one requiring a little bit more effort > > to

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

2016-09-28 Thread Andreas Hansson
> On Sept. 27, 2016, 8:53 p.m., Andreas Hansson wrote: > > Thanks again for getting this in shape. There are a few minor questions and > > comment on the patch, the most important one is related to the copyrights. > > > > Also, I assume you have used hg copy for the files that are more or less

Re: [gem5-dev] Review Request 3602: mem: Add DRAM low-power functionality

2016-09-28 Thread Éder F . Zulian
> On Sept. 19, 2016, 11:03 p.m., Éder F. Zulian wrote: > > The data members readEntries, writeEntries and outstandingEvents seem to be > > redundant information that could be extracted from queues (readQueue, > > writeQueue and respQueue, the last one requiring a little bit more effort > > to

Re: [gem5-dev] Review Request 3602: mem: Add DRAM low-power functionality

2016-09-28 Thread Éder F . Zulian
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3602/#review8748 --- Ship it! Ship It! - Éder F. Zulian On Aug. 11, 2016, 9:08 a.m., Curti

Re: [gem5-dev] Review Request 3602: mem: Add DRAM low-power functionality

2016-09-28 Thread Matthias Jung
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3602/#review8749 --- Again my comment, maybe refer to the mentioned paper, since other DRAM c

[gem5-dev] Review Request 3640: ruby: correct size for partial memory writes

2016-09-28 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3640/ --- Review request for Default. Repository: gem5 Description --- Changeset 11672

Re: [gem5-dev] Review Request 3601: mem: Add callback to compute stats prior to dump event

2016-09-28 Thread Wendy Elsasser
> On Aug. 16, 2016, 3:15 p.m., Jason Lowe-Power wrote: > > src/mem/dram_ctrl.cc, line 1928 > > > > > > Should this be here? Or in the constructor or ::init()? > > > > I'm not sure where the right place is.. but I d

Re: [gem5-dev] Review Request 3640: ruby: correct size for partial memory writes

2016-09-28 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3640/#review8751 --- Ship it! Ship It! - Jason Lowe-Power On Sept. 28, 2016, 10:16 p.m., T

Re: [gem5-dev] Review Request 3601: mem: Add callback to compute stats prior to dump event

2016-09-28 Thread Jason Lowe-Power
> On Aug. 16, 2016, 3:15 p.m., Jason Lowe-Power wrote: > > src/mem/dram_ctrl.cc, line 1928 > > > > > > Should this be here? Or in the constructor or ::init()? > > > > I'm not sure where the right place is.. but I d

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

2016-09-28 Thread Alec Roelke
> On Sept. 27, 2016, 8:53 p.m., Andreas Hansson wrote: > > Thanks again for getting this in shape. There are a few minor questions and > > comment on the patch, the most important one is related to the copyrights. > > > > Also, I assume you have used hg copy for the files that are more or less

[gem5-dev] changeset in gem5: ruby: correct size for partial memory writes

2016-09-28 Thread Brad Beckmann
changeset 49cbf4bb0d36 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=49cbf4bb0d36 description: ruby: correct size for partial memory writes Fixed AbstractController::queueMemoryWritePartial to specify the correct size for partial memory writes. diff

[gem5-dev] changeset in gem5: mem: minor dprintf fix to abstract mem

2016-09-28 Thread Brad Beckmann
changeset fab5e4523380 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=fab5e4523380 description: mem: minor dprintf fix to abstract mem print number of bytes written as a decimal number, not hex diffstat: src/mem/abstract_mem.cc | 2 +- 1 files changed, 1