Re: [Qemu-devel] [PULL 0/8] target/alpha cleanups

2017-07-20 Thread Paolo Bonzini
I am planning to refresh the series for 2.11. Paolo Il 20/lug/2017 11:07, "Peter Maydell" ha scritto: > On 19 July 2017 at 22:52, Richard Henderson wrote: > > On 07/19/2017 06:57 AM, Peter Maydell wrote: > >> > >> On 19 July 2017 at 05:45, Richard Henderson wrote: > >>> > >>> The new title ho

Re: [Qemu-devel] [PULL 0/8] target/alpha cleanups

2017-07-20 Thread Peter Maydell
On 19 July 2017 at 22:52, Richard Henderson wrote: > On 07/19/2017 06:57 AM, Peter Maydell wrote: >> >> On 19 July 2017 at 05:45, Richard Henderson wrote: >>> >>> The new title holder for perf top is helper_lookup_tb_ptr. >>> Those targets that have a complicated cpu_get_tb_cpu_state >>> function

Re: [Qemu-devel] [PULL 0/8] target/alpha cleanups

2017-07-19 Thread Richard Henderson
On 07/19/2017 06:57 AM, Peter Maydell wrote: On 19 July 2017 at 05:45, Richard Henderson wrote: The new title holder for perf top is helper_lookup_tb_ptr. Those targets that have a complicated cpu_get_tb_cpu_state function are going to regret that. Yeah, Paolo's pointed out (and had some patc

Re: [Qemu-devel] [PULL 0/8] target/alpha cleanups

2017-07-19 Thread Peter Maydell
On 19 July 2017 at 05:45, Richard Henderson wrote: > The new title holder for perf top is helper_lookup_tb_ptr. > Those targets that have a complicated cpu_get_tb_cpu_state > function are going to regret that. > > This cleans up the Alpha version of that function such that it is > just two loads a

Re: [Qemu-devel] [PULL 0/8] target/alpha cleanups

2017-07-19 Thread Peter Maydell
On 19 July 2017 at 05:45, Richard Henderson wrote: > The new title holder for perf top is helper_lookup_tb_ptr. > Those targets that have a complicated cpu_get_tb_cpu_state > function are going to regret that. Yeah, Paolo's pointed out (and had some patches for) ARM's rather complicated cpu_get_t

[Qemu-devel] [PULL 0/8] target/alpha cleanups

2017-07-18 Thread Richard Henderson
The new title holder for perf top is helper_lookup_tb_ptr. Those targets that have a complicated cpu_get_tb_cpu_state function are going to regret that. This cleans up the Alpha version of that function such that it is just two loads and one mask. Which is one practically-free mask away from bein