[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[qemu-xen master] tcg/cpu-exec: precise single-stepping after an interrupt



commit ba3c35d9c4026361fd380b269dc6def9510b7166
Author:     Richard Henderson <richard.henderson@xxxxxxxxxx>
AuthorDate: Fri Jul 17 09:26:59 2020 -0700
Commit:     Richard Henderson <richard.henderson@xxxxxxxxxx>
CommitDate: Fri Jul 17 11:09:34 2020 -0700

    tcg/cpu-exec: precise single-stepping after an interrupt
    
    When single-stepping with a debugger attached to QEMU, and when an
    interrupt is raised, the debugger misses the first instruction after
    the interrupt.
    
    Tested-by: Luc Michel <luc.michel@xxxxxxxxxxxxx>
    Reviewed-by: Luc Michel <luc.michel@xxxxxxxxxxxxx>
    Buglink: https://bugs.launchpad.net/qemu/+bug/757702
    Message-Id: <20200717163029.2737546-1-richard.henderson@xxxxxxxxxx>
    Signed-off-by: Richard Henderson <richard.henderson@xxxxxxxxxx>
---
 accel/tcg/cpu-exec.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c
index 6a3d3a3cfc..66d38f9d85 100644
--- a/accel/tcg/cpu-exec.c
+++ b/accel/tcg/cpu-exec.c
@@ -588,7 +588,13 @@ static inline bool cpu_handle_interrupt(CPUState *cpu,
         else {
             if (cc->cpu_exec_interrupt(cpu, interrupt_request)) {
                 replay_interrupt();
-                cpu->exception_index = -1;
+                /*
+                 * After processing the interrupt, ensure an EXCP_DEBUG is
+                 * raised when single-stepping so that GDB doesn't miss the
+                 * next instruction.
+                 */
+                cpu->exception_index =
+                    (cpu->singlestep_enabled ? EXCP_DEBUG : -1);
                 *last_tb = NULL;
             }
             /* The target hook may have updated the 'cpu->interrupt_request';
--
generated by git-patchbot for /home/xen/git/qemu-xen.git#master



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.