From nobody Mon May 20 15:26:57 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1666883806; cv=none; d=zohomail.com; s=zohoarc; b=CEJKzRAfdPy1F1hplh/W5aqeRz24KsA7zdtW3aCKUYBTo/jyxyprV7Bjf81zeBUKJFTsP5dNRizcI/sOY/3b3j0nbE8a7QQRTLq2zHhBHxUn7DXUqx12CPiRBK336Z9OmbP0Q9Pew5kibXc88TV3t5PFU5dvYwJ/U2RIpyK61Ko= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1666883806; h=Content-Type:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=KSp/4lHAXuOsrwqOC5Motkbb8RzEYnMS/koDLG8XNFw=; b=OH0V6CheLE1jkqetCy1Wp/ORmHN43cyq57gZ73gZx5mDrlvM87jlGvPNfALA3lTVsVUkNBroeslXlAQpdvQ2azneS1LSbiKZA+piaczQ3bBuYSjAVYVZdRYGm6wBPMj+7VZnIjNJYLvWiBTzIWaN9Qb2E69tphNSrvCRVU7md2I= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1666883806299897.016613396294; Thu, 27 Oct 2022 08:16:46 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oo4YC-0002cG-PH; Thu, 27 Oct 2022 11:12:08 -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 1oo4YA-0002DZ-Du for qemu-devel@nongnu.org; Thu, 27 Oct 2022 11:12:06 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oo4Y1-0002sm-HY for qemu-devel@nongnu.org; Thu, 27 Oct 2022 11:12:03 -0400 Received: from mail-wr1-f70.google.com (mail-wr1-f70.google.com [209.85.221.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-3-TcdLV-hEOU6BRLo7udSN8Q-1; Thu, 27 Oct 2022 11:11:52 -0400 Received: by mail-wr1-f70.google.com with SMTP id t12-20020adfa2cc000000b0022adcbb248bso492246wra.1 for ; Thu, 27 Oct 2022 08:11:52 -0700 (PDT) Received: from redhat.com ([2.52.15.189]) by smtp.gmail.com with ESMTPSA id a9-20020adffb89000000b0022cc0a2cbecsm1445443wrr.15.2022.10.27.08.11.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 27 Oct 2022 08:11:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1666883514; 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; bh=KSp/4lHAXuOsrwqOC5Motkbb8RzEYnMS/koDLG8XNFw=; b=QJEhdKyAQV0v3oRdS6MgnT/vLt92WkLD+tTg8OCbAUMbz4tFeOlcpTW2n8QJWQAv54s5nd tRj1V+Stfna6BcReoeFJQVngmXdbjcHnpC+9IEpCai+2EznzDwEQoUw9n7tmr0bclbD20F 2I4LMV++LOz3SWm5x/Xu/xu4lpdX520= X-MC-Unique: TcdLV-hEOU6BRLo7udSN8Q-1 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=KSp/4lHAXuOsrwqOC5Motkbb8RzEYnMS/koDLG8XNFw=; b=yLIf4jtgLorYnjjP4LxICxxntgdPPU38XaJH4yzM5M2fk7p0jjr/l5JLR5ImMVP19h Nvbl9zdqh+Lax+ZkQtqEvmABAfoBeDSGfRrPkLSha1Uy1AH34s/vSLI6MevQP/jGeArF HpDZ77YbBBZL6SBWTgJW0gM3MX9uEBwUPSqywmWBglXqRDFNpruJOcHkz8Z+ZJHH1KmW Hr2A7tpkHrlFeHLa3EaxyK93MHOq9KWvTDeZq8XMQYKIe5yz/vxbxVKwvOfcwwxHiEun nXWC+P80oa0YUDiG/jBlPuJK82gFw0PTBB8u/rOVKMf12zVUxZlzkyVwr7bl7JVxrbqB JmtQ== X-Gm-Message-State: ACrzQf1DrQ9zQXqZz9A1FAZrYboyo0rxpY/hehfbewrRGFkh3USzJ9UH ZUo3hHcSf+v1B4h6YNky/t0Z+4hjlATffUbAGchwtnp/RlF7X2NyEJ3cW10LBv1qpws2+mMHsSf AW1912U/ELTZTM9FId6621mIRo1ATNH8yM1OOrwFqb2O4l1GymwMFo5n/1EgT X-Received: by 2002:a05:600c:310b:b0:3cf:33c4:bc7 with SMTP id g11-20020a05600c310b00b003cf33c40bc7mr6376487wmo.25.1666883511343; Thu, 27 Oct 2022 08:11:51 -0700 (PDT) X-Google-Smtp-Source: AMsMyM4NWerlTyFG+pWysXe+HzOMnRWsN8RX4iXVZRayBiuoGHwZ3vTyITW24fEUg3k4HX2uTztmCQ== X-Received: by 2002:a05:600c:310b:b0:3cf:33c4:bc7 with SMTP id g11-20020a05600c310b00b003cf33c40bc7mr6376467wmo.25.1666883511103; Thu, 27 Oct 2022 08:11:51 -0700 (PDT) Date: Thu, 27 Oct 2022 11:11:48 -0400 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Igor Mammedov , Ani Sinha Subject: [PATCH] bios-tables-test: do not ignore allowed diff list Message-ID: <20221027151135.496368-1-mst@redhat.com> MIME-Version: 1.0 Content-Disposition: inline X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -25 X-Spam_score: -2.6 X-Spam_bar: -- X-Spam_report: (-2.6 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.515, 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 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: , Sender: "Qemu-devel" Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1666883807858100001 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" we had such a beautiful structure for updating expected files, designed to keep bisect working. It turns out that we ignored the result of the allow list checks unless all tables matched anyway. Sigh. Let's at least make it work going forward. Signed-off-by: Michael S. Tsirkin Reviewed-by: Ani Sinha --- tests/qtest/bios-tables-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c index e6096e7f73..a72f6ca326 100644 --- a/tests/qtest/bios-tables-test.c +++ b/tests/qtest/bios-tables-test.c @@ -458,7 +458,7 @@ static void test_acpi_asl(test_data *data) "for instructions on how to update expected files.\n", exp_sdt->aml, sdt->aml_file, exp_sdt->aml_file); =20 - all_tables_match =3D all_tables_match && + all_tables_match =3D all_tables_match || test_acpi_find_diff_allowed(exp_sdt); =20 /* --=20 MST