Re: [Qemu-devel] [PATCH 05/14] stream: Use BlockBackend for I/O

[Max Reitz]

On 04.05.2016 11:39, Kevin Wolf wrote:
> This changes the streaming block job to use the job's BlockBackend for
> performing the COR reads. job->bs isn't used by the streaming code any
> more afterwards.
> 
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
>  block/block-backend.c          | 15 +++++++++++++++
>  block/io.c                     |  9 ---------
>  block/stream.c                 | 14 ++++++++------
>  include/block/block.h          |  2 --
>  include/sysemu/block-backend.h |  2 ++
>  trace-events                   |  4 +++-
>  6 files changed, 28 insertions(+), 18 deletions(-)

Hm, I don't quite like adding a new sector-based BB function, but
changing *_co_copy_on_readv()'s interface doesn't seem too reasonable at
this point.

And I guess there are some bdrv_* functions in stream_run() for which a
blk_* equivalent exists, so it might have made sense to change those,
too (e.g. bdrv_getlength()).

Anyway:

Reviewed-by: Max Reitz <address@hidden>

signature.asc
Description: OpenPGP digital signature