> On Juni 23, 2016, 7:09 nachm., Matthias Jung wrote: > > Dear Christian, > > > > thank you for this nice contribution. I will have a look on your patch and > > maybe I can help you fixing your issue. > > > > Regards > > Matthias > > Andreas Hansson wrote: > Christian, Matthias, is there any sensible way we could actually get some > of this functionality tested? Including SystemC in ext seems problematic due > to the non-BSD compatible license, but perhaps we should do something like we > do with protobuf, and check if the host HAS_SYSTEMC or similar, and if so > compile the extra bits, and run the extra tests etc. > > This is tremendously useful functionality, and it feels like it is bolted > on rather than properly integrated at the moment (not because of the > functionality, but due to the non-default use). Any thoughts or suggestions?
What about an automated shell script that downloads SystemC from http://accellera.org/images/downloads/standards/systemc/systemc-2.3.1.tgz and compiles it? Its maybe a little complicated but makes things easy for people that don't have SystemC by default. Usually people install SystemC in /opt/systemc so thats a place that could be checked for existance. Christian? what do you think? - Matthias ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3528/#review8436 ----------------------------------------------------------- On Juni 24, 2016, 10:20 vorm., Christian Menard wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3528/ > ----------------------------------------------------------- > > (Updated Juni 24, 2016, 10:20 vorm.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > The current TLM code only provides a Slave Port that allows the gem5 world to > send requests to the the TLM world. This patch adds a Master Port that allows > the TLM world to send requests to the gem5 world. Furthermore, the patch > provides a simple example application based on a TLM traffic generator. > > As of now the example code does not run error free. It eventually results in > an erro "fatal: Missed an event at time ...". So for now I leave the patch > for discussion until the error is resolved. > > > Diffs > ----- > > util/tlm/README dd6dfd38b6c2 > util/tlm/examples/master_port/Makefile PRE-CREATION > util/tlm/examples/master_port/main.cc PRE-CREATION > util/tlm/examples/master_port/tlm.py PRE-CREATION > util/tlm/examples/slave_port/Makefile PRE-CREATION > util/tlm/sc_master_port.hh PRE-CREATION > util/tlm/sc_master_port.cc PRE-CREATION > util/tlm/sim_control.cc PRE-CREATION > > Diff: http://reviews.gem5.org/r/3528/diff/ > > > Testing > ------- > > A simple example application consisting of a TLM traffic generator and a gem5 > memory. Currently the test fails. > > > Thanks, > > Christian Menard > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
