> On 2010-11-21 01:26:15, Gabe Black wrote: > > I think this was supposed to go before the dtlb timing one.
no, it's supposed to go afterward or be merged together - Ali ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/322/#review501 ----------------------------------------------------------- On 2010-11-19 16:13:55, Ali Saidi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/322/ > ----------------------------------------------------------- > > (Updated 2010-11-19 16:13:55) > > > Review request for Default. > > > Summary > ------- > > O3: Change when the memory ordering violation is checked to after dtlb lookup. > > This patch is a follow-up to the dtlb_timing.patch. Separated intentionally to > be understood easily. > > Dtlb lookup now can finish (finishTranslation()) some time later from the tick > it was initiated (initiateAccess()), checking and handling of memory ordering > violation needs to happen after finishTranslation() is called for the > instruction. The code that handling this violation used to be in > executeInsts() > has been separated as handleMemOrderViolation(). > > Also, added cycleMemIssued variable to prevent memory instructions issued in > the > same cycle are falsely blamed to have caused ordering violation. > > > Diffs > ----- > > src/cpu/base_dyn_inst.hh 6286bb50127e > src/cpu/o3/iew.hh 6286bb50127e > src/cpu/o3/iew_impl.hh 6286bb50127e > src/cpu/o3/lsq_impl.hh 6286bb50127e > src/cpu/o3/lsq_unit.hh 6286bb50127e > src/cpu/o3/lsq_unit_impl.hh 6286bb50127e > > Diff: http://reviews.m5sim.org/r/322/diff > > > Testing > ------- > > > Thanks, > > Ali > >
_______________________________________________ m5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/m5-dev
