[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 15/18] target-i386: do not ignore error and f
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v2 15/18] target-i386: do not ignore error and fix apic parent |
Date: |
Mon, 4 Jul 2016 17:20:59 +0300 |
On Fri, Jun 24, 2016 at 06:06:03PM +0200, Igor Mammedov wrote:
> object_property_add_child() silently fails with error that it can't
> create duplicate propery 'apic' as we already have 'apic' property
I thought that one was 'apic-id'?
> registered for AIPC ID.
APIC
> As result generic device_realize puts
> apic as into unattached container.
>
> As it's programming error, abort on it and fix property name for
> apic_state to 'lapic', this way apic is a child of cpu instance.
>
> Signed-off-by: Igor Mammedov <address@hidden>
> ---
> target-i386/cpu.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index ebf4140..04c0b79 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -2763,8 +2763,9 @@ static void x86_cpu_apic_create(X86CPU *cpu, Error
> **errp)
>
> cpu->apic_state = DEVICE(object_new(apic_type));
>
> - object_property_add_child(OBJECT(cpu), "apic",
> - OBJECT(cpu->apic_state), NULL);
> + object_property_add_child(OBJECT(cpu), "lapic",
> + OBJECT(cpu->apic_state), &error_abort);
> +
> qdev_prop_set_uint8(cpu->apic_state, "id", cpu->apic_id);
> /* TODO: convert to link<> */
> apic = APIC_COMMON(cpu->apic_state);
> --
> 1.8.3.1
- Re: [Qemu-devel] [PATCH v2 15/18] target-i386: do not ignore error and fix apic parent,
Michael S. Tsirkin <=