Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-13 Thread Kevin Wolf
Am 07.07.2016 um 16:17 hat Kevin Wolf geschrieben: > Am 07.07.2016 um 14:59 hat Alberto Garcia geschrieben: > > On Thu 07 Jul 2016 02:11:27 PM CEST, Kevin Wolf wrote: > > > In order to remove the necessity to use BlockBackend names in the > > > external API, we want to allow node-names everywhere.

Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-08 Thread Eric Blake
On 07/08/2016 04:01 AM, Kevin Wolf wrote: > Am 08.07.2016 um 00:45 hat Eric Blake geschrieben: >> On 07/07/2016 06:11 AM, Kevin Wolf wrote: >>> In order to remove the necessity to use BlockBackend names in the >>> external API, we want to allow node-names everywhere. This converts >>> block-stream

Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-08 Thread Kevin Wolf
Am 08.07.2016 um 00:45 hat Eric Blake geschrieben: > On 07/07/2016 06:11 AM, Kevin Wolf wrote: > > In order to remove the necessity to use BlockBackend names in the > > external API, we want to allow node-names everywhere. This converts > > block-stream to accept a node-name without lifting the

Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-07 Thread Eric Blake
On 07/07/2016 06:11 AM, Kevin Wolf wrote: > In order to remove the necessity to use BlockBackend names in the > external API, we want to allow node-names everywhere. This converts > block-stream to accept a node-name without lifting the restriction that > we're operating at a root node. > > In

Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-07 Thread Kevin Wolf
Am 07.07.2016 um 16:39 hat Alberto Garcia geschrieben: > On Thu 07 Jul 2016 04:17:21 PM CEST, Kevin Wolf wrote: > >> > +static BlockDriverState *qmp_get_root_bs(const char *name, Error **errp) > >> > +{ > >> > +BlockDriverState *bs; > >> > + > >> > +bs = bdrv_lookup_bs(name, name, errp); >

Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-07 Thread Alberto Garcia
On Thu 07 Jul 2016 04:17:21 PM CEST, Kevin Wolf wrote: >> > +static BlockDriverState *qmp_get_root_bs(const char *name, Error **errp) >> > +{ >> > +BlockDriverState *bs; >> > + >> > +bs = bdrv_lookup_bs(name, name, errp); >> > +if (bs == NULL) { >> > +return NULL; >> > +}

Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-07 Thread Kevin Wolf
Am 07.07.2016 um 14:59 hat Alberto Garcia geschrieben: > On Thu 07 Jul 2016 02:11:27 PM CEST, Kevin Wolf wrote: > > In order to remove the necessity to use BlockBackend names in the > > external API, we want to allow node-names everywhere. This converts > > block-stream to accept a node-name

Re: [Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-07 Thread Alberto Garcia
On Thu 07 Jul 2016 02:11:27 PM CEST, Kevin Wolf wrote: > In order to remove the necessity to use BlockBackend names in the > external API, we want to allow node-names everywhere. This converts > block-stream to accept a node-name without lifting the restriction that > we're operating at a root

[Qemu-block] [PATCH v3 01/11] block: Accept node-name for block-stream

2016-07-07 Thread Kevin Wolf
In order to remove the necessity to use BlockBackend names in the external API, we want to allow node-names everywhere. This converts block-stream to accept a node-name without lifting the restriction that we're operating at a root node. In case of an invalid device name, the command returns the