From nobody Wed May 15 17:01:51 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=1601389621; cv=none; d=zohomail.com; s=zohoarc; b=hDvtNC6LBwmSDuvZftGsUws6mUF/0j/s9IjJI1lhsqJ+/ZhBHQrCkjDhY9gx6030VtwIgtuFcw+xAG953Nc8WdUDAzneqPcOIn8PD2aTrg2j81/rAcWig+CX5gBXrrSyiIk0kPJ5WTo/kheipohlFlSGdvo0q9OikRtnYs2KL0c= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1601389621; h=Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:Message-ID:Sender:Subject:To; bh=tw2BSAHxMyMXfXN20HZnVu2RCSj4CLVnFIN2vj4VPxI=; b=n5aIKTUcuOVLBY/ITrS4tRwiVjF2SsIoT68Q0KCQqb5Lf3MpiG24/cX6b0WQhcugr8jTyWzVK3lYR2/nEVUpeuyNfgpvg+zV5X4q2YChnmuPxCPfK/XELcXZ085Ngv9Rn/fs3yjlUmaUkD1Ts843TfUa44KUzSMD5+b9Euc6+IE= 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 16013896217811018.8306838474883; Tue, 29 Sep 2020 07:27:01 -0700 (PDT) Received: from localhost ([::1]:37972 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kNGaq-0002V6-54 for importer@patchew.org; Tue, 29 Sep 2020 10:27:00 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:53372) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kNGZB-00012g-Pn for qemu-devel@nongnu.org; Tue, 29 Sep 2020 10:25:17 -0400 Received: from mail-pg1-x543.google.com ([2607:f8b0:4864:20::543]:34008) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kNGZ9-00083h-Q5 for qemu-devel@nongnu.org; Tue, 29 Sep 2020 10:25:17 -0400 Received: by mail-pg1-x543.google.com with SMTP id u24so4026874pgi.1 for ; Tue, 29 Sep 2020 07:25:15 -0700 (PDT) Received: from localhost.localdomain ([115.96.125.11]) by smtp.googlemail.com with ESMTPSA id g1sm5631796pfm.124.2020.09.29.07.25.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 29 Sep 2020 07:25:12 -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=tw2BSAHxMyMXfXN20HZnVu2RCSj4CLVnFIN2vj4VPxI=; b=MVvCcs0dbTqCGedBriVQZxjd3D5OzbJxF10BuziepSaUP7z5PyUaF+d9kYv+J0hjSb 8sRta3s0OTqzORs/jj4ln/Yj8wvDUHXdrMrMqdtjAy3ZMSnQYK5hRkvHgDxFA2+WeA+7 p5lQlst8gYWArLBUOW9IYSRk8h1aO/+Gj/mRxfk42vrTiC7KFAv50KNBz6nRX/hXh92f EuNljg4CESSrhBXkdANgYAMrUKK85BuWl8UiGRn/39k0a1dtIt35pJYEwzTzh3IXSjPV vQpMVbEVrGTa6qdBclz8zXj2/bygQynOYvq/wvajnE/yywfucvQvXv+M+9JKwbApoOv1 maqA== 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=tw2BSAHxMyMXfXN20HZnVu2RCSj4CLVnFIN2vj4VPxI=; b=cd7Vy2xn489uJoWABvOXO1E1dMB5xRm77JzbVlJ56H4DpgwfStbRerFzzp2toJlnxy HM5Vlhrz4T1CIQrg71G+Th7Dtmhu2075sgJgg7zicJBa4CAoG/TUtpB2StzFS5EDP7pN cXnOaOxgplfqV15CV7wWn7bUG+abfeXr08gJi6TK9YlNH/HDNj1oGs4ZqF+kAeaUJr3V YH5OouxOTGnURAxpKuIh4ni8A4eEaOfxRQQZj94RASWEKSKLGlPbJKX9hUMwEDCVu5Eu jM6Y/6ds6rfuiMPwvby/OH9ZewXDTMnlbrr5OCZzBuWGlxnsAlgnL0aBtjrfP001zmeX j24Q== X-Gm-Message-State: AOAM533xnbk+ertZL327kunUShm3o6115RL7XMr72BqOvTE4Sd0eGXdI jH8KBNIQOU31vThvyq0EcDNfxA== X-Google-Smtp-Source: ABdhPJzikRuSNwFbS0XwHFw0G/0mjABCkLh29n8eDin2iJpJHDYOhOPMat1Zw4mcPkeI7f3Y2Yvfkw== X-Received: by 2002:aa7:8001:0:b029:142:2501:34e6 with SMTP id j1-20020aa780010000b0290142250134e6mr4355467pfi.63.1601389513985; Tue, 29 Sep 2020 07:25:13 -0700 (PDT) From: Ani Sinha To: mst@redhat.com, qemu-devel@nongnu.org Subject: [PATCH v5] Add a comment in bios-tables-test.c to clarify the reason behind approach Date: Tue, 29 Sep 2020 19:55:01 +0530 Message-Id: <20200929142501.1057-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::543; envelope-from=ani@anisinha.ca; helo=mail-pg1-x543.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 , 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 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 Reviewed-by: Michael S. Tsirkin --- tests/qtest/bios-tables-test.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) changelog: v5: incorporated suggestion from eric. v4: more updates as per Igor's suggestion. Dropped some comment lines. added ack'd by line. v3: review feedback incorporared and actual comment in the code reworded. v2: cosmetic - commit log reworded. diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c index 3c09b844f9..fc7aaaf82c 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,11 @@ * $(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. + * Expected binary updates needs to be a separate patch from the code t= hat + * introduces changes to ACPI tables. It lets the maintainer drop + * 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 + * disassembly). * 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