From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-17.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EEB1DC433F5 for ; Wed, 8 Sep 2021 16:11:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D4D0761153 for ; Wed, 8 Sep 2021 16:11:45 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230091AbhIHQMw (ORCPT ); Wed, 8 Sep 2021 12:12:52 -0400 Received: from foss.arm.com ([217.140.110.172]:48418 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237223AbhIHQMw (ORCPT ); Wed, 8 Sep 2021 12:12:52 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id E5AFA1FB; Wed, 8 Sep 2021 09:11:43 -0700 (PDT) Received: from [192.168.0.110] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 8EAD13F766; Wed, 8 Sep 2021 09:11:41 -0700 (PDT) Subject: Re: [kvm-unit-tests RFC PATCH 5/5] configure: Ignore --erratatxt when --target=kvmtool To: Andrew Jones Cc: thuth@redhat.com, pbonzini@redhat.com, lvivier@redhat.com, kvm-ppc@vger.kernel.org, david@redhat.com, frankja@linux.ibm.com, cohuck@redhat.com, imbrenda@linux.ibm.com, linux-s390@vger.kernel.org, kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, andre.przywara@arm.com, maz@kernel.org, vivek.gautam@arm.com References: <20210702163122.96110-1-alexandru.elisei@arm.com> <20210702163122.96110-6-alexandru.elisei@arm.com> <20210907102536.jhycvnazlmj7qyto@gator> From: Alexandru Elisei Message-ID: Date: Wed, 8 Sep 2021 17:13:04 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20210907102536.jhycvnazlmj7qyto@gator> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Content-Language: en-US Precedence: bulk List-ID: X-Mailing-List: linux-s390@vger.kernel.org Hi Drew, On 9/7/21 11:25 AM, Andrew Jones wrote: > On Fri, Jul 02, 2021 at 05:31:22PM +0100, Alexandru Elisei wrote: >> kvmtool runs a test using the -f/--firmware argument, which doesn't load an >> initrd, making specifying an errata file useless. Instead, configure forces >> all erratas to be enabled via the CONFIG_ERRATA_FORCE define in >> lib/config.h. >> >> Forbid the --erratatxt option when kvm-unit-tests is configured for kvmtool >> and let the user know that all erratas are enabled by default. >> >> Signed-off-by: Alexandru Elisei >> --- >> I'm not sure if printing an error is too strong here and a simple warning would >> suffice. Suggestions welcome! >> >> configure | 26 +++++++++++++++++++------- >> 1 file changed, 19 insertions(+), 7 deletions(-) >> >> diff --git a/configure b/configure >> index 395c809c9c02..acd288239f80 100755 >> --- a/configure >> +++ b/configure >> @@ -24,7 +24,8 @@ u32_long= >> wa_divide= >> target= >> errata_force=0 >> -erratatxt="$srcdir/errata.txt" >> +erratatxt_default="$srcdir/errata.txt" >> +erratatxt="_NO_FILE_4Uhere_" >> host_key_document= >> page_size= >> earlycon= >> @@ -50,7 +51,8 @@ usage() { >> enable or disable the generation of a default environ when >> no environ is provided by the user (enabled by default) >> --erratatxt=FILE specify a file to use instead of errata.txt. Use >> - '--erratatxt=' to ensure no file is used. >> + '--erratatxt=' to ensure no file is used. This option is >> + invalid for arm/arm64 when target=kvmtool. > Do we need to always specifiy arm/arm64 when talking about target=kvmtool? > How much more effort would an x86 kvmtool target be to add? Err.. there haven't been any commits to the x86 code, other than those caused by refactoring in the other parts of the code, since 2015. Ballpark estimate, it's going to be very hard even to make 50% of the tests run on kvmtool. I think adding adding --target=uefi is much more likely to happen for x86 than --target=kvmtool. > >> --host-key-document=HOST_KEY_DOCUMENT >> Specify the machine-specific host-key document for creating >> a PVM image with 'genprotimg' (s390x only) >> @@ -147,11 +149,6 @@ if [ -n "$host_key_document" ] && [ ! -f "$host_key_document" ]; then >> exit 1 >> fi >> >> -if [ "$erratatxt" ] && [ ! -f "$erratatxt" ]; then >> - echo "erratatxt: $erratatxt does not exist or is not a regular file" >> - exit 1 >> -fi >> - >> arch_name=$arch >> [ "$arch" = "aarch64" ] && arch="arm64" >> [ "$arch_name" = "arm64" ] && arch_name="aarch64" >> @@ -184,6 +181,21 @@ else >> fi >> fi >> >> +if [ "$target" = "kvmtool" ]; then >> + if [ "$erratatxt" ] && [ "$erratatxt" != "_NO_FILE_4Uhere_" ]; then >> + echo "--erratatxt is not supported for target=kvmtool (all erratas enabled by default)" >> + usage >> + fi >> +else >> + if [ "$erratatxt" = "_NO_FILE_4Uhere_" ]; then >> + erratatxt=$erratatxt_default >> + fi >> + if [ "$erratatxt" ] && [ ! -f "$erratatxt" ]; then >> + echo "erratatxt: $erratatxt does not exist or is not a regular file" >> + exit 1 >> + fi >> +fi > switch Sure. Thanks, Alex > >> + >> [ -z "$processor" ] && processor="$arch" >> >> if [ "$processor" = "arm64" ]; then >> -- >> 2.32.0 >> > Otherwise looks good to me. > > Thanks, > drew >