Em Wed, 29 Jan 2025 16:03:28 +0100
Igor Mammedov <imamm...@redhat.com> escreveu:
> On Wed, 29 Jan 2025 09:04:08 +0100
> Mauro Carvalho Chehab <mchehab+hua...@kernel.org> wrote:
>
> > The DSDT table will also be affected by such change.
> >
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+hua...@kernel.org>
>
> move it right before the patch that would actually make changes to tables
> (10/13)
Table changes happens on two patches:
- patch 03/13: acpi/ghes: add a firmware file with HEST address
HEST table was added here
- patch 10/13: arm/virt: Wire up a GED error device for ACPI / GHES
DSDT changes happen here.
If the idea is to avoid make check to fail between those two patches,
we need either to split them on 4 patches (one before/one after each
change) or do like I did on this series: whitelist before patch 3,
update after patch 10.
Regards,
Mauro
>
>
> > ---
> > tests/data/acpi/aarch64/virt/HEST | 0
> > tests/qtest/bios-tables-test-allowed-diff.h | 2 ++
> > 2 files changed, 2 insertions(+)
> > create mode 100644 tests/data/acpi/aarch64/virt/HEST
> >
> > diff --git a/tests/data/acpi/aarch64/virt/HEST
> > b/tests/data/acpi/aarch64/virt/HEST
> > new file mode 100644
> > index 000000000000..e69de29bb2d1
> > diff --git a/tests/qtest/bios-tables-test-allowed-diff.h
> > b/tests/qtest/bios-tables-test-allowed-diff.h
> > index dfb8523c8bf4..46298e38e7b8 100644
> > --- a/tests/qtest/bios-tables-test-allowed-diff.h
> > +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> > @@ -1 +1,3 @@
> > /* List of comma-separated changed AML files to ignore */
> > +"tests/data/acpi/aarch64/virt/HEST",
> > +"tests/data/acpi/aarch64/virt/DSDT",
>
> the list in not complete so 'make check-qtest' still fails
> [12/13] has complete list of changed files
>
Thanks,
Mauro
