1 From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
2 From: Dietmar Maurer <dietmar@proxmox.com>
3 Date: Mon, 6 Apr 2020 12:16:46 +0200
4 Subject: [PATCH] PVE: add savevm-async for background state snapshots
6 Put qemu_savevm_state_{header,setup} into the main loop and the rest
7 of the iteration into a coroutine. The former need to lock the
8 iothread (and we can't unlock it in the coroutine), and the latter
9 can't deal with being in a separate thread, so a coroutine it must
12 Truncate output file at 1024 boundary.
14 Do not block the VM and save the state on aborting a snapshot, as the
15 snapshot will be invalid anyway.
17 Also, when aborting, wait for the target file to be closed, otherwise a
18 client might run into race-conditions when trying to remove the file
21 Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
22 Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>
23 Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
25 register yank before migration_incoming_state_destroy]
26 Signed-off-by: Stefan Reiter <s.reiter@proxmox.com>
27 [FE: further improve aborting
28 adapt to removal of QEMUFileOps
29 improve condition for entering final stage
30 adapt to QAPI and other changes for 8.0]
31 Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
33 hmp-commands-info.hx | 13 +
34 hmp-commands.hx | 33 +++
35 include/migration/snapshot.h | 2 +
36 include/monitor/hmp.h | 5 +
37 migration/meson.build | 1 +
38 migration/savevm-async.c | 548 +++++++++++++++++++++++++++++++++++
39 monitor/hmp-cmds.c | 58 ++++
40 qapi/migration.json | 34 +++
41 qapi/misc.json | 32 ++
42 qemu-options.hx | 12 +
44 11 files changed, 748 insertions(+)
45 create mode 100644 migration/savevm-async.c
47 diff --git a/hmp-commands-info.hx b/hmp-commands-info.hx
48 index 47d63d26db..a166bff3d5 100644
49 --- a/hmp-commands-info.hx
50 +++ b/hmp-commands-info.hx
51 @@ -540,6 +540,19 @@ SRST
52 Show current migration parameters.
59 + .help = "show savevm status",
60 + .cmd = hmp_info_savevm,
71 diff --git a/hmp-commands.hx b/hmp-commands.hx
72 index bb85ee1d26..b66d7fc4ab 100644
75 @@ -1846,3 +1846,36 @@ SRST
76 List event channels in the guest
81 + .name = "savevm-start",
82 + .args_type = "statefile:s?",
83 + .params = "[statefile]",
84 + .help = "Prepare for snapshot and halt VM. Save VM state to statefile.",
85 + .cmd = hmp_savevm_start,
89 + .name = "snapshot-drive",
90 + .args_type = "device:s,name:s",
91 + .params = "device name",
92 + .help = "Create internal snapshot.",
93 + .cmd = hmp_snapshot_drive,
97 + .name = "delete-drive-snapshot",
98 + .args_type = "device:s,name:s",
99 + .params = "device name",
100 + .help = "Delete internal snapshot.",
101 + .cmd = hmp_delete_drive_snapshot,
105 + .name = "savevm-end",
108 + .help = "Resume VM after snaphot.",
109 + .cmd = hmp_savevm_end,
112 diff --git a/include/migration/snapshot.h b/include/migration/snapshot.h
113 index e72083b117..c846d37806 100644
114 --- a/include/migration/snapshot.h
115 +++ b/include/migration/snapshot.h
116 @@ -61,4 +61,6 @@ bool delete_snapshot(const char *name,
117 bool has_devices, strList *devices,
120 +int load_snapshot_from_blockdev(const char *filename, Error **errp);
123 diff --git a/include/monitor/hmp.h b/include/monitor/hmp.h
124 index fdb69b7f9c..c012bad741 100644
125 --- a/include/monitor/hmp.h
126 +++ b/include/monitor/hmp.h
127 @@ -28,6 +28,7 @@ void hmp_info_status(Monitor *mon, const QDict *qdict);
128 void hmp_info_uuid(Monitor *mon, const QDict *qdict);
129 void hmp_info_chardev(Monitor *mon, const QDict *qdict);
130 void hmp_info_mice(Monitor *mon, const QDict *qdict);
131 +void hmp_info_savevm(Monitor *mon, const QDict *qdict);
132 void hmp_info_migrate(Monitor *mon, const QDict *qdict);
133 void hmp_info_migrate_capabilities(Monitor *mon, const QDict *qdict);
134 void hmp_info_migrate_parameters(Monitor *mon, const QDict *qdict);
135 @@ -94,6 +95,10 @@ void hmp_closefd(Monitor *mon, const QDict *qdict);
136 void hmp_mouse_move(Monitor *mon, const QDict *qdict);
137 void hmp_mouse_button(Monitor *mon, const QDict *qdict);
138 void hmp_mouse_set(Monitor *mon, const QDict *qdict);
139 +void hmp_savevm_start(Monitor *mon, const QDict *qdict);
140 +void hmp_snapshot_drive(Monitor *mon, const QDict *qdict);
141 +void hmp_delete_drive_snapshot(Monitor *mon, const QDict *qdict);
142 +void hmp_savevm_end(Monitor *mon, const QDict *qdict);
143 void hmp_sendkey(Monitor *mon, const QDict *qdict);
144 void coroutine_fn hmp_screendump(Monitor *mon, const QDict *qdict);
145 void hmp_chardev_add(Monitor *mon, const QDict *qdict);
146 diff --git a/migration/meson.build b/migration/meson.build
147 index 8a142fc7a9..a7824b5266 100644
148 --- a/migration/meson.build
149 +++ b/migration/meson.build
150 @@ -25,6 +25,7 @@ softmmu_ss.add(files(
158 diff --git a/migration/savevm-async.c b/migration/savevm-async.c
160 index 0000000000..c5db9e9c1e
162 +++ b/migration/savevm-async.c
164 +#include "qemu/osdep.h"
165 +#include "migration/channel-savevm-async.h"
166 +#include "migration/migration.h"
167 +#include "migration/savevm.h"
168 +#include "migration/snapshot.h"
169 +#include "migration/global_state.h"
170 +#include "migration/ram.h"
171 +#include "migration/qemu-file.h"
172 +#include "sysemu/sysemu.h"
173 +#include "sysemu/runstate.h"
174 +#include "block/block.h"
175 +#include "sysemu/block-backend.h"
176 +#include "qapi/error.h"
177 +#include "qapi/qmp/qerror.h"
178 +#include "qapi/qmp/qdict.h"
179 +#include "qapi/qapi-commands-migration.h"
180 +#include "qapi/qapi-commands-misc.h"
181 +#include "qapi/qapi-commands-block.h"
182 +#include "qemu/cutils.h"
183 +#include "qemu/timer.h"
184 +#include "qemu/main-loop.h"
185 +#include "qemu/rcu.h"
186 +#include "qemu/yank.h"
188 +/* #define DEBUG_SAVEVM_STATE */
190 +#ifdef DEBUG_SAVEVM_STATE
191 +#define DPRINTF(fmt, ...) \
192 + do { printf("savevm-async: " fmt, ## __VA_ARGS__); } while (0)
194 +#define DPRINTF(fmt, ...) \
202 + SAVE_STATE_COMPLETED,
203 + SAVE_STATE_CANCELLED
207 +static struct SnapshotState {
208 + BlockBackend *target;
213 + int saved_vm_running;
215 + int64_t total_time;
216 + QEMUBH *finalize_bh;
218 + QemuCoSleep target_close_wait;
221 +static bool savevm_aborted(void)
223 + return snap_state.state == SAVE_STATE_CANCELLED ||
224 + snap_state.state == SAVE_STATE_ERROR;
227 +SaveVMInfo *qmp_query_savevm(Error **errp)
229 + SaveVMInfo *info = g_malloc0(sizeof(*info));
230 + struct SnapshotState *s = &snap_state;
232 + if (s->state != SAVE_STATE_DONE) {
233 + info->has_bytes = true;
234 + info->bytes = s->bs_pos;
235 + switch (s->state) {
236 + case SAVE_STATE_ERROR:
237 + info->status = g_strdup("failed");
238 + info->has_total_time = true;
239 + info->total_time = s->total_time;
241 + info->error = g_strdup(error_get_pretty(s->error));
244 + case SAVE_STATE_ACTIVE:
245 + info->status = g_strdup("active");
246 + info->has_total_time = true;
247 + info->total_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME)
250 + case SAVE_STATE_COMPLETED:
251 + info->status = g_strdup("completed");
252 + info->has_total_time = true;
253 + info->total_time = s->total_time;
261 +static int save_snapshot_cleanup(void)
265 + DPRINTF("save_snapshot_cleanup\n");
267 + snap_state.total_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME) -
268 + snap_state.total_time;
270 + if (snap_state.file) {
271 + ret = qemu_fclose(snap_state.file);
272 + snap_state.file = NULL;
275 + if (snap_state.target) {
276 + if (!savevm_aborted()) {
277 + /* try to truncate, but ignore errors (will fail on block devices).
278 + * note1: bdrv_read() need whole blocks, so we need to round up
279 + * note2: PVE requires 1024 (BDRV_SECTOR_SIZE*2) alignment
281 + size_t size = QEMU_ALIGN_UP(snap_state.bs_pos, BDRV_SECTOR_SIZE*2);
282 + blk_truncate(snap_state.target, size, false, PREALLOC_MODE_OFF, 0, NULL);
284 + blk_op_unblock_all(snap_state.target, snap_state.blocker);
285 + error_free(snap_state.blocker);
286 + snap_state.blocker = NULL;
287 + blk_unref(snap_state.target);
288 + snap_state.target = NULL;
290 + qemu_co_sleep_wake(&snap_state.target_close_wait);
296 +static void save_snapshot_error(const char *fmt, ...)
302 + msg = g_strdup_vprintf(fmt, ap);
305 + DPRINTF("save_snapshot_error: %s\n", msg);
307 + if (!snap_state.error) {
308 + error_set(&snap_state.error, ERROR_CLASS_GENERIC_ERROR, "%s", msg);
313 + snap_state.state = SAVE_STATE_ERROR;
316 +static void process_savevm_finalize(void *opaque)
319 + AioContext *iohandler_ctx = iohandler_get_aio_context();
320 + MigrationState *ms = migrate_get_current();
322 + bool aborted = savevm_aborted();
324 +#ifdef DEBUG_SAVEVM_STATE
325 + int64_t start_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME);
328 + qemu_bh_delete(snap_state.finalize_bh);
329 + snap_state.finalize_bh = NULL;
330 + snap_state.co = NULL;
332 + /* We need to own the target bdrv's context for the following functions,
333 + * so move it back. It can stay in the main context and live out its live
334 + * there, since we're done with it after this method ends anyway.
336 + aio_context_acquire(iohandler_ctx);
337 + blk_set_aio_context(snap_state.target, qemu_get_aio_context(), NULL);
338 + aio_context_release(iohandler_ctx);
340 + ret = vm_stop_force_state(RUN_STATE_FINISH_MIGRATE);
342 + save_snapshot_error("vm_stop_force_state error %d", ret);
346 + /* skip state saving if we aborted, snapshot will be invalid anyway */
347 + (void)qemu_savevm_state_complete_precopy(snap_state.file, false, false);
348 + ret = qemu_file_get_error(snap_state.file);
350 + save_snapshot_error("qemu_savevm_state_complete_precopy error %d", ret);
354 + DPRINTF("state saving complete\n");
355 + DPRINTF("timing: process_savevm_finalize (state saving) took %ld ms\n",
356 + qemu_clock_get_ms(QEMU_CLOCK_REALTIME) - start_time);
358 + /* clear migration state */
359 + migrate_set_state(&ms->state, MIGRATION_STATUS_SETUP,
360 + ret || aborted ? MIGRATION_STATUS_FAILED : MIGRATION_STATUS_COMPLETED);
361 + ms->to_dst_file = NULL;
363 + qemu_savevm_state_cleanup();
365 + ret = save_snapshot_cleanup();
367 + save_snapshot_error("save_snapshot_cleanup error %d", ret);
368 + } else if (snap_state.state == SAVE_STATE_ACTIVE) {
369 + snap_state.state = SAVE_STATE_COMPLETED;
370 + } else if (aborted) {
372 + * If there was an error, there's no need to set a new one here.
373 + * If the snapshot was canceled, leave setting the state to
374 + * qmp_savevm_end(), which is waked by save_snapshot_cleanup().
377 + save_snapshot_error("process_savevm_cleanup: invalid state: %d",
380 + if (snap_state.saved_vm_running) {
382 + snap_state.saved_vm_running = false;
385 + DPRINTF("timing: process_savevm_finalize (full) took %ld ms\n",
386 + qemu_clock_get_ms(QEMU_CLOCK_REALTIME) - start_time);
389 +static void coroutine_fn process_savevm_co(void *opaque)
393 + BdrvNextIterator it;
394 + BlockDriverState *bs = NULL;
396 +#ifdef DEBUG_SAVEVM_STATE
397 + int64_t start_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME);
400 + ret = qemu_file_get_error(snap_state.file);
402 + save_snapshot_error("qemu_savevm_state_setup failed");
406 + while (snap_state.state == SAVE_STATE_ACTIVE) {
407 + uint64_t pending_size, pend_precopy, pend_postcopy;
408 + uint64_t threshold = 400 * 1000;
411 + * pending_{estimate,exact} are expected to be called without iothread
412 + * lock. Similar to what is done in migration.c, call the exact variant
413 + * only once pend_precopy in the estimate is below the threshold.
415 + qemu_mutex_unlock_iothread();
416 + qemu_savevm_state_pending_estimate(&pend_precopy, &pend_postcopy);
417 + if (pend_precopy <= threshold) {
418 + qemu_savevm_state_pending_exact(&pend_precopy, &pend_postcopy);
420 + qemu_mutex_lock_iothread();
421 + pending_size = pend_precopy + pend_postcopy;
424 + * A guest reaching this cutoff is dirtying lots of RAM. It should be
425 + * large enough so that the guest can't dirty this much between the
426 + * check and the guest actually being stopped, but it should be small
427 + * enough to avoid long downtimes for non-hibernation snapshots.
429 + maxlen = blk_getlength(snap_state.target) - 100*1024*1024;
431 + /* Note that there is no progress for pend_postcopy when iterating */
432 + if (pend_precopy > threshold && snap_state.bs_pos + pending_size < maxlen) {
433 + ret = qemu_savevm_state_iterate(snap_state.file, false);
435 + save_snapshot_error("qemu_savevm_state_iterate error %d", ret);
438 + DPRINTF("savevm iterate pending size %lu ret %d\n", pending_size, ret);
440 + qemu_system_wakeup_request(QEMU_WAKEUP_REASON_OTHER, NULL);
441 + ret = global_state_store();
443 + save_snapshot_error("global_state_store error %d", ret);
447 + DPRINTF("savevm iterate complete\n");
452 + DPRINTF("timing: process_savevm_co took %ld ms\n",
453 + qemu_clock_get_ms(QEMU_CLOCK_REALTIME) - start_time);
455 +#ifdef DEBUG_SAVEVM_STATE
456 + int64_t start_time_flush = qemu_clock_get_ms(QEMU_CLOCK_REALTIME);
458 + /* If a drive runs in an IOThread we can flush it async, and only
459 + * need to sync-flush whatever IO happens between now and
460 + * vm_stop_force_state. bdrv_next can only be called from main AioContext,
461 + * so move there now and after every flush.
463 + aio_co_reschedule_self(qemu_get_aio_context());
464 + for (bs = bdrv_first(&it); bs; bs = bdrv_next(&it)) {
465 + /* target has BDRV_O_NO_FLUSH, no sense calling bdrv_flush on it */
466 + if (bs == blk_bs(snap_state.target)) {
470 + AioContext *bs_ctx = bdrv_get_aio_context(bs);
471 + if (bs_ctx != qemu_get_aio_context()) {
472 + DPRINTF("savevm: async flushing drive %s\n", bs->filename);
473 + aio_co_reschedule_self(bs_ctx);
474 + bdrv_graph_co_rdlock();
476 + bdrv_graph_co_rdunlock();
477 + aio_co_reschedule_self(qemu_get_aio_context());
481 + DPRINTF("timing: async flushing took %ld ms\n",
482 + qemu_clock_get_ms(QEMU_CLOCK_REALTIME) - start_time_flush);
484 + qemu_bh_schedule(snap_state.finalize_bh);
487 +void qmp_savevm_start(const char *statefile, Error **errp)
489 + Error *local_err = NULL;
490 + MigrationState *ms = migrate_get_current();
491 + AioContext *iohandler_ctx = iohandler_get_aio_context();
493 + int bdrv_oflags = BDRV_O_RDWR | BDRV_O_RESIZE | BDRV_O_NO_FLUSH;
495 + if (snap_state.state != SAVE_STATE_DONE) {
496 + error_set(errp, ERROR_CLASS_GENERIC_ERROR,
497 + "VM snapshot already started\n");
501 + if (migration_is_running(ms->state)) {
502 + error_set(errp, ERROR_CLASS_GENERIC_ERROR, QERR_MIGRATION_ACTIVE);
506 + if (migrate_use_block()) {
507 + error_set(errp, ERROR_CLASS_GENERIC_ERROR,
508 + "Block migration and snapshots are incompatible");
512 + /* initialize snapshot info */
513 + snap_state.saved_vm_running = runstate_is_running();
514 + snap_state.bs_pos = 0;
515 + snap_state.total_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME);
516 + snap_state.blocker = NULL;
517 + snap_state.target_close_wait = (QemuCoSleep){ .to_wake = NULL };
519 + if (snap_state.error) {
520 + error_free(snap_state.error);
521 + snap_state.error = NULL;
525 + vm_stop(RUN_STATE_SAVE_VM);
526 + snap_state.state = SAVE_STATE_COMPLETED;
530 + if (qemu_savevm_state_blocked(errp)) {
534 + /* Open the image */
535 + QDict *options = NULL;
536 + options = qdict_new();
537 + qdict_put_str(options, "driver", "raw");
538 + snap_state.target = blk_new_open(statefile, NULL, options, bdrv_oflags, &local_err);
539 + if (!snap_state.target) {
540 + error_set(errp, ERROR_CLASS_GENERIC_ERROR, "failed to open '%s'", statefile);
544 + QIOChannel *ioc = QIO_CHANNEL(qio_channel_savevm_async_new(snap_state.target,
545 + &snap_state.bs_pos));
546 + snap_state.file = qemu_file_new_output(ioc);
548 + if (!snap_state.file) {
549 + error_set(errp, ERROR_CLASS_GENERIC_ERROR, "failed to open '%s'", statefile);
554 + * qemu_savevm_* paths use migration code and expect a migration state.
555 + * State is cleared in process_savevm_co, but has to be initialized
556 + * here (blocking main thread, from QMP) to avoid race conditions.
559 + memset(&ram_counters, 0, sizeof(ram_counters));
560 + memset(&compression_counters, 0, sizeof(compression_counters));
561 + ms->to_dst_file = snap_state.file;
563 + error_setg(&snap_state.blocker, "block device is in use by savevm");
564 + blk_op_block_all(snap_state.target, snap_state.blocker);
566 + snap_state.state = SAVE_STATE_ACTIVE;
567 + snap_state.finalize_bh = qemu_bh_new(process_savevm_finalize, &snap_state);
568 + snap_state.co = qemu_coroutine_create(&process_savevm_co, NULL);
569 + qemu_mutex_unlock_iothread();
570 + qemu_savevm_state_header(snap_state.file);
571 + qemu_savevm_state_setup(snap_state.file);
572 + qemu_mutex_lock_iothread();
574 + /* Async processing from here on out happens in iohandler context, so let
575 + * the target bdrv have its home there.
577 + blk_set_aio_context(snap_state.target, iohandler_ctx, &local_err);
579 + aio_co_schedule(iohandler_ctx, snap_state.co);
585 + save_snapshot_error("setup failed");
587 + if (snap_state.saved_vm_running) {
589 + snap_state.saved_vm_running = false;
593 +void coroutine_fn qmp_savevm_end(Error **errp)
597 + if (snap_state.state == SAVE_STATE_DONE) {
598 + error_set(errp, ERROR_CLASS_GENERIC_ERROR,
599 + "VM snapshot not started\n");
603 + if (snap_state.state == SAVE_STATE_ACTIVE) {
604 + snap_state.state = SAVE_STATE_CANCELLED;
605 + goto wait_for_close;
608 + if (snap_state.saved_vm_running) {
610 + snap_state.saved_vm_running = false;
613 + snap_state.state = SAVE_STATE_DONE;
616 + if (!snap_state.target) {
617 + DPRINTF("savevm-end: no target file open\n");
621 + /* wait until cleanup is done before returning, this ensures that after this
622 + * call exits the statefile will be closed and can be removed immediately */
623 + DPRINTF("savevm-end: waiting for cleanup\n");
624 + timeout = 30L * 1000 * 1000 * 1000;
625 + qemu_co_sleep_ns_wakeable(&snap_state.target_close_wait,
626 + QEMU_CLOCK_REALTIME, timeout);
627 + if (snap_state.target) {
628 + save_snapshot_error("timeout waiting for target file close in "
630 + /* we cannot assume the snapshot finished in this case, so leave the
631 + * state alone - caller has to figure something out */
635 + // File closed and no other error, so ensure next snapshot can be started.
636 + if (snap_state.state != SAVE_STATE_ERROR) {
637 + snap_state.state = SAVE_STATE_DONE;
640 + DPRINTF("savevm-end: cleanup done\n");
643 +// FIXME: Deprecated
644 +void qmp_snapshot_drive(const char *device, const char *name, Error **errp)
646 + // Compatibility to older qemu-server.
647 + qmp_blockdev_snapshot_internal_sync(device, name, errp);
650 +// FIXME: Deprecated
651 +void qmp_delete_drive_snapshot(const char *device, const char *name,
654 + // Compatibility to older qemu-server.
655 + (void)qmp_blockdev_snapshot_delete_internal_sync(device, NULL, name, errp);
658 +int load_snapshot_from_blockdev(const char *filename, Error **errp)
661 + Error *local_err = NULL;
662 + Error *blocker = NULL;
668 + be = blk_new_open(filename, NULL, NULL, 0, &local_err);
671 + error_setg(errp, "Could not open VM state file");
675 + error_setg(&blocker, "block device is in use by load state");
676 + blk_op_block_all(be, blocker);
678 + /* restore the VM state */
679 + f = qemu_file_new_input(QIO_CHANNEL(qio_channel_savevm_async_new(be, &bs_pos)));
681 + error_setg(errp, "Could not open VM state file");
685 + qemu_system_reset(SHUTDOWN_CAUSE_NONE);
686 + ret = qemu_loadvm_state(f);
688 + /* dirty bitmap migration has a special case we need to trigger manually */
689 + dirty_bitmap_mig_before_vm_start();
693 + /* state_destroy assumes a real migration which would have added a yank */
694 + yank_register_instance(MIGRATION_YANK_INSTANCE, &error_abort);
696 + migration_incoming_state_destroy();
698 + error_setg_errno(errp, -ret, "Error while loading VM state");
706 + blk_op_unblock_all(be, blocker);
707 + error_free(blocker);
712 diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c
713 index 6c559b48c8..435f9334f9 100644
714 --- a/monitor/hmp-cmds.c
715 +++ b/monitor/hmp-cmds.c
717 #include "monitor/monitor-internal.h"
718 #include "qapi/error.h"
719 #include "qapi/qapi-commands-control.h"
720 +#include "qapi/qapi-commands-migration.h"
721 #include "qapi/qapi-commands-misc.h"
722 #include "qapi/qmp/qdict.h"
723 #include "qapi/qmp/qerror.h"
724 @@ -443,3 +444,60 @@ void hmp_info_mtree(Monitor *mon, const QDict *qdict)
726 mtree_info(flatview, dispatch_tree, owner, disabled);
729 +void hmp_savevm_start(Monitor *mon, const QDict *qdict)
731 + Error *errp = NULL;
732 + const char *statefile = qdict_get_try_str(qdict, "statefile");
734 + qmp_savevm_start(statefile, &errp);
735 + hmp_handle_error(mon, errp);
738 +void hmp_snapshot_drive(Monitor *mon, const QDict *qdict)
740 + Error *errp = NULL;
741 + const char *name = qdict_get_str(qdict, "name");
742 + const char *device = qdict_get_str(qdict, "device");
744 + qmp_snapshot_drive(device, name, &errp);
745 + hmp_handle_error(mon, errp);
748 +void hmp_delete_drive_snapshot(Monitor *mon, const QDict *qdict)
750 + Error *errp = NULL;
751 + const char *name = qdict_get_str(qdict, "name");
752 + const char *device = qdict_get_str(qdict, "device");
754 + qmp_delete_drive_snapshot(device, name, &errp);
755 + hmp_handle_error(mon, errp);
758 +void coroutine_fn hmp_savevm_end(Monitor *mon, const QDict *qdict)
760 + Error *errp = NULL;
762 + qmp_savevm_end(&errp);
763 + hmp_handle_error(mon, errp);
766 +void hmp_info_savevm(Monitor *mon, const QDict *qdict)
769 + info = qmp_query_savevm(NULL);
771 + if (info->status) {
772 + monitor_printf(mon, "savevm status: %s\n", info->status);
773 + monitor_printf(mon, "total time: %" PRIu64 " milliseconds\n",
776 + monitor_printf(mon, "savevm status: not running\n");
778 + if (info->has_bytes) {
779 + monitor_printf(mon, "Bytes saved: %"PRIu64"\n", info->bytes);
782 + monitor_printf(mon, "Error: %s\n", info->error);
785 diff --git a/qapi/migration.json b/qapi/migration.json
786 index c84fa10e86..1702b92553 100644
787 --- a/qapi/migration.json
788 +++ b/qapi/migration.json
790 '*compression': 'CompressionStats',
791 '*socket-address': ['SocketAddress'] } }
796 +# Information about current migration process.
798 +# @status: string describing the current savevm status.
799 +# This can be 'active', 'completed', 'failed'.
800 +# If this field is not returned, no savevm process
801 +# has been initiated
803 +# @error: string containing error message is status is failed.
805 +# @total-time: total amount of milliseconds since savevm started.
806 +# If savevm has ended, it returns the total save time
808 +# @bytes: total amount of data transfered
812 +{ 'struct': 'SaveVMInfo',
813 + 'data': {'*status': 'str', '*error': 'str',
814 + '*total-time': 'int', '*bytes': 'int'} }
819 +# Returns information about current savevm process.
821 +# Returns: @SaveVMInfo
825 +{ 'command': 'query-savevm', 'returns': 'SaveVMInfo' }
830 diff --git a/qapi/misc.json b/qapi/misc.json
831 index 6ddd16ea28..098c9bbe93 100644
836 { 'command': 'query-fdsets', 'returns': ['FdsetInfo'] }
841 +# Prepare for snapshot and halt VM. Save VM state to statefile.
844 +{ 'command': 'savevm-start', 'data': { '*statefile': 'str' } }
849 +# Create an internal drive snapshot.
852 +{ 'command': 'snapshot-drive', 'data': { 'device': 'str', 'name': 'str' } }
855 +# @delete-drive-snapshot:
857 +# Delete a drive snapshot.
860 +{ 'command': 'delete-drive-snapshot', 'data': { 'device': 'str', 'name': 'str' } }
865 +# Resume VM after a snapshot.
868 +{ 'command': 'savevm-end', 'coroutine': true }
871 # @CommandLineParameterType:
873 diff --git a/qemu-options.hx b/qemu-options.hx
874 index 59bdf67a2c..fc6cb23dd9 100644
875 --- a/qemu-options.hx
876 +++ b/qemu-options.hx
877 @@ -4378,6 +4378,18 @@ SRST
878 Start right away with a saved state (``loadvm`` in monitor)
881 +DEF("loadstate", HAS_ARG, QEMU_OPTION_loadstate, \
882 + "-loadstate file\n" \
883 + " start right away with a saved state\n",
887 + Start right away with a saved state. This option does not rollback
888 + disk state like @code{loadvm}, so user must make sure that disk
889 + have correct state. @var{file} can be any valid device URL. See the section
890 + for "Device URL Syntax" for more information.
894 DEF("daemonize", 0, QEMU_OPTION_daemonize, \
895 "-daemonize daemonize QEMU after initializing\n", QEMU_ARCH_ALL)
896 diff --git a/softmmu/vl.c b/softmmu/vl.c
897 index ea20b23e4c..0eabc71b68 100644
900 @@ -164,6 +164,7 @@ static const char *accelerators;
901 static bool have_custom_ram_size;
902 static const char *ram_memdev_id;
903 static QDict *machine_opts_dict;
904 +static const char *loadstate;
905 static QTAILQ_HEAD(, ObjectOption) object_opts = QTAILQ_HEAD_INITIALIZER(object_opts);
906 static QTAILQ_HEAD(, DeviceOption) device_opts = QTAILQ_HEAD_INITIALIZER(device_opts);
907 static int display_remote;
908 @@ -2612,6 +2613,12 @@ void qmp_x_exit_preconfig(Error **errp)
911 load_snapshot(loadvm, NULL, false, NULL, &error_fatal);
912 + } else if (loadstate) {
913 + Error *local_err = NULL;
914 + if (load_snapshot_from_blockdev(loadstate, &local_err) < 0) {
915 + error_report_err(local_err);
919 if (replay_mode != REPLAY_MODE_NONE) {
920 replay_vmstate_init();
921 @@ -3159,6 +3166,9 @@ void qemu_init(int argc, char **argv)
922 case QEMU_OPTION_loadvm:
925 + case QEMU_OPTION_loadstate:
926 + loadstate = optarg;
928 case QEMU_OPTION_full_screen:
929 dpy.has_full_screen = true;
930 dpy.full_screen = true;