From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34484) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fht7q-0003EA-AQ for qemu-devel@nongnu.org; Tue, 24 Jul 2018 04:56:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fht7m-0003Pb-7Q for qemu-devel@nongnu.org; Tue, 24 Jul 2018 04:56:58 -0400 Received: from mail-wr1-x444.google.com ([2a00:1450:4864:20::444]:33645) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fht7m-0003PM-0C for qemu-devel@nongnu.org; Tue, 24 Jul 2018 04:56:54 -0400 Received: by mail-wr1-x444.google.com with SMTP id g6-v6so3331448wrp.0 for ; Tue, 24 Jul 2018 01:56:53 -0700 (PDT) References: <20180717195553.9111-1-alex.bennee@linaro.org> <87o9eytgju.fsf@linaro.org> From: Alex =?utf-8?Q?Benn=C3=A9e?= In-reply-to: Date: Tue, 24 Jul 2018 09:56:51 +0100 Message-ID: <87efftt3jg.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v3 for 3.0 00/18] docker fixes (and one tcg test tweak) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng Cc: cota@braap.org, "Daniel P. Berrange" , Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= , Richard Henderson , balrogg@gmail.com, aurelien@aurel32.net, Alexander Graf , QEMU Developers Fam Zheng writes: > On Mon, Jul 23, 2018 at 6:03 PM Alex Benn=C3=A9e = wrote: >> >> >> Alex Benn=C3=A9e writes: >> >> > Hi, >> > >> > I've missed the boat for today's rc1 but I'd like to get this merged >> > before rc2. The new docker.py change is technically new functionality >> > but I'm counting it as a usability bug fix as it replaces a random >> > back trace failure with a preemptive failure and message mentioning >> > binfmt_misc configuration. This would have saved Richard a lot of head >> > scratching as he tried to setup a powerpc-user setup to test his >> > setcontext fix (he had a custom binfmt_misc pointing to his src tree). >> > >> > Finally we also drop the runcom test. It was cute that it got >> > resurrected but it is ultimately a pointless test for something I'm >> > sure no one actually uses. >> > >> > There will be a follow-up RFC series after this that cleans-up some of >> > the rough edges when your host is not an x86_64 box but that series >> > won't be targeting the 3.0 release. >> > >> > : The following patches need review >> > : patch docker/disable debian powerpc user cross.patch >> > : patch docker/drop QEMU_TARGET check fallback in EXECUTABLE.patch >> > : patch docker/Update debootstrap script after Debian migrat.patch >> > : patch docker/ignore distro versioning of debootstrap.patch >> > : patch docker/perform basic binfmt_misc validation in docke.patch >> > : patch tests/tcg remove runcom test.patch >> >> Ping? > > I had questions about patch 13 and 17. Otherwise looks good. > =E2=80=8B Maybe you can drop them for now (including patch 9) and send th= e PULL for > the rest.=E2=80=8B I can drop/replace 13 - but 17 was there for a better reporting of a failure case Peter found. We can certainly expand it and be smarter in future iterations. > I don't know if we can catch up -rc2 but I guess the testing fixes are > fairly safe to sneak in. :) > > (My development machine is affected by > an > office power outage, and I was > =E2=80=8B also=E2=80=8B > busy with upgrading patchew.org. Sorry for the > =E2=80=8B =E2=80=8B > late reply > !) > > Fam -- Alex Benn=C3=A9e