[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/4] cpu: Make cpu_init() return QOM object
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 0/4] cpu: Make cpu_init() return QOM object |
Date: |
Fri, 27 Feb 2015 15:40:38 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
Hi Eduardo,
Am 26.02.2015 um 21:37 schrieb Eduardo Habkost:
> This series changes cpu_init() to return a CPU QOM object, and changes
> existing
> arch-specific code to use the corresponding arch-specific function instead of
> cpu_init().
>
> With this, the only remaining users of cpu_init() are linux-user and bsd-user.
>
> Eduardo Habkost (4):
> target-unicore32: Make uc32_cpu_init() return UniCore32CPU
> m68k: Use cpu_m68k_init()
> unicore32: Use uc32_cpu_init()
This part looks good to me. At the time, I propagated *CPU only for
those machines that needed it for function calls or field accesses.
> cpu: Make cpu_init() return QOM object
As for this patch, the Coccinelle based approach looks cool! However I
would like to give this a bit more thought as to whether 1) this causes
churn with regards to the next steps I outlined, and 2) whether more
simplifications can be done while at it. Could be done as follow-ups.
Let's also keep in mind that target-tilegx patches are on the list.
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu,
Graham Norton; HRB 21284 (AG Nürnberg)