[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 4/5] QMP: Add support for Archipelago
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v6 4/5] QMP: Add support for Archipelago |
Date: |
Wed, 02 Jul 2014 08:22:59 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 |
On 07/02/2014 08:11 AM, Chrysostomos Nanakos wrote:
> On 07/02/2014 04:58 PM, Eric Blake wrote:
>> On 06/27/2014 02:24 AM, Chrysostomos Nanakos wrote:
>>> Introduce new enum BlockdevOptionsArchipelago.
>>>
>>> @@ -1347,6 +1379,7 @@
>>> 'base': 'BlockdevOptionsBase',
>>> 'discriminator': 'driver',
>>> 'data': {
>>> + 'archipelago':'BlockdevOptionsArchipelago',
>> Again, not necessarily your fault, but we ought to do a better job of
>> documenting when new union branches are added in later releases.
>
> Any proposition for this one?
Here's what was done for NetClientOptions:
# Since 1.2
#
# 'l2tpv3' - since 2.1
although doing it properly would best be done as a separate patch that
audits all qapi types. I'm not trying to force you to take on the
documentation cleanup patch, although getting just your addition to
mention when it was added will make that cleanup audit easier for
whoever takes that task on.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature