[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available |
Date: |
Thu, 23 Nov 2017 08:47:47 +0800 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Wed, 11/22 04:55, Jeff Cody wrote:
> On Wed, Nov 22, 2017 at 09:09:02AM +0100, Christian Borntraeger wrote:
> >
> >
> > On 11/22/2017 04:23 AM, Michael Roth wrote:
> > > Quoting Christian Borntraeger (2017-11-21 15:38:32)
> > >> forgot to cc qemu-devel....
> > >>
> > >> On 11/21/2017 10:37 PM, Christian Borntraeger wrote:
> > >>> a quick heads up . Rc2 now triggers
> > >>> +qemu-img: block/block-backend.c:2088: blk_root_drained_end: Assertion
> > >>> `blk->quiesce_counter' failed.
> > >>> for several qemu iotests.
> > >>>
> > >>> I have not looked into any details.
> > >
> > > It looks to be due to:
> > >
> > > 4afeffc8572f40d8844b946a30c00b10da4442b1
> > > blockjob: do not allow coroutine double entry or entry-after-completion
> >
> > Yes, I can confirm that reverting this patch gets rid of this assertion, but
> > I see things like
> >
> > --- /home/cborntra/REPOS/qemu/tests/qemu-iotests/020.out 2017-11-21
> > 20:19:34.785519323 +0100
> > +++ /home/cborntra/REPOS/qemu/build/tests/qemu-iotests/020.out.bad
> > 2017-11-22 09:04:50.127612500 +0100
> > @@ -537,7 +537,8 @@
> > wrote 65536/65536 bytes at offset 4295098368
> > 64 KiB, X ops; XX:XX:XX.X (XXX YYY/sec and XXX ops/sec)
> > No errors were found on the image.
> > -Image committed.
> > +qemu_aio_coroutine_enter: Co-routine was already scheduled in
> > 'co_aio_sleep_ns'
> > +./common.rc: line 61: 88002 Aborted (core dumped) ( exec
> > "$QEMU_IMG_PROG" $QEMU_IMG_OPTIONS "$@" )
> >
>
> That is from the subsequent patches in the series - you will want to revert
> the whole series to test, as the introduced aborts catch the illegal
> entries that the reverted patch sidestepped.
>
> The series patches are:
>
> 4afeffc
> 6133b39
> a233969
> d975301
>
> Of course, these new aborts prevent improper behavior, so we may want to
> figure out why this is getting hit.
>
> Unfortunately, I am traveling at the moment (waiting to board my flight), so
> will have limited connectivity.
I'll take a look at this today and the bottom line is we revert the series until
a proper fix is found.
Fa
- [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Michael Roth, 2017/11/21
- Message not available
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Christian Borntraeger, 2017/11/21
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Michael Roth, 2017/11/21
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Christian Borntraeger, 2017/11/22
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Jeff Cody, 2017/11/22
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available,
Fam Zheng <=
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Jeff Cody, 2017/11/22
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Fam Zheng, 2017/11/23
- Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available, Stefan Hajnoczi, 2017/11/23