From: Igor Mammedov <imammedo@redhat.com>
To: Max Reitz <mreitz@redhat.com>
Cc: Michal Privoznik <mprivozn@redhat.com>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] Preconfig state reachable without --preconfig given
Date: Mon, 4 Jun 2018 22:24:33 +0200 [thread overview]
Message-ID: <20180604222433.4d585846@redhat.com> (raw)
In-Reply-To: <787c23db-430b-dc20-10f7-00fdf119449c@redhat.com>
On Mon, 4 Jun 2018 16:04:33 +0200
Max Reitz <mreitz@redhat.com> wrote:
> On 2018-06-02 12:46, Michal Privoznik wrote:
> > On 06/01/2018 03:28 PM, Max Reitz wrote:
> >> Hi,
> >>
> >> The @preconfig RunState documentation states:
> >>
> >>> The state is reachable only if the --preconfig CLI option is used.
> >>
> >> However, this is not true:
> >>
> >> $ echo | x86_64-softmmu/qemu-system-x86_64 -monitor stdio
> >> QEMU 2.12.50 monitor - type 'help' for more information
> >> (qemu)
> >> HMP not available in preconfig state, use QMP instead
> >
> > Not sure if this is the same bug, but I've noticed libvirt having
> > troubles detecting capabilities of qemu and debugging lead to this patch:
> >
> > http://lists.nongnu.org/archive/html/qemu-devel/2018-06/msg00367.html
> >
> > (which by no means I claim is proper solution. It might be viewed as
> > workaround by expert qemu devels)
>
> I haven't investigated further, but that patch breaks iotest 091 (which
> tests migration). It just stops now after the migration has started, so
> it never completes.
>
> Actually, it seems to break all migration iotests (but 203, which
> migrates to /dev/null), so I suspect it breaks migration as a whole.
Manual migration to file and restore from it, was tested manually and
worked fine, migration-test from 'make check' is ok as well.
It's probably due broken to "echo foo | $QEMU"
There is another issue with -nodefaults option leading to indefinite
timeout even if --preconfig wasn't used. I've just posted fix for that
so that it won't be masked out by Michal/Daniel's fix.
But I've run iotests and -nodefaults fix doesn't affect outcome of the
tests (the same 4 failures).
>
> Max
>
next prev parent reply other threads:[~2018-06-04 20:24 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-01 13:28 [Qemu-devel] Preconfig state reachable without --preconfig given Max Reitz
2018-06-01 13:44 ` Max Reitz
2018-06-02 10:46 ` Michal Privoznik
2018-06-04 9:46 ` Max Reitz
2018-06-04 12:05 ` Igor Mammedov
2018-06-05 11:35 ` Dr. David Alan Gilbert
2018-06-04 14:04 ` Max Reitz
2018-06-04 20:24 ` Igor Mammedov [this message]
2018-06-06 10:35 ` Max Reitz
2018-06-06 11:00 ` Igor Mammedov
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=20180604222433.4d585846@redhat.com \
--to=imammedo@redhat.com \
--cc=mprivozn@redhat.com \
--cc=mreitz@redhat.com \
--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).