Re: [Qemu-block] [PATCH 00/17] block: Convert common I/O path to BdrvChild

2016-06-29 Thread Kevin Wolf
Am 28.06.2016 um 15:28 hat Stefan Hajnoczi geschrieben: > Do you want to take it through your tree to avoid > conflicts/dependencies? > > Acked-by: Stefan Hajnoczi Yes, thanks. I've applied it to my tree now (with a few fixes addressing the review comments). Kevin

Re: [Qemu-block] [PATCH 00/17] block: Convert common I/O path to BdrvChild

2016-06-28 Thread Stefan Hajnoczi
On Tue, Jun 21, 2016 at 11:21:17AM +0200, Kevin Wolf wrote: > This series converts all I/O function in the core block layer up to > bdrv_co_preadv/pwritev() to taking a BdrvChild as their first parameter > instead of a BlockDriverState. > > The original motivation for this change were op

Re: [Qemu-block] [PATCH 00/17] block: Convert common I/O path to BdrvChild

2016-06-22 Thread Fam Zheng
On Tue, 06/21 13:31, Kevin Wolf wrote: > Am 21.06.2016 um 13:01 hat Paolo Bonzini geschrieben: > > On 21/06/2016 12:56, Kevin Wolf wrote: > > > Am 21.06.2016 um 11:47 hat Paolo Bonzini geschrieben: > > >> I still fail to understand what is the rationale for this change. The > > >> API is weird;

Re: [Qemu-block] [PATCH 00/17] block: Convert common I/O path to BdrvChild

2016-06-21 Thread Kevin Wolf
Am 21.06.2016 um 13:01 hat Paolo Bonzini geschrieben: > On 21/06/2016 12:56, Kevin Wolf wrote: > > Am 21.06.2016 um 11:47 hat Paolo Bonzini geschrieben: > >> I still fail to understand what is the rationale for this change. The > >> API is weird; you read from a disk, not from an edge, and in

Re: [Qemu-block] [PATCH 00/17] block: Convert common I/O path to BdrvChild

2016-06-21 Thread Paolo Bonzini
On 21/06/2016 12:56, Kevin Wolf wrote: > Am 21.06.2016 um 11:47 hat Paolo Bonzini geschrieben: >> >> >> On 21/06/2016 11:21, Kevin Wolf wrote: >>> This series converts all I/O function in the core block layer up to >>> bdrv_co_preadv/pwritev() to taking a BdrvChild as their first parameter >>>

Re: [Qemu-block] [PATCH 00/17] block: Convert common I/O path to BdrvChild

2016-06-21 Thread Kevin Wolf
Am 21.06.2016 um 11:47 hat Paolo Bonzini geschrieben: > > > On 21/06/2016 11:21, Kevin Wolf wrote: > > This series converts all I/O function in the core block layer up to > > bdrv_co_preadv/pwritev() to taking a BdrvChild as their first parameter > > instead of a BlockDriverState. > > > > The

Re: [Qemu-block] [PATCH 00/17] block: Convert common I/O path to BdrvChild

2016-06-21 Thread Paolo Bonzini
On 21/06/2016 11:21, Kevin Wolf wrote: > This series converts all I/O function in the core block layer up to > bdrv_co_preadv/pwritev() to taking a BdrvChild as their first parameter > instead of a BlockDriverState. > > The original motivation for this change were op blockers, where one of >