diff options
| author | aurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162> | 2008-10-01 22:01:37 +0000 |
|---|---|---|
| committer | aurel32 <aurel32@c046a42c-6fe2-441c-8c8c-71466251a162> | 2008-10-01 22:01:37 +0000 |
| commit | ee600be6a6379697d04673fd9ab10b66732b42e1 (patch) | |
| tree | 2074314af1559f7dd9bfa5fc4ea2b80c13d0ebf9 /target-ppc | |
| parent | 3003b8bbf9a044e3f5f1c95c0fb1c35ff09f8385 (diff) | |
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
Diffstat (limited to 'target-ppc')
| -rw-r--r-- | target-ppc/translate.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/target-ppc/translate.c b/target-ppc/translate.c index 275f7807c9..00615d10d4 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); } } |