Re: [Qemu-devel] [PULL v2 00/22] Docker and block patches

[Fam Zheng]

Cc'ing Paolo.

On Fri, 06/02 16:50, Peter Maydell wrote:
> On 2 June 2017 at 09:10, Fam Zheng <address@hidden> wrote:
> > The following changes since commit 43771d5d92312504305c19abe29ec5bfabd55f01:
> >
> >   Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2017-05-31' 
> > into staging (2017-06-01 16:39:16 +0100)
> >
> > are available in the git repository at:
> >
> >   git://github.com/famz/qemu.git tags/docker-and-block-pull-request
> >
> > for you to fetch changes up to 4ab2cbc128f1938355a12970c9b6e419a63fcab6:
> >
> >   block: make accounting thread-safe (2017-06-02 15:59:32 +0800)
> >
> > ----------------------------------------------------------------
> >
> > v2: Fix building on OSX and BSD.
> >
> > ----------------------------------------------------------------
> >
> > Fam Zheng (4):
> >   docker: Run tests with current user
> >   docker: Add bzip2 and hostname to fedora image
> >   docker: Add libaio to fedora image
> >   docker: Add flex and bison to centos6 image
> >
> > Paolo Bonzini (18):
> >   block: access copy_on_read with atomic ops
> >   block: access quiesce_counter with atomic ops
> >   block: access io_limits_disabled with atomic ops
> >   block: access serialising_in_flight with atomic ops
> >   block: access wakeup with atomic ops
> >   block: access io_plugged with atomic ops
> >   throttle-groups: only start one coroutine from drained_begin
> >   throttle-groups: do not use qemu_co_enter_next
> >   throttle-groups: protect throttled requests with a CoMutex
> >   util: add stats64 module
> >   block: use Stat64 for wr_highest_offset
> >   block: access write_gen with atomics
> >   block: protect tracked_requests and flush_queue with reqs_lock
> >   block: introduce dirty_bitmap_mutex
> >   migration/block: reset dirty bitmap before reading
> >   block: protect modification of dirty bitmaps with a mutex
> >   block: introduce block_account_one_io
> >   block: make accounting thread-safe
> 
> Fails 'make check' on OSX:
>   GTESTER tests/test-blockjob
> qemu: qemu_mutex_destroy: Invalid argument
> 
> Backtrace:
> 
> (lldb) bt
> * thread #1: tid = 0x442e6f, 0x00007fffc611ad42
> libsystem_kernel.dylib`__pthread_kill + 10, queue =
> 'com.apple.main-thread', stop reason = signal SIGABRT
>   * frame #0: 0x00007fffc611ad42 libsystem_kernel.dylib`__pthread_kill + 10
>     frame #1: 0x00007fffc6208457 libsystem_pthread.dylib`pthread_kill + 90
>     frame #2: 0x00007fffc6080420 libsystem_c.dylib`abort + 129
>     frame #3: 0x00000001000c1335
> test-blockjob`error_exit(err=<unavailable>, msg=<unavailable>) + 53 at
> qemu-thread-posix.c:36
>     frame #4: 0x00000001000c135d
> test-blockjob`qemu_mutex_destroy(mutex=<unavailable>) + 29 at
> qemu-thread-posix.c:54
>     frame #5: 0x000000010004411d test-blockjob`blk_unref [inlined]
> blk_delete(blk=0x0000000100c0c360) + 215 at block-backend.c:298
>     frame #6: 0x0000000100044046
> test-blockjob`blk_unref(blk=0x0000000100c0c360) + 38 at
> block-backend.c:336
>     frame #7: 0x0000000100005ffa
> test-blockjob`bdrv_open_inherit(filename=<unavailable>,
> reference=<unavailable>, options=0x000000010100c000,
> flags=<unavailable>, parent=<unavailable>, child_role=<unavailable>,
> errp=<unavailable>) + 4346 at block.c:2546
>     frame #8: 0x0000000100006a07
> test-blockjob`bdrv_open(filename=<unavailable>,
> reference=<unavailable>, options=<unavailable>, flags=<unavailable>,
> errp=<unavailable>) + 23 at block.c:2632
>     frame #9: 0x0000000100001949
> test-blockjob`create_blk(name=0x0000000000000000) + 73 at
> test-blockjob.c:61
>     frame #10: 0x00000001000016f5 test-blockjob`test_job_ids + 21 at
> test-blockjob.c:92
>     frame #11: 0x000000010070d91d
> libglib-2.0.0.dylib`g_test_run_suite_internal + 626
>     frame #12: 0x000000010070dae1
> libglib-2.0.0.dylib`g_test_run_suite_internal + 1078
>     frame #13: 0x000000010070d198 libglib-2.0.0.dylib`g_test_run_suite + 266
>     frame #14: 0x00000001000016c8
> test-blockjob`main(argc=<unavailable>, argv=<unavailable>) + 88 at
> test-blockjob.c:152
>     frame #15: 0x00007fffc5fec235 libdyld.dylib`start + 1
>     frame #16: 0x00007fffc5fec235 libdyld.dylib`start + 1