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.gnu.org (lists.gnu.org [209.51.188.17]) (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 5FBD1C77B75 for ; Wed, 17 May 2023 14:37:28 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pzIGk-0004j9-FC; Wed, 17 May 2023 10:36:46 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pzIGh-0004ig-3r for qemu-devel@nongnu.org; Wed, 17 May 2023 10:36:43 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pzIGe-0008EG-Qw for qemu-devel@nongnu.org; Wed, 17 May 2023 10:36:42 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1684334199; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=jRX7ePMQ5y6vyphwwoCqI4DNBNeQzBgkvZVbDqKAl+Y=; b=OfXZzp5guvUtPPnDYBVXfjSxGywMfAv2ORmbhHTXG8lSvNr/5pBsy3kxRQoh1x6K897mCH tLkQmYw5eJrRw8wdlD2QrrEjnqjlXzoJiC4NzL3TNEeC+uV+g2e5smEUCwYX2/CPr3meji VT1/dhF5evvIhPpT1ELCJRYmUQeR/D8= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-388-bGHE_PWMP1Ci-KLevs-0Mg-1; Wed, 17 May 2023 10:36:38 -0400 X-MC-Unique: bGHE_PWMP1Ci-KLevs-0Mg-1 Received: by mail-wr1-f71.google.com with SMTP id ffacd0b85a97d-2f2981b8364so559202f8f.1 for ; Wed, 17 May 2023 07:36:38 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684334197; x=1686926197; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=jRX7ePMQ5y6vyphwwoCqI4DNBNeQzBgkvZVbDqKAl+Y=; b=j4X/hzHvNGgl6nLoABSJgtgjaZIEdOffMT1X+VsNRCqpnSAQbELagcqKdR9jfkjPss ZVbJcFgD9XLxJueyUAw+TXjunVrSaYufDN+CNDjFbc1Bw68spPsBLvz9Stb+XybrRG6P DdulOOPg5KYRW7RfSG0iP0iptKd7RefIeYIkkEODz9kmr/GnzgP34PwIsRIc8dc93dnl 8YmMwMiM5y2tfia6qSIMSACPZlqyrVVGvqES4lb8XX1zmcMYj7SVhpy6lHEw3cXlGmF5 piOjt+vROtB6u7Ib0kHOynct3sotPvBmwtjYQAKUqBCANZRTLZOOOtu5+g1CWPW7zuwd Z92Q== X-Gm-Message-State: AC+VfDxbv8Cz/Jfixdk+eLejCDGT229/ubIfi6AvLgt/E4FVdIko+ueI w0CpgpFDWqX0Md3UP4Qa2BmFruZQxWoChQ2yWSiTK06pckd6f6vciabSHVdgFBmdoU0cSYsYeki sPx6aAqzmCE//CYM= X-Received: by 2002:a5d:4403:0:b0:307:c1cb:425b with SMTP id z3-20020a5d4403000000b00307c1cb425bmr848224wrq.70.1684334196997; Wed, 17 May 2023 07:36:36 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6tuY+lItufnWuaGCMqkQznkPRspR80xVbgs0AFsC0D69spVjh99DFMJsdD+FpEqXYthE4QCA== X-Received: by 2002:a5d:4403:0:b0:307:c1cb:425b with SMTP id z3-20020a5d4403000000b00307c1cb425bmr848209wrq.70.1684334196566; Wed, 17 May 2023 07:36:36 -0700 (PDT) Received: from redhat.com ([109.253.194.87]) by smtp.gmail.com with ESMTPSA id u15-20020adfed4f000000b00307c46f4f08sm2956126wro.79.2023.05.17.07.36.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 17 May 2023 07:36:35 -0700 (PDT) Date: Wed, 17 May 2023 10:36:32 -0400 From: "Michael S. Tsirkin" To: Ani Sinha Cc: Igor Mammedov , qemu-devel@nongnu.org Subject: Re: [PATCH] acpi/tests/bios-tables-test: add an environment variable for iasl location Message-ID: <20230517103522-mutt-send-email-mst@kernel.org> References: <20230517120751.14679-1-anisinha@redhat.com> <20230517101523-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Received-SPF: pass client-ip=170.10.133.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org On Wed, May 17, 2023 at 07:57:53PM +0530, Ani Sinha wrote: > > > > On 17-May-2023, at 7:47 PM, Michael S. Tsirkin wrote: > > > > On Wed, May 17, 2023 at 05:37:51PM +0530, Ani Sinha wrote: > >> Currently the meson based QEMU build process locates the iasl binary from the > >> current PATH and other locations [1] and uses that to set CONFIG_IASL which is > >> then used by the test. > >> > >> This has two disadvantages: > >> - If iasl was not previously installed in the PATH, one has to install iasl > >> and rebuild QEMU in order to pick up the iasl location. One cannot simply > >> use the existing bios-tables-test binary because CONFIG_IASL is only set > >> during the QEMU build time by meson and then bios-tables-test has to be > >> rebuilt with CONFIG_IASL set in order to use iasl. > >> - Sometimes, the stock iasl that comes with distributions is simply not good > >> enough because it does not support the latest ACPI changes - newly > >> introduced tables or new table attributes etc. In order to test ACPI code > >> in QEMU, one has to clone the latest acpica upstream repository and > >> rebuild iasl in order to get support for it. In those cases, one may want > >> the test to use the iasl binary from a non-standard location. > >> > >> In order to overcome the above two disadvantages, we introduce a new > >> environment variable IASL_PATH that can be set by the tester pointing to an > >> possibly non-standard iasl binary location. Bios-tables-test then uses this > >> environment variable to set its iasl location, possibly also overriding the > >> location that was pointed to by CONFIG_IASL that was set by meson. This way > >> developers can not only use this new environment variable to set iasl > >> location to quickly run bios-tables-test but also can point the test to a > >> custom iasl if required. > >> > >> [1] https://mesonbuild.com/Reference-manual_functions.html#find_program > >> > >> Signed-off-by: Ani Sinha > > > > Well I think the point was originally that meson can > > also test the binary in a variety of ways. > > Never surfaced so maybe never mind. > > > > Would it be easier to just look iasl up on path > > But that’s what meson is also doing, only QEMU build time. So you were unhappy it's build time because it is not really part of build and you want flexibility, right? > > unless it was specified on command line? > > > >> --- > >> tests/qtest/bios-tables-test.c | 14 ++++++++++++++ > >> 1 file changed, 14 insertions(+) > >> > >> sample runs: > >> > >> $ QTEST_QEMU_BINARY=./qemu-system-x86_64 V=2 ./tests/qtest/bios-tables-test > >> ... > >> acpi-test: Warning! APIC binary file mismatch. Actual [aml:/tmp/aml-DLHA51], Expected [aml:tests/data/acpi/pc/APIC]. > >> See source file tests/qtest/bios-tables-test.c for instructions on how to update expected files. > >> Using iasl: /usr/bin/iasl > >> acpi-test: Warning! APIC mismatch. Actual [asl:/tmp/asl-L9GA51.dsl, aml:/tmp/aml-DLHA51], Expected [asl:/tmp/asl-10EA51.dsl, aml:tests/data/acpi/pc/APIC]. > >> > >> $ QTEST_QEMU_BINARY=./qemu-system-x86_64 V=2 IASL_PATH=/home/anisinha/workspace/acpica/generate/unix/bin/iasl ./tests/qtest/bios-tables-test > >> ... > >> acpi-test: Warning! APIC binary file mismatch. Actual [aml:/tmp/aml-5CQ341], Expected [aml:tests/data/acpi/pc/APIC]. > >> See source file tests/qtest/bios-tables-test.c for instructions on how to update expected files. > >> User has provided an iasl path, using that: /home/anisinha/workspace/acpica/generate/unix/bin/iasl > >> acpi-test: Warning! APIC mismatch. Actual [asl:/tmp/asl-2GQ341.dsl, aml:/tmp/aml-5CQ341], Expected [asl:/tmp/asl-IBR341.dsl, aml:tests/data/acpi/pc/APIC]. > >> > >> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c > >> index 7fd88b0e9c..37e8e484cb 100644 > >> --- a/tests/qtest/bios-tables-test.c > >> +++ b/tests/qtest/bios-tables-test.c > >> @@ -440,7 +440,12 @@ static void test_acpi_asl(test_data *data) > >> AcpiSdtTable *sdt, *exp_sdt; > >> test_data exp_data = {}; > >> gboolean exp_err, err, all_tables_match = true; > >> + const char *user_iasl_path = getenv("IASL_PATH"); > >> > >> + /* if user has provided a path to iasl, use that */ > >> + if (user_iasl_path) { > >> + iasl = user_iasl_path; > >> + } > >> exp_data.tables = load_expected_aml(data); > >> dump_aml_files(data, false); > >> for (i = 0; i < data->tables->len; ++i) { > >> @@ -473,6 +478,15 @@ static void test_acpi_asl(test_data *data) > >> continue; > >> } > >> > >> + if (verbosity_level >= 2) { > >> + if (user_iasl_path) { > >> + fprintf(stderr, "User has provided an iasl path," \ > >> + "using that: %s\n", user_iasl_path); > >> + } else { > >> + fprintf(stderr, "Using iasl: %s\n", iasl); > >> + } > >> + } > >> + > >> err = load_asl(data->tables, sdt); > >> asl = normalize_asl(sdt->asl); > >> > >> -- > >> 2.39.1