[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 10/10] scripts: qmp-shell: Add verbose flag
From: |
Luiz Capitulino |
Subject: |
[Qemu-devel] [PULL 10/10] scripts: qmp-shell: Add verbose flag |
Date: |
Fri, 8 May 2015 09:34:11 -0400 |
From: John Snow <address@hidden>
Add a verbose flag that shows the QMP command that was
constructed, to allow for later copy/pasting, reference,
debugging, etc.
The QMP is converted from a Python literal to JSON first,
to ensure that it is viable input to the actual QMP parser.
As a side-effect, this JSON output will helpfully show all
the necessary conversions that were performed on the input,
illustrating that "True" was transformed back into "true",
literal values are now escaped with "" instead of '', and so on.
Signed-off-by: John Snow <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Tested-by: Kashyap Chamarthy <address@hidden>
Signed-off-by: Luiz Capitulino <address@hidden>
---
scripts/qmp/qmp-shell | 24 ++++++++++++++++++------
1 file changed, 18 insertions(+), 6 deletions(-)
diff --git a/scripts/qmp/qmp-shell b/scripts/qmp/qmp-shell
index 1df2ca7..65280d2 100755
--- a/scripts/qmp/qmp-shell
+++ b/scripts/qmp/qmp-shell
@@ -195,6 +195,13 @@ class QMPShell(qmp.QEMUMonitorProtocol):
self.__cli_expr(cmdargs[1:], qmpcmd['arguments'])
return qmpcmd
+ def _print(self, qmp):
+ jsobj = json.dumps(qmp)
+ if self._pp is not None:
+ self._pp.pprint(jsobj)
+ else:
+ print str(jsobj)
+
def _execute_cmd(self, cmdline):
try:
qmpcmd = self.__build_cmd(cmdline)
@@ -206,15 +213,13 @@ class QMPShell(qmp.QEMUMonitorProtocol):
# For transaction mode, we may have just cached the action:
if qmpcmd is None:
return True
+ if self._verbose:
+ self._print(qmpcmd)
resp = self.cmd_obj(qmpcmd)
if resp is None:
print 'Disconnected'
return False
-
- if self._pp is not None:
- self._pp.pprint(resp)
- else:
- print resp
+ self._print(resp)
return True
def connect(self):
@@ -250,6 +255,9 @@ class QMPShell(qmp.QEMUMonitorProtocol):
else:
return self._execute_cmd(cmdline)
+ def set_verbosity(self, verbose):
+ self._verbose = verbose
+
class HMPShell(QMPShell):
def __init__(self, address):
QMPShell.__init__(self, address)
@@ -327,7 +335,7 @@ def die(msg):
def fail_cmdline(option=None):
if option:
sys.stderr.write('ERROR: bad command-line option \'%s\'\n' % option)
- sys.stderr.write('qemu-shell [ -p ] [ -H ] < UNIX socket path> | < TCP
address:port >\n')
+ sys.stderr.write('qemu-shell [ -v ] [ -p ] [ -H ] < UNIX socket path> | <
TCP address:port >\n')
sys.exit(1)
def main():
@@ -335,6 +343,7 @@ def main():
qemu = None
hmp = False
pp = None
+ verbose = False
try:
for arg in sys.argv[1:]:
@@ -346,6 +355,8 @@ def main():
if pp is not None:
fail_cmdline(arg)
pp = pprint.PrettyPrinter(indent=4)
+ elif arg == "-v":
+ verbose = True
else:
if qemu is not None:
fail_cmdline(arg)
@@ -370,6 +381,7 @@ def main():
die('Could not connect to %s' % addr)
qemu.show_banner()
+ qemu.set_verbosity(verbose)
while qemu.read_exec_command(qemu.get_prompt()):
pass
qemu.close()
--
1.9.3
- [Qemu-devel] [PULL 00/10] QMP queue, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 03/10] qobject: Clean up around qtype_code, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 02/10] QJSON: Use OBJECT_CHECK, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 05/10] json-parser: Accept 'null' in QMP, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 04/10] qobject: Add a special null QObject, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 07/10] scripts: qmp-shell: refactor helpers, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 06/10] MAINTAINERS: New maintainer for QMP and QAPI, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 01/10] monitor: add memory search commands s, sp, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 08/10] scripts: qmp-shell: Expand support for QMP expressions, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 09/10] scripts: qmp-shell: add transaction subshell, Luiz Capitulino, 2015/05/08
- [Qemu-devel] [PULL 10/10] scripts: qmp-shell: Add verbose flag,
Luiz Capitulino <=
- Re: [Qemu-devel] [PULL 00/10] QMP queue, Peter Maydell, 2015/05/11