From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45222) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gCrjI-0000J0-K1 for qemu-devel@nongnu.org; Wed, 17 Oct 2018 15:43:41 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gCrjE-0002yd-ND for qemu-devel@nongnu.org; Wed, 17 Oct 2018 15:43:40 -0400 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:48918 helo=mx0a-001b2d01.pphosted.com) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gCrjB-0002oC-R0 for qemu-devel@nongnu.org; Wed, 17 Oct 2018 15:43:34 -0400 Received: from pps.filterd (m0098420.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.22/8.16.0.22) with SMTP id w9HJYB3m026504 for ; Wed, 17 Oct 2018 15:43:23 -0400 Received: from e12.ny.us.ibm.com (e12.ny.us.ibm.com [129.33.205.202]) by mx0b-001b2d01.pphosted.com with ESMTP id 2n69adnn7y-1 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=NOT) for ; Wed, 17 Oct 2018 15:43:23 -0400 Received: from localhost by e12.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 17 Oct 2018 15:43:22 -0400 Date: Wed, 17 Oct 2018 16:43:15 -0300 From: Murilo Opsfelder Araujo References: <20181016232201.16829-1-crosa@redhat.com> <20181017162944.GF31060@habkost.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Message-Id: <20181017194315.GA18379@kermit-br-ibm-com> Subject: Re: [Qemu-devel] [PATCH] Acceptance tests: host arch to target arch name mapping List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Cleber Rosa , Caio Carrara , Philippe =?iso-8859-1?Q?Mathieu-Daud=E9?= , Eduardo Habkost , Wainer dos Santos Moschetta , QEMU Developers On Wed, Oct 17, 2018 at 07:40:51PM +0100, Peter Maydell wrote: > On 17 October 2018 at 18:38, Cleber Rosa wrote: > > > > > > On 10/17/18 12:29 PM, Eduardo Habkost wrote: > >> On Wed, Oct 17, 2018 at 01:34:41PM +0100, Peter Maydell wrote: > >>> So, why does the test code need to care? It's not clear > >>> from the patch... My expectation would be that you'd > >>> just test all the testable target architectures, > >>> regardless of what the host architecture is. > >> > >> I tend to agree. Maybe the right solution is to get rid of the > >> os.uname(). I think the default should be testing all QEMU > >> binaries that were built, and the host architecture shouldn't > >> matter. > > Yes, looking at os.uname() also seems like an odd thing > for the tests to be doing here. The test framework > should be as far as possible host-architecture agnostic. > (For some of the KVM cases there probably is a need to > care, but those are exceptions, not the rule.) > > > I'm in favor of exercising all built targets, but that seems to me to be > > on another layer, above the test themselves. This change is about the > > behavior of a test when not told about the target arch (and thus binary) > > it should use. > > At that level, I think the right answer is "tell the user > they need to specify the qemu executable they are trying to test". > In particular, there is no guarantee that the user has actually > built the executable for the target that corresponds to the > host, so it doesn't work to try to default to that anyway. > > thanks > -- PMM > I agree with Peter. We can make qemu_bin parameter mandatory. If it is not given, error out. Trying to guess it based on host architecture turns out to be troublesome. If we decide to follow this approach of not guessing QEMU binary, we should update docs/devel/testing.rst to make it crystal clear qemu_bin parameter is mandatory. -- Murilo