[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _H
From: |
Peter Maydell |
Subject: |
Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID |
Date: |
Thu, 16 Jan 2020 12:24:38 +0000 |
On Wed, 15 Jan 2020 at 10:55, Michael S. Tsirkin <address@hidden> wrote:
>
> Here's a hopefully better patch. Peter does this address the issue?
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
>
>
> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
> index f1ac2d7e96..3ab4872bd7 100644
> --- a/tests/qtest/bios-tables-test.c
> +++ b/tests/qtest/bios-tables-test.c
> @@ -16,7 +16,10 @@
> * 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:
> + *
> + * Contributor or ACPI Maintainer (steps 4-7 need to be redone to resolve
> conflicts
> + * in binary commit created in step 6):
> + *
> * 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:
> @@ -28,13 +31,23 @@
> * 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.
> + * Save the changes, preferably in form of ASL diff for the commit log in
> + * step 6.
> *
> * 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.
> + * 6. Now commit any changes to the expected binary, include diff from step 4
> + * in commit log.
> + * 7. Before sending patches to the list (Contributor)
> + * or before doing a pull request (Maintainer), make sure
> + * tests/bios-tables-test-allowed-diff.h is empty - this will ensure
> + * following changes to ACPI tables will be noticed.
> + *
> + * The resulting patchset/pull request then looks like this:
> + * - patch 1: list changed files in tests/bios-tables-test-allowed-diff.h.
> + * - patches 2 - n: real changes, may contain multiple patches.
> + * - patch n + 1: update golden master binaries and empty
> + * tests/bios-tables-test-allowed-diff.h
> */
I think that seems reasonable, but you're the ACPI expert.
As long as the patches on list:
* can be reviewed by somebody for whether their ACPI changes
are correct, including whether the golden-master changes are right
* can be applied by a maintainer without having to do anything
special
* don't break bisection
then I'm happy. It sounds like those steps will work for that.
> diff --git a/roms/seabios b/roms/seabios
> index f21b5a4aeb..c9ba5276e3 160000
> --- a/roms/seabios
> +++ b/roms/seabios
> @@ -1 +1 @@
> -Subproject commit f21b5a4aeb020f2a5e2c6503f906a9349dd2f069
> +Subproject commit c9ba5276e3217ac6a1ec772dbebf568ba3a8a55d
You have a stray submodule update in your patch, though :-)
thanks
-- PMM
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, (continued)
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Corey Minyard, 2020/01/05
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Peter Maydell, 2020/01/06
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Guoheyi, 2020/01/14
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Michael S. Tsirkin, 2020/01/15
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Guoheyi, 2020/01/15
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Michael S. Tsirkin, 2020/01/15
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Michael S. Tsirkin, 2020/01/15
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID,
Peter Maydell <=
- Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Guoheyi, 2020/01/16
Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Igor Mammedov, 2020/01/13
Re: [PATCH 2/2] arm/virt/acpi: remove _ADR from devices identified by _HID, Igor Mammedov, 2020/01/15