[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 2/2] qapi: Allow blockdev-add for NBD
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-block] [PATCH 2/2] qapi: Allow blockdev-add for NBD |
Date: |
Thu, 4 Feb 2016 12:58:37 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 |
On 03/02/2016 17:48, Eric Blake wrote:
> On 02/03/2016 09:33 AM, Max Reitz wrote:
>> We have to introduce a new object (BlockdevOptionsNbd) for
>> several reasons: - Neither of InetSocketAddress nor
>> UnixSocketAddress alone is sufficient, because both are
>> supported - We cannot use SocketAddress because NBD does not
>> support an fd, and because it is not a flat union which
>> BlockdevOptionsNbd is
>
> Can we do it anyways, and just error out/document that fd is
> unsupported?
Especially because there's no reason _not_ to support fd. Sure, it's
really fringe, but if qemu-socket APIs make it just work...
Paolo
[Qemu-block] [PATCH 1/2] block/nbd: Reject port parameter without host, Max Reitz, 2016/02/03