From: "Alex Bennée" <alex.bennee@linaro.org>
To: qemu-devel@nongnu.org
Cc: qemu-arm@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>
Subject: [Qemu-devel] [PATCH v2 09/16] tests: make docker.py check for persistent configs
Date: Thu, 7 Feb 2019 18:37:37 +0000 [thread overview]
Message-ID: <20190207183744.5054-10-alex.bennee@linaro.org> (raw)
In-Reply-To: <20190207183744.5054-1-alex.bennee@linaro.org>
binfmt_misc configured with the "F" flag opens the interpreter at
config time. This means it can use an already open file-descriptor to
run QEMU so there is no point trying to copy the binary into a
container.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/docker/docker.py | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/tests/docker/docker.py b/tests/docker/docker.py
index 30f463af9f..768728785f 100755
--- a/tests/docker/docker.py
+++ b/tests/docker/docker.py
@@ -127,6 +127,11 @@ def _check_binfmt_misc(executable):
with open(binfmt_entry) as x: entry = x.read()
+ if re.search("flags:.*F.*\n", entry):
+ print("binfmt_misc for %s uses persistent(F) mapping to host binary\n" %
+ (binary))
+ return None
+
m = re.search("interpreter (\S+)\n", entry)
interp = m.group(1)
if interp and interp != executable:
--
2.20.1
next prev parent reply other threads:[~2019-02-07 18:38 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-07 18:37 [Qemu-devel] [PATCH v2 00/16] current testing/next queue (with build fixes) Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 01/16] .cirrus.yml: basic compile and test for FreeBSD Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 02/16] .travis.yml: stop requesting libffi & gettext from homebrew Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 03/16] .travis.yml: separate tools and docs into another entry Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 04/16] .travis.yml: fold --disable-tcg into alternate coroutine builds Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 05/16] MAINTAINERS: Add an entry for scripts/archive-source.sh Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 06/16] archive-source.sh: Clone the submodules locally Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 07/16] docker: add debian-buster-arm64-cross Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 08/16] tests: make docker.py update use configured binfmt path Alex Bennée
2019-02-07 18:37 ` Alex Bennée [this message]
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 10/16] tests: docker.py be even smarter with persistent binfmt_misc Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 11/16] tests: PEP8 cleanup of docker.py, mostly white space Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 12/16] tests/vm: move images to $HOME/.cache/qemu-vm/images Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 13/16] tests/vm: call make check directly for netbsd/freebsd/ubuntu.i386 Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 14/16] tests/vm: add --build-target option Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 15/16] tests/vm: expose BUILD_TARGET, TARGET_LIST and EXTRA_CONFIGURE_OPTS Alex Bennée
2019-02-07 18:37 ` [Qemu-devel] [PATCH v2 16/16] scripts/qemu.py: allow arches use KVM for their 32bit cousins Alex Bennée
2019-02-07 22:01 ` [Qemu-devel] [PATCH v2 00/16] current testing/next queue (with build fixes) 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=20190207183744.5054-10-alex.bennee@linaro.org \
--to=alex.bennee@linaro.org \
--cc=qemu-arm@nongnu.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).