From: "Alex Bennée" <alex.bennee@linaro.org>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [PULL 0/9] final misc fixes for 5.1-rc0
Date: Wed, 15 Jul 2020 11:51:30 +0100 [thread overview]
Message-ID: <87lfjlxd8t.fsf@linaro.org> (raw)
In-Reply-To: <CAFEAcA-4U-LAjcBPWQNrtfk=Kr2zsejHGYCzLWSoYq97_o9j8g@mail.gmail.com>
Peter Maydell <peter.maydell@linaro.org> writes:
> On Tue, 14 Jul 2020 at 10:52, Alex Bennée <alex.bennee@linaro.org> wrote:
>>
>> The following changes since commit 20c1df5476e1e9b5d3f5b94f9f3ce01d21f14c46:
>>
>> Merge remote-tracking branch 'remotes/kraxel/tags/fixes-20200713-pull-request' into staging (2020-07-13 16:58:44 +0100)
>>
>> are available in the Git repository at:
>>
>> https://github.com/stsquad/qemu.git tags/pull-misc-for-rc0-140720-1
>>
>> for you to fetch changes up to 4e6400c97024c2660b6db8aab1f0677744449f36:
>>
>> plugins: expand the bb plugin to be thread safe and track per-cpu (2020-07-14 09:08:25 +0100)
>>
>> ----------------------------------------------------------------
>> Final fixes for 5.1-rc0
>>
>> - minor documentation nit
>> - clean all deps on re(configure)
>> - docker.py bootstrap fixes
>> - tweak containers.yml wildcards
>> - fix float16 nan detection
>> - conditional use of -Wpsabi
>> - fix missing iotlb data for plugins
>> - proper locking for helper based bb count
>
> Fails "make check" on aarch32 and aarch64 hosts, because
> eg qemu-system-arm segfaults on startup:
>
> $ gdb --args ./arm-softmmu/qemu-system-arm -M virt -display none
> [...]
> (gdb) r
> Starting program: /home/pm/qemu/build/all/arm-softmmu/qemu-system-arm
> -M virt -display none
> [Thread debugging using libthread_db enabled]
> Using host libthread_db library "/lib/aarch64-linux-gnu/libthread_db.so.1".
> [New Thread 0xffffe873e550 (LWP 1666261)]
> [New Thread 0xffffe7f3d550 (LWP 1666262)]
> [New Thread 0xffffe72e7550 (LWP 1666263)]
>
> Thread 4 "qemu-system-arm" received signal SIGSEGV, Segmentation fault.
> [Switching to Thread 0xffffe72e7550 (LWP 1666263)]
> 0x0000aaaaab1da004 in object_class_dynamic_cast_assert (class=0xaaaaabc209e0,
> typename=typename@entry=0xaaaaab33bfe8 "cpu",
> file=file@entry=0xaaaaab3201a0 "/home/pm/qemu/accel/tcg/cputlb.c",
> line=line@entry=1025, func=func@entry=0xaaaaab3202f0
> <__func__.35278> "tlb_fill") at /home/pm/qemu/qom/object.c:917
> 917 trace_object_class_dynamic_cast_assert(class ?
> class->type->name : "(null)",
> (gdb) bt
> #0 0x0000aaaaab1da004 in object_class_dynamic_cast_assert
> (class=0xaaaaabc209e0, typename=typename@entry=0xaaaaab33bfe8
> "cpu", file=file@entry=0xaaaaab3201a0
> "/home/pm/qemu/accel/tcg/cputlb.c", line=line@entry=1025,
> func=func@entry=0xaaaaab3202f0 <__func__.35278> "tlb_fill")
> at /home/pm/qemu/qom/object.c:917
> #1 0x0000aaaaaaded754 in tlb_fill
> (cpu=0xaaaaabd301c0, addr=0, size=0, access_type=MMU_INST_FETCH,
> mmu_idx=2, retaddr=0)
> at /home/pm/qemu/accel/tcg/cputlb.c:1025
> #2 0x0000aaaaaadf0e08 in get_page_addr_code_hostp
> (env=0xaaaaabd39870, addr=addr@entry=0, hostp=hostp@entry=0x0)
> at /home/pm/qemu/include/exec/cpu-all.h:451
> #3 0x0000aaaaaadf0e70 in get_page_addr_code (env=<optimized out>,
> addr=addr@entry=0)
> at /home/pm/qemu/accel/tcg/cputlb.c:1243
> #4 0x0000aaaaaae08c30 in tb_htable_lookup
> (cpu=cpu@entry=0xaaaaabd301b0, pc=0, cs_base=<optimized out>,
> flags=38928384, cf_mask=4278714368)
> at /home/pm/qemu/accel/tcg/cpu-exec.c:337
> #5 0x0000aaaaaae09b14 in tb_lookup__cpu_state
> (cf_mask=<optimized out>, flags=0xffffe72e6b48,
> cs_base=0xffffe72e6b40, pc=0xffffe72e6b44, cpu=0xaaaaabd301b0)
> at /home/pm/qemu/include/exec/tb-lookup.h:43
> #6 tb_find (cf_mask=<optimized out>, tb_exit=0, last_tb=0x0,
> cpu=0xaaaaabd301b0)
> at /home/pm/qemu/accel/tcg/cpu-exec.c:404
> #7 cpu_exec (cpu=cpu@entry=0xaaaaabd301b0) at
> /home/pm/qemu/accel/tcg/cpu-exec.c:731
> #8 0x0000aaaaaaec45c4 in tcg_cpu_exec (cpu=0xaaaaabd301b0) at
> /home/pm/qemu/softmmu/cpus.c:1356
> #9 0x0000aaaaaaec69ec in qemu_tcg_cpu_thread_fn
> (arg=arg@entry=0xaaaaabd301b0) at /home/pm/qemu/softmmu/cpus.c:1664
> #10 0x0000aaaaab2e1758 in qemu_thread_start (args=<optimized out>) at
> /home/pm/qemu/util/qemu-thread-posix.c:521
> #11 0x0000fffff57be4fc in start_thread (arg=0xffffffffe10f) at
> pthread_create.c:477
> #12 0x0000fffff5716f2c in thread_start () at
> ../sysdeps/unix/sysv/linux/aarch64/clone.S:78
This goes away after a make clean and rebuild. I suspect the commit:
configure: remove all dependencies on a (re)configure
just causes the build to miss more of the bits on a rebuild. I'll drop
it for now and just live with the random syscall_nr.h files.
--
Alex Bennée
next prev parent reply other threads:[~2020-07-15 10:52 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-14 9:52 [PULL 0/9] final misc fixes for 5.1-rc0 Alex Bennée
2020-07-14 9:52 ` [PULL 1/9] docs/devel: fix grammar in multi-thread-tcg Alex Bennée
2020-07-14 9:52 ` [PULL 2/9] configure: remove all dependencies on a (re)configure Alex Bennée
2020-07-14 9:52 ` [PULL 3/9] tests/docker: Remove the libssh workaround from the ubuntu 20.04 image Alex Bennée
2020-07-14 9:52 ` [PULL 4/9] docker.py: fix fetching of FROM layers Alex Bennée
2020-07-14 9:52 ` [PULL 5/9] gitlab-ci/containers: Add missing wildcard where we should look for changes Alex Bennée
2020-07-14 9:52 ` [PULL 6/9] fpu/softfloat: fix up float16 nan recognition Alex Bennée
2020-07-14 9:52 ` [PULL 7/9] tests/plugins: don't unconditionally add -Wpsabi Alex Bennée
2020-07-14 9:52 ` [PULL 8/9] cputlb: ensure we save the IOTLB data in case of reset Alex Bennée
2020-07-14 9:52 ` [PULL 9/9] plugins: expand the bb plugin to be thread safe and track per-cpu Alex Bennée
2020-07-15 8:06 ` [PULL 0/9] final misc fixes for 5.1-rc0 Peter Maydell
2020-07-15 10:51 ` Alex Bennée [this message]
2020-07-15 12:29 ` Philippe Mathieu-Daudé
2020-07-15 12:51 ` Peter Maydell
2020-07-15 12:54 ` Philippe Mathieu-Daudé
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87lfjlxd8t.fsf@linaro.org \
--to=alex.bennee@linaro.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).