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

2016-10-06 Thread Gabor Dozsa
> On Sept. 9, 2016, 2:53 p.m., Gabor Dozsa wrote: > > I like the idea of starting the sync on a pseudo op very much. I cannot see > > the usefulness of stopping the sync after it started though. Do you have a > > use case in mind? > > > > I have a few comments below (mainly minor nits). > >

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

2016-10-06 Thread Cron Daemon
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/tru64/simple-timing-ruby: passed. * build/ALPHA/tests/opt/quick/se/03.learning-gem5/alpha/linux/learning-gem5-p1-two-level: passed. * build/ALPHA/tests/opt/quick/se/00.hello/alpha/tru64/simple-atomic: passed. * build/ALPHA/tests/opt

Re: [gem5-dev] Review Request 3643: style: reduce include dependencies in some headers

2016-10-06 Thread Jason Lowe-Power
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3643/#review8772 --- Ship it! Seems reasonable to me. Could you explain in the commit message

Re: [gem5-dev] Review Request 3643: style: reduce include dependencies in some headers

2016-10-06 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3643/#review8773 --- Ship it! Ship It! - Tony Gutierrez On Oct. 5, 2016, 9:27 a.m., Brando

Re: [gem5-dev] Review Request 3643: style: reduce include dependencies in some headers

2016-10-06 Thread Alexandru Dutu
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3643/#review8774 --- Ship it! Ship It! - Alexandru Dutu On Oct. 5, 2016, 4:27 p.m., Brando

Re: [gem5-dev] Review Request 3644: gpu-compute: remove inst enums and use bit flag for attributes

2016-10-06 Thread Alexandru Dutu
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3644/#review8775 --- Ship it! Ship It! - Alexandru Dutu On Oct. 6, 2016, 2:14 a.m., Tony G

[gem5-dev] changeset in gem5: config: add a separate config file for the ne...

2016-10-06 Thread Tushar Krishna
changeset 004d34b65092 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=004d34b65092 description: config: add a separate config file for the network. This patch adds a new file configs/network/Network.py to setup the network, instead of doing that withi

[gem5-dev] changeset in gem5: ruby: rename ALPHA_Network_test protocol to G...

2016-10-06 Thread Tushar Krishna
changeset cfa97c37117a in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=cfa97c37117a description: ruby: rename ALPHA_Network_test protocol to Garnet_standalone. Over the past 6 years, we realized that the protocol is essentially used to run the garnet ne

[gem5-dev] changeset in gem5: config: make internal links in network topolo...

2016-10-06 Thread Tushar Krishna
changeset cf870cd20cfc in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=cf870cd20cfc description: config: make internal links in network topology unidirectional. This patch makes the internal links within the network topology unidirectional, thus allowin

[gem5-dev] changeset in gem5: ruby: rename networktest to garnet_synthetic_...

2016-10-06 Thread Tushar Krishna
changeset 2bc3962f59fe in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=2bc3962f59fe description: ruby: rename networktest to garnet_synthetic_traffic. networktest is essentially a collection of synthetic traffic patterns for the network. The protocol na

[gem5-dev] changeset in gem5: config: add port directions and per-router de...

2016-10-06 Thread Tushar Krishna
changeset 2365e9e396f7 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=2365e9e396f7 description: config: add port directions and per-router delay in topology. This patch adds port direction names to the links during topology creation, which can be used

[gem5-dev] changeset in gem5: ruby: garnet2.0

2016-10-06 Thread Tushar Krishna
changeset 10d59d546ea2 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=10d59d546ea2 description: ruby: garnet2.0 Revamped version of garnet with more optimized single-cycle routers, more configurability, and cleaner code. diffstat: configs/network/Ne

[gem5-dev] changeset in gem5: ruby: remove the original garnet code.

2016-10-06 Thread Tushar Krishna
changeset db895719c482 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=db895719c482 description: ruby: remove the original garnet code. Only garnet2.0 will be supported henceforth. diffstat: src/mem/ruby/network/garnet/BaseGarnetNetwork.cc |

[gem5-dev] Review Request 3645: hsail, gpu-compute: remove doGm/SmReturn from gm/lm pipe add completeAcc

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

[gem5-dev] Review Request 3646: gpu-compute, hsail: call discardFetch() from the WF

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

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

2016-10-06 Thread Matthew Poremba
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3602/#review8776 --- I suggest there be a parameter to enable or disable the low-power functi

Re: [gem5-dev] Review Request 3645: hsail, gpu-compute: remove doGm/SmReturn add completeAcc

2016-10-06 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3645/ --- (Updated Oct. 6, 2016, 4:42 p.m.) Review request for Default. Summary (updated) -

[gem5-dev] changeset in gem5: ruby: Add M5_VAR_USED before variables used o...

2016-10-06 Thread Tushar Krishna
changeset ebf2acd02fc5 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=ebf2acd02fc5 description: ruby: Add M5_VAR_USED before variables used only inside assert in garnet2.0. This removes errors when building gem5.fast diffstat: src/mem/ruby/network/garnet2.