qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 01/50] blockdev: Allow creation of BDS trees wit


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH 01/50] blockdev: Allow creation of BDS trees without BB
Date: Tue, 27 Jan 2015 11:22:08 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0

On 01/26/2015 09:02 AM, Max Reitz wrote:
> If the "id" field is missing from the options given to blockdev-add,
> just omit the BlockBackend and create the BlockDriverState tree alone.
> 
> Signed-off-by: Max Reitz <address@hidden>
> ---
>  blockdev.c                 | 38 +++++++++++++++++++++++++-------------
>  tests/qemu-iotests/087     | 20 --------------------
>  tests/qemu-iotests/087.out | 12 ------------
>  3 files changed, 25 insertions(+), 45 deletions(-)

I'd feel more comfortable if this patch also touched
qapi/block-core.json to mention the change in policy on
BlockdevOptionsBase.  Also, should 'node-name' be mandatory when
omitting 'id'?

But the idea makes sense.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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