[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 1/4] Remove unused code
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH 1/4] Remove unused code |
Date: |
Tue, 12 Jan 2010 12:50:33 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Liran Schour wrote:
> blk_mig_save_bulked_block is never called with sync flag. Remove the sync
> flag. Calculate bulk completion during blk_mig_save_bulked_block.
>
> Signed-off-by: Liran Schour <address@hidden>
> ---
> block-migration.c | 63 ++++++++++++++++++++--------------------------------
> 1 files changed, 24 insertions(+), 39 deletions(-)
>
> diff --git a/block-migration.c b/block-migration.c
> index 258a88a..6957909 100644
> --- a/block-migration.c
> +++ b/block-migration.c
> @@ -71,7 +71,8 @@ typedef struct BlkMigState {
> int read_done;
> int transferred;
> int64_t total_sector_sum;
> - int prev_progress;
> + int prev_progress;
> + int bulk_completed;
Trailing whitespaces - here and elsewhere in the series. Please make
sure to clean such things up. Almost all editors come with visualization
for trailing spaces and also tabs.
> } BlkMigState;
>
> static BlkMigState block_mig_state;
> @@ -138,7 +139,7 @@ static void blk_mig_read_cb(void *opaque, int ret)
> }
>
> static int mig_save_device_bulk(Monitor *mon, QEMUFile *f,
> - BlkMigDevState *bmds, int is_async)
> + BlkMigDevState *bmds)
> {
> int64_t total_sectors = bmds->total_sectors;
> int64_t cur_sector = bmds->cur_sector;
> @@ -175,27 +176,17 @@ static int mig_save_device_bulk(Monitor *mon, QEMUFile
> *f,
> blk->bmds = bmds;
> blk->sector = cur_sector;
>
> - if (is_async) {
> - blk->iov.iov_base = blk->buf;
> - blk->iov.iov_len = nr_sectors * BDRV_SECTOR_SIZE;
> - qemu_iovec_init_external(&blk->qiov, &blk->iov, 1);
> -
> - blk->aiocb = bdrv_aio_readv(bs, cur_sector, &blk->qiov,
> - nr_sectors, blk_mig_read_cb, blk);
> - if (!blk->aiocb) {
> - goto error;
> - }
> - block_mig_state.submitted++;
> - } else {
> - if (bdrv_read(bs, cur_sector, blk->buf, nr_sectors) < 0) {
> - goto error;
> - }
> - blk_send(f, blk);
> + blk->iov.iov_base = blk->buf;
> + blk->iov.iov_len = nr_sectors * BDRV_SECTOR_SIZE;
> + qemu_iovec_init_external(&blk->qiov, &blk->iov, 1);
>
> - qemu_free(blk->buf);
> - qemu_free(blk);
> + blk->aiocb = bdrv_aio_readv(bs, cur_sector, &blk->qiov,
> + nr_sectors, blk_mig_read_cb, blk);
> + if (!blk->aiocb) {
> + goto error;
> }
> -
> + block_mig_state.submitted++;
> +
> bdrv_reset_dirty(bs, cur_sector, nr_sectors);
> bmds->cur_sector = cur_sector + nr_sectors;
>
> @@ -229,6 +220,7 @@ static void init_blk_migration(Monitor *mon, QEMUFile *f)
> block_mig_state.transferred = 0;
> block_mig_state.total_sector_sum = 0;
> block_mig_state.prev_progress = -1;
> + block_mig_state.bulk_completed = 0;
>
> for (bs = bdrv_first; bs != NULL; bs = bs->next) {
> if (bs->type == BDRV_TYPE_HD) {
> @@ -260,7 +252,7 @@ static void init_blk_migration(Monitor *mon, QEMUFile *f)
> }
> }
>
> -static int blk_mig_save_bulked_block(Monitor *mon, QEMUFile *f, int is_async)
> +static int blk_mig_save_bulked_block(Monitor *mon, QEMUFile *f)
> {
> int64_t completed_sector_sum = 0;
> BlkMigDevState *bmds;
> @@ -269,7 +261,7 @@ static int blk_mig_save_bulked_block(Monitor *mon,
> QEMUFile *f, int is_async)
>
> QSIMPLEQ_FOREACH(bmds, &block_mig_state.bmds_list, entry) {
> if (bmds->bulk_completed == 0) {
> - if (mig_save_device_bulk(mon, f, bmds, is_async) == 1) {
> + if (mig_save_device_bulk(mon, f, bmds) == 1) {
> /* completed bulk section for this device */
> bmds->bulk_completed = 1;
> }
> @@ -289,7 +281,7 @@ static int blk_mig_save_bulked_block(Monitor *mon,
> QEMUFile *f, int is_async)
> monitor_printf(mon, "Completed %d %%\r", progress);
> monitor_flush(mon);
> }
> -
> +
Such hunks should always ring a bell... :)
> return ret;
> }
>
> @@ -362,19 +354,13 @@ static void flush_blks(QEMUFile* f)
>
> static int is_stage2_completed(void)
> {
> - BlkMigDevState *bmds;
>
> - if (block_mig_state.submitted > 0) {
> + if (block_mig_state.submitted == 0 &&
> + block_mig_state.bulk_completed == 1) {
> + return 1;
> + } else {
return (block_mig_state.submitted == 0 &&
block_mig_state.bulk_completed);
> return 0;
> }
> -
> - QSIMPLEQ_FOREACH(bmds, &block_mig_state.bmds_list, entry) {
> - if (bmds->bulk_completed == 0) {
> - return 0;
> - }
> - }
> -
> - return 1;
> }
>
> static void blk_mig_cleanup(Monitor *mon)
> @@ -432,8 +418,9 @@ static int block_save_live(Monitor *mon, QEMUFile *f, int
> stage, void *opaque)
> while ((block_mig_state.submitted +
> block_mig_state.read_done) * BLOCK_SIZE <
> qemu_file_get_rate_limit(f)) {
> - if (blk_mig_save_bulked_block(mon, f, 1) == 0) {
> - /* no more bulk blocks for now */
> + if (blk_mig_save_bulked_block(mon, f) == 0) {
> + /* finish saving bulk on all devices */
> + block_mig_state.bulk_completed = 1;
> break;
> }
> }
> @@ -446,9 +433,7 @@ static int block_save_live(Monitor *mon, QEMUFile *f, int
> stage, void *opaque)
> }
>
> if (stage == 3) {
> - while (blk_mig_save_bulked_block(mon, f, 0) != 0) {
> - /* empty */
> - }
> + /* we now for sure that save bulk is completed */
>
> blk_mig_save_dirty_blocks(mon, f);
> blk_mig_cleanup(mon);
Otherwise nice cleanup.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
[Qemu-devel] Re: [PATCH 1/4] Remove unused code,
Jan Kiszka <=