[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] tests: document how to update acpi tables
From: |
Thomas Huth |
Subject: |
Re: [PATCH] tests: document how to update acpi tables |
Date: |
Mon, 7 Oct 2019 17:06:01 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 05/10/2019 23.27, Michael S. Tsirkin wrote:
> Looks like no one understands how to do it.
> Document the process.
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
> ---
> tests/bios-tables-test.c | 27 +++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
> diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
> index 652a78773f..0b33fb265f 100644
> --- a/tests/bios-tables-test.c
> +++ b/tests/bios-tables-test.c
> @@ -10,6 +10,33 @@
> * See the COPYING file in the top-level directory.
> */
>
> +/*
> + * How to add or update the tests:
> + * Contributor:
> + * 1. add empty files for new tables, if any, under tests/data/acpi
> + * 2. list any changed files in tests/bios-tables-test-allowed-diff.h
> + * 3. commit the above *before* making changes that affect the tables
> + * Maintainer:
> + * After 1-3 above tests will pass but ignore differences with the expected
> files.
> + * You will also notice that tests/bios-tables-test-allowed-diff.h lists
> + * a bunch of files. This is your hint that you need to do the below:
> + * 4. Run
> + * make check V=1
> + * this will produce a bunch of warnings about differences
> + * beween actual and expected ACPI tables. If you have IASL installed,
s/beween/between/
With that typo fixed:
Acked-by: Thomas Huth <address@hidden>
> + * they will also be disassembled so you can look at the disassembled
> + * output. If not - disassemble them yourself in any way you like.
> + * Look at the differences - make sure they make sense and match what the
> + * changes you are merging are supposed to do.
> + *
> + * 5. From build directory, run:
> + * $(SRC_PATH)/tests/data/acpi/rebuild-expected-aml.sh
> + * 6. Now commit any changes.
> + * 7. Before doing a pull request, make sure
> tests/bios-tables-test-allowed-diff.h
> + * is empty - this will ensure following changes to ACPI tables will
> + * be noticed.
> + */
> +
> #include "qemu/osdep.h"
> #include <glib/gstdio.h>
> #include "qemu-common.h"
>