Sonic Zhang wrote: > I have to recall my last wrong patch. > > In blackfin, kgdb is running in delayed exception IRQ5 other than in > exception IRQ3 directly. Register reti other than retx in pt_regs is > the kgdb return address. So, don't put PC in gdb_regs into retx. > > Sonic Zhang > > Index: arch/blackfin/kernel/kgdb.c > ============================ > --- arch/blackfin/kernel/kgdb.c (revision 8105) > +++ arch/blackfin/kernel/kgdb.c (revision 8106) > @@ -147,7 +147,7 @@ > regs->lb1 = gdb_regs[BFIN_LB1]; > regs->usp = gdb_regs[BFIN_USP]; > regs->syscfg = gdb_regs[BFIN_SYSCFG]; > - regs->retx = gdb_regs[BFIN_PC]; > + regs->retx = gdb_regs[BFIN_RETX]; > regs->retn = gdb_regs[BFIN_RETN]; > regs->rete = gdb_regs[BFIN_RETE]; > regs->pc = gdb_regs[BFIN_PC]; > >
Sonic, I pulled in this patch, and made the changes the Mike talked about with respect to cleaning up the arch specific kgdb.c for blackfin. See the attached patch. Your patch and the one here will go into kgdb-next and onto linux-next. At some point you sent a patch for cpu switching that was blackfin specific. There is new generic code in the debug core for doing this without exiting the debug context. At some point you should test this and figure out if something further needs to be done that is blackfin specific. Thanks, Jason.
------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev
_______________________________________________ Kgdb-bugreport mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
