From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56144) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XQ1XD-0005yd-2A for qemu-devel@nongnu.org; Fri, 05 Sep 2014 17:59:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XQ1X7-0005vW-VV for qemu-devel@nongnu.org; Fri, 05 Sep 2014 17:59:11 -0400 Received: from smtp02.citrix.com ([66.165.176.63]:26071) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XQ1X7-0005vB-Oq for qemu-devel@nongnu.org; Fri, 05 Sep 2014 17:59:05 -0400 Date: Fri, 5 Sep 2014 22:57:22 +0100 From: Stefano Stabellini In-Reply-To: <1409909158-19243-24-git-send-email-wency@cn.fujitsu.com> Message-ID: References: <1409909158-19243-1-git-send-email-wency@cn.fujitsu.com> <1409909158-19243-24-git-send-email-wency@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Subject: Re: [Qemu-devel] [Xen-devel] [RFC Patch v3 23/22] Introduce "xen-load-devices-state" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Wen Congyang Cc: Ian Campbell , Stefano Stabellini , Ian Jackson , Jiang Yunhong , Dong Eddie , qemu-devel@nongnu.org, xen devel , Yang Hongyang , Lai Jiangshan On Fri, 5 Sep 2014, Wen Congyang wrote: > introduce a "xen-load-devices-state" QAPI command that can be used to load > the state of all devices, but not the RAM or the block devices of the > VM. Hello Wen, please CC qemu-devel too for QEMU patches. Could you please explain why do you need this command? Is the main issue that there are no QMP commands today to load the state of a VM? It looks like that for vm restore we are using the -incoming command line option, but there is no alternative over QMP. [...] > diff --git a/savevm.c b/savevm.c > index 22123be..c6aa502 100644 > --- a/savevm.c > +++ b/savevm.c > @@ -863,6 +863,105 @@ out: > return ret; > } > > +static int qemu_load_devices_state(QEMUFile *f) > +{ > + uint8_t section_type; > + unsigned int v; > + int ret; > + > + if (qemu_savevm_state_blocked(NULL)) { > + return -EINVAL; > + } > + > + v = qemu_get_be32(f); > + if (v != QEMU_VM_FILE_MAGIC) { > + return -EINVAL; > + } > + > + v = qemu_get_be32(f); > + if (v == QEMU_VM_FILE_VERSION_COMPAT) { > + fprintf(stderr, "SaveVM v2 format is obsolete and don't work anymore\n"); > + return -ENOTSUP; > + } > + if (v != QEMU_VM_FILE_VERSION) { > + return -ENOTSUP; > + } > + > + while ((section_type = qemu_get_byte(f)) != QEMU_VM_EOF) { > + uint32_t instance_id, version_id, section_id; > + SaveStateEntry *se; > + char idstr[257]; > + int len; > + > + switch (section_type) { > + case QEMU_VM_SECTION_FULL: > + /* Read section start */ > + section_id = qemu_get_be32(f); > + len = qemu_get_byte(f); > + qemu_get_buffer(f, (uint8_t *)idstr, len); > + idstr[len] = 0; > + instance_id = qemu_get_be32(f); > + version_id = qemu_get_be32(f); > + > + /* Find savevm section */ > + se = find_se(idstr, instance_id); > + if (se == NULL) { > + fprintf(stderr, "Unknown savevm section or instance '%s' %d\n", > + idstr, instance_id); > + ret = -EINVAL; > + goto out; > + } > + > + /* Validate version */ > + if (version_id > se->version_id) { > + fprintf(stderr, "loadvm: unsupported version %d for '%s' v%d\n", > + version_id, idstr, se->version_id); > + ret = -EINVAL; > + goto out; > + } > + > + /* Validate if it is a device's state */ > + if (se->is_ram) { > + fprintf(stderr, "loadvm: %s is not devices state\n", idstr); > + ret = -EINVAL; > + goto out; > + } > + > + ret = vmstate_load(f, se, version_id); > + if (ret < 0) { > + fprintf(stderr, "qemu: warning: error while loading state for instance 0x%x of device '%s'\n", > + instance_id, idstr); > + goto out; > + } > + break; > + case QEMU_VM_SECTION_START: > + case QEMU_VM_SECTION_PART: > + case QEMU_VM_SECTION_END: > + /* > + * The file is saved by the command xen-save-devices-state, > + * So it should not contain section start/part/end. > + */ > + default: > + fprintf(stderr, "Unknown savevm section type %d\n", section_type); > + ret = -EINVAL; > + goto out; > + } > + } > + > + cpu_synchronize_all_post_init(); > + > + ret = 0; > + > +out: > + if (ret == 0) { > + if (qemu_file_get_error(f)) { > + ret = -EIO; > + } > + } > + > + return ret; > +} Assuming that the state file only contains device states, why don't you just call qemu_loadvm_state to implement the command? > static BlockDriverState *find_vmstate_bs(void) > { > BlockDriverState *bs = NULL; > @@ -1027,6 +1126,33 @@ void qmp_xen_save_devices_state(const char *filename, Error **errp) > } > } > > +void qmp_xen_load_devices_state(const char *filename, Error **errp) > +{ > + QEMUFile *f; > + int saved_vm_running; > + int ret; > + > + saved_vm_running = runstate_is_running(); > + vm_stop(RUN_STATE_RESTORE_VM); > + > + f = qemu_fopen(filename, "rb"); > + if (!f) { > + error_setg_file_open(errp, errno, filename); > + goto out; > + } > + > + ret = qemu_load_devices_state(f); > + qemu_fclose(f); > + if (ret < 0) { > + error_set(errp, QERR_IO_ERROR); > + } > + > +out: > + if (saved_vm_running) { > + vm_start(); > + } > +} > + > int load_vmstate(const char *name) > { > BlockDriverState *bs, *bs_vm_state; > -- > 1.9.0 > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel >