qemu_driver: Add calc_mode for dirtyrate statistics

Add calc_mode for dirtyrate statistics retured by
virsh domstats --dirtyrate api, also add vcpu dirtyrate
if dirty-ring mode was used in last measurement.

Signed-off-by: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Hyman Huang(黄勇) 2022-02-20 21:28:15 +08:00 committed by Michal Privoznik
parent 72e60ddf1b
commit 42d36b65a3
4 changed files with 88 additions and 3 deletions

View File

@ -11980,6 +11980,12 @@ virConnectGetDomainCapabilities(virConnectPtr conn,
* "dirtyrate.megabytes_per_second" - the calculated memory dirty rate in
* MiB/s as long long. It is produced
* only if the calc_status is measured.
* "dirtyrate.calc_mode" - the calculation mode used last measurement, either
* of these 3 'page-sampling,dirty-bitmap,dirty-ring'
* values returned.
* "dirtyrate.vcpu.<num>.megabytes_per_second" - the calculated memory dirty
* rate for a virtual cpu as
* unsigned long long.
*
* Note that entire stats groups or individual stat fields may be missing from
* the output in case they are not supported by the given hypervisor, are not

View File

@ -18556,11 +18556,27 @@ qemuDomainGetStatsDirtyRate(virQEMUDriver *driver,
"dirtyrate.calc_period") < 0)
return -1;
if ((info.status == VIR_DOMAIN_DIRTYRATE_MEASURED) &&
virTypedParamListAddLLong(params, info.dirtyRate,
"dirtyrate.megabytes_per_second") < 0)
if (virTypedParamListAddString(params,
qemuMonitorDirtyRateCalcModeTypeToString(info.mode),
"dirtyrate.calc_mode") < 0)
return -1;
if (info.status == VIR_DOMAIN_DIRTYRATE_MEASURED) {
if (virTypedParamListAddLLong(params, info.dirtyRate,
"dirtyrate.megabytes_per_second") < 0)
return -1;
if (info.mode == QEMU_MONITOR_DIRTYRATE_CALC_MODE_DIRTY_RING) {
size_t i;
for (i = 0; i < info.nvcpus; i++) {
if (virTypedParamListAddULLong(params, info.rates[i].value,
"dirtyrate.vcpu.%d.megabytes_per_second",
info.rates[i].idx) < 0)
return -1;
}
}
}
return 0;
}

View File

@ -1553,6 +1553,12 @@ qemuMonitorStartDirtyRateCalc(qemuMonitor *mon,
int seconds,
qemuMonitorDirtyRateCalcMode mode);
typedef struct _qemuMonitorDirtyRateVcpu qemuMonitorDirtyRateVcpu;
struct _qemuMonitorDirtyRateVcpu {
int idx; /* virtual cpu index */
unsigned long long value; /* virtual cpu dirty page rate in MB/s */
};
typedef struct _qemuMonitorDirtyRateInfo qemuMonitorDirtyRateInfo;
struct _qemuMonitorDirtyRateInfo {
int status; /* the status of last dirtyrate calculation,
@ -1560,6 +1566,10 @@ struct _qemuMonitorDirtyRateInfo {
int calcTime; /* the period of dirtyrate calculation */
long long startTime; /* the start time of dirtyrate calculation */
long long dirtyRate; /* the dirtyrate in MiB/s */
qemuMonitorDirtyRateCalcMode mode; /* calculation mode used in
last measurement */
size_t nvcpus; /* number of virtual cpu */
qemuMonitorDirtyRateVcpu *rates; /* array of dirty page rate */
};
int

View File

@ -8781,12 +8781,46 @@ VIR_ENUM_IMPL(qemuMonitorDirtyRateStatus,
"measuring",
"measured");
static int
qemuMonitorJSONExtractVcpuDirtyRate(virJSONValue *data,
qemuMonitorDirtyRateInfo *info)
{
size_t nvcpus;
size_t i;
nvcpus = virJSONValueArraySize(data);
info->nvcpus = nvcpus;
info->rates = g_new0(qemuMonitorDirtyRateVcpu, nvcpus);
for (i = 0; i < nvcpus; i++) {
virJSONValue *entry = virJSONValueArrayGet(data, i);
if (virJSONValueObjectGetNumberInt(entry, "id",
&info->rates[i].idx) < 0) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("query-dirty-rate reply was missing 'id' data"));
return -1;
}
if (virJSONValueObjectGetNumberUlong(entry, "dirty-rate",
&info->rates[i].value) < 0) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("query-dirty-rate reply was missing 'dirty-rate' data"));
return -1;
}
}
return 0;
}
static int
qemuMonitorJSONExtractDirtyRateInfo(virJSONValue *data,
qemuMonitorDirtyRateInfo *info)
{
const char *statusstr;
const char *modestr;
int status;
int mode;
virJSONValue *rates = NULL;
if (!(statusstr = virJSONValueObjectGetString(data, "status"))) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
@ -8823,6 +8857,25 @@ qemuMonitorJSONExtractDirtyRateInfo(virJSONValue *data,
return -1;
}
if ((modestr = virJSONValueObjectGetString(data, "mode"))) {
if ((mode = qemuMonitorDirtyRateCalcModeTypeFromString(modestr)) < 0) {
virReportError(VIR_ERR_INTERNAL_ERROR,
_("Unknown dirty page rate calculation mode: %s"), modestr);
return -1;
}
info->mode = mode;
} else {
info->mode = QEMU_MONITOR_DIRTYRATE_CALC_MODE_PAGE_SAMPLING;
}
if ((rates = virJSONValueObjectGetArray(data, "vcpu-dirty-rate"))) {
if (qemuMonitorJSONExtractVcpuDirtyRate(rates, info) < 0) {
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("query-dirty-rate parsing 'vcpu-dirty-rate' in failure"));
return -1;
}
}
return 0;
}