mbox series

[v3,0/2] backup: Use copy offloading

Message ID 20180605140631.4696-1-famz@redhat.com
Headers show
Series backup: Use copy offloading | expand

Message

Fam Zheng June 5, 2018, 2:06 p.m. UTC
Based-on: <20180529055959.32002-1-famz@redhat.com>
([PATCH v7 00/10] qemu-img convert with copy offloading)

This enhances the backup job to make use of the copy offloading API. It
eliminates the necessity to use the bounce buffer as well as speeding up the
copy operation when the backend supports it.

v3: Don't forget coroutine_fn. [Stefan]
    Don't reset job->use_copy_range redundantly. [Stefan]

v2: Use helper functions. [Stefan]

Fam Zheng (2):
  block: Honour BDRV_REQ_NO_SERIALISING in copy range
  backup: Use copy offloading

 block/backup.c        | 150 ++++++++++++++++++++++++++++++------------
 block/io.c            |   6 +-
 block/trace-events    |   1 +
 include/block/block.h |   5 +-
 4 files changed, 117 insertions(+), 45 deletions(-)

Comments

Stefan Hajnoczi June 7, 2018, 12:45 p.m. UTC | #1
On Tue, Jun 05, 2018 at 10:06:29PM +0800, Fam Zheng wrote:
> Based-on: <20180529055959.32002-1-famz@redhat.com>
> ([PATCH v7 00/10] qemu-img convert with copy offloading)
> 
> This enhances the backup job to make use of the copy offloading API. It
> eliminates the necessity to use the bounce buffer as well as speeding up the
> copy operation when the backend supports it.
> 
> v3: Don't forget coroutine_fn. [Stefan]
>     Don't reset job->use_copy_range redundantly. [Stefan]
> 
> v2: Use helper functions. [Stefan]
> 
> Fam Zheng (2):
>   block: Honour BDRV_REQ_NO_SERIALISING in copy range
>   backup: Use copy offloading
> 
>  block/backup.c        | 150 ++++++++++++++++++++++++++++++------------
>  block/io.c            |   6 +-
>  block/trace-events    |   1 +
>  include/block/block.h |   5 +-
>  4 files changed, 117 insertions(+), 45 deletions(-)
> 
> -- 
> 2.17.0
> 
> 

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Fam Zheng July 2, 2018, 9:04 a.m. UTC | #2
Jeff: ping? Can we have this in 3.0?

On Tue, 06/05 22:06, Fam Zheng wrote:
> Based-on: <20180529055959.32002-1-famz@redhat.com>
> ([PATCH v7 00/10] qemu-img convert with copy offloading)
> 
> This enhances the backup job to make use of the copy offloading API. It
> eliminates the necessity to use the bounce buffer as well as speeding up the
> copy operation when the backend supports it.
> 
> v3: Don't forget coroutine_fn. [Stefan]
>     Don't reset job->use_copy_range redundantly. [Stefan]
> 
> v2: Use helper functions. [Stefan]
> 
> Fam Zheng (2):
>   block: Honour BDRV_REQ_NO_SERIALISING in copy range
>   backup: Use copy offloading
> 
>  block/backup.c        | 150 ++++++++++++++++++++++++++++++------------
>  block/io.c            |   6 +-
>  block/trace-events    |   1 +
>  include/block/block.h |   5 +-
>  4 files changed, 117 insertions(+), 45 deletions(-)
> 
> -- 
> 2.17.0
>
Jeff Cody July 2, 2018, 12:02 p.m. UTC | #3
On Mon, Jul 02, 2018 at 05:04:20PM +0800, Fam Zheng wrote:
> Jeff: ping? Can we have this in 3.0?
>

Yes - can you rebase on current master?  I can resolve the conflicts myself
if not.

> On Tue, 06/05 22:06, Fam Zheng wrote:
> > Based-on: <20180529055959.32002-1-famz@redhat.com>
> > ([PATCH v7 00/10] qemu-img convert with copy offloading)
> > 
> > This enhances the backup job to make use of the copy offloading API. It
> > eliminates the necessity to use the bounce buffer as well as speeding up the
> > copy operation when the backend supports it.
> > 
> > v3: Don't forget coroutine_fn. [Stefan]
> >     Don't reset job->use_copy_range redundantly. [Stefan]
> > 
> > v2: Use helper functions. [Stefan]
> > 
> > Fam Zheng (2):
> >   block: Honour BDRV_REQ_NO_SERIALISING in copy range
> >   backup: Use copy offloading
> > 
> >  block/backup.c        | 150 ++++++++++++++++++++++++++++++------------
> >  block/io.c            |   6 +-
> >  block/trace-events    |   1 +
> >  include/block/block.h |   5 +-
> >  4 files changed, 117 insertions(+), 45 deletions(-)
> > 
> > -- 
> > 2.17.0
> >