[libvirt] [PATCH] OpenVZ: implement suspend/resume driver APIs
Daniel Veillard
veillard at redhat.com
Wed Aug 4 12:02:55 UTC 2010
On Fri, Jul 30, 2010 at 05:48:49PM +0200, Jean-Baptiste Rouault wrote:
> ---
> src/openvz/openvz_driver.c | 84 ++++++++++++++++++++++++++++++++++++++++++-
> 1 files changed, 82 insertions(+), 2 deletions(-)
>
> diff --git a/src/openvz/openvz_driver.c b/src/openvz/openvz_driver.c
> index e5bbdd0..bdc0e92 100644
> --- a/src/openvz/openvz_driver.c
> +++ b/src/openvz/openvz_driver.c
> @@ -503,6 +503,86 @@ static void openvzSetProgramSentinal(const char **prog, const char *key)
> }
> }
>
> +static int openvzDomainSuspend(virDomainPtr dom) {
> + struct openvz_driver *driver = dom->conn->privateData;
> + virDomainObjPtr vm;
> + const char *prog[] = {VZCTL, "--quiet", "chkpnt", PROGRAM_SENTINAL, "--suspend", NULL};
> + int ret = -1;
> +
> + openvzDriverLock(driver);
> + vm = virDomainFindByUUID(&driver->domains, dom->uuid);
> + openvzDriverUnlock(driver);
> +
> + if (!vm) {
> + openvzError(VIR_ERR_INVALID_DOMAIN, "%s",
> + _("no domain with matching uuid"));
> + goto cleanup;
> + }
> +
> + if (!virDomainObjIsActive(vm)) {
> + openvzError(VIR_ERR_OPERATION_INVALID, "%s",
> + _("Domain is not running"));
> + goto cleanup;
> + }
> +
> + if (vm->state != VIR_DOMAIN_PAUSED) {
> + openvzSetProgramSentinal(prog, vm->def->name);
> + if (virRun(prog, NULL) < 0) {
> + openvzError(VIR_ERR_OPERATION_FAILED, "%s",
> + _("Suspend operation failed"));
> + goto cleanup;
> + }
> + vm->state = VIR_DOMAIN_PAUSED;
> + }
> +
> + ret = 0;
> +
> +cleanup:
> + if (vm)
> + virDomainObjUnlock(vm);
> + return ret;
> +}
> +
> +static int openvzDomainResume(virDomainPtr dom) {
> + struct openvz_driver *driver = dom->conn->privateData;
> + virDomainObjPtr vm;
> + const char *prog[] = {VZCTL, "--quiet", "chkpnt", PROGRAM_SENTINAL, "--resume", NULL};
> + int ret = -1;
> +
> + openvzDriverLock(driver);
> + vm = virDomainFindByUUID(&driver->domains, dom->uuid);
> + openvzDriverUnlock(driver);
> +
> + if (!vm) {
> + openvzError(VIR_ERR_INVALID_DOMAIN, "%s",
> + _("no domain with matching uuid"));
> + goto cleanup;
> + }
> +
> + if (!virDomainObjIsActive(vm)) {
> + openvzError(VIR_ERR_OPERATION_INVALID, "%s",
> + _("Domain is not running"));
> + goto cleanup;
> + }
> +
> + if (vm->state == VIR_DOMAIN_PAUSED) {
> + openvzSetProgramSentinal(prog, vm->def->name);
> + if (virRun(prog, NULL) < 0) {
> + openvzError(VIR_ERR_OPERATION_FAILED, "%s",
> + _("Resume operation failed"));
> + goto cleanup;
> + }
> + vm->state = VIR_DOMAIN_RUNNING;
> + }
> + ret = 0;
> +
> +cleanup:
> + if (vm)
> + virDomainObjUnlock(vm);
> + return ret;
> +}
> +
> static int openvzDomainShutdown(virDomainPtr dom) {
> struct openvz_driver *driver = dom->conn->privateData;
> virDomainObjPtr vm;
> @@ -1491,8 +1571,8 @@ static virDriver openvzDriver = {
> openvzDomainLookupByID, /* domainLookupByID */
> openvzDomainLookupByUUID, /* domainLookupByUUID */
> openvzDomainLookupByName, /* domainLookupByName */
> - NULL, /* domainSuspend */
> - NULL, /* domainResume */
> + openvzDomainSuspend, /* domainSuspend */
> + openvzDomainResume, /* domainResume */
> openvzDomainShutdown, /* domainShutdown */
> openvzDomainReboot, /* domainReboot */
> openvzDomainShutdown, /* domainDestroy */
This patch looks fine, simple enough and should not introduce
regressions, so I'm pushing it to be part of 0.8.3
thanks !
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel at veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
More information about the libvir-list
mailing list