[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual C
From: |
Peter Xu |
Subject: |
Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU |
Date: |
Fri, 24 Dec 2021 13:14:21 +0800 |
On Tue, Dec 14, 2021 at 07:07:34PM +0800, huangy81@chinatelecom.cn wrote:
> +void qmp_vcpu_dirty_limit(bool enable,
> + bool has_cpu_index,
> + uint64_t cpu_index,
> + bool has_dirty_rate,
> + uint64_t dirty_rate,
> + Error **errp)
> +{
> + static bool initialized;
IMHO this is not needed; if we're with a global state pointer then it's the
same to check against that.
The rest looks mostly good (besides the last proposal on API design which you
got confirmation from Markus).
--
Peter Xu
- Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU, (continued)
- Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU, Markus Armbruster, 2021/12/15
- Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU, Markus Armbruster, 2021/12/15
- Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU, Peter Xu, 2021/12/16
- Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU, Hyman Huang, 2021/12/16
- Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU, Markus Armbruster, 2021/12/16
- Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU, Peter Xu, 2021/12/24
Re: [PATCH v10 3/3] cpus-common: implement dirty page limit on virtual CPU,
Peter Xu <=
Re: [PATCH v10 0/3] support dirty restraint on vCPU, Peter Xu, 2021/12/24