[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] target-ppc: Add versions to server CPU descript

From: Andreas Färber
Subject: Re: [Qemu-devel] [PATCH] target-ppc: Add versions to server CPU descriptions
Date: Wed, 04 Mar 2015 15:14:14 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0


Am 04.03.2015 um 13:28 schrieb Alexander Graf:
> On 04.03.15 02:31, Alexey Kardashevskiy wrote:
>> 5b79b1c "target-ppc: Create versionless CPU class per family if KVM" added
>> a dynamic CPU class registration with the name of the CPU family which
>> QEMU is running on. For example, this allowed specifying "-cpu POWER7"
>> on every version of POWER7 machine, not just the one which POWER7 was
>> an alias of. I.e. before 5b79b1c, "-cpu POWER7" would not work on real
>> POWER7 2.1 and would work on POWER7 2.3 only. The same story for POWER8.
>> However that patch broke POWER5+ support as POWER5+ CPU uses the same
>> name as the CPU class so dynamic registering of the POWER5+ class failed.
>> This redefines POWER5+ server CPUs by adding a version to them and adding
>> an alias for TCG case. KVM will use dynamically registered CPUs.
>> While we are here, do the same for 970 CPU.
>> Signed-off-by: Alexey Kardashevskiy <address@hidden>
> Thanks, applied to ppc-next.

As Alexey predicted, I object.

The 970 part looks good and could be applied immediately if it were a
separate patch.

But the POWER5+ part I have my doubts about: Was there really a v0.0???
Others start with v1.0 and I have:

revision        : 2.1 (pvr 003b 0201)

Also, this is still just bandaid and not a fix to the code that I
pointed out. But let's keep that discussion over at the other patch.


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)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]