[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH ppc-next v2 00/52] PowerPCCPU subclas
From: |
Alexander Graf |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH ppc-next v2 00/52] PowerPCCPU subclasses, remixed |
Date: |
Mon, 25 Feb 2013 14:33:53 +0100 |
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
- [Qemu-ppc] [PATCH ppc-next v2 42/52] target-ppc: Convert CPU definitions, (continued)
[Qemu-ppc] [PATCH ppc-next v2 38/52] target-ppc: Extract 405GPe alias, Andreas Färber, 2013/02/18
Re: [Qemu-ppc] [PATCH ppc-next v2 00/52] PowerPCCPU subclasses, remixed, Alexander Graf, 2013/02/22