[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 13/13] vm-snapshot-save: add force parameter
From: |
Pavel Hrdina |
Subject: |
[Qemu-devel] [PATCH 13/13] vm-snapshot-save: add force parameter |
Date: |
Wed, 9 Jan 2013 16:18:07 +0100 |
HMP command "savevm" now takes extra optional force parameter to specifi whether
replace existing snapshot or not.
QMP command "vm-snapshot-save" has also extra optional force parameter and
name parameter isn't optional anymore.
Signed-off-by: Pavel Hrdina <address@hidden>
---
hmp-commands.hx | 15 ++++++++-------
hmp.c | 25 ++++++++++++++++++++++++-
qapi-schema.json | 9 ++++++---
qmp-commands.hx | 15 +++++++++------
savevm.c | 36 ++++++++++++++----------------------
5 files changed, 61 insertions(+), 39 deletions(-)
diff --git a/hmp-commands.hx b/hmp-commands.hx
index 3b781f7..d857682 100644
--- a/hmp-commands.hx
+++ b/hmp-commands.hx
@@ -307,18 +307,19 @@ ETEXI
{
.name = "savevm",
- .args_type = "name:s?",
- .params = "[tag|id]",
- .help = "save a VM snapshot. If no tag or id are provided, a new
snapshot is created",
+ .args_type = "force:-f,name:s",
+ .params = "[-f] [tag|id]",
+ .help = "save a VM snapshot. To replace existing snapshot use
force flag.",
.mhandler.cmd = hmp_vm_snapshot_save,
},
STEXI
address@hidden savevm address@hidden|@var{id}]
address@hidden savevm address@hidden @var{tag}|@var{id}
@findex savevm
-Create a snapshot of the whole virtual machine. If @var{tag} is
-provided, it is used as human readable identifier. If there is already
-a snapshot with the same @var{tag} or @var{id}, it is replaced. More info at
+Create a snapshot of the whole virtual machine. Paramtere "name" is optional.
+If @var{tag} is provided, it is used as human readable identifier. If there is
+already a snapshot with the same @var{tag} or @var{id}, @var{-f} flag needs to
+be specified. More info at @ref{vm_snapshots}.
@ref{vm_snapshots}.
ETEXI
diff --git a/hmp.c b/hmp.c
index 2d8600a..02493ca 100644
--- a/hmp.c
+++ b/hmp.c
@@ -1340,8 +1340,31 @@ void hmp_nbd_server_stop(Monitor *mon, const QDict
*qdict)
void hmp_vm_snapshot_save(Monitor *mon, const QDict *qdict)
{
const char *name = qdict_get_try_str(qdict, "name");
+ char new_name[256];
+ bool force = qdict_get_try_bool(qdict, "force", 0);
Error *err = NULL;
+#ifdef _WIN32
+ struct _timeb tb;
+ struct tm *ptm;
+#else
+ struct timeval tv;
+ struct tm tm;
+#endif
+
+ if (!name) {
+#ifdef _WIN32
+ time_t t = tb.time;
+ ptm = localtime(&t);
+ strftime(new_name, sizeof(new_name), "vm-%Y%m%d%H%M%S", ptm);
+#else
+ /* cast below needed for OpenBSD where tv_sec is still 'long' */
+ localtime_r((const time_t *)&tv.tv_sec, &tm);
+ strftime(new_name, sizeof(new_name), "vm-%Y%m%d%H%M%S", &tm);
+#endif
+ } else {
+ pstrcpy(new_name, sizeof(new_name), name);
+ }
- qmp_vm_snapshot_save(!!name, name, &err);
+ qmp_vm_snapshot_save(new_name, !!force, force, &err);
hmp_handle_error(mon, &err);
}
diff --git a/qapi-schema.json b/qapi-schema.json
index a6f4cb0..7f1639c 100644
--- a/qapi-schema.json
+++ b/qapi-schema.json
@@ -3023,15 +3023,18 @@
#
# Create a snapshot of the whole virtual machine. If tag is provided as @name,
# it is used as human readable identifier. If there is already a snapshot
-# with the same tag or ID, it is replaced.
+# with the same tag or id, the force argument needs to be true to replace it.
#
# The VM is automatically stopped and resumed and saving a snapshot can take
# a long time.
#
-# @name: #optional tag of new snapshot or tag|id of existing snapshot
+# @name: tag of new snapshot or tag|id of existing snapshot
+#
+# @force: #optional specify whether existing snapshot is replaced or not,
+# default is false
#
# Returns: Nothing on success
#
# Since: 1.4.0
##
-{ 'command': 'vm-snapshot-save', 'data': {'*name': 'str'} }
+{ 'command': 'vm-snapshot-save', 'data': {'name': 'str', '*force': 'bool'} }
diff --git a/qmp-commands.hx b/qmp-commands.hx
index 9a0b566..1bd16f1 100644
--- a/qmp-commands.hx
+++ b/qmp-commands.hx
@@ -1349,9 +1349,9 @@ Example:
EQMP
{
.name = "vm-snapshot-save",
- .args_type = "name:s?",
- .params = "name",
- .help = "save a VM snapshot. If no tag or id are provided, a new
snapshot is created",
+ .args_type = "name:s,force:b?",
+ .params = "name [force]",
+ .help = "save a VM snapshot. To replace existing snapshot use
force parameter.",
.mhandler.cmd_new = qmp_marshal_input_vm_snapshot_save
},
@@ -1360,15 +1360,18 @@ vm-snapshot-save
------
Create a snapshot of the whole virtual machine. If tag is provided as name,
-it is used as human readable identifier. If there is already a snapshot
-with the same tag or id, it is replaced.
+it is used as human readable identifier. If there is already a snapshot with
+the same tag, the force argument needs to be true to replace it.
The VM is automatically stopped and resumed and saving a snapshot can take
a long time.
Arguments:
-- "name": #optional tag of new snapshot or tag|id of existing snapshot
(json-string, optional)
+- "name": tag of new snapshot or tag|id of existing snapshot (json-string)
+
+- "force": specify whether existing snapshot is replaced or not,
+ default is false (json-bool, optional)
Example:
diff --git a/savevm.c b/savevm.c
index a7254b2..090ab1f 100644
--- a/savevm.c
+++ b/savevm.c
@@ -2102,7 +2102,7 @@ static int del_existing_snapshots(const char *name, Error
**errp)
return 0;
}
-void qmp_vm_snapshot_save(bool has_name, const char *name, Error **errp)
+void qmp_vm_snapshot_save(const char *name, bool has_force, bool force, Error
**errp)
{
BlockDriverState *bs, *bs1;
QEMUSnapshotInfo sn1, *sn = &sn1, old_sn1, *old_sn = &old_sn1;
@@ -2112,10 +2112,8 @@ void qmp_vm_snapshot_save(bool has_name, const char
*name, Error **errp)
uint64_t vm_state_size;
#ifdef _WIN32
struct _timeb tb;
- struct tm *ptm;
#else
struct timeval tv;
- struct tm tm;
#endif
Error *local_err = NULL;
@@ -2157,30 +2155,24 @@ void qmp_vm_snapshot_save(bool has_name, const char
*name, Error **errp)
#endif
sn->vm_clock_nsec = qemu_get_clock_ns(vm_clock);
- if (has_name) {
- ret = bdrv_snapshot_find(bs, old_sn, name);
- if (ret >= 0) {
+ ret = bdrv_snapshot_find(bs, old_sn, name);
+ if (ret >= 0) {
+ if (has_force && force) {
pstrcpy(sn->name, sizeof(sn->name), old_sn->name);
pstrcpy(sn->id_str, sizeof(sn->id_str), old_sn->id_str);
+
+ /* Delete old snapshots of the same name */
+ if (del_existing_snapshots(name, &local_err) < 0) {
+ error_propagate(errp, local_err);
+ goto the_end;
+ }
} else {
- pstrcpy(sn->name, sizeof(sn->name), name);
+ error_setg(errp, "Snapshot '%s' exist. For override add "
+ "'force' parameter.", name);
+ goto the_end;
}
} else {
-#ifdef _WIN32
- time_t t = tb.time;
- ptm = localtime(&t);
- strftime(sn->name, sizeof(sn->name), "vm-%Y%m%d%H%M%S", ptm);
-#else
- /* cast below needed for OpenBSD where tv_sec is still 'long' */
- localtime_r((const time_t *)&tv.tv_sec, &tm);
- strftime(sn->name, sizeof(sn->name), "vm-%Y%m%d%H%M%S", &tm);
-#endif
- }
-
- /* Delete old snapshots of the same name */
- if (has_name && del_existing_snapshots(name, &local_err) < 0) {
- error_propagate(errp, local_err);
- goto the_end;
+ pstrcpy(sn->name, sizeof(sn->name), name);
}
/* save the VM state */
--
1.8.1
- Re: [Qemu-devel] [PATCH 07/13] qapi: Convert savevm, (continued)
- [Qemu-devel] [PATCH 10/13] savevm: update return value from qemu_savevm_state_begin, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 08/13] qemu-img: introduce qemu_img_handle_error, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 09/13] block: update return value from bdrv_snapshot_create, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 06/13] savevm: add error parameter to qemu_savevm_state(), Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 11/13] savevm: update return value from qemu_savevm_state_complete, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 12/13] savevm: update return value from qemu_savevm_state, Pavel Hrdina, 2013/01/09
- [Qemu-devel] [PATCH 13/13] vm-snapshot-save: add force parameter,
Pavel Hrdina <=
- Re: [Qemu-devel] [PATCH 00/13] convert savevm to use qapi and introduce qmp command, Pavel Hrdina, 2013/01/09