[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 09/16] migration: We don't need the field rate_limit_used
|
From: |
Juan Quintela |
|
Subject: |
Re: [PATCH v2 09/16] migration: We don't need the field rate_limit_used anymore |
|
Date: |
Fri, 26 May 2023 10:18:24 +0200 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Leonardo Brás <leobras@redhat.com> wrote:
> On Mon, 2023-05-15 at 21:57 +0200, Juan Quintela wrote:
>> Since previous commit, we calculate how much data we have send with
>> migration_transferred_bytes() so no need to maintain this counter and
>> remember to always update it.
>>
>> Signed-off-by: Juan Quintela <quintela@redhat.com>
>> Reviewed-by: Cédric Le Goater <clg@kaod.org>
> I reviewed this one together with 8/16.
> It makes sense for me to squash them together, but anyway:
Already in tree.
See explanation for the split on previous patch.
The other reason for the split is that this part of the patch is trivial
to review O:-)
Later, Juan.
>
> Reviewed-by: Leonardo Bras <leobras@redhat.com>
- Re: [PATCH v2 04/16] qemu-file: Account for rate_limit usage on qemu_fflush(), (continued)
[PATCH v2 07/16] migration: Add a trace for migration_transferred_bytes, Juan Quintela, 2023/05/15
[PATCH v2 08/16] migration: Use migration_transferred_bytes() to calculate rate_limit, Juan Quintela, 2023/05/15
[PATCH v2 09/16] migration: We don't need the field rate_limit_used anymore, Juan Quintela, 2023/05/15
[PATCH v2 06/16] migration: Move migration_total_bytes() to migration-stats.c, Juan Quintela, 2023/05/15
[PATCH v2 10/16] migration: Don't abuse qemu_file transferred for RDMA, Juan Quintela, 2023/05/15
[PATCH v2 11/16] migration/RDMA: It is accounting for zero/normal pages in two places, Juan Quintela, 2023/05/15
[PATCH v2 13/16] migration/rdma: Don't use imaginary transfers, Juan Quintela, 2023/05/15
[PATCH v2 12/16] migration/rdma: Remove QEMUFile parameter when not used, Juan Quintela, 2023/05/15