[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v6 7/8] sparc/sun4m: Use start-powered-off CPUState property

From: Thiago Jung Bauermann
Subject: Re: [PATCH v6 7/8] sparc/sun4m: Use start-powered-off CPUState property
Date: Wed, 19 Aug 2020 20:54:33 -0300
User-agent: mu4e 1.2.0; emacs 26.3

Philippe Mathieu-Daudé <philmd@redhat.com> writes:

> On 8/19/20 6:43 PM, Thiago Jung Bauermann wrote:
>> Instead of setting CPUState::halted to 1 in secondary_cpu_reset(), use the
>> start-powered-off property which makes cpu_common_reset() initialize it
>> to 1 in common code.
>> This makes secondary_cpu_reset() unnecessary, so remove it.
>> Also remove setting of cs->halted from cpu_devinit(), which seems out of
>> place when compared to similar code in other architectures (e.g.,
>> ppce500_init() in hw/ppc/e500.c).
>> Finally, change creation of CPU object from cpu_create() to object_new()
>> and qdev_realize_and_unref() because cpu_create() realizes the CPU and it's
>> not possible to set a property after the object is realized.
>> Suggested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Signed-off-by: Thiago Jung Bauermann <bauerman@linux.ibm.com>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>


Thiago Jung Bauermann
IBM Linux Technology Center

reply via email to

[Prev in Thread] Current Thread [Next in Thread]