Config: Add ruby_fs.py's functionality to fs.py

Information
Submitter:	Nilay Vaish
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Config: Add ruby_fs.py's functionality to fs.py
This patch combines fs.py and ruby_fs.py and removes ruby_fs.py. Using
ruby with fs.py would require mentioning --ruby on the command line. This
same as se.py.

Testing Done

I just have a couple minor edits to make to the comments.  However, I wonder what is the real benefit for combining the two fs files.  I don't believe there is a lot of duplicate code between the two.  Most of the added options in fs.py are not supported/related to Ruby.  Also with your change, the main body of fs.py is now a large if/else statement with the if being the functionality from ruby_fs.py and the else being the fs.py functionality.  Overall, I feel this is an unnecessary change that will negatively impact those of us who have other internal patches that modify these files.  Why do it?

Nilay Vaish Aug. 9, 2011, 12:35 p.m. (Aug. 9, 2011, 12:35 p.m.)

There are a couple of reasons. We should not have different set of options for
ruby_fs.py and fs.py. Since we have already combined se.py and ruby_se.py, we
should do the same for fs scripts as well. The main body of fs.py being a large
if/else is either because I have not taken out all those things that are common,
and that we still lack in the integration that Ruby and M5 should achieve. As 
far as internal patches are concerned, unless the patches can be updated, they 
should be used with the version with which they were created. Going ahead, we 
are only going to add more features, sooner or later these features will step
on the code used in the patches.

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

Remove "ALPHA_FS" from comment.  Ruby now works with both ALPHA and X86.

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

Change comment to say the driver system is always atomic with the Classic memory system.

You have a pending review.

Review Board 2.0.15

This change has been discarded.

Screenshots

Files

Status: Discarded