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=unavailable 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 DA433C433F5 for ; Wed, 8 Sep 2021 16:11:48 +0000 (UTC) Received: from mm01.cs.columbia.edu (mm01.cs.columbia.edu [128.59.11.253]) by mail.kernel.org (Postfix) with ESMTP id 5907261154 for ; Wed, 8 Sep 2021 16:11:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 5907261154 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.cs.columbia.edu Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id CCBDA4B0F1; Wed, 8 Sep 2021 12:11:47 -0400 (EDT) X-Virus-Scanned: at lists.cs.columbia.edu Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Nyg5y8c8VRln; Wed, 8 Sep 2021 12:11:46 -0400 (EDT) Received: from mm01.cs.columbia.edu (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 9CD874AC80; Wed, 8 Sep 2021 12:11:46 -0400 (EDT) Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id B33C04A524 for ; Wed, 8 Sep 2021 12:11:45 -0400 (EDT) X-Virus-Scanned: at lists.cs.columbia.edu Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ymh1CYNs8rzw for ; Wed, 8 Sep 2021 12:11:44 -0400 (EDT) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 63CB8407D1 for ; Wed, 8 Sep 2021 12:11:44 -0400 (EDT) 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 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-Language: en-US Cc: lvivier@redhat.com, linux-s390@vger.kernel.org, thuth@redhat.com, frankja@linux.ibm.com, kvm@vger.kernel.org, david@redhat.com, andre.przywara@arm.com, cohuck@redhat.com, kvm-ppc@vger.kernel.org, vivek.gautam@arm.com, maz@kernel.org, pbonzini@redhat.com, imbrenda@linux.ibm.com, kvmarm@lists.cs.columbia.edu X-BeenThere: kvmarm@lists.cs.columbia.edu X-Mailman-Version: 2.1.14 Precedence: list List-Id: Where KVM/ARM decisions are made List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu 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 > _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm