[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1] migration: change blocktime type to uint32_t
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v1] migration: change blocktime type to uint32_t |
Date: |
Thu, 25 Jan 2018 14:37:33 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 |
On 01/25/2018 10:43 AM, Alexey Perevalov wrote:
> Initially int64_t was used, but on PowerPC architecture,
> clang doesn't have atomic_*_8 function, so it produces
> link time error.
>
> QEMU is working with time as with 64bit value, but by
> fact 32 bit is enough with CLOCK_REALTIME. In this case
> blocktime will keep only 1200 hours time interval.
>
> Signed-off-by: Alexey Perevalov <address@hidden>
> ---
> +++ b/qapi/migration.json
> @@ -175,8 +175,8 @@
> '*setup-time': 'int',
> '*cpu-throttle-percentage': 'int',
> '*error-desc': 'str',
> - '*postcopy-blocktime' : 'int64',
> - '*postcopy-vcpu-blocktime': ['int64']} }
> + '*postcopy-blocktime' : 'uint32',
> + '*postcopy-vcpu-blocktime': ['uint32']} }
Questionable to change types if this had been released already - but
these fields are marked 'since 2.12' so you can change them at will up
until the release.
Acked-by: Eric Blake <address@hidden>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature