dist: config file for distributed switch

Information
Submitter:	Mohammad Alian
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 11293:a360fd989d96
---------------------------
dist: config file for distributed switch

Distributed gem5 is the result of the convergence effort between
multi-gem5 and pd-gem5. It relies on the base multi-gem5 infrastructure
for packet forwarding, synchronisation and checkpointing but combines
those with the elaborated network switch model from pd-gem5.

Testing Done

Issue Summary

Description	From	Last Updated	Status
The name of this file is not particularly descriptive. Also, perhaps we should add a configs/dist/ subdirectory? I guess there ...	Andreas Hansson	April 19, 2016, 1:32 p.m.	Resolved

Ship It!

The description should be <80 char lines. I am also not sure about the license header. Is it intentionally the "extended" BSD rather than just the plain BSD?

Mohammad Alian Jan. 9, 2016, 1:55 p.m. (Jan. 9, 2016, 1:55 p.m.)

I fixed the description. Regarding the licence header, I didn't do it intentionally. Please let me know how should I change it.

Mohammad Alian Jan. 10, 2016, 10:10 p.m. (Jan. 10, 2016, 10:10 p.m.)
```
Fixed the licence part.
```

configs/example/sw.py (Diff revision 1)

The name of this file is not particularly descriptive. Also, perhaps we should add a configs/dist/ subdirectory? I guess there will be more scripts related to distributed gem5?

I also don't understand how this script interacts with the other scripts. Is this a top-level? If so, where is the rest of the system?

Show all issues

Mohammad Alian Jan. 9, 2016, 1:55 p.m. (Jan. 9, 2016, 1:55 p.m.)

Just like fs.py which creates a full system, sw.py creates and simulate an Ethenet switch-box. It means that the whole gem5 process just simulates an Ethernet switch, nothing else. It seems inefficience but the purpose of this script is just to provide an example of how to create/simulate a switch box in dist-gem5. A more efficient way is to simulate the switch alongside a full-system.
For now this is the only script specifically for dist-gem5. I'm not sure if we need the dist/ subdir now.

Description:

~		Distributed gem5 is the result of the convergence effort between multi-gem5 and pd-gem5. It relies on the base multi-gem5 infrastructure for packet forwarding, synchronisation and checkpointing but combines those with the elaborated network switch model from pd-gem5.
	~	dist: config file for distributed switch
	+
	+	Distributed gem5 is the result of the convergence effort between
	+	multi-gem5 and pd-gem5. It relies on the base multi-gem5 infrastructure
	+	for packet forwarding, synchronisation and checkpointing but combines
	+	those with the elaborated network switch model from pd-gem5.

Description:

~		dist: config file for distributed switch
	~	Changeset 11293:fb17215842f0
	+	---------------------------
	+	dist: config file for distributed switch

~		Distributed gem5 is the result of the convergence effort between
~		multi-gem5 and pd-gem5. It relies on the base multi-gem5 infrastructure
~		for packet forwarding, synchronisation and checkpointing but combines
	~	Distributed gem5 is the result of the convergence effort between
	~	multi-gem5 and pd-gem5. It relies on the base multi-gem5 infrastructure
	~	for packet forwarding, synchronisation and checkpointing but combines
		those with the elaborated network switch model from pd-gem5.

Diff:

Revision 2 (+84)

Show changes

configs/example/sw.py

Description:

~		Changeset 11293:fb17215842f0
	~	Changeset 11293:a360fd989d96
		---------------------------
		dist: config file for distributed switch

		Distributed gem5 is the result of the convergence effort between
		multi-gem5 and pd-gem5. It relies on the base multi-gem5 infrastructure
		for packet forwarding, synchronisation and checkpointing but combines
		those with the elaborated network switch model from pd-gem5.

Diff:

Revision 3 (+75)

Show changes

configs/example/sw.py

Andreas, are you ok with this being shipped now? I am planning on shipping the switch, and it'd be nice if this can go in as well.

Andreas Hansson Feb. 8, 2016, 2:36 p.m. (Feb. 8, 2016, 2:36 p.m.)

No objections. That said, I do think that a separate directory is more sensible, since the use of this script is unique to the "dist" use-case (as far as I understand).

Tony Gutierrez Feb. 8, 2016, 3:19 p.m. (Feb. 8, 2016, 3:19 p.m.)
```
Ok, I will put the file in a separate dir before shipping.
```

Mohammad Alian April 12, 2016, 9:24 a.m. (April 12, 2016, 9:24 a.m.)

Could you please commit this patch? It has been here waiting for 2 month. Thanks

Andreas Hansson April 19, 2016, 1:33 p.m. (April 19, 2016, 1:33 p.m.)
```
Should be marked submitted?
```

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Issue Summary

Description:

Description:

Diff:

Description:

Diff:

Status: Closed (submitted)