From: "Alex Bennée" <alex.bennee@linaro.org>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Thomas Huth <thuth@redhat.com>,
qemu-s390x@nongnu.org, Cornelia Huck <cohuck@redhat.com>,
qemu-devel@nongnu.org, Laurent Vivier <laurent@vivier.eu>
Subject: Re: [PULL 00/31] testing/next and other misc fixes
Date: Thu, 13 Jan 2022 10:13:39 +0000 [thread overview]
Message-ID: <87h7a89bnm.fsf@linaro.org> (raw)
In-Reply-To: <CAFEAcA--rEZ0+JxFzeSFD1j6FM65_x3CXcn_oTnaNkNVU1hW0Q@mail.gmail.com>
Peter Maydell <peter.maydell@linaro.org> writes:
(adding the s390x people to the CC if they have any clues)
> On Wed, 12 Jan 2022 at 11:27, Alex Bennée <alex.bennee@linaro.org> wrote:
>>
>> The following changes since commit bf99e0ec9a51976868d7a8334620716df15fe7fe:
>>
>> Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2022-01-11 10:12:29 +0000)
>>
>> are available in the Git repository at:
>>
>> https://github.com/stsquad/qemu.git tags/pull-for-7.0-110122-1
>>
>> for you to fetch changes up to dbd30b7abee963f4fb08892a7d7f920bb76ece58:
>>
>> linux-user: Remove the deprecated ppc64abi32 target (2022-01-11 13:00:53 +0000)
>>
<snip>
> This seems to fail the ubuntu-18.04-s390x-all-linux-static job
> with segfaults running linux-user binaries (not always the same
> binary), eg:
> https://gitlab.com/qemu-project/qemu/-/jobs/1968789446
> https://gitlab.com/qemu-project/qemu/-/jobs/1968080419
*sigh*
So the regression is caused by:
linux-user: don't adjust base of found hole
However it only occurs when pgb_static starts base at a low address. For
example:
pgb_find_hole: base @ 13dd000 for 17432080 bytes
pgb_static: base @ 13dd000 for 17432080 bytes
Locating guest address space @ 0x13dd000
fails whereas:
pgb_find_hole: base @ 41f97000 for 17432080 bytes
pgb_static: base @ 41f97000 for 17432080 bytes
Locating guest address space @ 0x41f97000
works.
What I find confusing is why we end up with different addresses when
both QEMU and the test binary are static allocations. However the
varying allocation occurs before the change but without triggering the
crash:
pgb_static: base @ 3dd000 for 17432080 bytes
pgb_static: base @ 3dd000 for 17432080 bytes
pgb_static: base @ 41246000 for 17432080 bytes
pgb_static: base @ 3dd000 for 17432080 bytes
pgb_static: base @ 40a2a000 for 17432080 bytes
pgb_static: base @ 3dd000 for 17432080 bytes
pgb_static: base @ 3dd000 for 17432080 bytes
pgb_static: base @ 4060c000 for 17432080 bytes
pgb_static: base @ 3dd000 for 17432080 bytes
pgb_static: base @ 3dd000 for 17432080 bytes
pgb_static: base @ 3dd000 for 17432080 bytes
>
>
> thanks
> -- PMM
--
Alex Bennée
next prev parent reply other threads:[~2022-01-13 10:26 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-12 11:26 [PULL 00/31] testing/next and other misc fixes Alex Bennée
2022-01-12 11:26 ` [PULL 01/31] ui: avoid compiler warnings from unused clipboard info variable Alex Bennée
2022-01-12 11:26 ` [PULL 02/31] spice: Update QXLInterface for spice >= 0.15.0 Alex Bennée
2022-01-12 11:26 ` [PULL 03/31] meson: require liburing >= 0.3 Alex Bennée
2022-01-12 11:26 ` [PULL 04/31] ui: avoid warnings about directdb on Alpine / musl libc Alex Bennée
2022-01-12 11:26 ` [PULL 05/31] ci: explicitly skip I/O tests on alpine Alex Bennée
2022-01-12 11:26 ` [PULL 06/31] tests/docker: switch fedora image to release 35 Alex Bennée
2022-01-12 11:26 ` [PULL 07/31] tests: integrate lcitool for generating build env manifests Alex Bennée
2022-01-12 11:26 ` [PULL 08/31] tests/docker: auto-generate centos8.docker with lcitool Alex Bennée
2022-01-12 11:27 ` [PULL 09/31] tests/docker: auto-generate fedora.docker " Alex Bennée
2022-01-12 11:27 ` [PULL 10/31] tests/docker: auto-generate ubuntu1804.docker " Alex Bennée
2022-01-12 11:27 ` [PULL 11/31] tests/docker: auto-generate ubuntu2004.docker " Alex Bennée
2022-01-12 11:27 ` [PULL 12/31] tests/docker: auto-generate opensuse-leap.docker " Alex Bennée
2022-01-12 11:27 ` [PULL 13/31] tests/docker: remove ubuntu.docker container Alex Bennée
2022-01-12 11:27 ` [PULL 14/31] .gitlab-ci.d/cirrus: auto-generate variables with lcitool Alex Bennée
2022-01-12 11:27 ` [PULL 15/31] tests/docker: updates to alpine package list Alex Bennée
2022-01-12 11:27 ` [PULL 16/31] tests/docker: fix sorting of alpine image package lists Alex Bennée
2022-01-12 11:27 ` [PULL 17/31] tests/docker: fully expand the alpine package list Alex Bennée
2022-01-12 11:27 ` [PULL 18/31] tests/docker: auto-generate alpine.docker with lcitool Alex Bennée
2022-01-12 11:27 ` [PULL 19/31] tests/tcg: use CONFIG_LINUX_USER, not CONFIG_LINUX Alex Bennée
2022-01-12 11:27 ` [PULL 20/31] tests/docker: add libfuse3 development headers Alex Bennée
2022-01-12 11:27 ` [PULL 21/31] hw/arm: add control knob to disable kaslr_seed via DTB Alex Bennée
2022-01-12 11:27 ` [PULL 22/31] monitor: move x-query-profile into accel/tcg to fix build Alex Bennée
2022-01-12 11:27 ` [PULL 23/31] docs/devel: update C standard to C11 Alex Bennée
2022-01-12 11:27 ` [PULL 24/31] docs/devel: more documentation on the use of suffixes Alex Bennée
2022-01-12 11:27 ` [PULL 25/31] linux-user/elfload: add extra logging for hole finding Alex Bennée
2022-01-12 11:27 ` [PULL 26/31] linux-user: don't adjust base of found hole Alex Bennée
2022-01-12 11:27 ` [PULL 27/31] tests/tcg/multiarch: Read fp flags before printf Alex Bennée
2022-01-12 11:27 ` [PULL 28/31] test/tcg/ppc64le: Add float reference files Alex Bennée
2022-01-12 11:27 ` [PULL 29/31] FreeBSD: Upgrade to 12.3 release Alex Bennée
2022-01-12 11:27 ` [PULL 30/31] docker: include bison in debian-tricore-cross Alex Bennée
2022-01-12 11:27 ` [PULL 31/31] linux-user: Remove the deprecated ppc64abi32 target Alex Bennée
2022-02-07 8:51 ` Thomas Huth
2022-02-07 10:20 ` Alex Bennée
2022-01-12 21:20 ` [PULL 00/31] testing/next and other misc fixes Peter Maydell
2022-01-13 10:13 ` Alex Bennée [this message]
2022-01-13 12:49 ` Alex Bennée
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=87h7a89bnm.fsf@linaro.org \
--to=alex.bennee@linaro.org \
--cc=cohuck@redhat.com \
--cc=laurent@vivier.eu \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=thuth@redhat.com \
/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).