> On Nov. 20, 2014, 9:58 a.m., Andreas Hansson wrote: > > src/mem/packet.hh, line 964 > > <http://reviews.gem5.org/r/2498/diff/1/?file=42567#file42567line964> > > > > It's used to check against MSHRs. I honestly do not know the historical > > reasons for this (which is also why I haven't changed it...)
It's an ancient and rarely used facility I added a long time ago (Jan 2008, according to the log, see http://repo.gem5.org/gem5/rev/e902f12a3af1 and http://repo.gem5.org/gem5/rev/30997e988446). Basically it builds on the functional access framework that traverses the whole memory system looking for copies of a cache block, but instead of reading/updating those copies, it prints information about those copies (where they live, what their state is, etc.). I found an email I sent doumenting this feature to the m5-dev list, apparently shortly before we started archiving it, so I can't even send you a link :). I'll put it up on the wiki later; if no one finds it useful, we could get rid of it. - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2498/#review5507 ----------------------------------------------------------- On Nov. 16, 2014, 10:16 p.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2498/ > ----------------------------------------------------------- > > (Updated Nov. 16, 2014, 10:16 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10548:2b33b6c6b656 > --------------------------- > mem: Cleanup Packet::checkFunctional and hasData usage > > This patch cleans up the use of hasData and checkFunctional in the > packet. The latter function is also tidied up to avoid name overloading. > > > Diffs > ----- > > src/mem/cache/cache_impl.hh 1a9e235cab09 > src/mem/packet.hh 1a9e235cab09 > src/mem/packet.cc 1a9e235cab09 > > Diff: http://reviews.gem5.org/r/2498/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
