On 01.03.2017 12:36, Alex Bennée wrote: > > Peter Maydell <peter.mayd...@linaro.org> writes: > >> I got a make check failure on aarch64 host running a sparc64 test: >> >> >> TEST: tests/prom-env-test... (pid=13573) >> /sparc64/prom-env/sun4u: ** >> ERROR:/home/pm215/qemu/translate-common.c:34:tcg_handle_interrupt: >> assertion failed: (qemu_mutex_iothread_locked()) > > So the assertions where added with MTTCG. The design specifies which > bits should be protected by the BQL and cpu->interrupt_request is one of > them. This is because cpu->interrupt_request is often a cross-vCPU > action (one vCPU triggering an interrupt on another) so there is a > chance of racing if not protected. > > It's odd this is showing up on a aarch64 host though when it didn't hit > on my x86_64 host while testing. > > As most of this stuff is triggered by hardware emulation the BQL should > be in effect when handling MMIO for device emulation. There where other > entry points in ARM which could trigger stuff which is why we add > locking for things like ARM_CP_IO which are co-processor register > accesses which trigger other things in the system. > > What will be useful for all these reports is the backtrace. Then it's > fairly simple to identify the thing triggering the interrupt and > identify the correct place for the locking.
Here are the backtraces from the s390x moon buggy image: Thread 3 (Thread 0x7fffdc608700 (LWP 14468)): #0 0x00007ffff18ef1d7 in raise () at /lib64/libc.so.6 #1 0x00007ffff18f08c8 in abort () at /lib64/libc.so.6 #2 0x00007ffff2f642a5 in g_assertion_message () at /lib64/libglib-2.0.so.0 #3 0x00007ffff2f6433a in g_assertion_message_expr () at /lib64/libglib-2.0.so.0 #4 0x000055555560bd31 in tcg_handle_interrupt (cpu=0x55555612fc40, mask=2) at /home/thuth/devel/qemu/translate-common.c:34 #5 0x000055555568fe03 in css_do_ssch (sch=sch@entry=0x5555561740d0, orb=orb@entry=0x7fffdc607400) at /home/thuth/devel/qemu/hw/s390x/css.c:945 #6 0x00005555556b99ad in ioinst_handle_ssch (cpu=0x55555612fc40, reg1=<optimized out>, ipb=<optimized out>) at /home/thuth/devel/qemu/target/s390x/ioinst.c:238 #7 0x00007fffe60957be in code_gen_buffer () #8 0x000055555560b49d in cpu_exec (itb=<optimized out>, itb=<optimized out>, cpu=0x7fffe52dc790) at /home/thuth/devel/qemu/cpu-exec.c:165 #9 0x000055555560b49d in cpu_exec (sc=0x7fffdc6079b0, tb_exit=<synthetic pointer>, last_tb=<synthetic pointer>, tb=<optimized out>, cpu=0x7fffe52dc790) at /home/thuth/devel/qemu/cpu-exec.c:584 #10 0x000055555560b49d in cpu_exec (cpu=cpu@entry=0x55555612fc40) at /home/thuth/devel/qemu/cpu-exec.c:686 #11 0x000055555563677a in tcg_cpu_exec (cpu=0x55555612fc40) at /home/thuth/devel/qemu/cpus.c:1251 #12 0x0000555555636ab4 in qemu_tcg_rr_cpu_thread_fn (arg=<optimized out>) at /home/thuth/devel/qemu/cpus.c:1347 #13 0x00007ffff53b3dc5 in start_thread () at /lib64/libpthread.so.0 #14 0x00007ffff19b173d in clone () at /lib64/libc.so.6 Thread 2 (Thread 0x7fffe82b5700 (LWP 14467)): #0 0x00007ffff19abbf9 in syscall () at /lib64/libc.so.6 #1 0x0000555555853896 in qemu_event_wait (val=<optimized out>, f=<optimized out>) at /home/thuth/devel/qemu/include/qemu/futex.h:26 #2 0x0000555555853896 in qemu_event_wait (ev=ev@entry=0x555556082284 <rcu_call_ready_event>) at /home/thuth/devel/qemu/util/qemu-thread-posix.c:399 #3 0x000055555586243e in call_rcu_thread (opaque=<optimized out>) at /home/thuth/devel/qemu/util/rcu.c:249 #4 0x00007ffff53b3dc5 in start_thread () at /lib64/libpthread.so.0 #5 0x00007ffff19b173d in clone () at /lib64/libc.so.6 Thread 1 (Thread 0x7ffff7f91c00 (LWP 14463)): #0 0x00007ffff19a6ebf in ppoll () at /lib64/libc.so.6 #1 0x000055555584f819 in qemu_poll_ns (__ss=0x0, __timeout=0x7fffffffda20, __nfds=<optimized out>, __fds=<optimized out>) at /usr/include/bits/poll2.h:77 #2 0x000055555584f819 in qemu_poll_ns (fds=<optimized out>, nfds=<optimized out>, timeout=timeout@entry=9897590) at /home/thuth/devel/qemu/util/qemu-timer.c:333 #3 0x00005555558505e8 in main_loop_wait (timeout=9897590) at /home/thuth/devel/qemu/util/main-loop.c:254 #4 0x00005555558505e8 in main_loop_wait (nonblocking=<optimized out>) at /home/thuth/devel/qemu/util/main-loop.c:508 #5 0x00005555555f83b9 in main () at /home/thuth/devel/qemu/vl.c:1897 #6 0x00005555555f83b9 in main (argc=<optimized out>, argv=<optimized out>, envp=<optimized out>) at /home/thuth/devel/qemu/vl.c:4675 HTH2, Thomas