[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->di
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization |
Date: |
Thu, 23 Jul 2015 21:25:57 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 |
On 23/07/2015 21:19, Christian Borntraeger wrote:
> Am 23.07.2015 um 20:19 schrieb Paolo Bonzini:
>>
>>
>> On 23/07/2015 19:20, Paolo Bonzini wrote:
>>>
>>>
>>> On 23/07/2015 16:14, Cornelia Huck wrote:
>>>> (gdb) bt
>>>> #0 0x000003fffc5871b4 in pthread_cond_wait@@GLIBC_2.3.2 ()
>>>> from /lib64/libpthread.so.0
>>>> #1 0x000000008024cfca in qemu_cond_wait (address@hidden,
>>>> address@hidden)
>>>> at /data/git/yyy/qemu/util/qemu-thread-posix.c:132
>>>> #2 0x000000008025e83a in rfifolock_lock (r=0x9717d920)
>>>> at /data/git/yyy/qemu/util/rfifolock.c:59
>>>> #3 0x00000000801b78fa in aio_context_acquire (ctx=<optimized out>)
>>>> at /data/git/yyy/qemu/async.c:331
>>>> #4 0x000000008007ceb4 in virtio_blk_data_plane_start (s=0x9717d710)
>>>> at /data/git/yyy/qemu/hw/block/dataplane/virtio-blk.c:285
>>>> #5 0x000000008007c64a in virtio_blk_handle_output (vdev=<optimized out>,
>>>> vq=<optimized out>) at /data/git/yyy/qemu/hw/block/virtio-blk.c:599
>>>> #6 0x00000000801c56dc in qemu_iohandler_poll (pollfds=0x97142800,
>>>> address@hidden) at /data/git/yyy/qemu/iohandler.c:126
>>>> #7 0x00000000801c5178 in main_loop_wait (nonblocking=<optimized out>)
>>>> at /data/git/yyy/qemu/main-loop.c:494
>>>> #8 0x0000000080013ee2 in main_loop () at /data/git/yyy/qemu/vl.c:1902
>>>> #9 main (argc=<optimized out>, argv=<optimized out>, envp=<optimized out>)
>>>> at /data/git/yyy/qemu/vl.c:4653
>>>>
>>>> I've stripped down the setup to the following commandline:
>>>>
>>>> /data/git/yyy/qemu/build/s390x-softmmu/qemu-system-s390x -machine
>>>> s390-ccw-virtio-2.4,accel=kvm,usb=off -m 1024 -smp
>>>> 4,sockets=4,cores=1,threads=1 -nographic -drive
>>>> file=/dev/sda,if=none,id=drive-virtio-disk0,format=raw,serial=ccwzfcp1,cache=none,aio=native
>>>> -device
>>>> virtio-blk-ccw,scsi=off,drive=drive-virtio-disk0,id=virtio-disk0,bootindex=1,x-data-plane=on
>>>
>>> What's the backtrace like for the other threads? This is almost
>>> definitely a latent bug somewhere else.
>>
>> BTW, I can reproduce this---I'm asking because I cannot even attach gdb
>> to the hung process.
>>
>> The simplest workaround is to reintroduce commit a0710f7995 (iothread:
>> release iothread around aio_poll, 2015-02-20), though it also comes with
>> some risk. It avoids the bug because it limits the contention on the
>> RFifoLock.
>
> I can reproduce this with the following backtrace (with --enable-debug info
> added qemu is the tag v2.4.0-rc2)
Can you check that cherry-picking a0710f7995 works for you?
Paolo
> Thread 4 (Thread 0x3fffb4ce910 (LWP 57750)):
> #0 0x000003fffc185e8e in syscall () from /lib64/libc.so.6
> #1 0x00000000803578ee in futex_wait (ev=0x8098486c <rcu_call_ready_event>,
> val=4294967295) at /home/cborntra/REPOS/qemu/util/qemu-thread-posix.c:301
> #2 0x0000000080357ae8 in qemu_event_wait (ev=0x8098486c
> <rcu_call_ready_event>) at
> /home/cborntra/REPOS/qemu/util/qemu-thread-posix.c:399
> #3 0x000000008036e202 in call_rcu_thread (opaque=0x0) at
> /home/cborntra/REPOS/qemu/util/rcu.c:233
> #4 0x000003fffc2374e6 in start_thread () from /lib64/libpthread.so.0
> #5 0x000003fffc18b0fa in thread_start () from /lib64/libc.so.6
>
> Thread 3 (Thread 0x3fffacce910 (LWP 57751)):
> #0 0x000003fffc17f5d6 in ppoll () from /lib64/libc.so.6
> #1 0x0000000080294c2e in qemu_poll_ns (fds=0x3fff40008c0, nfds=1,
> timeout=-1) at /home/cborntra/REPOS/qemu/qemu-timer.c:310
> #2 0x0000000080296788 in aio_poll (ctx=0x809c2830, blocking=true) at
> /home/cborntra/REPOS/qemu/aio-posix.c:271
> #3 0x0000000080137f58 in iothread_run (opaque=0x809c2450) at
> /home/cborntra/REPOS/qemu/iothread.c:42
> #4 0x000003fffc2374e6 in start_thread () from /lib64/libpthread.so.0
> #5 0x000003fffc18b0fa in thread_start () from /lib64/libc.so.6
>
> Thread 2 (Thread 0x3fff9411910 (LWP 57754)):
> #0 0x000003fffc23e662 in __lll_lock_wait () from /lib64/libpthread.so.0
> #1 0x000003fffc2394a4 in pthread_mutex_lock () from /lib64/libpthread.so.0
> #2 0x000000008035706e in qemu_mutex_lock (mutex=0x80559288
> <qemu_global_mutex>) at /home/cborntra/REPOS/qemu/util/qemu-thread-posix.c:73
> #3 0x000000008005ae46 in qemu_mutex_lock_iothread () at
> /home/cborntra/REPOS/qemu/cpus.c:1164
> #4 0x000000008012f44e in kvm_arch_handle_exit (cs=0x82141930,
> run=0x3fffd31a000) at /home/cborntra/REPOS/qemu/target-s390x/kvm.c:2010
> #5 0x00000000800782f8 in kvm_cpu_exec (cpu=0x82141930) at
> /home/cborntra/REPOS/qemu/kvm-all.c:1901
> #6 0x000000008005a73c in qemu_kvm_cpu_thread_fn (arg=0x82141930) at
> /home/cborntra/REPOS/qemu/cpus.c:977
> #7 0x000003fffc2374e6 in start_thread () from /lib64/libpthread.so.0
> #8 0x000003fffc18b0fa in thread_start () from /lib64/libc.so.6
>
> Thread 1 (Thread 0x3fffb4d0bd0 (LWP 57736)):
> #0 0x000003fffc23b57c in pthread_cond_wait@@GLIBC_2.3.2 () from
> /lib64/libpthread.so.0
> #1 0x00000000803572fa in qemu_cond_wait (cond=0x809c28c0, mutex=0x809c2890)
> at /home/cborntra/REPOS/qemu/util/qemu-thread-posix.c:132
> #2 0x000000008036dc3e in rfifolock_lock (r=0x809c2890) at
> /home/cborntra/REPOS/qemu/util/rfifolock.c:59
> #3 0x0000000080281162 in aio_context_acquire (ctx=0x809c2830) at
> /home/cborntra/REPOS/qemu/async.c:331
> #4 0x00000000800a2f08 in virtio_blk_data_plane_start (s=0x80a0d6f0) at
> /home/cborntra/REPOS/qemu/hw/block/dataplane/virtio-blk.c:285
> #5 0x00000000800a0bfe in virtio_blk_handle_output (vdev=0x809b5e18,
> vq=0x80a70940) at /home/cborntra/REPOS/qemu/hw/block/virtio-blk.c:599
> #6 0x00000000800d065c in virtio_queue_notify_vq (vq=0x80a70940) at
> /home/cborntra/REPOS/qemu/hw/virtio/virtio.c:921
> #7 0x00000000800d29b4 in virtio_queue_host_notifier_read (n=0x80a70988) at
> /home/cborntra/REPOS/qemu/hw/virtio/virtio.c:1480
> #8 0x0000000080293b76 in qemu_iohandler_poll (pollfds=0x809b3600, ret=1) at
> /home/cborntra/REPOS/qemu/iohandler.c:126
> #9 0x0000000080293714 in main_loop_wait (nonblocking=0) at
> /home/cborntra/REPOS/qemu/main-loop.c:494
> #10 0x000000008014dc1c in main_loop () at /home/cborntra/REPOS/qemu/vl.c:1902
> #11 0x000000008015627c in main (argc=44, argv=0x3ffffcd66e8,
> envp=0x3ffffcd6850) at /home/cborntra/REPOS/qemu/vl.c:4653
>
>
>
- [Qemu-devel] [PULL v2 for-2.4 v2 2/7] qemu-timer: initialize "timers_done_ev" to set, (continued)
- [Qemu-devel] [PULL v2 for-2.4 v2 2/7] qemu-timer: initialize "timers_done_ev" to set, Stefan Hajnoczi, 2015/07/22
- [Qemu-devel] [PULL v2 for-2.4 v2 3/7] tests: remove irrelevant assertions from test-aio, Stefan Hajnoczi, 2015/07/22
- [Qemu-devel] [PULL v2 for-2.4 v2 4/7] aio-win32: reorganize polling loop, Stefan Hajnoczi, 2015/07/22
- [Qemu-devel] [PULL v2 for-2.4 v2 6/7] AioContext: fix broken placement of event_notifier_test_and_clear, Stefan Hajnoczi, 2015/07/22
- [Qemu-devel] [PULL v2 for-2.4 v2 7/7] AioContext: optimize clearing the EventNotifier, Stefan Hajnoczi, 2015/07/22
- [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Stefan Hajnoczi, 2015/07/22
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Cornelia Huck, 2015/07/23
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Paolo Bonzini, 2015/07/23
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Paolo Bonzini, 2015/07/23
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Christian Borntraeger, 2015/07/23
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization,
Paolo Bonzini <=
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Christian Borntraeger, 2015/07/23
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Cornelia Huck, 2015/07/24
- Re: [Qemu-devel] [PULL v2 for-2.4 v2 5/7] AioContext: fix broken ctx->dispatching optimization, Stefan Hajnoczi, 2015/07/27
Re: [Qemu-devel] [PULL v2 for-2.4 v2 0/7] Block patches, Peter Maydell, 2015/07/22