----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1355/#review3401 -----------------------------------------------------------
Looks good overall, thanks! configs/example/se.py <http://reviews.gem5.org/r/1355/#comment3467> I'm not sure it makes sense to have multiple CPUs with an SMT workload... in the long run we probably want some way to assign multiple different threads on each of multiple CPUs, but in the meantime we might want to just make it an error to have SMT and multiple CPUs. - Steve Reinhardt On Aug. 17, 2012, 9:39 a.m., Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1355/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2012, 9:39 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9155:752d18560fe9 > --------------------------- > se.py: support specifying multiple programs via command line > This patch allows for specifying multiple programs via command line. It also > adds an option for specifying whether to use of SMT. But SMT does not work for > the o3 cpu as of now. > > > Diffs > ----- > > configs/common/Options.py 198352d722e4 > configs/example/se.py 198352d722e4 > > Diff: http://reviews.gem5.org/r/1355/diff/ > > > Testing > ------- > > > Thanks, > > Nilay Vaish > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
