mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-22 17:34:18 +03:00
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:
parent
72e60ddf1b
commit
42d36b65a3
@ -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
|
||||
|
@ -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;
|
||||
}
|
||||
|
||||
|
@ -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
|
||||
|
@ -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;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user