[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 3/8] migration: Introduce dirty-limit capability
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v1 3/8] migration: Introduce dirty-limit capability |
Date: |
Fri, 02 Sep 2022 10:07:45 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
huangy81@chinatelecom.cn writes:
> From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
>
> Introduce migration dirty-limit capability, which can
> be turned on before live migration and limit dirty
> page rate durty live migration.
>
> Introduce migrate_dirty_limit function to help check
> if dirty-limit capability enabled during live migration.
>
> Meanwhile, refactor vcpu_dirty_rate_stat_collect
> so that period can be configured instead of hardcoded.
>
> dirty-limit capability is kind of like auto-converge
> but using dirty limit instead of traditional cpu-throttle
> to throttle guest down. To enable this feature, turn on
> the dirty-limit capability before live migration using
> migratioin-set-capabilities, and set the parameters
migrate-set-capabilities
> "x-vcpu-dirty-limit-period", "vcpu-dirty-limit" suitably
"x-vcpu-dirty-limit"
> to speed up convergence.
>
> Signed-off-by: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
Hmm. You make dirty-limiting as a whole a stable interface (evidence:
capability "dirty-limit" is stable), but keep its two parameters
unstable. Rationale behind that?
- Re: [PATCH v1 4/8] migration: Implement dirty-limit convergence algo, (continued)
[PATCH v1 1/8] qapi/migration: Introduce x-vcpu-dirty-limit-period parameter, huangy81, 2022/09/01
[PATCH v1 2/8] qapi/migration: Introduce x-vcpu-dirty-limit parameters, huangy81, 2022/09/01
[PATCH v1 5/8] migration: Export dirty-limit time info, huangy81, 2022/09/01
[PATCH v1 3/8] migration: Introduce dirty-limit capability, huangy81, 2022/09/01
- Re: [PATCH v1 3/8] migration: Introduce dirty-limit capability,
Markus Armbruster <=
[PATCH v1 6/8] tests: Add migration dirty-limit capability test, huangy81, 2022/09/01
[PATCH v1 7/8] tests/migration: Introduce dirty-ring-size option into guestperf, huangy81, 2022/09/01
[PATCH v1 8/8] tests/migration: Introduce dirty-limit into guestperf, huangy81, 2022/09/01
Re: [PATCH v1 0/8] migration: introduce dirtylimit capability, Peter Xu, 2022/09/06