Re: [PATCH v6 01/10] ACPI ERST: bios-tables-test.c steps 1 and 2

[Eric DeVolder]

On 9/20/21 8:05 AM, Igor Mammedov wrote:
> On Thu,  5 Aug 2021 18:30:30 -0400
> Eric DeVolder <eric.devolder@oracle.com> wrote:
>
> > Following the guidelines in tests/qtest/bios-tables-test.c, this
> > change adds empty placeholder files per step 1 for the new ERST
> > table, and excludes resulting changed files in bios-tables-test-allowed-diff.h
> > per step 2.
>
> I'd move this right before 10/10

done

> > Signed-off-by: Eric DeVolder <eric.devolder@oracle.com>
>
> Acked-by: Igor Mammedov <imammedo@redhat.com>
>
>
> > ---
> >   tests/data/acpi/microvm/ERST                | 0
> >   tests/data/acpi/pc/ERST                     | 0
> >   tests/data/acpi/q35/ERST                    | 0
> >   tests/qtest/bios-tables-test-allowed-diff.h | 6 ++++++
> >   4 files changed, 6 insertions(+)
> >   create mode 100644 tests/data/acpi/microvm/ERST
> >   create mode 100644 tests/data/acpi/pc/ERST
> >   create mode 100644 tests/data/acpi/q35/ERST
> >
> > diff --git a/tests/data/acpi/microvm/ERST b/tests/data/acpi/microvm/ERST
> > new file mode 100644
> > index 0000000..e69de29
> > diff --git a/tests/data/acpi/pc/ERST b/tests/data/acpi/pc/ERST
> > new file mode 100644
> > index 0000000..e69de29
> > diff --git a/tests/data/acpi/q35/ERST b/tests/data/acpi/q35/ERST
> > new file mode 100644
> > index 0000000..e69de29
> > diff --git a/tests/qtest/bios-tables-test-allowed-diff.h 
> > b/tests/qtest/bios-tables-test-allowed-diff.h
> > index dfb8523..b3aaf76 100644
> > --- a/tests/qtest/bios-tables-test-allowed-diff.h
> > +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> > @@ -1 +1,7 @@
> >   /* List of comma-separated changed AML files to ignore */
> > +"tests/data/acpi/pc/ERST",
> > +"tests/data/acpi/q35/ERST",
> > +"tests/data/acpi/microvm/ERST",
> > +"tests/data/acpi/pc/DSDT",
> > +"tests/data/acpi/q35/DSDT",
> > +"tests/data/acpi/microvm/DSDT",