All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jason Baron <jbaron@redhat.com>
To: qemu-devel@nongnu.org
Cc: kwolf@redhat.com, aliguori@us.ibm.com, quintela@redhat.com,
	blauwirbel@gmail.com, andreas.faerber@web.de,
	pbonzini@redhat.com
Subject: [Qemu-devel] [PATCH v2 0/3] qtest: add migration testing
Date: Thu, 20 Dec 2012 12:14:48 -0500	[thread overview]
Message-ID: <cover.1356022464.git.jbaron@redhat.com> (raw)

Hi,

Add a basic qtest for migration testing. Currently, it just tests a migrate of
machine 'pc' on the same host. Would be nice to extend to multiple machine
versions, but that requires multiple binaries, which could be done, but is
perhaps a bit awkward from qtest? Testing different machine versions within
the same binary doesn't seem like a real world test case to me. Currently,
the test aborts, if the migrate takes more than 2 minutes.

In any case, the test currently fails for q35, since ahci migration suport
isn't in place. Thus, I intend to add q35 testing here, once those ahci
migration patches are accepted.

Thanks,

-Jason

v2:
 * move core migration functions to libqtest.c
 * simplify Makefile dependencies
 * use 'mkdtemp()' instead of child pid to identify qemu instances

Jason Baron (3):
  qtest: Enable creation of multiple qemu instances
  qtest: extend qtest_qmp() to fill in the reply
  qtest: add migrate-test

 tests/Makefile       |    4 +-
 tests/libqtest.c     |  130 ++++++++++++++++++++++++++++++++++++++++++++------
 tests/libqtest.h     |   30 ++++++++++--
 tests/migrate-test.c |   68 ++++++++++++++++++++++++++
 4 files changed, 212 insertions(+), 20 deletions(-)
 create mode 100644 tests/migrate-test.c

             reply	other threads:[~2012-12-20 17:14 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-20 17:14 Jason Baron [this message]
2012-12-20 17:14 ` [Qemu-devel] [PATCH v2 1/3] qtest: Enable creation of multiple qemu instances Jason Baron
2012-12-20 20:07   ` Blue Swirl
2012-12-20 20:26     ` Jason Baron
2012-12-20 20:38       ` Blue Swirl
2012-12-21  7:47       ` Markus Armbruster
2012-12-20 17:14 ` [Qemu-devel] [PATCH v2 3/3] qtest: add migrate-test Jason Baron
2012-12-20 20:17   ` Blue Swirl
2012-12-20 17:14 ` [Qemu-devel] [PATCH v2 2/3] qtest: extend qtest_qmp() to fill in the reply Jason Baron
2013-01-07 20: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=cover.1356022464.git.jbaron@redhat.com \
    --to=jbaron@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=andreas.faerber@web.de \
    --cc=blauwirbel@gmail.com \
    --cc=kwolf@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.