On Tue, Jan 19, 2010 at 11:24 PM, David Brownell <[email protected]> wrote: > On Tuesday 19 January 2010, Øyvind Harboe wrote: >> It should be fixed before 0.4 as this is a clear cut regression. > > Let's fix the underlying bug though, not just paper over symptoms.
Agreed. The patch is just confirmation that we've identified the problem. >> With the str710 we have something nice to test against since the >> there are at least three sector sizes and two banks. Pretty messy :-) > > You sent the following memory map: > >> (gdb) info mem >> 0 y 0x00000000 0x40000000 rw nocache >> 1 y 0x40000000 0x40040000 flash blocksize 0x2000 nocache >> 2 y 0x40040000 0x400c0000 rw nocache >> 3 y 0x400c0000 0x400c4000 flash blocksize 0x2000 nocache >> 4 y 0x400c4000 0x100000000 rw nocache > > That shows two banks, yes ... but only one sector size, not three. That's because the memory map is incorrectly generated in gdb_server.c. gdb_server.c needs to iterate over banks & then sectors within each bank, rather than assuming that all sectors in a bank are of equal size. > SEPARATE ISSUE ... I think we should consider (for 0.5) adding > ways to declare where RAM/SRAM regions exist. Both from C code > (microcontrollers can often autoprobe that stuff) and Tcl. It's > unwise to just anything about non-flash regions. That would be an excellent refinement. Also we can "cache" such regions in order to improve gdb performance, another refinement. -- Øyvind Harboe US toll free 1-866-980-3434 / International +47 51 63 25 00 http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer _______________________________________________ Openocd-development mailing list [email protected] https://lists.berlios.de/mailman/listinfo/openocd-development
