From nobody Thu May 2 10:29:24 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; 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 ARC-Seal: i=1; a=rsa-sha256; t=1600769446; cv=none; d=zohomail.com; s=zohoarc; b=lX2SZfOlmOrXAdp4UOBuWgzsoE1+JYgbcOItia3Z25dZkcdfiqJnpyu2WUY6SaoUve92xWdeuH8FFCGlncgQQ1CCPGsIOvmYa3bLm+blxDWeRvs68P9jMHpWrq6GWDTEecZ7ischh+1FdXJc+F1L/zphUt0ylaM8P0UlSHElMm8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1600769446; h=Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:Message-ID:Sender:Subject:To; bh=c24pFn9TuOtHvIccTwrvpvAxyZ0ejqYC9CUKyj/sVGI=; b=G8xY4Z9/Jn8HiOFdZiKizXhVIUeOh0cv99DCmM9knyC/4cjgL+d1hUJPSiBuYwsgUeop+thEBrAuhtMMvEMhrPmnX/Mdx3ovzrj09B89Tcz9HT99S88MSZH2+oDCplTpZgateIfiz8f3KxrNWExDNrlZp7uhsI/seANo+aBEtFM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; 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 Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1600769446093368.56949487044074; Tue, 22 Sep 2020 03:10:46 -0700 (PDT) Received: from localhost ([::1]:56828 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kKfFz-0003YH-VN for importer@patchew.org; Tue, 22 Sep 2020 06:10:43 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:49092) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kKfEt-00030V-SK for qemu-devel@nongnu.org; Tue, 22 Sep 2020 06:09:36 -0400 Received: from mail-pf1-x444.google.com ([2607:f8b0:4864:20::444]:34927) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kKfEs-0003Q5-7m for qemu-devel@nongnu.org; Tue, 22 Sep 2020 06:09:35 -0400 Received: by mail-pf1-x444.google.com with SMTP id k8so11997276pfk.2 for ; Tue, 22 Sep 2020 03:09:33 -0700 (PDT) Received: from localhost.localdomain ([115.96.140.229]) by smtp.googlemail.com with ESMTPSA id y195sm14897186pfc.137.2020.09.22.03.09.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Sep 2020 03:09:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=anisinha-ca.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id; bh=c24pFn9TuOtHvIccTwrvpvAxyZ0ejqYC9CUKyj/sVGI=; b=S4vkU7oUJ3WAFIVBGOJzwyw0AfofWsOagx9Uq3cGT/Vf7MQ8p5bgSiKHH+qSW/Fc4+ bcoJrmWjOgrDs2CpeW2jPSy8Z3CRpfMlltzlmp6GuoLYj3FNO+WalhCrGyAOJZ8/9Y/I N4OiYJeS5qoBMmb5l3GGHSVXrY06qpqh3Yf3ED7J6Mahzz7BGzcuoOlKmsYerb/n8ALR VFP9sCXSx+xpeeMcJl6D9m1DVQHO3cMjMyfMQbxHSZEPGdySyFoXT5jS1NpFQqVN9TeC 0SUi/v0dzPb6ksDFODNDowdcEsdiiOebNQ/ldrYYsbNUI4+7+nJrn1ItNBrKkdnTbIO5 Cf+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=c24pFn9TuOtHvIccTwrvpvAxyZ0ejqYC9CUKyj/sVGI=; b=ner5GWmlZ+mzsyc0dmaBbxyjLvYB1dEQ9o/BELX2i8BY56V37IrXj+jUM2ws55B2db yWobM6f4yEsbIqSr2igPb6aD5huSGlTM/ASQBw5CIyWpHEZhc8+Ff270ncUALTl83wxj XC0YB6ar76c7gjrZ6IsThqjGRwrqkjK3IwcVLZqy/76QRIw54gCxpbh0PUNbqrRLwRFz CYfPyxAMYYtxfUl1qj+Rra9amAOJPd843C4bzIq4VCJGnRGLV3BWjDiCsddfhkvxqbG/ 6bPDdHLfSe1gDcfsmB+/5CNT9bsz8efEYxZ8PSKHe4lajWyv2Pbqb6EOwf6JkaXtumxo IKFw== X-Gm-Message-State: AOAM530HyPJkVERwmix2YD7uTms+WxY+qIOOQAvWkiH7omheyyQ/pJDl MKUuDAeN03OYamM/ZmHPXx3GktM4FrbeUqos X-Google-Smtp-Source: ABdhPJy7NhN2Alu82J72QkbQ/BRfP6bPC0SVERL6XGGsWRnIZLoERVRq8gHqOwViaXtZcKejhrWIyg== X-Received: by 2002:a17:902:b404:b029:d2:43a9:9f5c with SMTP id x4-20020a170902b404b02900d243a99f5cmr1008259plr.38.1600769371679; Tue, 22 Sep 2020 03:09:31 -0700 (PDT) From: Ani Sinha To: qemu-devel@nongnu.org Subject: [PATCH v3] Add a comment in bios-tables-test.c to clarify the reason behind approach Date: Tue, 22 Sep 2020 15:39:18 +0530 Message-Id: <20200922100918.4200-1-ani@anisinha.ca> X-Mailer: git-send-email 2.17.1 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: none client-ip=2607:f8b0:4864:20::444; envelope-from=ani@anisinha.ca; helo=mail-pf1-x444.google.com X-detected-operating-system: by eggs.gnu.org: No matching host in p0f cache. That's all we know. X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_NONE=0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Laurent Vivier , Thomas Huth , "Michael S. Tsirkin" , Paolo Bonzini , Ani Sinha , Igor Mammedov Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" A comment blob is added in bios-tables-test.c that explains the reasoning behind the process of updating the ACPI table blobs when new tests are added or old tests are modified or code is committed that affect tests. The explanation would help future contributors follow the correct process when making code changes that affect ACPI tables. Signed-off-by: Ani Sinha Acked-by: Igor Mammedov --- tests/qtest/bios-tables-test.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) Changelog: v2: cosmetic - commit log reworded. v3: review feedback incorporared and actual comment in the code reworded. diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c index b514b70b62..1e8838c8c6 100644 --- a/tests/qtest/bios-tables-test.c +++ b/tests/qtest/bios-tables-test.c @@ -11,7 +11,7 @@ */ =20 /* - * How to add or update the tests: + * How to add or update the tests or commit changes that affect ACPI table= s: * Contributor: * 1. add empty files for new tables, if any, under tests/data/acpi * 2. list any changed files in tests/qtest/bios-tables-test-allowed-diff.h @@ -38,6 +38,17 @@ * $(SRC_PATH)/tests/data/acpi/rebuild-expected-aml.sh * 6. Now commit any changes to the expected binary, include diff from ste= p 4 * in commit log. + * The reason why we follow the above process is described below: + * After every commit we make sure that the unit tests are not broken. = To do + * that, we could have committed the table updates along with the patch= es. + * However, expected binary updates needs to be a separate patch from t= he + * code that introduces changes to ACPI tables. It lets maintainer to d= rop + * and regenerate binary updates in case of merge conflicts. Further, a= code + * change is easily reviewable but a binary blob is not (without doing a + * diassemly). Listing the modified table files additionally helps in + * bisection in case things are broken. + * Hence, we have a multi-step process and at each step unit tests cont= inue + * to pass. * 7. Before sending patches to the list (Contributor) * or before doing a pull request (Maintainer), make sure * tests/qtest/bios-tables-test-allowed-diff.h is empty - this will ens= ure --=20 2.17.1