qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v3 02/25] qapi: backup: add perf.use-copy-range parameter


From: Max Reitz
Subject: Re: [PATCH v3 02/25] qapi: backup: add perf.use-copy-range parameter
Date: Mon, 11 Jan 2021 17:07:06 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.5.0

On 26.10.20 18:17, Vladimir Sementsov-Ogievskiy wrote:
Experiments show, that copy_range is not always making things faster.
So, to make experimentation simpler, let's add a parameter. Some more
perf parameters will be added soon, so here is a new struct.

For now, add new backup qmp parameter with x- prefix for the following
reasons:

  - We are going to add more performance parameters, some will be
    related to the whole block-copy process, some only to background
    copying in backup (ignored for copy-before-write operations).
  - On the other hand, we are going to use block-copy interface in other
    block jobs, which will need performance options as well.. And it
    should be the same structure or at least somehow related.

So, there are too much unclean things about how the interface and now
we need the new options mostly for testing. Let's keep them
experimental for a while.

In do_backup_common() new x-perf parameter handled in a way to
make further options addition simpler.

We add use-copy-range with default=true, and we'll change the default
in further patch, after moving backup to use block-copy.

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
---
  qapi/block-core.json       | 17 ++++++++++++++++-
  block/backup-top.h         |  1 +
  include/block/block-copy.h |  2 +-
  include/block/block_int.h  |  3 +++
  block/backup-top.c         |  4 +++-
  block/backup.c             |  6 +++++-
  block/block-copy.c         |  4 ++--
  block/replication.c        |  2 ++
  blockdev.c                 |  8 ++++++++
  9 files changed, 41 insertions(+), 6 deletions(-)

Reviewed-by: Max Reitz <mreitz@redhat.com>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]