> On Sept. 30, 2014, 7:57 p.m., Andreas Hansson wrote: > > Should this not be folded into the original patch? > > Alexandru Dutu wrote: > I am not sure, that patch has already been commited. I thought once > commited, a patch becomes a commit and one can't modify it.
My bad. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2460/#review5386 ----------------------------------------------------------- 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
