Re: [PATCH v6 00/20] target/arm: Reduce overhead of cpu_get_tb_cpu_state

2019-10-17 Thread Peter Maydell
On Thu, 17 Oct 2019 at 17:25, Richard Henderson wrote: > > On 10/17/19 8:26 AM, Peter Maydell wrote: > > Don't we also need to do something to rebuild the hflags > > for M-profile writes to the memory mapped system registers? > > For instance rebuild_hflags_m32() bakes in state which > > cares

Re: [PATCH v6 00/20] target/arm: Reduce overhead of cpu_get_tb_cpu_state

2019-10-17 Thread Richard Henderson
On 10/17/19 8:26 AM, Peter Maydell wrote: > Don't we also need to do something to rebuild the hflags > for M-profile writes to the memory mapped system registers? > For instance rebuild_hflags_m32() bakes in state which > cares about env->v7m.ccr, which is set via nvic_writel(), > but I don't see

Re: [PATCH v6 00/20] target/arm: Reduce overhead of cpu_get_tb_cpu_state

2019-10-17 Thread Peter Maydell
On Fri, 11 Oct 2019 at 16:55, Richard Henderson wrote: > Richard Henderson (20): > target/arm: Split out rebuild_hflags_common > target/arm: Split out rebuild_hflags_a64 > target/arm: Split out rebuild_hflags_common_32 > target/arm: Split arm_cpu_data_is_big_endian > target/arm: Split

[PATCH v6 00/20] target/arm: Reduce overhead of cpu_get_tb_cpu_state

2019-10-11 Thread Richard Henderson
Changes since v5: * Fix the debug assertion ifdef in the final patch. * Add more calls to arm_rebuild_hflags: CPSR and M-profile These become two new patches, 18 & 19. * Update some comments per review. (Alex) Changes since v4: * Split patch 1 into 15 smaller patches. * Cache the