[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH QEMU v5 8/8] tests: Add migration dirty-limit capability test
From: |
Juan Quintela |
Subject: |
Re: [PATCH QEMU v5 8/8] tests: Add migration dirty-limit capability test |
Date: |
Tue, 13 Jun 2023 20:09:01 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
~hyman <hyman@git.sr.ht> wrote:
> From: Hyman Huang(黄勇) <yong.huang@smartx.com>
>
> Add migration dirty-limit capability test if kernel support
> dirty ring.
>
> Migration dirty-limit capability introduce dirty limit
> capability, two parameters: x-vcpu-dirty-limit-period and
> vcpu-dirty-limit are introduced to implement the live
> migration with dirty limit.
>
> The test case does the following things:
> 1. start src, dst vm and enable dirty-limit capability
> 2. start migrate and set cancel it to check if dirty limit
> stop working.
> 3. restart dst vm
> 4. start migrate and enable dirty-limit capability
> 5. check if migration satisfy the convergence condition
> during pre-switchover phase.
>
> Signed-off-by: Hyman Huang(黄勇) <yong.huang@smartx.com>
> +static void test_migrate_dirty_limit(void)
> +{
> + g_autofree char *uri = g_strdup_printf("unix:%s/migsocket", tmpfs);
> + QTestState *from, *to;
> + int64_t remaining, throttle_us_per_full;
See comments on previous patch about int64_t vs uint64_t.
Except if negative values have some meaning, it should be uint64_t.
Rest seems correct.
Later, Juan.
- Re: [PATCH QEMU v5 1/8] softmmu/dirtylimit: Add parameter check for hmp "set_vcpu_dirty_limit", (continued)
- [PATCH QEMU v5 5/8] migration: Refactor auto-converge capability logic, ~hyman, 2023/06/08
- [PATCH QEMU v5 2/8] qapi/migration: Introduce x-vcpu-dirty-limit-period parameter, ~hyman, 2023/06/08
- [PATCH QEMU v5 7/8] migration: Extend query-migrate to provide dirty page limit info, ~hyman, 2023/06/08
- [PATCH QEMU v5 8/8] tests: Add migration dirty-limit capability test, ~hyman, 2023/06/08
- Re: [PATCH QEMU v5 8/8] tests: Add migration dirty-limit capability test,
Juan Quintela <=
- [PATCH QEMU v5 4/8] migration: Introduce dirty-limit capability, ~hyman, 2023/06/08
- [PATCH QEMU v5 3/8] qapi/migration: Introduce vcpu-dirty-limit parameters, ~hyman, 2023/06/08
- [PATCH QEMU v5 6/8] migration: Implement dirty-limit convergence algo, ~hyman, 2023/06/08