Re: [gem5-dev] Review Request 3558: ruby: garnet2.0

2016-10-03 Thread Krishna, Tushar
Hi Jieming,
I have been updating the wiki so that I can point to it as soon as I release 
the patches.
I am having trouble uploading images to the wiki. I get this error:

Warning: 
fopen(/z/www/htdocs/wiki/images/lockdir/jny8zw5whr3mvxbze7rj32hk8ghgbgf.lock): 
failed to open stream: Permission denied in 
/z/www/htdocs/wiki/includes/filebackend/lockmanager/FSLockManager.php on line 
125
[51edc250] 2016-10-02 20:30:33: Fatal exception of type "MWException"

It seems to be from the server hosting the wiki.
Do you know how to resolve this?

Thanks,
Tushar



On Oct 3, 2016, at 1:55 PM, Jieming Yin 
> wrote:

Hi Tushar,

Thanks for addressing Zicong's comment. Do you feel comfortable checking in the 
patches?

Jieming

On Sat, Sep 24, 2016 at 9:18 AM, Krishna, Tushar 
> wrote:
I found the bug and fixed it. [Basically by default garnet should use the 
routing table, the src_outport and dst_inport names are for supporting special 
routing algorithms that the user may want to implement. The default routing 
algo was set to the user-specified one in the config file].
Thanks Zicong!


On Sep 24, 2016, at 12:11 PM, Zicong Wang 
>>
 wrote:

This is an automatically generated e-mail. To reply, visit: 
http://reviews.gem5.org/r/3558/


On 九月 23rd, 2016, 1:47 p.m. CST, Zicong Wang wrote:

Missing src_outport & dst_inport when create the mesh links in 
MeshDirCorners_XY.py, which will cause segmentation fault(tested in 
X86_MESI_Two_Level)

On 九月 23rd, 2016, 11:58 p.m. CST, Tushar Krishna wrote:

Hi Zicong,
Thanks for testing the patch on your end. Can you send me the command you used 
that gave you the segmentation fault?
src_outport and dst_inport are supposed to be optional parameters.
I can add them in to MeshDirCorners_XY but want to find out why they gave an 
error.
Thanks,
Tushar

Hi, Tushar,
Thanks for your working garnet2.0, and the network becomes easier to use for 
researching.
The command I used is showed as below:

./build/X86_MESI_Two_Level/gem5.debug \
configs/example/se.py \
--cmd=tests/test-progs/hello/bin/x86/linux/hello \
--cpu-type=detailed --ruby --num-cpus=16 \
--caches --cacheline_size=128 \
--l1i_size=16kB --l1i_assoc=2 \
--l1d_size=16kB --l1d_assoc=2 \
--l2cache --l2_size=128kB --l2_assoc=4 --num-l2caches=16 \
--topology=MeshDirCorners_XY --mesh-rows=4 \
--num-dirs=4 --mem-size=4GB \
--sys-clock=1GHz --ruby-clock=1GHz --cpu-clock=1GHz \
--maxinsts=1 \
--network=garnet2.0 |tee m5out/runscript.log

I test it on a latest changeset(fd783bff017c) which patched with 
rb35{49,50,54,55,56,57,58}.patch. In addition, I got the same segmentation 
fault running on my own gem5 project(based on a changeset several month before 
released). I checked the core file and found out nullptr at 
NetworkInterface.cc>:178
 caused the breakdown(a packet injected to vnet 0 incorrectly enqueued at NI 0).
What's more, It works well in X86_MESI_Two_Level(Mesh_XY) and 
Garnet_standalone. When I add src_outport & dst_inport into MeshDirCorners_XY, 
it works as well as Mesh_XY.
I hope the information are useful to your working.

Zicong


- Zicong


On 八月 24th, 2016, 11:24 p.m. CST, Tushar Krishna wrote:

Review request for Default, Andreas Hansson, Brad Beckmann, Jieming Yin, and 
Matthew Poremba.
By Tushar Krishna.

