[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 6/7] monitor: "i": Add ARM specifics

From: Richard Henderson
Subject: Re: [Qemu-devel] [PATCH 6/7] monitor: "i": Add ARM specifics
Date: Wed, 06 May 2015 07:12:37 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0

On 05/06/2015 12:06 AM, Peter Crosthwaite wrote:
> Can
> we rely on the env/CPUState always being up to date during
> target_disas (which happens at translate time?) or will we need to go
> field by field to make sure any env updates explicitly occur before
> target_disas?

I *think* so, but it's a near thing.  The path goes

  cpu_get_tb_cpu_state, fill fill in flags for TB from current ENV state.
    tb_gen_code, using those same flags.

There's the edge case of re-translation, but I'm going to assert that cpu mode
changes ought not happen in that context.  Doing otherwise means that the
kernel has just switched modes, the translator has failed to end the TB, and
the new code has faulted immediately.

What I don't know is if we can, at the moment, canonicalize on TB flags.  If
the monitor were to use cpu_get_tb_cpu_state, I know it would work when using
TCG, but I don't know if we keep all the same data up-to-date in KVM or XEN 


reply via email to

[Prev in Thread] Current Thread [Next in Thread]