Re: [Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-03 Thread Peter Maydell
On 2 September 2012 18:33, Blue Swirl blauwir...@gmail.com wrote: Add an explicit CPUState parameter instead of relying on AREG0 and switch to AREG0 free mode. My cheesy test harness for running a popular embedded benchmark in system mode (x86-64 host, ARM guest) shows mostly slowdowns of

Re: [Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-03 Thread Blue Swirl
On Mon, Sep 3, 2012 at 12:03 AM, Peter Maydell peter.mayd...@linaro.org wrote: On 3 September 2012 01:01, Peter Maydell peter.mayd...@linaro.org wrote: On 2 September 2012 18:33, Blue Swirl blauwir...@gmail.com wrote: Add an explicit CPUState parameter instead of relying on AREG0 and switch to

Re: [Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-03 Thread Peter Maydell
On 3 September 2012 19:58, Blue Swirl blauwir...@gmail.com wrote: On Mon, Sep 3, 2012 at 12:03 AM, Peter Maydell peter.mayd...@linaro.org wrote: On 3 September 2012 01:01, Peter Maydell peter.mayd...@linaro.org wrote: That's quite hard to cross-reference when the patch is this big. I think

Re: [Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-03 Thread Blue Swirl
On Mon, Sep 3, 2012 at 7:54 PM, Peter Maydell peter.mayd...@linaro.org wrote: On 3 September 2012 19:58, Blue Swirl blauwir...@gmail.com wrote: On Mon, Sep 3, 2012 at 12:03 AM, Peter Maydell peter.mayd...@linaro.org wrote: On 3 September 2012 01:01, Peter Maydell peter.mayd...@linaro.org

Re: [Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-03 Thread Peter Maydell
On 3 September 2012 21:10, Blue Swirl blauwir...@gmail.com wrote: On Mon, Sep 3, 2012 at 7:54 PM, Peter Maydell peter.mayd...@linaro.org wrote: I don't want the *file* split, I'd just like to see this *patch* as 4 or 5 separate patches, not one big one. While converting, it's easier to work

[Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-02 Thread Blue Swirl
Add an explicit CPUState parameter instead of relying on AREG0 and switch to AREG0 free mode. Signed-off-by: Blue Swirl blauwir...@gmail.com --- configure|2 +- target-arm/Makefile.objs |2 - target-arm/cpu.h | 10 ++- target-arm/helper.c |8 +-

Re: [Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-02 Thread Peter Maydell
On 2 September 2012 18:33, Blue Swirl blauwir...@gmail.com wrote: Add an explicit CPUState parameter instead of relying on AREG0 and switch to AREG0 free mode. Signed-off-by: Blue Swirl blauwir...@gmail.com --- configure|2 +- target-arm/Makefile.objs |2 -

Re: [Qemu-devel] [PATCH 16/21] target-arm: switch to AREG0 free mode

2012-09-02 Thread Peter Maydell
On 3 September 2012 01:01, Peter Maydell peter.mayd...@linaro.org wrote: On 2 September 2012 18:33, Blue Swirl blauwir...@gmail.com wrote: Add an explicit CPUState parameter instead of relying on AREG0 and switch to AREG0 free mode. Signed-off-by: Blue Swirl blauwir...@gmail.com ---