Commit 0965a41e authored by Vladimir Sementsov-Ogievskiy's avatar Vladimir Sementsov-Ogievskiy Committed by Jeff Cody
Browse files

mirror: double performance of the bulk stage if the disc is full 



Mirror can do up to 16 in-flight requests, but actually on full copy
(the whole source disk is non-zero) in-flight is always 1. This happens
as the request is not limited in size: the data occupies maximum available
capacity of s->buf.

The patch limits the size of the request to some artificial constant
(1 Mb here), which is not that big or small. This effectively enables
back parallelism in mirror code as it was designed.

The result is important: the time to migrate 10 Gb disk is reduced from
~350 sec to 170 sec.

Signed-off-by: default avatarVladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Signed-off-by: default avatarDenis V. Lunev <den@openvz.org>
Reviewed-by: default avatarMax Reitz <mreitz@redhat.com>
Reviewed-by: default avatarJeff Cody <jcody@redhat.com>
Message-id: 1468516741-82174-1-git-send-email-vsementsov@virtuozzo.com
CC: Stefan Hajnoczi <stefanha@redhat.com>
CC: Fam Zheng <famz@redhat.com>
CC: Kevin Wolf <kwolf@redhat.com>
CC: Max Reitz <mreitz@redhat.com>
CC: Jeff Cody <jcody@redhat.com>
CC: Eric Blake <eblake@redhat.com>
Signed-off-by: default avatarJeff Cody <jcody@redhat.com>
parent 0a189ffb

block/mirror.c

+8 −2
Original line number Diff line number Diff line
@@ -23,7 +23,9 @@ 


#define SLICE_TIME    100000000ULL /* ns */

#define MAX_IN_FLIGHT 16

#define DEFAULT_MIRROR_BUF_SIZE   (10 << 20)

#define MAX_IO_SECTORS ((1 << 20) >> BDRV_SECTOR_BITS) /* 1 Mb */

#define DEFAULT_MIRROR_BUF_SIZE \

    (MAX_IN_FLIGHT * MAX_IO_SECTORS * BDRV_SECTOR_SIZE)



/* The mirroring buffer is a list of granularity-sized chunks.

 * Free chunks are organized in a list.
@@ -325,6 +327,8 @@ static uint64_t coroutine_fn mirror_iteration(MirrorBlockJob *s) 
    int64_t end = s->bdev_length / BDRV_SECTOR_SIZE;

    int sectors_per_chunk = s->granularity >> BDRV_SECTOR_BITS;

    bool write_zeroes_ok = bdrv_can_write_zeroes_with_unmap(blk_bs(s->target));

    int max_io_sectors = MAX((s->buf_size >> BDRV_SECTOR_BITS) / MAX_IN_FLIGHT,

                             MAX_IO_SECTORS);



    sector_num = hbitmap_iter_next(&s->hbi);

    if (sector_num < 0) {
@@ -388,7 +392,9 @@ static uint64_t coroutine_fn mirror_iteration(MirrorBlockJob *s) 
                                          nb_chunks * sectors_per_chunk,

                                          &io_sectors, &file);

        if (ret < 0) {

            io_sectors = nb_chunks * sectors_per_chunk;

            io_sectors = MIN(nb_chunks * sectors_per_chunk, max_io_sectors);

        } else if (ret & BDRV_BLOCK_DATA) {

            io_sectors = MIN(io_sectors, max_io_sectors);

        }



        io_sectors -= io_sectors % sectors_per_chunk;