[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 2/4] libqtest: Ignore QMP events when parsing the res
From: |
Dr. David Alan Gilbert (git) |
Subject: |
[Qemu-devel] [PULL 2/4] libqtest: Ignore QMP events when parsing the response for HMP commands |
Date: |
Tue, 25 Apr 2017 11:41:14 +0100 |
From: Thomas Huth <address@hidden>
When running certain HMP commands (like "device_del") via QMP, we
can sometimes get a QMP event in the response first, so that the
"g_assert(ret)" statement in qtest_hmp() triggers and the test
fails. Fix this by ignoring such QMP events while looking for the
real return value from QMP.
Signed-off-by: Thomas Huth <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>
Added note to qtest_hmp/qtest_hmpv's header description to say
it discards events
---
tests/libqtest.c | 6 ++++++
tests/libqtest.h | 4 +++-
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/tests/libqtest.c b/tests/libqtest.c
index 99b1195355..0b0bf1d460 100644
--- a/tests/libqtest.c
+++ b/tests/libqtest.c
@@ -588,6 +588,12 @@ char *qtest_hmpv(QTestState *s, const char *fmt, va_list
ap)
" 'arguments': {'command-line': %s}}",
cmd);
ret = g_strdup(qdict_get_try_str(resp, "return"));
+ while (ret == NULL && qdict_get_try_str(resp, "event")) {
+ /* Ignore asynchronous QMP events */
+ QDECREF(resp);
+ resp = qtest_qmp_receive(s);
+ ret = g_strdup(qdict_get_try_str(resp, "return"));
+ }
g_assert(ret);
QDECREF(resp);
g_free(cmd);
diff --git a/tests/libqtest.h b/tests/libqtest.h
index 2c9962d94f..ee237448da 100644
--- a/tests/libqtest.h
+++ b/tests/libqtest.h
@@ -132,11 +132,12 @@ void qtest_qmp_eventwait(QTestState *s, const char
*event);
QDict *qtest_qmp_eventwait_ref(QTestState *s, const char *event);
/**
- * qtest_hmpv:
+ * qtest_hmp:
* @s: #QTestState instance to operate on.
* @fmt...: HMP command to send to QEMU
*
* Send HMP command to QEMU via QMP's human-monitor-command.
+ * QMP events are discarded.
*
* Returns: the command's output. The caller should g_free() it.
*/
@@ -149,6 +150,7 @@ char *qtest_hmp(QTestState *s, const char *fmt, ...);
* @ap: HMP command arguments
*
* Send HMP command to QEMU via QMP's human-monitor-command.
+ * QMP events are discarded.
*
* Returns: the command's output. The caller should g_free() it.
*/
--
2.12.2
- [Qemu-devel] [PULL 2/4] libqtest: Ignore QMP events when parsing the response for HMP commands, (continued)
[Qemu-devel] [PULL 0/4] hmp queue, Dr. David Alan Gilbert (git), 2017/04/25
- [Qemu-devel] [PULL 2/4] libqtest: Ignore QMP events when parsing the response for HMP commands,
Dr. David Alan Gilbert (git) <=
- [Qemu-devel] [PULL 1/4] hmp: gpa2hva and gpa2hpa hostaddr command, Dr. David Alan Gilbert (git), 2017/04/25
- [Qemu-devel] [PULL 3/4] libqtest: Add a generic function to run a callback function for every machine, Dr. David Alan Gilbert (git), 2017/04/25
- [Qemu-devel] [PULL 4/4] tests: Add a tester for HMP commands, Dr. David Alan Gilbert (git), 2017/04/25
- Re: [Qemu-devel] [PULL 0/4] hmp queue, Peter Maydell, 2017/04/25
- Re: [Qemu-devel] [PULL 0/4] hmp queue, Dr. David Alan Gilbert, 2017/04/25
- Re: [Qemu-devel] [PULL 0/4] hmp queue, Thomas Huth, 2017/04/25
- Re: [Qemu-devel] [PULL 0/4] hmp queue, Peter Maydell, 2017/04/25
- Re: [Qemu-devel] [PULL 0/4] hmp queue, Peter Maydell, 2017/04/25
- Re: [Qemu-devel] [PULL 0/4] hmp queue, Dr. David Alan Gilbert, 2017/04/25