All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: Luiz Capitulino <lcapitulino@redhat.com>
Cc: Max Reitz <mreitz@redhat.com>, Qemu-block <qemu-block@nongnu.org>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Subject: Re: [Qemu-devel] Aborts in iotest 169
Date: Wed, 23 Jan 2019 16:35:54 +0000	[thread overview]
Message-ID: <20190123163553.GD2193@work-vm> (raw)
In-Reply-To: <20190123112420.0fe8fb0c@doriath>

* Luiz Capitulino (lcapitulino@redhat.com) wrote:
> On Wed, 23 Jan 2019 17:12:35 +0100
> Max Reitz <mreitz@redhat.com> wrote:
> 
> > On 23.01.19 17:04, Luiz Capitulino wrote:
> > > On Wed, 23 Jan 2019 16:48:49 +0100
> > > Max Reitz <mreitz@redhat.com> wrote:
> > >   
> > >> Hi,
> > >>
> > >> When running 169 in parallel (e.g. like so:
> > >>
> > >> $ while TEST_DIR=/tmp/t0 ./check -T -qcow2 169; do; done
> > >> $ while TEST_DIR=/tmp/t1 ./check -T -qcow2 169; do; done
> > >> $ while TEST_DIR=/tmp/t2 ./check -T -qcow2 169; do; done
> > >> $ while TEST_DIR=/tmp/t3 ./check -T -qcow2 169; do; done
> > >>
> > >> in four different shells), I get aborts:  
> > > 
> > > OK, is this part of a test-suite that's also running migration
> > > tests in parallel or in sequence? In other words, what does
> > > iotests have to do with migration (sorry if this is stupid
> > > question, but it's been years I don't do qemu).  
> > 
> > They run migration tests in sequence, but if you run multiple test
> > instances in parallel, well, then they will be run in parallel.
> > 
> > The only reason I CC'd you was because you were so prominent in git
> > blame. O:-)
> 
> Yeah, that's often the case with me :-)
> 
> > > When this happened in the past it meant some QEMU code skipped a
> > > transition, but I can't tell what this has to do with iotests.  
> > 
> > Well, this iotest (which tests a migration configuration) sometimes
> > apparently results in this invalid transition.  But that can't be just
> > the test's fault, as qemu should handle that gracefully.
> 
> Does iotest run a guest or does it only executes parts of qemu
> code? If it's the latter, then I'd guess the test code is missing
> calling qemu code that sets the appropriate state between
> running and postmigrate states.
> 
> > It's probably an issue in the migration code and not so much in vl.c, yes...
> 
> Yeah, I'll let the migration people jump in.

Can we get a log of the qmp commands when it fails?
A 'running->postmigrate' transition is a bit weird; you could get a
something->postmigrate if you migrate a VM twice, and the source is
already in postmigrate, so it tells the destination to go into
postmigrate.   That's odd in itself, but not *that* odd.
But then why is it going running->postmigrate?  That suggests
the source is currently running when it receives an incoming migration,
and that really shouldn't happen.

Dave
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK

  reply	other threads:[~2019-01-23 16:41 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-23 15:48 [Qemu-devel] Aborts in iotest 169 Max Reitz
2019-01-23 16:04 ` Luiz Capitulino
2019-01-23 16:12   ` Max Reitz
2019-01-23 16:24     ` Luiz Capitulino
2019-01-23 16:35       ` Dr. David Alan Gilbert [this message]
2019-01-23 17:16         ` Max Reitz
2019-01-23 18:08           ` Dr. David Alan Gilbert
2019-01-24  8:08             ` Vladimir Sementsov-Ogievskiy
2019-01-24  9:29 ` Vladimir Sementsov-Ogievskiy
2019-01-24  9:49   ` Vladimir Sementsov-Ogievskiy
2019-01-24 10:10     ` Dr. David Alan Gilbert
2019-01-24 10:23       ` Vladimir Sementsov-Ogievskiy
2019-01-24 10:52         ` Dr. David Alan Gilbert
2019-01-24 10:15   ` [Qemu-devel] [Qemu-block] " Kevin Wolf
2019-01-24 10:32     ` Vladimir Sementsov-Ogievskiy
2019-01-24 10:45       ` Kevin Wolf
2019-01-24 10:49     ` Dr. David Alan Gilbert
2019-01-24 11:58       ` Kevin Wolf
2019-01-24 20:15         ` Dr. David Alan Gilbert
2019-01-24 11:53     ` Vladimir Sementsov-Ogievskiy
2019-01-24 17:58     ` Eric Blake
2019-04-08 14:57 ` [Qemu-devel] " Vladimir Sementsov-Ogievskiy
2019-04-08 15:08   ` Max Reitz

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=20190123163553.GD2193@work-vm \
    --to=dgilbert@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=vsementsov@virtuozzo.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.