From nobody Tue May 7 02:48:28 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=1600485689; cv=none; d=zohomail.com; s=zohoarc; b=Dz2/EMT1o3eEHRrD6wi5SSjWN1YKLdX21RsYSWlywiGSbaBRaLL9KZnzDiSrxRs/8KPD8asoEv/H1vgk4Ykz9fdyI8HyIcSOeATjmH+EEALb3bYACNucj3pfLRkTvjBAGwq2ShIDOiB5D2lmnq/ScqQASDkUgrKbrcbnNxiKOHI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1600485689; h=Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:Message-ID:Sender:Subject:To; bh=hjOQGgY8Q3eCsOfBzNckldz8eBMsuaAC9MCuD88EQd8=; b=d+Jo9Nk7t7lZeas1ocBzr96Z3saKiNvCckotvzSSoTCxcpMy/jn1Jiqv/VjgavKdNZ4R6kdpdWq5V7S4ZqIBoO/B9Nk0ANN12SXJSDlxMTAUzWe7fAVxZZSwwoZcwT+y3ZhxGimAz5npV6PAOtXJy6/aZd0DXrL6iek9wQSc2pc= 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 1600485689509806.7151726031256; Fri, 18 Sep 2020 20:21:29 -0700 (PDT) Received: from localhost ([::1]:48930 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kJTRH-0000Og-C1 for importer@patchew.org; Fri, 18 Sep 2020 23:21:27 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:53284) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kJTPr-0008J8-MV for qemu-devel@nongnu.org; Fri, 18 Sep 2020 23:19:59 -0400 Received: from mail-pg1-x544.google.com ([2607:f8b0:4864:20::544]:35152) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kJTPp-0006h5-Kb for qemu-devel@nongnu.org; Fri, 18 Sep 2020 23:19:59 -0400 Received: by mail-pg1-x544.google.com with SMTP id g29so4595427pgl.2 for ; Fri, 18 Sep 2020 20:19:56 -0700 (PDT) Received: from localhost.localdomain ([115.96.138.96]) by smtp.googlemail.com with ESMTPSA id g26sm4738467pfr.105.2020.09.18.20.19.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 18 Sep 2020 20:19:54 -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=hjOQGgY8Q3eCsOfBzNckldz8eBMsuaAC9MCuD88EQd8=; b=xoPhLAEB4lJDobtdeopNYudIygmhnvJYiKMS4KCdGAE9LRbtEm0sBuLsMMldbStAeI 10WMe6DUf4LJB2s0YxE7t/8dQbd8bTNcQEKdbTTqOGqIr4RbJu3z+ZD18pWUXSbXzyc7 iMP2R+G8R/nYrIBwqhvhGfridH56tbD8MBLHLo38t/kTJiY3Af6zCGRth9n/PsdqMFDM BhZ4ZNtRBkY2nI8CWP0zWBjKXUTYzegsnHJVMJ0wwef+f5UPeuS7ISt9G6UE4XUFY3IM x7altBxSkE1DYONiN7N42hHWrWN5y8udUudIVClJq0S4NRj+qrNjEEd4uovlt4QcgsNo y8vA== 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=hjOQGgY8Q3eCsOfBzNckldz8eBMsuaAC9MCuD88EQd8=; b=ZOj2DKqkZWEL54i9/nwuOtFwHj1JBJr/aHz7ZPb46zP/KotRLtaXeSQtPnPn0pC4ld 77G2H/BjwQBmaGPGw7UjD8sFTdbJH5+zx2D2vjS+PO2adIepKgTjOl6YMIlUgltBINZA UGfCAnI11QQ5wTM4MzPGbomyDd7k4HntmN/EXKM+9paoLtc2uUO27XiZgixdJlaEqz9p +fVTF0cbk9foQi6MzDbUSnam04j1ew9pNOO52EJyUSYw15w6fQNknseU6KRG30gT44g/ AMeGB9aJlBH9wscAa726TGmLL0Yw140MLGDs8BA9HvQUTAVJyvZIMr4FumFG8+mnDq+D 7Cbg== X-Gm-Message-State: AOAM530/qMqTVCeb6NBtIb18X8aW3qKjKvkqehOSma1ibrn9I+mTwi9f dfMhmBYkuOYb0t2jBlnA6SQEyI6cQqozRrAJ X-Google-Smtp-Source: ABdhPJzDE6Z2VUC7z+Gm09eOgfJzCunsmQ11MrCdPoxSxqFEflc1mQAFm4XyqZcc8t6gC4Ep7ChwDw== X-Received: by 2002:a62:7b55:0:b029:13e:7367:2b2e with SMTP id w82-20020a627b550000b029013e73672b2emr33782582pfc.7.1600485595030; Fri, 18 Sep 2020 20:19:55 -0700 (PDT) From: Ani Sinha To: qemu-devel@nongnu.org Subject: [PATCH v2] Add a comment in bios-tables-test.c to clarify the reason behind approach Date: Sat, 19 Sep 2020 08:49:40 +0530 Message-Id: <20200919031941.2664-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::544; envelope-from=ani@anisinha.ca; helo=mail-pg1-x544.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 --- tests/qtest/bios-tables-test.c | 20 +++++++++++++++++++- 1 file changed, 19 insertions(+), 1 deletion(-) changelog: v1: initial patch v2: cosmetic - commit log reworded. diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c index 3f7f1a8107..e51ea26ae8 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 tables: * 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 @@ -48,6 +48,24 @@ * - patches 2 - n: real changes, may contain multiple patches. * - patch n + 1: update golden master binaries and empty * tests/qtest/bios-tables-test-allowed-diff.h + * + * There is a reason why the above process is followed. After every commit= we + * make sure that the unit tests are not broken. + * Listing changed files in patch 1 makes sure every commit that follows w= hich + * affect the tests (patches 2 - n) does not break tests. + * This is followed by the actual changes (test changes or code changes) t= hat + * actually affect the acpi tables. + * Finally in patch n + 1, we update the golden master blobs as well as re= vert + * the file additions in bios-tables-test-allowed-diff.h. This makes sure = that + * the test continues to pass because of updated table blobs while the sta= te of + * bios-tables-test-allowed-diff.h is reverted back to the default empty f= ile + * condition. + * + * We could have committed the table updates along with the patches. Howev= er, + * whereas a code change is easily reviewable and in case of conflicts, ea= sily + * addressible, a binary blob is not. Hence, its better to commmit the bin= ary + * blob updates as a separate independent commit. Listing the modified tab= le + * files additionally helps in bisection in case things are broken. */ =20 #include "qemu/osdep.h" --=20 2.17.1