summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authoraurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162>2008-10-01 22:01:37 +0000
committeraurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162>2008-10-01 22:01:37 +0000
commitee600be6a6379697d04673fd9ab10b66732b42e1 (patch)
tree2074314af1559f7dd9bfa5fc4ea2b80c13d0ebf9
parent3003b8bbf9a044e3f5f1c95c0fb1c35ff09f8385 (diff)
downloadhqemu-ee600be6a6379697d04673fd9ab10b66732b42e1.zip
hqemu-ee600be6a6379697d04673fd9ab10b66732b42e1.tar.gz
ppc: fix crash in ppc system single step support
There was a bogus case where two system debug ops get generated. This patch removes the broken system debug op. This was a left over after making some changes to correctly generate debug ops on branch operations inside gen_goto_tb(); The test case against this patch is to turn on single stepping with timers, boot a linux kernel, set a breakpoint a do_fork and in gdb execute "si 3000". Then qemu-system-ppc will fault executing a debug op, which should not have been executed. Signed-off-by: Jason Wessel <jason.wessel@windriver.com> Signed-off-by: Aurelien Jarno <aurelien@aurel32.net> git-svn-id: svn://svn.savannah.nongnu.org/qemu/trunk@5391 c046a42c-6fe2-441c-8c8c-71466251a162
-rw-r--r--target-ppc/translate.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/target-ppc/translate.c b/target-ppc/translate.c
index 275f780..00615d1 100644
--- a/target-ppc/translate.c
+++ b/target-ppc/translate.c
@@ -3048,10 +3048,6 @@ static always_inline void gen_bcond (DisasContext *ctx, int type)
#endif
gen_op_btest_T1(ctx->nip);
no_test:
- if (ctx->singlestep_enabled & GDBSTUB_SINGLE_STEP) {
- gen_update_nip(ctx, ctx->nip);
- gen_op_debug();
- }
tcg_gen_exit_tb(0);
}
}
OpenPOWER on IntegriCloud