| Message ID | 1628202639-16361-2-git-send-email-eric.devolder@oracle.com |
|---|---|
| State | New |
| Headers | show |
| Series | acpi: Error Record Serialization Table, ERST, support for QEMU | expand |
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 > 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",
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", >
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",
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. Signed-off-by: Eric DeVolder <eric.devolder@oracle.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