[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] target-arm: cache tbflags in CPUARMState
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] target-arm: cache tbflags in CPUARMState |
Date: |
Thu, 10 Nov 2016 13:19:26 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
On 10/11/2016 12:42, Alex Bennée wrote:
>
> Paolo Bonzini <address@hidden> writes:
>
>> Computing TranslationBlock flags is pretty expensive on ARM, especially
>> 32-bit. Because tbflags are computed on every tb lookup, it is not
>> unlikely to see cpu_get_tb_cpu_state close to the top of the profile
>> now that QHT makes the hash table much more efficient.
>>
>> However, most tbflags only change when the EL is switched or after
>> MSR instructions. Based on this observation, this series caches these
>> tbflags in CPUARMState, resulting in a 10-15% speedup on 32-bit code.
>
> Hi,
>
> I'm starting to clear out my review queue but I notice these now longer
> apply cleanly to master. Where you going to re-issue the series once
> you'd addressed Peter's concerns?
Yes, I didn't want to send them two days before soft freeze and Peter's
vacation. :)
Paolo