diff options
| author | Stefan Hajnoczi | 2014-01-29 12:17:31 +0100 |
|---|---|---|
| committer | Luiz Capitulino | 2014-02-17 17:57:23 +0100 |
| commit | ff9ec34de8f6a37bd29ac72c0c4c94bd5d43d7b0 (patch) | |
| tree | fbca8fa2816d803a682a034072c5b1f772c1cf6c /scripts/qmp | |
| parent | hmp: migrate command (without -d) now blocks correctly (diff) | |
| download | qemu-ff9ec34de8f6a37bd29ac72c0c4c94bd5d43d7b0.tar.gz qemu-ff9ec34de8f6a37bd29ac72c0c4c94bd5d43d7b0.tar.xz qemu-ff9ec34de8f6a37bd29ac72c0c4c94bd5d43d7b0.zip | |
QMP: allow JSON dict arguments in qmp-shell
qmp-shell hides the QMP wire protocol JSON encoding from the user. Most
of the time this is helpful and makes the command-line human-friendly.
Some QMP commands take a dict as an argument. In order to express this
we need to revert back to JSON notation.
This patch allows JSON dict arguments in qmp-shell so commands like
blockdev-add and nbd-server-start can be invoked:
(QEMU) blockdev-add options={"driver":"file","id":"drive1",...}
Note that spaces are not allowed since str.split() is used to break up
the command-line arguments first.
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
Diffstat (limited to 'scripts/qmp')
| -rwxr-xr-x | scripts/qmp/qmp-shell | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/scripts/qmp/qmp-shell b/scripts/qmp/qmp-shell index d6b420f18a..d374b357aa 100755 --- a/scripts/qmp/qmp-shell +++ b/scripts/qmp/qmp-shell @@ -31,6 +31,7 @@ # (QEMU) import qmp +import json import readline import sys import pprint @@ -107,6 +108,8 @@ class QMPShell(qmp.QEMUMonitorProtocol): value = True elif opt[1] == 'false': value = False + elif opt[1].startswith('{'): + value = json.loads(opt[1]) else: value = opt[1] qmpcmd['arguments'][opt[0]] = value |