Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-26 Thread Alex Bennée
Andrew Jones drjo...@redhat.com writes: On Wed, Jun 24, 2015 at 08:12:52PM +0100, Peter Maydell wrote: On 24 June 2015 at 18:18, Alex Bennée alex.ben...@linaro.org wrote: Paolo Bonzini pbonz...@redhat.com writes: On 24/06/2015 17:34, Alex Bennée wrote: Testing with Alexander's bare

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-26 Thread Andrew Jones
On Fri, Jun 26, 2015 at 08:06:55AM +0100, Alex Bennée wrote: Andrew Jones drjo...@redhat.com writes: On Wed, Jun 24, 2015 at 08:12:52PM +0100, Peter Maydell wrote: On 24 June 2015 at 18:18, Alex Bennée alex.ben...@linaro.org wrote: Paolo Bonzini pbonz...@redhat.com writes: On

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-25 Thread Frederic Konrad
On 25/06/2015 01:55, Alexander Spyridakis wrote: On 24 June 2015 at 17:34, Alex Bennée alex.ben...@linaro.org wrote: Testing with Alexander's bare metal syncronisation tests fails in MTTCG leaving one CPU spinning forever waiting for the second CPU to wake up. We simply need to poke the

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-25 Thread Andrew Jones
On Wed, Jun 24, 2015 at 08:12:52PM +0100, Peter Maydell wrote: On 24 June 2015 at 18:18, Alex Bennée alex.ben...@linaro.org wrote: Paolo Bonzini pbonz...@redhat.com writes: On 24/06/2015 17:34, Alex Bennée wrote: Testing with Alexander's bare metal syncronisation tests fails in MTTCG

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-25 Thread Alex Bennée
Alexander Spyridakis a.spyrida...@virtualopensystems.com writes: On 24 June 2015 at 17:34, Alex Bennée alex.ben...@linaro.org wrote: Testing with Alexander's bare metal syncronisation tests fails in MTTCG leaving one CPU spinning forever waiting for the second CPU to wake up. We simply need

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-24 Thread Alexander Spyridakis
On 24 June 2015 at 17:34, Alex Bennée alex.ben...@linaro.org wrote: Testing with Alexander's bare metal syncronisation tests fails in MTTCG leaving one CPU spinning forever waiting for the second CPU to wake up. We simply need to poke the halt_cond once we have processed the PSCI power on

[Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-24 Thread Alex Bennée
Testing with Alexander's bare metal syncronisation tests fails in MTTCG leaving one CPU spinning forever waiting for the second CPU to wake up. We simply need to poke the halt_cond once we have processed the PSCI power on call. Tested-by: Alex Bennée alex.ben...@linaro.org CC: Alexander

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-24 Thread Paolo Bonzini
On 24/06/2015 17:34, Alex Bennée wrote: Testing with Alexander's bare metal syncronisation tests fails in MTTCG leaving one CPU spinning forever waiting for the second CPU to wake up. We simply need to poke the halt_cond once we have processed the PSCI power on call. Tested-by: Alex

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-24 Thread Alex Bennée
Paolo Bonzini pbonz...@redhat.com writes: On 24/06/2015 19:18, Alex Bennée wrote: @@ -196,6 +196,8 @@ void arm_handle_psci_call(ARMCPU *cpu) } target_cpu_class-set_pc(target_cpu_state, entry); +qemu_cond_signal(target_cpu_state-halt_cond); That's called

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-24 Thread Paolo Bonzini
On 24/06/2015 19:18, Alex Bennée wrote: @@ -196,6 +196,8 @@ void arm_handle_psci_call(ARMCPU *cpu) } target_cpu_class-set_pc(target_cpu_state, entry); +qemu_cond_signal(target_cpu_state-halt_cond); That's called qemu_cpu_kick(target_cpu_state). :) The

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-24 Thread Alex Bennée
Paolo Bonzini pbonz...@redhat.com writes: On 24/06/2015 17:34, Alex Bennée wrote: Testing with Alexander's bare metal syncronisation tests fails in MTTCG leaving one CPU spinning forever waiting for the second CPU to wake up. We simply need to poke the halt_cond once we have processed the

Re: [Qemu-devel] [RFC PATCH] target-arm/psci.c: wake up sleeping CPUs (MTTCG)

2015-06-24 Thread Peter Maydell
On 24 June 2015 at 18:18, Alex Bennée alex.ben...@linaro.org wrote: Paolo Bonzini pbonz...@redhat.com writes: On 24/06/2015 17:34, Alex Bennée wrote: Testing with Alexander's bare metal syncronisation tests fails in MTTCG leaving one CPU spinning forever waiting for the second CPU to wake