From 2a667c34cb3c7887bad59a9e99f264514cf7af70 Mon Sep 17 00:00:00 2001 From: Wen Congyang Date: Mon, 25 Jul 2011 13:38:51 +0800 Subject: [PATCH] rename cfs_* to vcpu_* In the XML file we now have 1024 90000 0 But the schedinfo parameter are being named cpu_shares: 1024 cfs_period: 90000 cfs_quota: 0 The period/quota is per-vcpu value, so these new tunables should be named 'vcpu_period' and 'vcpu_quota'. --- src/qemu/qemu_driver.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 5df58b11dc..130f474382 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -6188,10 +6188,10 @@ static int qemuSetSchedulerParametersFlags(virDomainPtr dom, if (flags & VIR_DOMAIN_AFFECT_CONFIG) { vmdef->cputune.shares = params[i].value.ul; } - } else if (STREQ(param->field, "cfs_period")) { + } else if (STREQ(param->field, "vcpu_period")) { if (param->type != VIR_TYPED_PARAM_ULLONG) { qemuReportError(VIR_ERR_INVALID_ARG, "%s", - _("invalid type for cfs_period tunable," + _("invalid type for vcpu_period tunable," " expected a 'ullong'")); goto cleanup; } @@ -6208,10 +6208,10 @@ static int qemuSetSchedulerParametersFlags(virDomainPtr dom, if (flags & VIR_DOMAIN_AFFECT_CONFIG) { vmdef->cputune.period = params[i].value.ul; } - } else if (STREQ(param->field, "cfs_quota")) { + } else if (STREQ(param->field, "vcpu_quota")) { if (param->type != VIR_TYPED_PARAM_LLONG) { qemuReportError(VIR_ERR_INVALID_ARG, "%s", - _("invalid type for cfs_quota tunable," + _("invalid type for vcpu_quota tunable," " expected a 'llong'")); goto cleanup; } @@ -6470,10 +6470,10 @@ out: if (*nparams > saved_nparams) { params[1].value.ul = period; params[1].type = VIR_TYPED_PARAM_ULLONG; - if (virStrcpyStatic(params[1].field, "cfs_period") == NULL) { + if (virStrcpyStatic(params[1].field, "vcpu_period") == NULL) { qemuReportError(VIR_ERR_INTERNAL_ERROR, "%s", - _("Field cfs_period too long for destination")); + _("Field vcpu_period too long for destination")); goto cleanup; } saved_nparams++; @@ -6482,10 +6482,10 @@ out: if (*nparams > saved_nparams) { params[2].value.ul = quota; params[2].type = VIR_TYPED_PARAM_LLONG; - if (virStrcpyStatic(params[2].field, "cfs_quota") == NULL) { + if (virStrcpyStatic(params[2].field, "vcpu_quota") == NULL) { qemuReportError(VIR_ERR_INTERNAL_ERROR, "%s", - _("Field cfs_quota too long for destination")); + _("Field vcpu_quota too long for destination")); goto cleanup; } saved_nparams++;