On Thu, Oct 29, 2009 at 7:58 PM, Nicolas Pitre <n...@fluxnic.net> wrote: > On Thu, 29 Oct 2009, Øyvind Harboe wrote: > >> In OpenOCD there is a bit of clever handling of working areas. >> >> You can essentially allocate a piece of working memory on the target and >> store e.g. an algorithm into that working area over an extended period >> of time to improve performance. >> >> If a reset or similar happens, then OpenOCD will iterate through all the >> working areas and null out the pointers to the working area so that >> e.g. the flash routines know to reallocate and repopulate the working >> areas. >> >> The neat side effect of this is that it will leave algorithms in >> target memory for a long time, increasing performance. >> >> The nasty side effect is that it *really* complicates the code path... >> >> I have an unsubstantiated suspicion that this breaks e.g. flash >> programming on arm926ejs. >> >> If I can catch this handling of working memory red-handed in causing >> bugs, my instinct would be to remove it. I'm hopeful that e.g. flash >> programming routines would be able to recover the performance >> with slight rewriting.... > > My instinct tells me otherwise. But you know, now that we have Git, you > can do that rewrite in a branch of your own and let people try it out in > parallel with the master branch. Only when factual results are > confirmed by enough people that the rewrite is really better then should > that branch be merged in the master branch. > > Please let's not recreate that tantrum in the repository like what > happened a few months ago.
I've made a personal not to avoid it. You'll be happy to hear that I've got two branches mrcmcr and mmureadonly. I've even worked on rebase -i'ing them + improving the comments to meet your standards ;-) After 0.3 I'll post them again. If the particular issue above is caught red-handed we'll be able to evaluate whether or not we have something which is optimized too much for marginal speed gains and too little for stability... > > > Nicolas > > -- Øyvind Harboe http://www.zylin.com/zy1000.html ARM7 ARM9 ARM11 XScale Cortex JTAG debugger and flash programmer _______________________________________________ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
