> On Sept. 30, 2014, 12: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. > > Andreas Hansson wrote: > My bad.
This is exactly what I asked Alex when he posted this internally... then he pointed out that the original patch has already been committed: http://repo.gem5.org/gem5/rev/77af86f37337 - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2460/#review5386 ----------------------------------------------------------- On Sept. 30, 2014, 12: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, 12: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
