From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43720) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1evoaa-0002Ge-7Z for qemu-devel@nongnu.org; Tue, 13 Mar 2018 14:23:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1evoaX-0005OO-1K for qemu-devel@nongnu.org; Tue, 13 Mar 2018 14:23:56 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:40238 helo=mx1.redhat.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1evoaW-0005O6-RV for qemu-devel@nongnu.org; Tue, 13 Mar 2018 14:23:52 -0400 Date: Tue, 13 Mar 2018 18:23:43 +0000 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Message-ID: <20180313182343.GF3784@redhat.com> Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= References: <20180312201305.16972-1-berrange@redhat.com> <20180313162708.GB3784@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20180313162708.GB3784@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL 0/9] Socket next patches List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Markus Armbruster , QEMU Developers , Gerd Hoffmann , =?utf-8?Q?Marc-Andr=C3=A9?= Lureau , Paolo Bonzini On Tue, Mar 13, 2018 at 04:27:08PM +0000, Daniel P. Berrang=C3=A9 wrote: > On Tue, Mar 13, 2018 at 04:20:19PM +0000, Peter Maydell wrote: > > On 12 March 2018 at 20:12, Daniel P. Berrang=C3=A9 wrote: > > > The following changes since commit 819fd4699c7b36d574292bcbd8bc25e9= d716c84b: > > > > > > Merge remote-tracking branch 'remotes/dgilbert/tags/pull-migratio= n-20180309a' into staging (2018-03-12 13:21:53 +0000) > > > > > > are available in the Git repository at: > > > > > > https://github.com/berrange/qemu tags/socket-next-pull-request > > > > > > for you to fetch changes up to f4d2a296de2ccf5ff80ddd343c09f0075d10= 583a: > > > > > > char: allow passing pre-opened socket file descriptor at startup = (2018-03-12 17:50:52 +0000) > > > > > > ---------------------------------------------------------------- > > > > > > ---------------------------------------------------------------- > > > > > > Daniel P. Berrang=C3=A9 (9): > > > char: don't silently skip tn3270 protocol init when TLS is enable= d > > > cutils: add qemu_strtoi & qemu_strtoui parsers for int/unsigned i= nt > > > types > > > sockets: pull code for testing IP availability out of specific te= st > > > sockets: strengthen test suite IP protocol availability checks > > > sockets: move fd_is_socket() into common sockets code > > > sockets: check that the named file descriptor is a socket > > > sockets: allow SocketAddress 'fd' to reference numeric file > > > descriptors > > > char: refactor parsing of socket address information > > > char: allow passing pre-opened socket file descriptor at startup > > > > >=20 > > Test failure, 32-bit arm: > >=20 > > MALLOC_PERTURB_=3D${MALLOC_PERTURB_:-$(( ${RANDOM:-0} % 255 + 1))} > > gtester -k --verbose -m=3Dquick tests/test-c > > utils > > TEST: tests/test-cutils... (pid=3D21029) > > /cutils/parse_uint/null: = OK > > [etc] > > /cutils/parse_uint_full/trailing: = ** > > ERROR:/home/peter.maydell/qemu/tests/test-cutils.c:715:test_qemu_strt= oui_underflow: > > assertion failed (err =3D > > =3D -ERANGE): (0 =3D=3D -34) >=20 > This will be a genuine bug. Not sure if its in the test case or code > yet.... >=20 > > OK > > /cutils/parse_uint_full/correct: = OK > >=20 > > Test failure, x86-64 Linux: > >=20 > > TEST: tests/boot-serial-test... (pid=3D11205) > > /sparc64/boot-serial/sun4u: = ** > > ERROR:/home/petmay01/linaro/qemu-for-merges/tests/boot-serial-test.c:= 137:check_guest_output: > > assertion failed: (output_ok) > > FAIL > > GTester: last random seed: R02S99166ae475faeb3280608eeed6d61c4f > > (pid=3D15216) >=20 > This feels unrelated to the series, so possibly a non-deterministic > failure Looking at the test source code, it has a 60 second wait for the emulator to print expected data on the serial port. My guess would be that when running with a highly parallel make check we're sometimes exceeding that 60 seconds. I don't see a reason why my series would have affected this in a way that only failed with sparc64 qtest, and none others. >=20 > >=20 > > Test failure, 64-bit arm: > >=20 > > ERROR:/home/pm215/qemu/tests/test-aio-multithread.c:365:test_multi_fa= ir_mutex: > > assertion failed (counter =3D=3D > > atomic_counter): (59827 =3D=3D 59828) >=20 > I'm sure I've seen this non-deterministic failure before, unrelated to > this series. Regards, Daniel --=20 |: https://berrange.com -o- https://www.flickr.com/photos/dberran= ge :| |: https://libvirt.org -o- https://fstop138.berrange.c= om :| |: https://entangle-photo.org -o- https://www.instagram.com/dberran= ge :|