From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53105) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1boUmy-000445-A1 for qemu-devel@nongnu.org; Mon, 26 Sep 2016 08:13:44 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1boUmt-0005CH-WE for qemu-devel@nongnu.org; Mon, 26 Sep 2016 08:13:40 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38278) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1boUmt-0005Bz-Mb for qemu-devel@nongnu.org; Mon, 26 Sep 2016 08:13:35 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 341B746291 for ; Mon, 26 Sep 2016 12:13:34 +0000 (UTC) Date: Mon, 26 Sep 2016 09:13:31 -0300 From: Eduardo Habkost Message-ID: <20160926121331.GF3877@thinpad.lan.raisama.net> References: <20160923153612.GU3233@thinpad.lan.raisama.net> <20160923184040-mutt-send-email-mst@kernel.org> <29325c65-7ee6-22ed-31b3-211bcaceef50@redhat.com> <4f7bb8d9-a9da-5c06-a0b7-20dea8b802f8@redhat.com> <274086782.1698878.1474836953815.JavaMail.zimbra@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <274086782.1698878.1474836953815.JavaMail.zimbra@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] vhost-user-test failure List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?Marc-Andr=E9?= Lureau Cc: "Michael S. Tsirkin" , qemu-devel@nongnu.org, =?iso-8859-1?Q?Marc-Andr=E9?= Lureau , Maxime Coquelin On Sun, Sep 25, 2016 at 04:55:53PM -0400, Marc-Andr=E9 Lureau wrote: > Hi >=20 > ----- Original Message ----- > > This time with Marc-Andr=E9 in cc:... > >=20 > > On 09/23/2016 07:40 PM, Maxime Coquelin wrote: > > > > > > > > > On 09/23/2016 05:41 PM, Michael S. Tsirkin wrote: > > >> On Fri, Sep 23, 2016 at 12:36:12PM -0300, Eduardo Habkost wrote: > > >>> Hi, > > >>> > > >>> I hit a weird vhost-user-test failure on travis-ci recently, on a > > >>> branch where I didn't touch any vhost-related code. From a quick > > >>> look at the code, it looks like the vhost-user code is unhappy to > > >>> see a disconnected socket. > > >>> > > >>> I wasn't able to reproduce it. It seems to be a hard to reproduce > > >>> race between vhost-user code and socket reconnection. > > >>> > > >>> The failure can be seen at: > > >>> > > >>> https://travis-ci.org/ehabkost/qemu-hacks/jobs/162077239 > > >> > > >> Maxime looked at something similiar. Any idea? > > > No, not really. > > > Marc-Andr=E9 contributed a lot to these tests, I add him in cc: in = case > > > he has an idea. > > > > > > I will have a look in the mean time. > > > >=20 > I am unable to reproduce locally (over 500x iterations), and I > have no clue what's going on: the warnings there aren't the > problem (that's the main reason why we use the subprocess, to > silence those). Do you have a local reproducer or is it only on > travis? Afaik, there are no other reports of this test failing, > are you sure its not related to changes on your branch? I don't have a local reproducer, I could only see it once on travis-ci. Maybe it is not possible to reproduce it if the machine isn't loaded enough to make the right thread/process be delayed. I am pretty sure it's not related to my changes. Below is the diffstat between master and the commit that was being tested. All the changes were limited to x86 CPUID code (which shouldn't affect qtest code at all). $ git diff --stat master...8de32e0 include/hw/i386/pc.h | 7 +- include/sysemu/cpus.h | 5 +- target-i386/cpu.c | 567 ++++++++++++++++++++++++++++++++++++= +++++++++++++++++++++++++++++++++++++++----------------------------------= ----------------------- target-i386/cpu.h | 15 +++- target-ppc/translate_init.c | 3 +- tests/Makefile.include | 2 + tests/test-x86-cpuid-compat.c | 171 ++++++++++++++++++++++++++++++++++++= ++++ 7 files changed, 516 insertions(+), 254 deletions(-) --=20 Eduardo