From patchwork Thu Nov 15 00:11:46 2012
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [2/2] kvm: Actually remove software breakpoints from list on cleanup
Date: Wed, 14 Nov 2012 14:11:46 -0000
From: Marcelo Tosatti <mtosatti@redhat.com>
X-Patchwork-Id: 199095
Message-Id: <78021d6d8dc50ed4d67442c3ca11eab78220e9ef.1352938305.git.mtosatti@redhat.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: Jan Kiszka <jan.kiszka@siemens.com>,
 Marcelo Tosatti <mtosatti@redhat.com>, 
 qemu-devel@nongnu.org, kvm@vger.kernel.org

From: Jan Kiszka <jan.kiszka@siemens.com>

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>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>

---
kvm-all.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

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();