----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/740/#review1315 -----------------------------------------------------------
You can make it a control instruction -- that shouldn't cause any issues, but making it non-serializing is nearly hopeless. HWREI changes the processor state and effects translation among other things. All PAL code instructions that exist before it in the pipe must be complete before other instructions. This is especially true since we use ev5 pal code. I'm pretty sure that o3 does inst->mispredicted() on every instruction. I'm not sure if this is any faster than first calling isControl() on it, but i doubt it. - Ali On 2011-06-08 23:15:21, Korey Sewell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/740/ > ----------------------------------------------------------- > > (Updated 2011-06-08 23:15:21) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > alpha: make hwrei a control inst > this always changes the PC and is basically an impromptu branch instruction. > why > not speculate on this instead of always be forced to mispredict/squash after > the > hwrei gets resolved? > > The InOrder model needs this marked as "isControl" so it knows to update the > PC > after the ALU executes it. If this isnt marked as control, then it's going to > force the model to check the PC of every instruction at commit (what O3 > does?), > and that would be a wasteful check for a very high percentage of instructions. > > > Diffs > ----- > > src/arch/alpha/isa/decoder.isa 77d12d8f7971 > > Diff: http://reviews.m5sim.org/r/740/diff > > > Testing > ------- > > > Thanks, > > Korey > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
