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 Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B50EEC02183 for ; Wed, 15 Jan 2025 09:55:57 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [127.0.0.1]) by lists.ozlabs.org (Postfix) with ESMTP id 4YY1bJ1Wkvz30h5; Wed, 15 Jan 2025 20:55:56 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; arc=none smtp.remote-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1736934956; cv=none; b=DcZiFGZWJIP4poQfU74btfj9qXHvJvLzQyYp5eC0mLz4cjowfjtjssKCCNmhh3aZNWVHpKW1qzuKhv6n7K6jsU0CZ+wwVMdngDRHCYLEstRtgecXuVyqKokcWAYYem4XGWw+ton2u+HFi0Xy6h1SyqOIIirs59DFkyCJP4DHXFyDAroH6OyCNvKeD1K368IUU2Ts8xosQLpE1Zhr2gQjIHA3rSh3PvxOGtVD3+UwsgJSyh7Ac6F8W/bCAF0NaOJwGP8GepPZDCr9O1icFHb3iRfAkWiXkL5un9H29qil6fXX11zlcvVY7wvLjPwj/tsxnJJV51VbLX8mkd2YWet/mw== ARC-Message-Signature: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1736934956; c=relaxed/relaxed; bh=rT32sodiwTBuZaDBSI6rgB1m4++QP8tHXloSXLLwdvs=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=U12KD/5r2vwX/NAmopOp8C5pr5ZVZm/G7q4ZEHet7cBGdMXX7e1FlNQ2mZvG5nz1IbYdxbdgbzdBUk9h56gZcGbQ4MmMN2h8MJAqMZ7MyTEgJ8Kdl7SbM2/cWQjEBl66EmJ7GsJQUQgyVv2j7kObU1/02TKJQ6DXws6uyen0edAW+JLtocPIYQ3mDUcnxIhXnjlLkN1nR6/dnBEmq3LjZ1M+1tPL7u9zhhsfD4djTHJ/TNCsVm+U+S0NCxDYlFR79R9C9/LwxcMK/abmjZZjKcytCLOd9dBrDU4nG0I0B5e4zOP2quI7lvNGrSzG5vllhT+SOJjLztzGaDrJyw+Vpw== ARC-Authentication-Results: i=1; lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass (client-ip=217.140.110.172; helo=foss.arm.com; envelope-from=alexandru.elisei@arm.com; receiver=lists.ozlabs.org) smtp.mailfrom=arm.com Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=arm.com (client-ip=217.140.110.172; helo=foss.arm.com; envelope-from=alexandru.elisei@arm.com; receiver=lists.ozlabs.org) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by lists.ozlabs.org (Postfix) with ESMTP id 4YY1bG6lZzz30fM for ; Wed, 15 Jan 2025 20:55:53 +1100 (AEDT) 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 CD09212FC; Wed, 15 Jan 2025 01:55:49 -0800 (PST) Received: from arm.com (e134078.arm.com [10.32.102.60]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 680D23F51B; Wed, 15 Jan 2025 01:55:18 -0800 (PST) Date: Wed, 15 Jan 2025 09:55:14 +0000 From: Alexandru Elisei To: Andrew Jones Cc: eric.auger@redhat.com, lvivier@redhat.com, thuth@redhat.com, frankja@linux.ibm.com, imbrenda@linux.ibm.com, nrb@linux.ibm.com, david@redhat.com, pbonzini@redhat.com, kvmarm@lists.linux.dev, linuxppc-dev@lists.ozlabs.org, kvm-riscv@lists.infradead.org, linux-s390@vger.kernel.org, vladimir.murzin@arm.com Subject: Re: [kvm-unit-tests PATCH v1 2/5] configure: Display the default processor for arm and arm64 Message-ID: References: <20250110135848.35465-1-alexandru.elisei@arm.com> <20250110135848.35465-3-alexandru.elisei@arm.com> <20250113-45b57478be2241a35ffa1b67@orel> <20250114-a36510d222fc3410b9b7654e@orel> X-Mailing-List: linuxppc-dev@lists.ozlabs.org List-Id: List-Help: List-Owner: List-Post: List-Archive: , List-Subscribe: , , List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250114-a36510d222fc3410b9b7654e@orel> Hi Drew, On Tue, Jan 14, 2025 at 07:51:04PM +0100, Andrew Jones wrote: > On Tue, Jan 14, 2025 at 05:17:28PM +0000, Alexandru Elisei wrote: > ... > > > > +# $arch will have changed when cross-compiling. > > > > +[ -z "$processor" ] && processor=$(get_default_processor $arch) > > > > > > The fact that $arch and $processor are wrong until they've had a chance to > > > > $processor is never wrong. $processor is unset until either the user sets it > > with --processor, or until this line. This patch introduces $default_processor > > only for the purpose of having an accurate help text, it doesn't change when and > > how $processor is assigned. > > I should have said "The fact that $arch and $default_processor are wrong..." > > > > > > be converted might be another reason for the $do_help idea. But it'll > > > always be fragile since another change that does some sort of conversion > > > could end up getting added after the '[ $do_help ] && usage' someday. > > > > configure needs to distinguish between: > > > > 1. The user not having specified --processor when doing ./configure. > > 2. The user having set --processor. > > > > If 1, then kvm-unit-tests can use the default $processor value for $arch, > > which could have also been specified by the user. > > > > If 2, then kvm-unit-tests should not touch $processor because that's what the > > user wants. > > > > Do you see something wrong with that reasoning? > > If we output $default_processor in usage() before it's had a chance to be > set correctly based on a given cross arch, then it won't display the > correct name. > > > > > Also, I don't understand why you say it's fragile, since configure doesn't > > I wrote "it'll always be fragile" where 'it' refers to the most recent > object of my paragraph ("the $do_help idea"). But, TBH, I'm not sure > how important it is to get the help text accurate, so we can just not > care if we call usage() with the wrong strings sometimes. Got it now, thanks for explaining it. My opinion is that a help text is there to help the user, and in my experience an inaccurate help text can be very frustrating - think comments that say one thing, and the code does something else. How about this: diff --git a/configure b/configure index 3ab0ec208e10..5dbe189816b2 100755 --- a/configure +++ b/configure @@ -51,7 +51,6 @@ page_size= earlycon= efi= efi_direct= -default_processor=$(get_default_processor $arch) # Enable -Werror by default for git repositories only (i.e. developer builds) if [ -e "$srcdir"/.git ]; then @@ -61,13 +60,14 @@ else fi usage() { + [ -z "$processor" ] && processor=$(get_default_processor $arch) cat <<-EOF Usage: $0 [options] Options include: --arch=ARCH architecture to compile for ($arch). ARCH can be one of: arm, arm64/aarch64, i386, ppc64, riscv32, riscv64, s390x, x86_64 - --processor=PROCESSOR processor to compile for ($default_processor). For arm and arm64, the + --processor=PROCESSOR processor to compile for ($processor). For arm and arm64, the value 'max' is special and it will be passed directly to qemu, bypassing the compiler. In this case, --cflags can be used to compile for a specific processor. Should be accurate enough, as far as I can tell. And I don't think there's a need for $do_help: if the user does ./configure --help --arch=arm64, then I think it's reasonable to expect that --help will be interpreted before --arch is parsed. Thanks, Alex