cortex_m3: add auto maskisrv0.5.0-rc2
authorPeter Horn <peter.horn@bluewin.ch>
Mon, 4 Jul 2011 10:25:26 +0000 (11:25 +0100)
committerSpencer Oliver <ntfreak@users.sourceforge.net>
Mon, 4 Jul 2011 10:31:52 +0000 (11:31 +0100)
commitd4cd6f032015552f00bf4b5a90f25f5f958e9d9e
tree7d60728bcf75fbc84867ebceb5fc0784145fff09
parent89fa8ce2d8c58707f3dfda397138f8ee336e1a47
cortex_m3: add auto maskisr

This patch extends the cortex_m3 maskisr command by a new option 'auto'.
The 'auto' option handles interrupts during stepping in a way they are
processed but don't disturb the program flow during debugging.

Before one had to choose to either enable or disable interrupts. The former
steps into interrupt handlers when they trigger. This disturbs the flow during
debugging, making it hard to follow some piece of code when interrupts occur
often.

When interrupts are disabled, the flow isn't disturbed but code relying on
interrupt handlers to be processed will stop working. For example a delay
function counting the number of timer interrupts will never complete, RTOS
task switching will not occur and output I/O queues of interrupt driven
I/O will stall or overflow.

Using the 'maskisr' command also typically requires gdb hooks to be supplied
by the user to switch interrupts off during the step and to enable them again
afterward.

The new 'auto' option of the 'maskisr' command solves the above problems. When
set, the step command allows pending interrupt handlers to be executed before
the step, then the step is taken with interrupts disabled and finally interrupts
are enabled again. This way interrupt processing stays in the background without
disturbing the flow of debugging. No gdb hooks are required. The 'auto'
option is the default, since it's believed that handling interrupts in this
way is suitable for most users.

The principle used for interrupt handling could probably be used for other
targets too.

Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
doc/openocd.texi
src/target/cortex_m3.c
src/target/cortex_m3.h