[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 7/8] s390x/kvm: software breakpoint support
From: |
Cornelia Huck |
Subject: |
[Qemu-devel] [PATCH 7/8] s390x/kvm: software breakpoint support |
Date: |
Fri, 16 May 2014 14:41:35 +0200 |
From: David Hildenbrand <address@hidden>
This patch allows to insert and remove sw breakpoints using the QEMU gdbserver
on s390 as well as to interrupt execution on a breakpoint hit when running
with KVM enabled.
Whenever a software breakpoint is inserted, common code calls kvm ioctl
KVM_UPDATE_GUEST_DEBUG. As this method's default on s390 is to return an error
if not implement, the insertion will fail. Therefore, KVM also has to be
updated in order to make use of software breakpoints.
Signed-off-by: David Hildenbrand <address@hidden>
Signed-off-by: Jens Freimann <address@hidden>
Reviewed-by: Cornelia Huck <address@hidden>
Signed-off-by: Cornelia Huck <address@hidden>
---
target-s390x/kvm.c | 25 ++++++++++++++++++++++---
1 file changed, 22 insertions(+), 3 deletions(-)
diff --git a/target-s390x/kvm.c b/target-s390x/kvm.c
index 4d12f70..6d47637 100644
--- a/target-s390x/kvm.c
+++ b/target-s390x/kvm.c
@@ -583,6 +583,22 @@ static void kvm_handle_diag_308(S390CPU *cpu, struct
kvm_run *run)
handle_diag_308(&cpu->env, r1, r3);
}
+static int handle_sw_breakpoint(S390CPU *cpu, struct kvm_run *run)
+{
+ CPUS390XState *env = &cpu->env;
+ unsigned long pc;
+
+ cpu_synchronize_state(CPU(cpu));
+
+ pc = env->psw.addr - 4;
+ if (kvm_find_sw_breakpoint(CPU(cpu), pc)) {
+ env->psw.addr = pc;
+ return EXCP_DEBUG;
+ }
+
+ return -ENOENT;
+}
+
#define DIAG_KVM_CODE_MASK 0x000000000000ffff
static int handle_diag(S390CPU *cpu, struct kvm_run *run, uint32_t ipb)
@@ -603,7 +619,7 @@ static int handle_diag(S390CPU *cpu, struct kvm_run *run,
uint32_t ipb)
r = handle_hypercall(cpu, run);
break;
case DIAG_KVM_BREAKPOINT:
- sleep(10);
+ r = handle_sw_breakpoint(cpu, run);
break;
default:
DPRINTF("KVM: unknown DIAG: 0x%x\n", func_code);
@@ -705,7 +721,7 @@ out:
return 0;
}
-static void handle_instruction(S390CPU *cpu, struct kvm_run *run)
+static int handle_instruction(S390CPU *cpu, struct kvm_run *run)
{
unsigned int ipa0 = (run->s390_sieic.ipa & 0xff00);
uint8_t ipa1 = run->s390_sieic.ipa & 0x00ff;
@@ -732,8 +748,11 @@ static void handle_instruction(S390CPU *cpu, struct
kvm_run *run)
}
if (r < 0) {
+ r = 0;
enter_pgmcheck(cpu, 0x0001);
}
+
+ return r;
}
static bool is_special_wait_psw(CPUState *cs)
@@ -753,7 +772,7 @@ static int handle_intercept(S390CPU *cpu)
(long)cs->kvm_run->psw_addr);
switch (icpt_code) {
case ICPT_INSTRUCTION:
- handle_instruction(cpu, run);
+ r = handle_instruction(cpu, run);
break;
case ICPT_WAITPSW:
/* disabled wait, since enabled wait is handled in kernel */
--
1.7.9.5
- [Qemu-devel] [PATCH 0/8] s390: pending patches, Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 1/8] s390x: split flic into kvm and non-kvm parts, Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 2/8] s390x: Add I/O adapter registration., Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 5/8] linux-headers: update, Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 8/8] s390x/kvm: hw debugging support via guest PER facility, Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 6/8] s390x: remove duplicate definitions of DIAG 501, Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 4/8] s390x/virtio-ccw: Wire up irq routing and irqfds., Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 3/8] s390x/virtio-ccw: reference-counted indicators, Cornelia Huck, 2014/05/16
- [Qemu-devel] [PATCH 7/8] s390x/kvm: software breakpoint support,
Cornelia Huck <=
- Re: [Qemu-devel] [PATCH 0/8] s390: pending patches, Andreas Färber, 2014/05/16