[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 10/12] migration: Use migration_transferred_bytes()
|
From: |
Fabiano Rosas |
|
Subject: |
Re: [PATCH 10/12] migration: Use migration_transferred_bytes() |
|
Date: |
Tue, 24 Oct 2023 14:46:24 -0300 |
Juan Quintela <quintela@redhat.com> writes:
> There are only two differnces with the old value:
>
> - the amount of QEMUFile that hasn't yet been flushed. It can be
> discussed what is more exact, the new or the old one.
> - the amount of transferred bytes that we forgot to account for (the
> newer is better, i.e. exact).
>
> Notice that this two values are used to:
> a - present to the user
> b - calculate the rate_limit
>
> So a few KB here and there is not going to make a difference.
>
> Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Fabiano Rosas <farosas@suse.de>
- Re: [PATCH 02/12] qemu_file: Use a stat64 for qemu_file_transferred, (continued)
- [PATCH 03/12] qemu_file: total_transferred is not used anymore, Juan Quintela, 2023/10/24
- [PATCH 09/12] qemu-file: Simplify qemu_file_get_error(), Juan Quintela, 2023/10/24
- [PATCH 11/12] migration: Remove transferred atomic counter, Juan Quintela, 2023/10/24
- [PATCH 08/12] migration: migration_rate_limit_reset() don't need the QEMUFile, Juan Quintela, 2023/10/24
- [PATCH 10/12] migration: Use migration_transferred_bytes(), Juan Quintela, 2023/10/24
- Re: [PATCH 10/12] migration: Use migration_transferred_bytes(),
Fabiano Rosas <=
- [PATCH 12/12] qemu-file: Make qemu_fflush() return errors, Juan Quintela, 2023/10/24
- [PATCH 07/12] migration: migration_transferred_bytes() don't need the QEMUFile, Juan Quintela, 2023/10/24
- [PATCH 06/12] qemu-file: Remove _noflush from qemu_file_transferred_noflush(), Juan Quintela, 2023/10/24