mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-08-30 12:31:16 +00:00
libxl: move libxlFreeMem to libxl_domain
Move libxlFreeMem from libxl_driver to libxl_domain for use by other libxl modules. For consistency, rename to libxlDomainFreeMem. Signed-off-by: Jim Fehlig <jfehlig@suse.com>
This commit is contained in:
parent
e394d39ed8
commit
024e4bb400
@ -854,3 +854,42 @@ cleanup:
|
|||||||
VIR_FREE(cpumap);
|
VIR_FREE(cpumap);
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
int
|
||||||
|
libxlDomainFreeMem(libxlDomainObjPrivatePtr priv, libxl_domain_config *d_config)
|
||||||
|
{
|
||||||
|
uint32_t needed_mem;
|
||||||
|
uint32_t free_mem;
|
||||||
|
size_t i;
|
||||||
|
int ret = -1;
|
||||||
|
int tries = 3;
|
||||||
|
int wait_secs = 10;
|
||||||
|
|
||||||
|
if ((ret = libxl_domain_need_memory(priv->ctx, &d_config->b_info,
|
||||||
|
&needed_mem)) >= 0) {
|
||||||
|
for (i = 0; i < tries; ++i) {
|
||||||
|
if ((ret = libxl_get_free_memory(priv->ctx, &free_mem)) < 0)
|
||||||
|
break;
|
||||||
|
|
||||||
|
if (free_mem >= needed_mem) {
|
||||||
|
ret = 0;
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
|
if ((ret = libxl_set_memory_target(priv->ctx, 0,
|
||||||
|
free_mem - needed_mem,
|
||||||
|
/* relative */ 1, 0)) < 0)
|
||||||
|
break;
|
||||||
|
|
||||||
|
ret = libxl_wait_for_free_memory(priv->ctx, 0, needed_mem,
|
||||||
|
wait_secs);
|
||||||
|
if (ret == 0 || ret != ERROR_NOMEM)
|
||||||
|
break;
|
||||||
|
|
||||||
|
if ((ret = libxl_wait_for_memory_target(priv->ctx, 0, 1)) < 0)
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
@ -127,4 +127,8 @@ int
|
|||||||
libxlDomainSetVcpuAffinities(libxlDriverPrivatePtr driver,
|
libxlDomainSetVcpuAffinities(libxlDriverPrivatePtr driver,
|
||||||
virDomainObjPtr vm);
|
virDomainObjPtr vm);
|
||||||
|
|
||||||
|
int
|
||||||
|
libxlDomainFreeMem(libxlDomainObjPrivatePtr priv,
|
||||||
|
libxl_domain_config *d_config);
|
||||||
|
|
||||||
#endif /* LIBXL_DOMAIN_H */
|
#endif /* LIBXL_DOMAIN_H */
|
||||||
|
@ -311,45 +311,6 @@ const struct libxl_event_hooks ev_hooks = {
|
|||||||
.disaster = NULL,
|
.disaster = NULL,
|
||||||
};
|
};
|
||||||
|
|
||||||
static int
|
|
||||||
libxlFreeMem(libxlDomainObjPrivatePtr priv, libxl_domain_config *d_config)
|
|
||||||
{
|
|
||||||
uint32_t needed_mem;
|
|
||||||
uint32_t free_mem;
|
|
||||||
size_t i;
|
|
||||||
int ret = -1;
|
|
||||||
int tries = 3;
|
|
||||||
int wait_secs = 10;
|
|
||||||
|
|
||||||
if ((ret = libxl_domain_need_memory(priv->ctx, &d_config->b_info,
|
|
||||||
&needed_mem)) >= 0) {
|
|
||||||
for (i = 0; i < tries; ++i) {
|
|
||||||
if ((ret = libxl_get_free_memory(priv->ctx, &free_mem)) < 0)
|
|
||||||
break;
|
|
||||||
|
|
||||||
if (free_mem >= needed_mem) {
|
|
||||||
ret = 0;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
if ((ret = libxl_set_memory_target(priv->ctx, 0,
|
|
||||||
free_mem - needed_mem,
|
|
||||||
/* relative */ 1, 0)) < 0)
|
|
||||||
break;
|
|
||||||
|
|
||||||
ret = libxl_wait_for_free_memory(priv->ctx, 0, needed_mem,
|
|
||||||
wait_secs);
|
|
||||||
if (ret == 0 || ret != ERROR_NOMEM)
|
|
||||||
break;
|
|
||||||
|
|
||||||
if ((ret = libxl_wait_for_memory_target(priv->ctx, 0, 1)) < 0)
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Start a domain through libxenlight.
|
* Start a domain through libxenlight.
|
||||||
*
|
*
|
||||||
@ -429,7 +390,7 @@ libxlVmStart(libxlDriverPrivatePtr driver, virDomainObjPtr vm,
|
|||||||
if (libxlBuildDomainConfig(driver, vm, &d_config) < 0)
|
if (libxlBuildDomainConfig(driver, vm, &d_config) < 0)
|
||||||
goto endjob;
|
goto endjob;
|
||||||
|
|
||||||
if (cfg->autoballoon && libxlFreeMem(priv, &d_config) < 0) {
|
if (cfg->autoballoon && libxlDomainFreeMem(priv, &d_config) < 0) {
|
||||||
virReportError(VIR_ERR_INTERNAL_ERROR,
|
virReportError(VIR_ERR_INTERNAL_ERROR,
|
||||||
_("libxenlight failed to get free memory for domain '%s'"),
|
_("libxenlight failed to get free memory for domain '%s'"),
|
||||||
d_config.c_info.name);
|
d_config.c_info.name);
|
||||||
|
Loading…
Reference in New Issue
Block a user