Re: [Qemu-devel] [PATCH 1/2] qom-test: Run for all available machines

["Andreas Färber" <afaerber@suse.de>]

Am 10.01.2014 15:17, schrieb Andreas Färber:
> Am 10.01.2014 14:42, schrieb Markus Armbruster:
>> armbru@redhat.com writes:
>>
>>> From: Markus Armbruster <armbru@redhat.com>
>>>
>>> Get available machines via QMP instead of hardcoding a list that's
>>> perpetually out of date.
>>>
>>> A few machines don't work out of the box:
>>>
>>> * Several ppcemb machines can't initialize their CPU.
> 
> That's why we had the rather complicated ppc machine grouping:
> 
> http://git.qemu.org/?p=qemu.git;a=commit;h=7c41f2177e280dec1f1d4c5cd72333c5c55943af
> 
>> Alex, any chance these could be fixed?
>>
>> ppce500     Unable to initialize CPU!
>> mac99       Unable to find PowerPC CPU definition
>> g3beige     Unable to find PowerPC CPU definition
>> mpc8544ds   Unable to initialize CPU!
> 
> => shouldn't be compiled into ppcemb

Still TBD, Alex has not volunteered :(

> 
> easiest: add #ifndef TARGET_PPCEMB in machine registration functions
> better: use CONFIG_* in Makefile.objs and ppcemb-softmmu.mak
> 
>> ref405ep    Unable to find PowerPC 405ep CPU definition
>> taihu       Unable to find PowerPC 405ep CPU definition
> 
> http://lists.nongnu.org/archive/html/qemu-devel/2013-08/msg03697.html
> 
> As per Alex, these should be made to work. How???

Unanswered!

> -> target-ppc/translate_init.c
> 
>> prep        le to find PowerPC CPU definition
> 
> => shouldn't be compiled in, will look into this one as maintainer

Done: http://patchwork.ozlabs.org/patch/313620/

TBD: Macs and ppc405

As indicated on IRC, I want to get those fixed before transforming one
notation to another. Especially this patch is throwing them all in one
bucket dropping any annotation hinting at why and how it may be fixed.

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg