Since Alpha actually has a machine check fault and alpha/tlb.cc is an Alpha specific file, I decided not to use M5PanicFault there and go with Alpha's MachineCheckFault. The other version allowed adding a message to the fault, and if that's preferable I can go back to that.
Gabe On 09/24/11 18:09, Gabe Black wrote: > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/881/ > ----------------------------------------------------------- > > (Updated 2011-09-24 18:09:29.511654) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > Faults: Replace calls to genMachineCheckFault with M5PanicFault. > > > Diffs (updated) > ----- > > src/arch/alpha/faults.hh e21224136182 > src/arch/alpha/tlb.cc e21224136182 > src/arch/arm/faults.hh e21224136182 > src/arch/mips/faults.hh e21224136182 > src/arch/power/faults.hh e21224136182 > src/arch/sparc/faults.hh e21224136182 > src/arch/x86/faults.hh e21224136182 > src/cpu/o3/lsq_unit.hh e21224136182 > src/cpu/o3/lsq_unit_impl.hh e21224136182 > > Diff: http://reviews.m5sim.org/r/881/diff > > > Testing > ------- > > > Thanks, > > Gabe > > _______________________________________________ > gem5-dev mailing list > [email protected] > http://m5sim.org/mailman/listinfo/gem5-dev _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
