[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [Qemu-devel] [PATCH v6 6/6] target: [tcg, arm] Port to ge
From: |
Emilio G. Cota |
Subject: |
Re: [Qemu-arm] [Qemu-devel] [PATCH v6 6/6] target: [tcg, arm] Port to generic translation framework |
Date: |
Thu, 15 Jun 2017 20:18:42 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, Jun 12, 2017 at 17:54:30 +0300, Lluís Vilanova wrote:
> Signed-off-by: Lluís Vilanova <address@hidden>
> ---
> target/arm/translate-a64.c | 346 ++++++++++-----------
> target/arm/translate.c | 720
> ++++++++++++++++++++++----------------------
> target/arm/translate.h | 46 ++-
> 3 files changed, 560 insertions(+), 552 deletions(-)
This one makes my arm-softmmu hang while booting debian. The last line
I see is:
> Freeing unused kernel memory: 300K (80669000 - 806b4000)
Note that this happens even after disabling goto_ptr, so it shouldn't
be related to that feature.
The problem might be with the rebase. For instance, the
hunk below was modified by commit 542b3478a ("armv7m: Replace armv7m.hack
with unassigned_access handler"). Might be a good idea to go over
the latest changes to arm/translate.c. What commit was your last working
version based on?
(snip)
> diff --git a/target/arm/translate.c b/target/arm/translate.c
> index 96272a9888..06f207a5f6 100644
> --- a/target/arm/translate.c
> +++ b/target/arm/translate.c
> +#else
> + if (dc->base.pc_next >= 0xfffffff0 && arm_dc_feature(dc, ARM_FEATURE_M))
> {
> + /* We always get here via a jump, so know we are not in a
> + conditional execution block. */
> + gen_exception_internal(EXCP_EXCEPTION_EXIT);
> + dc->base.jmp_type = DJ_EXC;
> + }
> +#endif
> +}
Thanks,
Emilio