----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/763/#review2135 -----------------------------------------------------------
Ship it! Korey, does this seem fine? Could you commit it. - Ali Saidi On June 30, 2011, 7:39 a.m., Maximilien Breughe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/763/ > ----------------------------------------------------------- > > (Updated June 30, 2011, 7:39 a.m.) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, Nathan > Binkert, and Korey Sewell. > > > Description > ------- > > Fixed the Multiply Divide Unit (MDU) to support latencies higher than 11. > Before the CPU went in an idle state after 11 cycles waiting on the MDU. > When the MDU gets finished after e.g. 15 cycles, the CPU didn't woke up. > > > This addresses bug MDU. > http://flyspray.gem5.org/task/MDU > > > Diffs > ----- > > src/cpu/inorder/resources/mult_div_unit.cc 4adb1148ef73 > > Diff: http://reviews.gem5.org/r/763/diff/diff > > > Testing > ------- > > Tested it with MiBench's rsynth benchmark on large input. > > > Thanks, > > Maximilien Breughe > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
