[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/9] target-cris: sync CC state at load/stores.
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH 5/9] target-cris: sync CC state at load/stores. |
Date: |
Tue, 6 Sep 2016 23:52:38 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, Sep 05, 2016 at 12:02:14PM -0700, Richard Henderson wrote:
> On 09/05/2016 04:54 AM, Rabin Vincent wrote:
> >From: "Edgar E. Iglesias" <address@hidden>
> >
> >Icount may choose to abort and recompile a TB at any load/store. We
> >need to sync the CC state at these insns.
> >
> >Signed-off-by: Edgar E. Iglesias <address@hidden>
> >Signed-off-by: Rabin Vincent <address@hidden>
> >---
> > target-cris/translate.c | 9 +++++++++
> > target-cris/translate_v10.c | 3 +++
> > 2 files changed, 12 insertions(+)
> >
> >diff --git a/target-cris/translate.c b/target-cris/translate.c
> >index f4a8d7d..c280e24 100644
> >--- a/target-cris/translate.c
> >+++ b/target-cris/translate.c
> >@@ -1098,6 +1098,9 @@ static void gen_load64(DisasContext *dc, TCGv_i64 dst,
> >TCGv addr)
> > {
> > int mem_index = cpu_mmu_index(&dc->cpu->env, false);
> >
> >+ /* Due to icount, we need to update the CC flags on load/stores. */
> >+ cris_evaluate_flags(dc);
> >+
>
> This is not the proper way to handle this. You should arrange to sync the
> CC flags in restore_state_to_opc. There are plenty of examples in the tree
> you can examine. It looks like there's plenty of room for cleanup for cris
> here.
>
Thanks, agreed.
Rabin, most of the CRIS patches that I left in the AXIS tree without upstreaming
need more work. I don't mind you posting them but keep in mind that there's
not much that can go up without tidy up.
Best regards,
Edgar
- Re: [Qemu-devel] [PATCH 9/9] tests: cris: add v17 ADDC test, (continued)
- [Qemu-devel] [PATCH 5/9] target-cris: sync CC state at load/stores., Rabin Vincent, 2016/09/05
- [Qemu-devel] [PATCH 6/9] target-cris: reduce v32isms from v10 log dumps, Rabin Vincent, 2016/09/05
- [Qemu-devel] [PATCH 4/9] tests: cris: remove check_time1, Rabin Vincent, 2016/09/05
- Re: [Qemu-devel] [PATCH 1/9] tests: cris: force inlining, Edgar E. Iglesias, 2016/09/06
- Re: [Qemu-devel] [PATCH 1/9] tests: cris: force inlining, Peter Maydell, 2016/09/08