diff options
author | Chris Johns <chrisj@rtems.org> | 2017-04-14 17:12:44 +1000 |
---|---|---|
committer | Chris Johns <chrisj@rtems.org> | 2017-04-15 08:18:25 +1000 |
commit | b53ad4615a16f65119e40f7084f9bacd88dc42cd (patch) | |
tree | d1f56f04b019b77aef1a150f0ab3e32870027e9c /cpukit/libdebugger | |
parent | 50b1269626e5dbb7badc8d397430b15ad7bd2387 (diff) |
libdebugger: Work around assert when using _Thread_Executing.
Using _Thread_Executing with RTEMS_DEBUG results in an `assert` if
the server accesses invalid memory.
Updates #2993.
Diffstat (limited to 'cpukit/libdebugger')
-rw-r--r-- | cpukit/libdebugger/rtems-debugger-arm.c | 3 | ||||
-rw-r--r-- | cpukit/libdebugger/rtems-debugger-target.c | 7 |
2 files changed, 8 insertions, 2 deletions
diff --git a/cpukit/libdebugger/rtems-debugger-arm.c b/cpukit/libdebugger/rtems-debugger-arm.c index 8126c1471e..200d758a6b 100644 --- a/cpukit/libdebugger/rtems-debugger-arm.c +++ b/cpukit/libdebugger/rtems-debugger-arm.c @@ -717,7 +717,8 @@ target_exception(CPU_Exception_frame* frame) "mov r3, #0\n" \ "str r3, [r2]\n" /* clear the FPU frame pointer */ \ EXCEPTION_ENTRY_FPU(frame_fpu_size) \ - "msr cpsr, r1\n" /* restore the irq mask */ \ + "bic r1, r1, %[psr_i]\n" /* clear irq mask, debug checks */ \ + "msr cpsr, r1\n" /* restore the state with irq mask clear */ \ ASM_THUMB_MODE \ : ARM_SWITCH_REG_ASM, \ [o_frame] "=r" (_frame) \ diff --git a/cpukit/libdebugger/rtems-debugger-target.c b/cpukit/libdebugger/rtems-debugger-target.c index cd1565463e..65b878469d 100644 --- a/cpukit/libdebugger/rtems-debugger-target.c +++ b/cpukit/libdebugger/rtems-debugger-target.c @@ -284,7 +284,12 @@ rtems_debugger_target_exception(CPU_Exception_frame* frame) if (!rtems_interrupt_is_in_progress()) { rtems_debugger_threads* threads = rtems_debugger->threads; - Thread_Control* thread = _Thread_Executing; + #if USE_THREAD_EXECUTING + Thread_Control* thread = _Thread_Executing; + #else + const Per_CPU_Control* cpu = _Per_CPU_Get_snapshot(); + Thread_Control* thread = _Per_CPU_Get_executing(cpu); + #endif rtems_id* excludes; const rtems_id tid = thread->Object.id; DB_UINT pc; |