Re: [PATCH v6 4/8] ppc/e500: Use start-powered-off CPUState property

2020-08-25 Thread Thiago Jung Bauermann
Philippe Mathieu-Daudé writes: > On 8/22/20 10:59 AM, Cédric Le Goater wrote: >> Hello, >> >> On 8/19/20 6:43 PM, Thiago Jung Bauermann wrote: >>> Instead of setting CPUState::halted to 1 in ppce500_cpu_reset_sec(), use >>> the start-powered-off property which makes cpu_common_reset()

Re: [PATCH v6 4/8] ppc/e500: Use start-powered-off CPUState property

2020-08-24 Thread Thiago Jung Bauermann
Philippe Mathieu-Daudé writes: > On 8/22/20 10:59 AM, Cédric Le Goater wrote: >> Hello, >> >> On 8/19/20 6:43 PM, Thiago Jung Bauermann wrote: >>> Instead of setting CPUState::halted to 1 in ppce500_cpu_reset_sec(), use >>> the start-powered-off property which makes cpu_common_reset()

Re: [PATCH v6 4/8] ppc/e500: Use start-powered-off CPUState property

2020-08-24 Thread Thiago Jung Bauermann
David Gibson writes: > On Sat, Aug 22, 2020 at 10:59:56AM +0200, Cédric Le Goater wrote: >> Hello, >> >> On 8/19/20 6:43 PM, Thiago Jung Bauermann wrote: >> > Instead of setting CPUState::halted to 1 in ppce500_cpu_reset_sec(), use >> > the start-powered-off property which makes

Re: [PATCH v6 4/8] ppc/e500: Use start-powered-off CPUState property

2020-08-23 Thread David Gibson
On Sat, Aug 22, 2020 at 10:59:56AM +0200, Cédric Le Goater wrote: > Hello, > > On 8/19/20 6:43 PM, Thiago Jung Bauermann wrote: > > Instead of setting CPUState::halted to 1 in ppce500_cpu_reset_sec(), use > > the start-powered-off property which makes cpu_common_reset() initialize it > > to 1 in

Re: [PATCH v6 4/8] ppc/e500: Use start-powered-off CPUState property

2020-08-22 Thread Philippe Mathieu-Daudé
On 8/22/20 10:59 AM, Cédric Le Goater wrote: > Hello, > > On 8/19/20 6:43 PM, Thiago Jung Bauermann wrote: >> Instead of setting CPUState::halted to 1 in ppce500_cpu_reset_sec(), use >> the start-powered-off property which makes cpu_common_reset() initialize it >> to 1 in common code. >> >> Also

Re: [PATCH v6 4/8] ppc/e500: Use start-powered-off CPUState property

2020-08-22 Thread Cédric Le Goater
Hello, On 8/19/20 6:43 PM, Thiago Jung Bauermann wrote: > Instead of setting CPUState::halted to 1 in ppce500_cpu_reset_sec(), use > the start-powered-off property which makes cpu_common_reset() initialize it > to 1 in common code. > > Also change creation of CPU object from cpu_create() to

[PATCH v6 4/8] ppc/e500: Use start-powered-off CPUState property

2020-08-19 Thread Thiago Jung Bauermann
Instead of setting CPUState::halted to 1 in ppce500_cpu_reset_sec(), use the start-powered-off property which makes cpu_common_reset() initialize it to 1 in common code. Also change creation of CPU object from cpu_create() to object_new() and qdev_realize_and_unref() because cpu_create() realizes