[libvirt] [PATCH 5/9] Use virDomainObjGetDefs in lxcDomainGetMemoryParameters
John Ferlan
jferlan at redhat.com
Wed Jun 8 12:44:51 UTC 2016
On 06/06/2016 04:08 AM, Ján Tomko wrote:
> Instead of virDomainLiveConfigHelperMethod.
> ---
> src/lxc/lxc_driver.c | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c
> index c39b4b4..00b4df2 100644
> --- a/src/lxc/lxc_driver.c
> +++ b/src/lxc/lxc_driver.c
> @@ -941,9 +941,9 @@ lxcDomainGetMemoryParameters(virDomainPtr dom,
> {
> virCapsPtr caps = NULL;
^^
Unused now too and could be removed.
John
> virDomainDefPtr persistentDef = NULL;
> + virDomainDefPtr def = NULL;
> virDomainObjPtr vm = NULL;
> virLXCDomainObjPrivatePtr priv = NULL;
> - virLXCDriverPtr driver = dom->conn->privateData;
> unsigned long long val;
> int ret = -1;
> size_t i;
> @@ -960,13 +960,13 @@ lxcDomainGetMemoryParameters(virDomainPtr dom,
>
> priv = vm->privateData;
>
> - if (virDomainGetMemoryParametersEnsureACL(dom->conn, vm->def) < 0 ||
> - !(caps = virLXCDriverGetCapabilities(driver, false)) ||
> - virDomainLiveConfigHelperMethod(caps, driver->xmlopt,
> - vm, &flags, &persistentDef) < 0)
> + if (virDomainGetMemoryParametersEnsureACL(dom->conn, vm->def) < 0)
> goto cleanup;
>
> - if (flags & VIR_DOMAIN_AFFECT_LIVE &&
> + if (virDomainObjGetDefs(vm, flags, &def, &persistentDef) < 0)
> + goto cleanup;
> +
> + if (def &&
> !virCgroupHasController(priv->cgroup, VIR_CGROUP_CONTROLLER_MEMORY)) {
> virReportError(VIR_ERR_OPERATION_INVALID,
> "%s", _("cgroup memory controller is not mounted"));
> @@ -986,7 +986,7 @@ lxcDomainGetMemoryParameters(virDomainPtr dom,
>
> switch (i) {
> case 0: /* fill memory hard limit here */
> - if (flags & VIR_DOMAIN_AFFECT_CONFIG) {
> + if (persistentDef) {
> val = persistentDef->mem.hard_limit;
> } else if (virCgroupGetMemoryHardLimit(priv->cgroup, &val) < 0) {
> goto cleanup;
> @@ -996,7 +996,7 @@ lxcDomainGetMemoryParameters(virDomainPtr dom,
> goto cleanup;
> break;
> case 1: /* fill memory soft limit here */
> - if (flags & VIR_DOMAIN_AFFECT_CONFIG) {
> + if (persistentDef) {
> val = persistentDef->mem.soft_limit;
> } else if (virCgroupGetMemorySoftLimit(priv->cgroup, &val) < 0) {
> goto cleanup;
> @@ -1006,7 +1006,7 @@ lxcDomainGetMemoryParameters(virDomainPtr dom,
> goto cleanup;
> break;
> case 2: /* fill swap hard limit here */
> - if (flags & VIR_DOMAIN_AFFECT_CONFIG) {
> + if (persistentDef) {
> val = persistentDef->mem.swap_hard_limit;
> } else if (virCgroupGetMemSwapHardLimit(priv->cgroup, &val) < 0) {
> goto cleanup;
>
More information about the libvir-list
mailing list