[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 7/7] migration/dirtyrate: implement dirty-ring dirtyrate c
From: |
Peter Xu |
Subject: |
Re: [PATCH v7 7/7] migration/dirtyrate: implement dirty-ring dirtyrate calculation |
Date: |
Thu, 17 Jun 2021 11:41:22 -0400 |
On Thu, Jun 17, 2021 at 10:12:08PM +0800, huangy81@chinatelecom.cn wrote:
> @@ -487,12 +646,23 @@ void hmp_info_dirty_rate(Monitor *mon, const QDict
> *qdict)
> info->sample_pages);
> monitor_printf(mon, "Period: %"PRIi64" (sec)\n",
> info->calc_time);
> + monitor_printf(mon, "Mode: %s\n",
> + DirtyRateMeasureMode_str(info->mode));
> monitor_printf(mon, "Dirty rate: ");
> if (info->has_dirty_rate) {
> monitor_printf(mon, "%"PRIi64" (MB/s)\n", info->dirty_rate);
> + if (info->has_vcpu_dirty_rate) {
> + DirtyRateVcpuList *rate, *head = info->vcpu_dirty_rate;
> + for (rate = head; rate != NULL; rate = rate->next) {
> + monitor_printf(mon, "vcpu[%"PRIi64"], Dirty rate:
> %"PRIi64"\n",
Maybe ending it with " (MB/s)" to match the dirty rate line?
Otherwise this patch looks good to me, thanks. It's just that I still have a
few trivial comments out there, so this patch still may need some adjustment.
> + rate->value->id, rate->value->dirty_rate);
> + }
> + }
> } else {
> monitor_printf(mon, "(not ready)\n");
> }
> +
> + qapi_free_DirtyRateVcpuList(info->vcpu_dirty_rate);
> g_free(info);
> }
--
Peter Xu
- [PATCH v7 2/7] memory: rename global_dirty_log to global_dirty_tracking, (continued)
- [PATCH v7 2/7] memory: rename global_dirty_log to global_dirty_tracking, huangy81, 2021/06/17
- [PATCH v7 3/7] memory: make global_dirty_tracking a bitmask, huangy81, 2021/06/17
- [PATCH v7 4/7] migration/dirtyrate: introduce struct and adjust DirtyRateStat, huangy81, 2021/06/17
- [PATCH v7 5/7] migration/dirtyrate: adjust order of registering thread, huangy81, 2021/06/17
- [PATCH v7 7/7] migration/dirtyrate: implement dirty-ring dirtyrate calculation, huangy81, 2021/06/17
- Re: [PATCH v7 7/7] migration/dirtyrate: implement dirty-ring dirtyrate calculation,
Peter Xu <=
- [PATCH v7 6/7] migration/dirtyrate: move init step of calculation to main thread, huangy81, 2021/06/17