[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1.1] target-xtensa: fix CCOUNT for conditional b
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 1.1] target-xtensa: fix CCOUNT for conditional branches |
Date: |
Sat, 9 Jun 2012 11:30:38 +0000 |
Thanks, applied.
On Sun, May 27, 2012 at 11:18 PM, Max Filippov <address@hidden> wrote:
> Taken conditional branches fail to update CCOUNT register because
> accumulated ccount_delta is reset during translation of non-taken
> branch. To fix it only update CCOUNT once per conditional branch
> instruction translation.
>
> This fixes guest linux freeze on LTP waitpid06 test.
>
> Signed-off-by: Max Filippov <address@hidden>
> ---
> target-xtensa/translate.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/target-xtensa/translate.c b/target-xtensa/translate.c
> index 521c0e6..a542a31 100644
> --- a/target-xtensa/translate.c
> +++ b/target-xtensa/translate.c
> @@ -388,6 +388,7 @@ static bool gen_check_loop_end(DisasContext *dc, int slot)
> dc->next_pc == dc->lend) {
> int label = gen_new_label();
>
> + gen_advance_ccount(dc);
> tcg_gen_brcondi_i32(TCG_COND_EQ, cpu_SR[LCOUNT], 0, label);
> tcg_gen_subi_i32(cpu_SR[LCOUNT], cpu_SR[LCOUNT], 1);
> gen_jumpi(dc, dc->lbeg, slot);
> @@ -410,6 +411,7 @@ static void gen_brcond(DisasContext *dc, TCGCond cond,
> {
> int label = gen_new_label();
>
> + gen_advance_ccount(dc);
> tcg_gen_brcond_i32(cond, t0, t1, label);
> gen_jumpi_check_loop_end(dc, 0);
> gen_set_label(label);
> --
> 1.7.7.6
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 1.1] target-xtensa: fix CCOUNT for conditional branches,
Blue Swirl <=