Updated 八月 24, 2016, 11:24 p.m.

Repository: gem5
Description

ruby: garnet2.0
Revamped version of garnet with more optimized single-cycle routers,
more configurability, and cleaner code.


Diffs

  *   src/mem/ruby/network/garnet2.0/GarnetLink.py (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/GarnetNetwork.hh (PRE-CREATION)
  *   
src/mem/ruby/network/garnet2.0/GarnetNetwork.cc>
 (PRE-CREATION)
  *   configs/topologies/Crossbar.py (d726d0cea027)
  *   configs/topologies/MeshDirCorners_XY.py (PRE-CREATION)
  *   configs/topologies/Mesh_XY.py (PRE-CREATION)
  *   configs/topologies/Mesh_westfirst.py (PRE-CREATION)
  *   configs/topologies/Pt2Pt.py (d726d0cea027)
  *   
src/base/statistics.cc>
 (d726d0cea027)
  *   src/mem/ruby/network/BasicRouter.py (d726d0cea027)
  *   src/mem/ruby/network/garnet2.0/CommonTypes.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/Credit.hh (PRE-CREATION)
  *   
src/mem/ruby/network/garnet2.0/Credit.cc>
 (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/CreditLink.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/CrossbarSwitch.hh (PRE-CREATION)
  *   
src/mem/ruby/network/garnet2.0/CrossbarSwitch.cc>
 

Re: [gem5-dev] Review Request 3558: ruby: garnet2.0

2016-10-03 Thread Jieming Yin
Hi Tushar,

Thanks for addressing Zicong's comment. Do you feel comfortable checking in
the patches?

Jieming

On Sat, Sep 24, 2016 at 9:18 AM, Krishna, Tushar 
wrote:

> I found the bug and fixed it. [Basically by default garnet should use the
> routing table, the src_outport and dst_inport names are for supporting
> special routing algorithms that the user may want to implement. The default
> routing algo was set to the user-specified one in the config file].
> Thanks Zicong!
>
>
> On Sep 24, 2016, at 12:11 PM, Zicong Wang > wrote:
>
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/3558/
>
>
> On 九月 23rd, 2016, 1:47 p.m. CST, Zicong Wang wrote:
>
> Missing src_outport & dst_inport when create the mesh links in
> MeshDirCorners_XY.py, which will cause segmentation fault(tested in
> X86_MESI_Two_Level)
>
> On 九月 23rd, 2016, 11:58 p.m. CST, Tushar Krishna wrote:
>
> Hi Zicong,
> Thanks for testing the patch on your end. Can you send me the command you
> used that gave you the segmentation fault?
> src_outport and dst_inport are supposed to be optional parameters.
> I can add them in to MeshDirCorners_XY but want to find out why they gave
> an error.
> Thanks,
> Tushar
>
> Hi, Tushar,
> Thanks for your working garnet2.0, and the network becomes easier to use
> for researching.
> The command I used is showed as below:
>
> ./build/X86_MESI_Two_Level/gem5.debug \
> configs/example/se.py \
> --cmd=tests/test-progs/hello/bin/x86/linux/hello \
> --cpu-type=detailed --ruby --num-cpus=16 \
> --caches --cacheline_size=128 \
> --l1i_size=16kB --l1i_assoc=2 \
> --l1d_size=16kB --l1d_assoc=2 \
> --l2cache --l2_size=128kB --l2_assoc=4 --num-l2caches=16 \
> --topology=MeshDirCorners_XY --mesh-rows=4 \
> --num-dirs=4 --mem-size=4GB \
> --sys-clock=1GHz --ruby-clock=1GHz --cpu-clock=1GHz \
> --maxinsts=1 \
> --network=garnet2.0 |tee m5out/runscript.log
>
> I test it on a latest changeset(fd783bff017c) which patched with
> rb35{49,50,54,55,56,57,58}.patch. In addition, I got the same
> segmentation fault running on my own gem5 project(based on a changeset
> several month before released). I checked the core file and found out
> nullptr at NetworkInterface.cc:178 caused the
> breakdown(a packet injected to vnet 0 incorrectly enqueued at NI 0).
> What's more, It works well in X86_MESI_Two_Level(Mesh_XY) and
> Garnet_standalone. When I add src_outport & dst_inport into
> MeshDirCorners_XY, it works as well as Mesh_XY.
> I hope the information are useful to your working.
>
> Zicong
>
>
> - Zicong
>
>
> On 八月 24th, 2016, 11:24 p.m. CST, Tushar Krishna wrote:
>
> Review request for Default, Andreas Hansson, Brad Beckmann, Jieming Yin,
> and Matthew Poremba.
> By Tushar Krishna.
>
> Updated 八月 24, 2016, 11:24 p.m.
>
> Repository: gem5
> Description
>
> ruby: garnet2.0
> Revamped version of garnet with more optimized single-cycle routers,
> more configurability, and cleaner code.
>
>
> Diffs
>
>   *   src/mem/ruby/network/garnet2.0/GarnetLink.py (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/GarnetNetwork.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/GarnetNetwork.cc garnetnetwork.cc> (PRE-CREATION)
>   *   configs/topologies/Crossbar.py (d726d0cea027)
>   *   configs/topologies/MeshDirCorners_XY.py (PRE-CREATION)
>   *   configs/topologies/Mesh_XY.py (PRE-CREATION)
>   *   configs/topologies/Mesh_westfirst.py (PRE-CREATION)
>   *   configs/topologies/Pt2Pt.py (d726d0cea027)
>   *   src/base/statistics.cc (d726d0cea027)
>   *   src/mem/ruby/network/BasicRouter.py (d726d0cea027)
>   *   src/mem/ruby/network/garnet2.0/CommonTypes.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/Credit.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/Credit.cc
> (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/CreditLink.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/CrossbarSwitch.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/CrossbarSwitch.cc crossbarswitch.cc> (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/GarnetLink.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/GarnetLink.cc
> (PRE-CREATION)
>   *   configs/network/Network.py (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/Router.cc
> (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/RoutingUnit.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/RoutingUnit.cc
> (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/SConscript (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/SwitchAllocator.hh (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/SwitchAllocator.cc switchallocator.cc> (PRE-CREATION)
>   *   src/mem/ruby/network/garnet2.0/VirtualChannel.hh (PRE-CREATION)
>   *   

Re: [gem5-dev] Review Request 3558: ruby: garnet2.0

2016-09-24 Thread Krishna, Tushar
I found the bug and fixed it. [Basically by default garnet should use the 
routing table, the src_outport and dst_inport names are for supporting special 
routing algorithms that the user may want to implement. The default routing 
algo was set to the user-specified one in the config file].
Thanks Zicong!


On Sep 24, 2016, at 12:11 PM, Zicong Wang 
> wrote:

This is an automatically generated e-mail. To reply, visit: 
http://reviews.gem5.org/r/3558/


On 九月 23rd, 2016, 1:47 p.m. CST, Zicong Wang wrote:

Missing src_outport & dst_inport when create the mesh links in 
MeshDirCorners_XY.py, which will cause segmentation fault(tested in 
X86_MESI_Two_Level)

On 九月 23rd, 2016, 11:58 p.m. CST, Tushar Krishna wrote:

Hi Zicong,
Thanks for testing the patch on your end. Can you send me the command you used 
that gave you the segmentation fault?
src_outport and dst_inport are supposed to be optional parameters.
I can add them in to MeshDirCorners_XY but want to find out why they gave an 
error.
Thanks,
Tushar

Hi, Tushar,
Thanks for your working garnet2.0, and the network becomes easier to use for 
researching.
The command I used is showed as below:

./build/X86_MESI_Two_Level/gem5.debug \
configs/example/se.py \
--cmd=tests/test-progs/hello/bin/x86/linux/hello \
--cpu-type=detailed --ruby --num-cpus=16 \
--caches --cacheline_size=128 \
--l1i_size=16kB --l1i_assoc=2 \
--l1d_size=16kB --l1d_assoc=2 \
--l2cache --l2_size=128kB --l2_assoc=4 --num-l2caches=16 \
--topology=MeshDirCorners_XY --mesh-rows=4 \
--num-dirs=4 --mem-size=4GB \
--sys-clock=1GHz --ruby-clock=1GHz --cpu-clock=1GHz \
--maxinsts=1 \
--network=garnet2.0 |tee m5out/runscript.log

I test it on a latest changeset(fd783bff017c) which patched with 
rb35{49,50,54,55,56,57,58}.patch. In addition, I got the same segmentation 
fault running on my own gem5 project(based on a changeset several month before 
released). I checked the core file and found out nullptr at 
NetworkInterface.cc:178 caused the breakdown(a 
packet injected to vnet 0 incorrectly enqueued at NI 0).
What's more, It works well in X86_MESI_Two_Level(Mesh_XY) and 
Garnet_standalone. When I add src_outport & dst_inport into MeshDirCorners_XY, 
it works as well as Mesh_XY.
I hope the information are useful to your working.

Zicong


- Zicong


On 八月 24th, 2016, 11:24 p.m. CST, Tushar Krishna wrote:

Review request for Default, Andreas Hansson, Brad Beckmann, Jieming Yin, and 
Matthew Poremba.
By Tushar Krishna.

Updated 八月 24, 2016, 11:24 p.m.

Repository: gem5
Description

ruby: garnet2.0
Revamped version of garnet with more optimized single-cycle routers,
more configurability, and cleaner code.


Diffs

  *   src/mem/ruby/network/garnet2.0/GarnetLink.py (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/GarnetNetwork.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/GarnetNetwork.cc 
(PRE-CREATION)
  *   configs/topologies/Crossbar.py (d726d0cea027)
  *   configs/topologies/MeshDirCorners_XY.py (PRE-CREATION)
  *   configs/topologies/Mesh_XY.py (PRE-CREATION)
  *   configs/topologies/Mesh_westfirst.py (PRE-CREATION)
  *   configs/topologies/Pt2Pt.py (d726d0cea027)
  *   src/base/statistics.cc (d726d0cea027)
  *   src/mem/ruby/network/BasicRouter.py (d726d0cea027)
  *   src/mem/ruby/network/garnet2.0/CommonTypes.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/Credit.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/Credit.cc (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/CreditLink.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/CrossbarSwitch.hh (PRE-CREATION)
  *   
src/mem/ruby/network/garnet2.0/CrossbarSwitch.cc 
(PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/GarnetLink.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/GarnetLink.cc 
(PRE-CREATION)
  *   configs/network/Network.py (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/Router.cc (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/RoutingUnit.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/RoutingUnit.cc 
(PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/SConscript (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/SwitchAllocator.hh (PRE-CREATION)
  *   
src/mem/ruby/network/garnet2.0/SwitchAllocator.cc 
(PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/VirtualChannel.hh (PRE-CREATION)
  *   
src/mem/ruby/network/garnet2.0/VirtualChannel.cc 
(PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/flit.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/flit.cc (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/flitBuffer.hh (PRE-CREATION)
  *   src/mem/ruby/network/garnet2.0/flitBuffer.cc 
(PRE-CREATION)
  *   

Re: [gem5-dev] Review Request 3558: ruby: garnet2.0

2016-09-23 Thread Tushar Krishna


> On Sept. 23, 2016, 5:47 a.m., Zicong Wang wrote:
> > Missing src_outport & dst_inport when create the mesh links in 
> > MeshDirCorners_XY.py, which will cause segmentation fault(tested in 
> > X86_MESI_Two_Level)

Hi Zicong,
Thanks for testing the patch on your end. Can you send me the command you used 
that gave you the segmentation fault?
src_outport and dst_inport are supposed to be optional parameters.
I can add them in to MeshDirCorners_XY but want to find out why they gave an 
error.

Thanks,
Tushar


- Tushar


---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3558/#review8734
---


On Aug. 24, 2016, 3:24 p.m., Tushar Krishna wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/3558/
> ---
> 
> (Updated Aug. 24, 2016, 3:24 p.m.)
> 
> 
> Review request for Default, Andreas Hansson, Brad Beckmann, Jieming Yin, and 
> Matthew Poremba.
> 
> 
> Repository: gem5
> 
> 
> Description
> ---
> 
> ruby: garnet2.0
> Revamped version of garnet with more optimized single-cycle routers,
> more configurability, and cleaner code.
> 
> 
> Diffs
> -
> 
>   src/mem/ruby/network/garnet2.0/GarnetLink.py PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetNetwork.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetNetwork.cc PRE-CREATION 
>   configs/topologies/Crossbar.py d726d0cea027 
>   configs/topologies/MeshDirCorners_XY.py PRE-CREATION 
>   configs/topologies/Mesh_XY.py PRE-CREATION 
>   configs/topologies/Mesh_westfirst.py PRE-CREATION 
>   configs/topologies/Pt2Pt.py d726d0cea027 
>   src/base/statistics.cc d726d0cea027 
>   src/mem/ruby/network/BasicRouter.py d726d0cea027 
>   src/mem/ruby/network/garnet2.0/CommonTypes.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Credit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Credit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/CreditLink.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/CrossbarSwitch.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/CrossbarSwitch.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetLink.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetLink.cc PRE-CREATION 
>   configs/network/Network.py PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Router.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/RoutingUnit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/RoutingUnit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/SConscript PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/SwitchAllocator.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/SwitchAllocator.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/VirtualChannel.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/VirtualChannel.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flitBuffer.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flitBuffer.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutputUnit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutputUnit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/README.txt PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Router.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkLink.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutVcState.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutVcState.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkLink.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetNetwork.py PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/InputUnit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/InputUnit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkInterface.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkInterface.cc PRE-CREATION 
> 
> Diff: http://reviews.gem5.org/r/3558/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Tushar Krishna
> 
>

___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev


Re: [gem5-dev] Review Request 3558: ruby: garnet2.0

2016-09-14 Thread Jieming Yin

---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3558/#review8721
---

Ship it!


Looks good overall.


configs/topologies/Mesh_westfirst.py (line 106)


white space



configs/topologies/Pt2Pt.py (line 51)


white space



src/mem/ruby/network/garnet2.0/SwitchAllocator.cc (line 293)


white space



src/mem/ruby/network/garnet2.0/SwitchAllocator.cc (line 299)


empty line



src/mem/ruby/network/garnet2.0/SwitchAllocator.cc (line 317)


empty line



src/mem/ruby/network/garnet2.0/SwitchAllocator.cc (line 321)


white space


- Jieming Yin


On Aug. 24, 2016, 3:24 p.m., Tushar Krishna wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/3558/
> ---
> 
> (Updated Aug. 24, 2016, 3:24 p.m.)
> 
> 
> Review request for Default, Andreas Hansson, Brad Beckmann, Jieming Yin, and 
> Matthew Poremba.
> 
> 
> Repository: gem5
> 
> 
> Description
> ---
> 
> ruby: garnet2.0
> Revamped version of garnet with more optimized single-cycle routers,
> more configurability, and cleaner code.
> 
> 
> Diffs
> -
> 
>   src/mem/ruby/network/garnet2.0/GarnetLink.py PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetNetwork.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetNetwork.cc PRE-CREATION 
>   configs/topologies/Crossbar.py d726d0cea027 
>   configs/topologies/MeshDirCorners_XY.py PRE-CREATION 
>   configs/topologies/Mesh_XY.py PRE-CREATION 
>   configs/topologies/Mesh_westfirst.py PRE-CREATION 
>   configs/topologies/Pt2Pt.py d726d0cea027 
>   src/base/statistics.cc d726d0cea027 
>   src/mem/ruby/network/BasicRouter.py d726d0cea027 
>   src/mem/ruby/network/garnet2.0/CommonTypes.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Credit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Credit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/CreditLink.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/CrossbarSwitch.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/CrossbarSwitch.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetLink.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetLink.cc PRE-CREATION 
>   configs/network/Network.py PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Router.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/RoutingUnit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/RoutingUnit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/SConscript PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/SwitchAllocator.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/SwitchAllocator.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/VirtualChannel.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/VirtualChannel.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flitBuffer.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/flitBuffer.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutputUnit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutputUnit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/README.txt PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/Router.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkLink.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutVcState.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/OutVcState.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkLink.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/GarnetNetwork.py PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/InputUnit.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/InputUnit.cc PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkInterface.hh PRE-CREATION 
>   src/mem/ruby/network/garnet2.0/NetworkInterface.cc PRE-CREATION 
> 
> Diff: http://reviews.gem5.org/r/3558/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Tushar Krishna
> 
>

___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev


Re: [gem5-dev] Review Request 3558: ruby: garnet2.0

2016-08-24 Thread Tushar Krishna

---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/3558/
---

(Updated Aug. 24, 2016, 3:24 p.m.)


Review request for Default, Andreas Hansson, Brad Beckmann, Jieming Yin, and 
Matthew Poremba.


Summary (updated)
-

ruby: garnet2.0


Repository: gem5


Description (updated)
---

ruby: garnet2.0
Revamped version of garnet with more optimized single-cycle routers,
more configurability, and cleaner code.


Diffs (updated)
-

  src/mem/ruby/network/garnet2.0/GarnetLink.py PRE-CREATION 
  src/mem/ruby/network/garnet2.0/GarnetNetwork.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/GarnetNetwork.cc PRE-CREATION 
  configs/topologies/Crossbar.py d726d0cea027 
  configs/topologies/MeshDirCorners_XY.py PRE-CREATION 
  configs/topologies/Mesh_XY.py PRE-CREATION 
  configs/topologies/Mesh_westfirst.py PRE-CREATION 
  configs/topologies/Pt2Pt.py d726d0cea027 
  src/base/statistics.cc d726d0cea027 
  src/mem/ruby/network/BasicRouter.py d726d0cea027 
  src/mem/ruby/network/garnet2.0/CommonTypes.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/Credit.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/Credit.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/CreditLink.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/CrossbarSwitch.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/CrossbarSwitch.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/GarnetLink.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/GarnetLink.cc PRE-CREATION 
  configs/network/Network.py PRE-CREATION 
  src/mem/ruby/network/garnet2.0/Router.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/RoutingUnit.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/RoutingUnit.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/SConscript PRE-CREATION 
  src/mem/ruby/network/garnet2.0/SwitchAllocator.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/SwitchAllocator.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/VirtualChannel.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/VirtualChannel.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/flit.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/flit.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/flitBuffer.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/flitBuffer.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/OutputUnit.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/OutputUnit.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/README.txt PRE-CREATION 
  src/mem/ruby/network/garnet2.0/Router.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/NetworkLink.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/OutVcState.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/OutVcState.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/NetworkLink.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/GarnetNetwork.py PRE-CREATION 
  src/mem/ruby/network/garnet2.0/InputUnit.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/InputUnit.cc PRE-CREATION 
  src/mem/ruby/network/garnet2.0/NetworkInterface.hh PRE-CREATION 
  src/mem/ruby/network/garnet2.0/NetworkInterface.cc PRE-CREATION 

Diff: http://reviews.gem5.org/r/3558/diff/


Testing
---


Thanks,

Tushar Krishna

___
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev