From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54452) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eHRl0-0001Au-9U for qemu-devel@nongnu.org; Wed, 22 Nov 2017 04:55:52 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eHRkw-0001Iy-6V for qemu-devel@nongnu.org; Wed, 22 Nov 2017 04:55:50 -0500 Received: from mx1.redhat.com ([209.132.183.28]:60440) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1eHRkv-0001Ie-TM for qemu-devel@nongnu.org; Wed, 22 Nov 2017 04:55:46 -0500 Date: Wed, 22 Nov 2017 04:55:41 -0500 From: Jeff Cody Message-ID: <20171122095541.GA8572@localhost.localdomain> References: <151129128964.22232.12085565834223304351@sif> <255b996e-cfe4-2499-c6aa-8122461eb1dc@de.ibm.com> <151132098079.31225.2285990377569459449@sif> <6739de79-67e2-1fc7-3f6a-774e7d2bc08f@de.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <6739de79-67e2-1fc7-3f6a-774e7d2bc08f@de.ibm.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [ANNOUNCE] QEMU 2.11.0-rc2 is now available List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Christian Borntraeger Cc: Michael Roth , Kevin Wolf , Max Reitz , QingFeng Hao , qemu-devel , Stefan Hajnoczi On Wed, Nov 22, 2017 at 09:09:02AM +0100, Christian Borntraeger wrote: >=20 >=20 > 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: Assert= ion `blk->quiesce_counter' failed. > >>> for several qemu iotests.=20 > >>> > >>> I have not looked into any details. > >=20 > > It looks to be due to: > >=20 > > 4afeffc8572f40d8844b946a30c00b10da4442b1 > > blockjob: do not allow coroutine double entry or entry-after-completi= on >=20 > Yes, I can confirm that reverting this patch gets rid of this assertion= , but > I see things like >=20 > --- /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) ( ex= ec "$QEMU_IMG_PROG" $QEMU_IMG_OPTIONS "$@" ) >=20 That is from the subsequent patches in the series - you will want to reve= rt 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. >=20 > >=20 > >>> > >>> Christian > >>> > >>> On 11/21/2017 08:08 PM, Michael Roth wrote: > >>>> Hello, > >>>> > >>>> On behalf of the QEMU Team, I'd like to announce the availability = of the > >>>> third release candidate for the QEMU 2.11 release. This release i= s meant > >>>> for testing purposes and should not be used in a production enviro= nment. > >>>> > >>>> http://download.qemu-project.org/qemu-2.11.0-rc2.tar.xz > >>>> http://download.qemu-project.org/qemu-2.11.0-rc2.tar.xz.sig > >>>> > >>>> You can help improve the quality of the QEMU 2.11 release by testi= ng this > >>>> release and reporting bugs on Launchpad: > >>>> > >>>> https://bugs.launchpad.net/qemu/ > >>>> > >>>> The release plan, as well a documented known issues for release > >>>> candidates, are available at: > >>>> > >>>> http://wiki.qemu.org/Planning/2.11 > >>>> > >>>> Please add entries to the ChangeLog for the 2.11 release below: > >>>> > >>>> http://wiki.qemu.org/ChangeLog/2.11 > >>>> > >>>> Changes since rc1: > >>>> > >>>> a15d835f00: Update version for v2.11.0-rc2 release (Peter Maydell) > >>>> d975301dc8: qemu-iotest: add test for blockjob coroutine race cond= ition (Jeff Cody) > >>>> a2339699c3: qemu-iotests: add option in common.qemu for mismatch o= nly (Jeff Cody) > >>>> 6133b39f3c: coroutine: abort if we try to schedule or enter a pend= ing coroutine (Jeff Cody) > >>>> 4afeffc857: blockjob: do not allow coroutine double entry or entry= -after-completion (Jeff Cody) > >>>> 7c3d1917fd: build: disarm the TCG unit test trap (Daniel P. Berran= ge) > >>>> 2807746ff1: iotests: Fix 176 on 32-bit host (Eric Blake) > >>>> 50a3efb0f0: block: Close a BlockDriverState completely even when b= s->drv is NULL (Alberto Garcia) > >>>> 70a5afedd6: block: Error out on load_vm with active dirty bitmaps = (Kevin Wolf) > >>>> 2b624fe079: block: Add errp to bdrv_all_goto_snapshot() (Kevin Wol= f) > >>>> 0b62bcbc61: block: Add errp to bdrv_snapshot_goto() (Kevin Wolf) > >>>> 1f4ad7d3b8: block: Don't request I/O permission with BDRV_O_NO_IO = (Kevin Wolf) > >>>> dacaa16238: block: Don't use BLK_PERM_CONSISTENT_READ for format p= robing (Kevin Wolf) > >>>> 70b7fba987: Use HTTPS for qemu.org and other domains (Stefan Hajno= czi) > >>>> 1b3bbc6887: Use qemu.org domain name (Stefan Hajnoczi) > >>>> d0dead3b6d: scripts/make-release: ship u-boot source as a tarball = (Michael Roth) > >>>> 34d49937e4: accel/tcg: Handle atomic accesses to notdirty memory c= orrectly (Peter Maydell) > >>>> 2726627197: exec.c: Factor out before/after actions for notdirty m= emory writes (Peter Maydell) > >>>> df83eabd52: qga: replace GetIfEntry with GetIfEntry2 for interface= stats (ZhiPeng Lu) > >>>> 6cbf1ec85c: pc-bios/s390-ccw.img: update image (Cornelia Huck) > >>>> f516511ea8: linux-user: Fix calculation of auxv length (Peter Mayd= ell) > >>>> b350ae138f: hw/arm: Silence xlnx-ep108 deprecation warning during = tests (Thomas Huth) > >>>> b6e70d1d7f: hw/arm/aspeed: Unlock SCU when running kernel (Joel St= anley) > >>>> 50cd71b0d3: arm: check regime, not current state, for ATS write PA= R format (Peter Maydell) > >>>> 2b75ef01ca: nvic: Fix ARMv7M MPU_RBAR reads (Peter Maydell) > >>>> 96a8b92ed8: target/arm: Report GICv3 sysregs present in ID registe= rs if needed (Peter Maydell) > >>>> b11ce33fe0: Revert "cpu-exec: don't overwrite exception_index" (Pe= ter Maydell) > >>>> 8775d91a0f: pc-bios/s390-ccw: Fix problem with invalid virtio-scsi= LUN when rebooting (Thomas Huth) > >>>> 7337c6eb98: s390x/tcg: fix DIAG 308 with > 1 VCPU (MTTCG) (David H= ildenbrand) > >>>> dc0bbef5e6: s390x: fix storing CPU status (again) (David Hildenbra= nd) > >>>> c527e0afcd: hw/net/vmxnet3: Fix code to work on big endian hosts, = too (Thomas Huth) > >>>> 0dacea92d2: net: Transmit zero UDP checksum as 0xFFFF (Ed Swierk) > >>>> ebc2327f07: MAINTAINERS: Add missing entry for eepro100 emulation = (Stefan Weil) > >>>> 8f8e8053d9: hw/net/eepro100: Fix endianness problem on big endian = hosts (Thomas Huth) > >>>> 2ec405524e: Revert "Add new PCI ID for i82559a" (Jason Wang) > >>>> 4d36623520: colo-compare: fix the dangerous assignment (Mao Zhongy= i) > >>>> 8251248394: spapr: reset DRCs after devices (Greg Kurz) > >>>> 7abd43baec: target/ppc: Update setting of cpu features to account = for compat modes (Suraj Jitindar Singh) > >>>> c0012e9a22: iotests: Make 087 pass without AIO enabled (Max Reitz) > >>>> 5e003f17ec: block: Make bdrv_next() keep strong references (Max Re= itz) > >>>> 08546bcfb2: qcow2: Fix overly broad madvise() (Max Reitz) > >>>> 4efb1f7c61: qcow2: Refuse to get unaligned offsets from cache (Max= Reitz) > >>>> 23482f8a60: qcow2: Add bounds check to get_refblock_offset() (Max = Reitz) > >>>> d470ad42ac: block: Guard against NULL bs->drv (Max Reitz) > >>>> 93bbaf03ff: qcow2: Unaligned zero cluster in handle_alloc() (Max R= eitz) > >>>> 791fff504c: qcow2: check_errors are fatal (Max Reitz) > >>>> 3e3b838ffe: qcow2: reject unaligned offsets in write compressed (A= nton Nefedov) > >>>> 2b7731938d: iotests: Add test for failing qemu-img commit (Max Rei= tz) > >>>> 1b76e8389b: tests: Add check-qobject for equality tests (Max Reitz= ) > >>>> 791cbccc94: iotests: Add test for non-string option reopening (Max= Reitz) > >>>> 54fd1b0d26: block: qobject_is_equal() in bdrv_reopen_prepare() (Ma= x Reitz) > >>>> b38dd678a2: qapi: Add qobject_is_equal() (Max Reitz) > >>>> 254bf807e5: qapi/qlist: Add qlist_append_null() macro (Max Reitz) > >>>> 84be629d55: qapi/qnull: Add own header (Max Reitz) > >>>> 4096974e18: qcow2: fix image corruption on commit with persistent = bitmap (Eric Blake) > >>>> 3590cd0f04: iotests: test clearing unknown autoclear_features by q= cow2 (Vladimir Sementsov-Ogievskiy) > >>>> dafe096057: block: Fix permissions in image activation (Kevin Wolf= ) > >>>> fed5f8f820: nbd/server: Fix error reporting for bad requests (Eric= Blake) > >>>> 01b05c66a3: nbd/client: Don't hard-disconnect on ESHUTDOWN from se= rver (Eric Blake) > >>>> cb6b1a3fc3: nbd/client: Use error_prepend() correctly (Eric Blake) > >>>> 08ace1d753: nbd: Don't crash when server reports NBD_CMD_READ fail= ure (Eric Blake) > >>>> f06033295b: qcow2: fix image corruption after committing qcow2 ima= ge into base (Daniel P. Berrange) > >>>> 398e6ad014: block: Deprecate bdrv_set_read_only() and users (Kevin= Wolf) > >>>> f66afbe26f: qcow2: don't permit changing encryption parameters (Da= niel P. Berrange) > >>>> 6473069416: block: Fix error path in bdrv_backing_update_filename(= ) (Kevin Wolf) > >>>> c60f6fcfbd: qemu-iotests: Use -nographic in 182 (Kevin Wolf) > >>>> 611e0653ad: replication: Fix replication open fail (Wang Guang) > >>>> 3831c07b89: tests/bios-tables-test: Fix endianess problems when pa= ssing data to iasl (Thomas Huth) > >>>> f865da7c36: build-sys: restrict vmcoreinfo to fw_cfg+dma capable t= argets (Marc-Andr=E9 Lureau) > >>>> b948bb55da: vmcoreinfo: put it in the 'misc' device category (Marc= -Andr=E9 Lureau) > >>>> 7b8be49d36: NUMA: Enable adding NUMA node implicitly (Dou Liyang) > >>>> 45bd4b1c09: tests/acpi-test-data: update _CRS in DSDT (Michael S. = Tsirkin) > >>>> 2d0f99ed38: hw/pcie-pci-bridge: restrict to X86 and ARM (Marcel Ap= felbaum) > >>>> 9fa99d2519: hw/pci-host: Fix x86 Host Bridges 64bit PCI hole (Marc= el Apfelbaum) > >>>> d06bce95ff: pci: Initialize pci_dev->name before use (Alexey Karda= shevskiy) > >>>> 7abea552ab: fix: unrealize virtio device if we fail to hotplug it = (linzhecheng) > >>>> 341e0b5658: throttle-groups: forget timer and schedule next TGM on= detach (Stefan Hajnoczi) > >>>> bcf43cdc17: sdl2: Fix broken display updating after the window is = hidden (Jindrich Makovicka) > >>>> 24952847ad: sdl2: Do not leave grab when fullscreen (Jindrich Mako= vicka) > >>>> cd6d78b718: sdl2: Fix dead keyboard after fullsceen (Jindrich Mako= vicka) > >>>> 253347e100: sdl2: Use the same pointer show/hide logic for absolut= e and relative mode (Jindrich Makovicka) > >>>> fc49e7279d: sdl2: Do not quit the emulator when an auxilliary wind= ow is closed (Jindrich Makovicka) > >>>> 4950b1a766: fix scripts/update-linux-headers.sh here document (Ger= d Hoffmann) > >>>> 07c114bbf3: exec: Do not resolve subpage in mru_section (Paolo Bon= zini) > >>>> 6cd65969da: tpm_tis: Return 0 for every register in case of failur= e mode (Stefan Berger) > >>>> ad4aca69bb: tpm_tis: Return TPM_VERSION_UNSPEC in case of BE failu= re (Stefan Berger) > >>>> 17b1af773e: tpm-emulator: protect concurrent ctrl_chr access (Marc= -Andr=E9 Lureau) > >>>> 83e83c34b2: specs: Extend TPM spec with TPM emulator description (= Stefan Berger) > >>>> 3c5f9c3f35: target/arm: Fix GETPC usage in do_paired_cmpxchg64_l/b= e (Richard Henderson) > >>>> 3bdb5fcc9a: target/arm: Use helper_retaddr in stxp helpers (Richar= d Henderson) > >>>> ec603b5584: tcg: Record code_gen_buffer address for user-only memo= ry helpers (Richard Henderson) > >>>> 26a5db322b: util/stats64: Fix min/max comparisons (Max Reitz) > >>>> 17b50b0c29: cpu-exec: avoid cpu_exec_nocache infinite loop with re= cord/replay (Pavel Dovgalyuk) > >>>> e01cecabf3: cpu-exec: don't overwrite exception_index (Pavel Dovga= lyuk) > >>>> 9200361060: vhost-user-scsi: add missing virtqueue_size param (Dar= iusz Stojaczyk) > >>>> 6976af663d: target-i386: adds PV_TLB_FLUSH CPUID feature bit (Wanp= eng Li) > >>>> 54113dd5eb: thread-posix: fix qemu_rec_mutex_trylock macro (Emilio= G. Cota) > >>>> b8e535ae8a: Makefile: simpler/faster "make help" (Philippe Mathieu= -Daud=E9) > >>>> a2e6ffab97: ioapic/tracing: Remove last DPRINTFs (Dr. David Alan G= ilbert) > >>>> 78bfef72fb: linux-user: Handle rt_sigaction correctly for SPARC (P= eter Maydell) > >>>> 8d8cb956e0: linux-user/sparc: Put address for data faults where li= nux-user expects it (Peter Maydell) > >>>> 15e692a6fc: linux-user/ppc: Report correct fault address for data = faults (Peter Maydell) > >>>> f2d34df3c1: linux-user/s390x: Mask si_addr for SIGSEGV (Peter Mayd= ell) > >>>> a8b154a637: linux-user: return EINVAL from prctl(PR_*_SECCOMP) (Ja= mes Cowgill) > >>>> a4dd3d5172: linux-user: fix 'finshed' typo in comment (Emilio G. C= ota) > >>>> 8bf8e9df4a: linux-user/syscall.c: Handle SH4's exceptional alignme= nt for p{read, write}64 (James Clarke) > >>>> 541e169042: linux-user: Handle TARGET_MAP_STACK and TARGET_MAP_HUG= ETLB (Helge Deller) > >>>> 3d60c84dea: linux-user/hppa: Fix TARGET_F_RDLCK, TARGET_F_WRLCK, T= ARGET_F_UNLCK (Helge Deller) > >>>> e65be6a7cf: linux-user/hppa: Fix TARGET_MAP_TYPE (Helge Deller) > >>>> 92afb15310: linux-user/hppa: Fix typo for TARGET_NR_epoll_wait (He= lge Deller) > >>>> fc37dcfe1a: linux-user/hppa: Fix cpu_clone_regs (Richard Henderson= ) > >>>> 2da6e76cca: linux-user/hppa: Fix TARGET_SA_* defines (Helge Deller= ) > >>>> 7f047de18c: linux-user: Restrict usage of sa_restorer (Richard Hen= derson) > >>>> e5a11847c4: Enable 8-byte wide MMIO for 16550 serial devices (Mike= Nawrocki) > >>>> > >>>> > >> >=20