Re: [Qemu-devel] [Qemu-ppc] [PATCH ppc-next v2 00/52] PowerPCCPU subclasses, remixed

[Alexander Graf]

On 23.02.2013, at 19:27, Andreas Färber wrote:

> Am 23.02.2013 17:15, schrieb Andreas Färber:
>> Am 22.02.2013 16:21, schrieb Alexander Graf:
>>> After this patch set, the -cpu ? list is split into 2 segments.
>>> 
>>> I would prefer if we could in the help output revert to the old behavior of 
>>> only a single list ordered by PVR. Let me show you what I'm thinking of:
>>> 
>>>> PowerPC 601_v1           PVR 00010001
>>> [...]
>>>> PowerPC 740_v3.1         PVR 00080301
>>> 740 (alias to "PowerPC 740_v3.1)
>>>> PowerPC 750cx_v1.0       PVR 00082100
>> 
>> How do you imagine this to be implemented then? Per ObjectClass walk the
>> aliases list for aliases that resolve to the same type?
> 
> Didn't come up with a more efficient implementation:
> http://patchwork.ozlabs.org/patch/222738/

Yeah, it does exactly what I would like it to do, just that it's a bit slow :). 
On my 970 system it takes almost 6 seconds to show the CPU list.

$ time ./ppc-softmmu/qemu-system-ppc -cpu '?'
[...]
real    0m5.899s
user    0m5.856s
sys     0m0.025s

I don't think it's a major issue - you don't show the CPU list every day. But 
if you can come up with something faster, I wouldn't mind it ;). For the time 
being, this version is good.


Alex