> On 2011-07-11 11:38:24, Brad Beckmann wrote: > > Is it necessary for the fault model to be configured using a text file? > > Can you do this in python instead? For instance, if you used scripts to > > generate the files, it would be best if you implemented those scripts in > > python and used the standard sim_object methodology. Also is the one text > > file just a "temp" file? If so, should it be checked it?
These 2 text files are a small database generated by my infrastructure (PrimeTime+SPICE), and is used to calculate the fault vector based on the routers' configuration and the current environmental conditions. Specifically, DB.conf.txt stores the fault vectors for a number of baseline configurations, while DB.temp.txt stores weights applied to vectors (these weights reflect how much higher the fault probabilities will be) while temperatures increase. Thus, these two files are a part of the fault model's "code" and not a user input. The fault model's user is not supposed to ever change these files, that's why I do not consider a good idea to put them together with gem5 inputs. In practice, I (or whoever maintains this tool in the future) should only edit these files. For instance, if in the future I want to incorporate additional technologies (eg 32nm) or additional router configurations in the fault model, I can do so by using my infrastructure to generate 2 updated DB files and provide you another gem5 patch. The actual input to my fault model is the network configuration (routers' input/output ports, buffers, VCs, etc) which is read directly from Garnet. Currently, the only actual input that the user can control is the enable/disable flag (i.e., enable_fault_model in src/mem/ruby/network/garnet/BaseGarnetNetwork.py) Let me know if you think my argument makes sense, or you still think the database files should be a python input. - Konstantinos ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/776/#review1408 ----------------------------------------------------------- On 2011-07-08 17:46:10, Konstantinos Aisopos wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/776/ > ----------------------------------------------------------- > > (Updated 2011-07-08 17:46:10) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, Nathan > Binkert, Brad Beckmann, and Tushar Krishna. > > > Summary > ------- > > GARNET: adding a fault model for resilient on-chip network research. > > This patch adds a fault model, which provides the probability for a number of > architectural faults in the interconnection network (e.g., data corruption, > misrouting). These probabilities can be used to realistically inject faults > in GARNET and faithfully evaluate the effectiveness of novel resilient NoC > architectures. > > > Diffs > ----- > > src/mem/ruby/network/fault_model/DB.conf.txt PRE-CREATION > src/mem/ruby/network/fault_model/DB.temp.txt PRE-CREATION > src/mem/ruby/network/fault_model/FaultModel.hh PRE-CREATION > src/mem/ruby/network/fault_model/FaultModel.cc PRE-CREATION > src/mem/ruby/network/fault_model/README PRE-CREATION > src/mem/ruby/network/fault_model/SConscript PRE-CREATION > src/mem/ruby/network/garnet/BaseGarnetNetwork.hh 7907b19fbe80 > src/mem/ruby/network/garnet/BaseGarnetNetwork.cc 7907b19fbe80 > src/mem/ruby/network/garnet/BaseGarnetNetwork.py 7907b19fbe80 > src/mem/ruby/network/garnet/fixed-pipeline/GarnetNetwork_d.cc 7907b19fbe80 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.hh 7907b19fbe80 > src/mem/ruby/network/garnet/fixed-pipeline/Router_d.cc 7907b19fbe80 > > Diff: http://reviews.m5sim.org/r/776/diff > > > Testing > ------- > > > Thanks, > > Konstantinos > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
