> On 2010-11-21 01:26:15, Gabe Black wrote: > > I think this was supposed to go before the dtlb timing one. > > Ali Saidi wrote: > no, it's supposed to go afterward or be merged together
In that case, in this diff on line 285 of src/cpu/base_dyn_inst.hh you're declaring Tick cycleMemIssued, and in the DTLB change on line 957 of src/cpu/base_dyn_inst.hh that variable is being set. Because the DTLB change goes first it's using a variable that doesn't exist yet, unless that comes in from some other bit of code. That's why I thought these were out of order. - Gabe ----------------------------------------------------------- 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
