[PATCH v3 1/3] tests/acpi: pc: allow DSDT acpi table changes

Ricardo Ribalda posted 3 patches 5 months, 2 weeks ago
There is a newer version of this series
[PATCH v3 1/3] tests/acpi: pc: allow DSDT acpi table changes
Posted by Ricardo Ribalda 5 months, 2 weeks ago
Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
---
 tests/qtest/bios-tables-test-allowed-diff.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..b2c2c10cbc 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1 +1,2 @@
 /* List of comma-separated changed AML files to ignore */
+"tests/data/acpi/pc/DSDT",
-- 
2.45.2.505.gda0bf45e8d-goog
Re: [PATCH v3 1/3] tests/acpi: pc: allow DSDT acpi table changes
Posted by Igor Mammedov 4 months, 4 weeks ago
On Fri,  7 Jun 2024 14:17:24 +0000
Ricardo Ribalda <ribalda@chromium.org> wrote:

> Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
> ---
>  tests/qtest/bios-tables-test-allowed-diff.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
> index dfb8523c8b..b2c2c10cbc 100644
> --- a/tests/qtest/bios-tables-test-allowed-diff.h
> +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> @@ -1 +1,2 @@
>  /* List of comma-separated changed AML files to ignore */
> +"tests/data/acpi/pc/DSDT",

that's no enough, a lot more tables expected blobs are affected by
the next patch.

before posting, make sure that 'make check-qtest' passes fine
Re: [PATCH v3 1/3] tests/acpi: pc: allow DSDT acpi table changes
Posted by Ricardo Ribalda 4 months, 4 weeks ago
Hi Igor



On Fri, 28 Jun 2024 at 13:25, Igor Mammedov <imammedo@redhat.com> wrote:
>
> On Fri,  7 Jun 2024 14:17:24 +0000
> Ricardo Ribalda <ribalda@chromium.org> wrote:
>
> > Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
> > ---
> >  tests/qtest/bios-tables-test-allowed-diff.h | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
> > index dfb8523c8b..b2c2c10cbc 100644
> > --- a/tests/qtest/bios-tables-test-allowed-diff.h
> > +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> > @@ -1 +1,2 @@
> >  /* List of comma-separated changed AML files to ignore */
> > +"tests/data/acpi/pc/DSDT",
>
> that's no enough, a lot more tables expected blobs are affected by
> the next patch.
>

Sorry about that, I did not realise that the check was quitting after
the first different file was found.

will post a new version soon

Thanks!

>
> before posting, make sure that 'make check-qtest' passes fine
>


-- 
Ricardo Ribalda