Re: [Qemu-devel] [PATCH v6 6/6] target: [tcg, arm] Port to generic translation framework

[Lluís Vilanova]

Emilio G Cota writes:

> 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?

I didn't write it down on the series docs, so I don't really know. I'll break
the patch down into multiple as you suggested, and review them manually to see
if I'm able to catch the error (all the exit reason code is quite burdensome to
port to the new infrastructure).

Is there any ready-made kernel or disk image I can test with?


Thanks,
  Lluis