[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] dirty-bitmaps: deprecate @status field
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] dirty-bitmaps: deprecate @status field |
Date: |
Tue, 12 Feb 2019 12:12:35 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 2/11/19 7:02 PM, John Snow wrote:
> The current internal meanings of "locked", "user_locked",
> "qmp_locked", "frozen", "enabled", and "disabled" are all
> a little muddled.
>
> Deprecate the @status field in favor of two new booleans
> that carry very specific meanings. Then, rename and rework
> some of the internal semantics to help make the API a bit
> more clear and easier to read.
So far, my libvirt patches are not relying on the contents of the status
field, so deprecating things should not hurt libvirt. Using the full
2-cycle process is still probably wise for other users, though, even if
it means for some awkward back-compat computation of status for a while
longer.
>
> Well, in my opinion.
>
> Based on my current bitmaps branch (includes Eric's patch
> and my documentation update patch.)
>
> John Snow (5):
> block/dirty-bitmap: add recording and busy properties
> block/dirty-bitmaps: rename frozen predicate helper
> block/dirty-bitmap: change semantics of enabled predicate
> block/dirty-bitmap: explicitly lock bitmaps with successors
> block/dirty-bitmaps: unify qmp_locked and user_locked calls
>
> block/dirty-bitmap.c | 74 +++++++++++++++++++---------------
> blockdev.c | 18 ++++-----
> include/block/dirty-bitmap.h | 7 ++--
> migration/block-dirty-bitmap.c | 8 ++--
> nbd/server.c | 6 +--
> qapi/block-core.json | 9 ++++-
> tests/qemu-iotests/236.out | 28 +++++++++++++
> 7 files changed, 96 insertions(+), 54 deletions(-)
No change to qemu-deprecated.texi to start the clock ticking, so we can
remove the old field?
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH 2/5] block/dirty-bitmaps: rename frozen predicate helper, (continued)