----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2460/#review5391 -----------------------------------------------------------
src/mem/multi_level_page_table_impl.hh <http://reviews.gem5.org/r/2460/#comment4885> Is fatal really suitable here? Is it a user error, or should it rather be a panic? - Andreas Hansson On Sept. 30, 2014, 7:18 p.m., Alexandru Dutu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2460/ > ----------------------------------------------------------- > > (Updated Sept. 30, 2014, 7:18 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10424:853fff736800 > --------------------------- > mem: Multi Level Page Table bug fix > > The multi level page table was giving false positives for already mapped > translations. This patch fixes the bogus behavior. > > > Diffs > ----- > > src/mem/multi_level_page_table_impl.hh > 28b31101d9e6e5e75d04448451986d6318383f3c > > Diff: http://reviews.gem5.org/r/2460/diff/ > > > Testing > ------- > > Quick regressions passed. > > > Thanks, > > Alexandru Dutu > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
