----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3200/#review7536 -----------------------------------------------------------
Ship it! The technical aspects of this patch look fine. Could you change the commit message to comply with gem5's commit message policy? See http://www.m5sim.org/Submitting_Contributions. - Andreas Sandberg On Nov. 7, 2015, 9:22 p.m., Palle Lyckegaard wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3200/ > ----------------------------------------------------------- > > (Updated Nov. 7, 2015, 9:22 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > sparc: Make remote debugging with gdb work - remove sparc V8 TBR register > from list of registers since it is not part of sparc V9. This brings the > number of registers in sync with what gdb expects > > > Diffs > ----- > > src/arch/sparc/remote_gdb.hh UNKNOWN > > Diff: http://reviews.gem5.org/r/3200/diff/ > > > Testing > ------- > > Without this patch gdb complains about receoved packet too long. > > with this patch gdb is able to work properly with gem5 for remote debugging. > > Note: gdb is version 7.8 > Note: gdb is configured with --target=sparc64-sun-solaris2.8 > > > Thanks, > > Palle Lyckegaard > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
