Patchwork kvm: Actually remove software breakpoints from list on cleanup

login
register
mail settings
Submitter Jan Kiszka
Date Nov. 12, 2012, 2:04 p.m.
Message ID <50A101F3.9040908@siemens.com>
Download mbox | patch
Permalink /patch/198404/
State New
Headers show

Comments

Jan Kiszka - Nov. 12, 2012, 2:04 p.m.
So far we only removed them from the guest, leaving its states in the
list. This made it impossible for gdb to re-enable breakpoints on the
same address after re-attaching.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 kvm-all.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Patch

diff --git a/kvm-all.c b/kvm-all.c
index b6d0483..3bc3347 100644
--- a/kvm-all.c
+++ b/kvm-all.c
@@ -1905,6 +1905,8 @@  void kvm_remove_all_breakpoints(CPUArchState *current_env)
                 }
             }
         }
+        QTAILQ_REMOVE(&s->kvm_sw_breakpoints, bp, entry);
+        g_free(bp);
     }
     kvm_arch_remove_all_hw_breakpoints();