[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] block: Support streaming to an intermediate
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] block: Support streaming to an intermediate layer |
Date: |
Thu, 5 Mar 2015 16:47:47 +0100 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Thu, Mar 05, 2015 at 04:15:52PM +0100, Kevin Wolf wrote:
> > > 3. As the BDS that owns the job
> > >
> > > My question is whether we can't simply call stream_start()
> > > with an intermediate node as bs instead of introducing a new
> > > parameter. I'm not completely sure about the consequences of
> > > 3., i.e. moving ownership of a block job to some BDS somewhere
> > > down the chain, but otherwise it should be possible and seems
> > > cleaner.
> >
> > Since I was also not sure about the consequences of such a change
> > I opted for the conservative approach.
>
> I see. I'm worried about the external API. If we let the root node
> own the job, we may paint ourselves into a corner with respect to
> nodes with multiple users and therefore multiple possible root
> nodes.
I'll try to move the job to the intermediate node and see where I can
get then.
Berto