[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v3 06/11] tcg: Introduce tb_mark_invalid() and tb_
From: |
Alex Bennée |
Subject: |
Re: [Qemu-arm] [PATCH v3 06/11] tcg: Introduce tb_mark_invalid() and tb_is_invalid() |
Date: |
Thu, 14 Jul 2016 13:04:25 +0100 |
User-agent: |
mu4e 0.9.17; emacs 25.0.95.9 |
Paolo Bonzini <address@hidden> writes:
> On 14/07/2016 13:10, Sergey Fedorov wrote:
>> > This has merge conflicts with the current state of master. Is there
>> > anyway to have a common implementation that is specialised only when
>> > needed?
>>
>> The point was to put the assumptions on invalid CPU TB state as close to
>> cpu_get_tb_cpu_state() definitions as possible. So that if anyone make
>> changes they can notice those assumptions and correct them if necessary.
>
> It causes some repetition indeed, but I think it's a good idea.
>
> restore_state_to_opc is another case where most implementations have the
> same simple "env->pc = data[0]" implementation.
Yeah, now I've seen cpu_get_tb_cpu_state jump up the hot-path I tend to
agree ;-)
--
Alex Bennée