qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: aliguori@us.ibm.com, jan.kiszka@siemens.com, mjt@tls.msk.ru,
	alex.williamson@redhat.com, aviksil@linux.vnet.ibm.com,
	afaerber@suse.de
Subject: [Qemu-devel] [PATCH v4 01/12] qtest: Don't reset on qtest chardev connect
Date: Wed, 26 Jun 2013 15:52:12 +0200	[thread overview]
Message-ID: <1372254743-15808-2-git-send-email-armbru@redhat.com> (raw)
In-Reply-To: <1372254743-15808-1-git-send-email-armbru@redhat.com>

libqtest's qtest_init() connecting to the qtest socket triggers reset.
This was coded in the hope we could use the same QEMU process for
multiple tests that way.  Never used.  Injects an extra reset even
when it's not used, and that can mess up tests such as the one of
-boot once I'm about to add.  Drop it.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 qtest.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/qtest.c b/qtest.c
index 07a9612..74f1842 100644
--- a/qtest.c
+++ b/qtest.c
@@ -472,7 +472,12 @@ static void qtest_event(void *opaque, int event)
 
     switch (event) {
     case CHR_EVENT_OPENED:
-        qemu_system_reset(false);
+        /*
+         * We used to call qemu_system_reset() here, hoping we could
+         * use the same process for multiple tests that way.  Never
+         * used.  Injects an extra reset even when it's not used, and
+         * that can mess up tests, e.g. -boot once.
+         */
         for (i = 0; i < ARRAY_SIZE(irq_levels); i++) {
             irq_levels[i] = 0;
         }
-- 
1.7.11.7

  reply	other threads:[~2013-06-26 13:52 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-26 13:52 [Qemu-devel] [PATCH v4 00/12] Boot order tests Markus Armbruster
2013-06-26 13:52 ` Markus Armbruster [this message]
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 02/12] boot-order-test: New; covering just PC for now Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 03/12] libqos: include dependencies Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 04/12] libqos: Add support for memory-mapped fw_cfg Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 05/12] boot-order-test: Add tests for PowerMacs Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 06/12] boot-order-test: Cover -boot once in ppc tests Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 07/12] boot-order-test: Better separate target-specific and generic parts Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 08/12] boot-order-test: Code motion for better readability Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 09/12] boot-order-test: Add tests for PowerPC PREP Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 10/12] boot-order-test: Add tests for Sun4m Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 11/12] libqos: Generalize I/O-mapped fw_cfg Markus Armbruster
2013-06-26 13:52 ` [Qemu-devel] [PATCH v4 12/12] boot-order-test: Add tests for Sun4u Markus Armbruster
2013-07-18 16:33 ` [Qemu-devel] [PATCH v4 00/12] Boot order tests Markus Armbruster
2013-07-23 19:04 ` Anthony Liguori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1372254743-15808-2-git-send-email-armbru@redhat.com \
    --to=armbru@redhat.com \
    --cc=afaerber@suse.de \
    --cc=alex.williamson@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=aviksil@linux.vnet.ibm.com \
    --cc=jan.kiszka@siemens.com \
    --cc=mjt@tls.msk.ru \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).