----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/397/#review651 -----------------------------------------------------------
src/dev/x86/i82094aa.hh <http://reviews.m5sim.org/r/397/#comment884> You don't need "virtual" here. src/dev/x86/i82094aa.cc <http://reviews.m5sim.org/r/397/#comment885> This comment probably could go in /* */s. If you want to stick with //s, please get rid of the blank lines that are commented. src/dev/x86/intdev.hh <http://reviews.m5sim.org/r/397/#comment886> I really want one of the folks that knows the memory system better (Steve, Ali, Nate) to comment on whether this is right. I'm worried that by only broadcasting its address range only at init it won't go out at all sometimes (checkpoint restore?) or somebody will miss somebody else's status change. - Gabe On 2011-01-06 16:12:56, Brad Beckmann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/397/ > ----------------------------------------------------------- > > (Updated 2011-01-06 16:12:56) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > dev: fixed bugs to extend interrupt capability beyond 15 cores > > > Diffs > ----- > > src/arch/x86/interrupts.cc 9f9e10967912 > src/dev/x86/i82094aa.hh 9f9e10967912 > src/dev/x86/i82094aa.cc 9f9e10967912 > src/dev/x86/intdev.hh 9f9e10967912 > src/dev/x86/intdev.cc 9f9e10967912 > > Diff: http://reviews.m5sim.org/r/397/diff > > > Testing > ------- > > > Thanks, > > Brad > >
_______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev