From: Fam Zheng <famz@redhat.com>
To: Roman Penyaev <roman.penyaev@profitbricks.com>
Cc: Stefan Hajnoczi <stefanha@redhat.com>,
Kevin Wolf <kwolf@redhat.com>, qemu-devel <qemu-devel@nongnu.org>,
qemu-block@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 3/3] linux-aio: fix re-entrant completion processing
Date: Wed, 28 Sep 2016 11:01:18 +0800 [thread overview]
Message-ID: <20160928030118.GH1284@lemon> (raw)
In-Reply-To: <CAJrWOzBfVJaAjwj2wA-EFKv9v-4u14oPcVo2WJyjB6qmBWku2w@mail.gmail.com>
On Tue, 09/27 19:55, Roman Penyaev wrote:
> > The bug is 100% deterministic. Just boot up a guest with -drive
> > format=qcow2,aio=native.
>
> It turns out to be that everything is broken. I started all my
> tests with format=raw,aio=native and immediately got coroutine
> recursive. That is completely weird.
>
> So, what I did is the following:
>
> 1. Took latest master (nothing works)
> 2. Did interactive rebase to 12c8720
> 12c8720 2016-06-28 | Merge remote-tracking branch
> 'remotes/stefanha/tags/block-pull-request' into staging [Peter
> Maydell]
>
> this merge request includes all your patches related to
> virtio-blk and MQ support.
>
> 3. Applied 0ed93d84edab. Everything works fine.
Have you tried qcow2 at this point? raw crashes with 1a62d0accdf85 doesn't mean
qcow2 is fine without it.
Fam
>
> 4. Rebased up till 0647d47:
> 0647d47 2016-09-13 | qcow2: avoid memcpy(dst, NULL, len) [Stefan Hajnoczi]
>
> this is the point, after which 0ed93d84edab was applied
> on master.
>
> Got recursive coroutine, so nothing works.
>
> 5. Did a besect, which shows this commit:
>
> --
> commit 1a62d0accdf85fbeac149018ee8d1728e754de73
> Author: Eric Blake <eblake@redhat.com>
> Date: Fri Jul 15 12:31:59 2016 -0600
>
> block: Fragment reads to max transfer length
> --
>
> So after this commit my commit 0ed93d84edab stops working.
> And now for me is completely not clear what is happening there.
>
> --
> Roman
>
next prev parent reply other threads:[~2016-09-28 3:01 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-27 14:06 [Qemu-devel] [PATCH 0/3] linux-aio: fix "Co-routine re-entered recursively" error Stefan Hajnoczi
2016-09-27 14:06 ` [Qemu-devel] [PATCH 1/3] coroutine: add qemu_coroutine_entered() function Stefan Hajnoczi
2016-09-27 16:20 ` Paolo Bonzini
2016-09-27 16:29 ` Stefan Hajnoczi
2016-09-27 16:55 ` Paolo Bonzini
2016-09-28 9:50 ` Kevin Wolf
2016-09-27 14:06 ` [Qemu-devel] [PATCH 2/3] test-coroutine: test qemu_coroutine_entered() Stefan Hajnoczi
2016-09-27 14:06 ` [Qemu-devel] [PATCH 3/3] linux-aio: fix re-entrant completion processing Stefan Hajnoczi
2016-09-27 14:29 ` Roman Penyaev
2016-09-27 15:25 ` Stefan Hajnoczi
2016-09-27 17:55 ` Roman Penyaev
2016-09-28 3:01 ` Fam Zheng [this message]
2016-09-28 9:14 ` Roman Penyaev
2016-09-28 9:34 ` Fam Zheng
2016-09-28 9:38 ` Roman Penyaev
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=20160928030118.GH1284@lemon \
--to=famz@redhat.com \
--cc=kwolf@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=roman.penyaev@profitbricks.com \
--cc=stefanha@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 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).