[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v1] target/s390x: change CPU type name to "s390x
From: |
David Hildenbrand |
Subject: |
Re: [qemu-s390x] [PATCH v1] target/s390x: change CPU type name to "s390x-cpu" |
Date: |
Fri, 20 Oct 2017 17:23:12 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 20.10.2017 14:44, Thomas Huth wrote:
> On 20.10.2017 13:58, David Hildenbrand wrote:
>> For now, e.g. host-s390-cpu wasn't exposed to the user. cpu-add, -cpu
>> and the CPU model qmp interfaces didn't care about the actual type,
>> as that information was hidden.
>>
>> This changed with CPU hotplug via device_add. Now the type is visible to
>> the user. Before we get that supported in a stable version, this is our
>> last chance to change it.
>>
>> So change it form "s390-cpu" to "s390x-cpu", to match the architecture
>> name. Example names are then e.g. z14-s390x-cpu or qemu-s390x-cpu.
>
> Basically fine for me ... but just to be sure: Did you test whether
> migration still works fine from older versions to a version that
> contains this patch?
>
> Thomas
>
Very basic migration testing (initrd guest with 4 cpus). That name
should not pup up in the migration stream.
--
Thanks,
David