Re: [PATCH v5 03/10] acpi/tests/avocado/bits: disable acpi PSS tests that are failing in biosbits

[Daniel P . Berrangé]

On Wed, Oct 19, 2022 at 04:30:57PM +0100, Alex Bennée wrote:
> 
> Ani Sinha <ani@anisinha.ca> writes:
> 
> > PSS tests in acpi test suite seems to be failing in biosbits. This is 
> > because
> > the test is unable to find PSS support in QEMU bios. Let us disable
> > them for now so that make check does not fail. We can fix the tests and
> > re-enable them later.
> >
> > Example failure:
> >
> > ---- ACPI _PSS (Pstate) table conformance tests ----
> > [assert] _PSS must exist FAIL
> >   \_SB_.CPUS.C000
> >   No _PSS exists
> > Summary: 1 passed, 1 failed
> > ---- ACPI _PSS (Pstate) runtime tests ----
> > [assert] _PSS must exist FAIL
> >   \_SB_.CPUS.C000
> >   No _PSS exists
> > Summary: 0 passed, 1 failed
> >
> > Cc: Daniel P. Berrangé <berrange@redhat.com>
> > Cc: Paolo Bonzini <pbonzini@redhat.com>
> > Cc: Maydell Peter <peter.maydell@linaro.org>
> > Cc: John Snow <jsnow@redhat.com>
> > Cc: Thomas Huth <thuth@redhat.com>
> > Cc: Alex Bennée <alex.bennee@linaro.org>
> > Cc: Igor Mammedov <imammedo@redhat.com>
> > Cc: Michael Tsirkin <mst@redhat.com>
> > Signed-off-by: Ani Sinha <ani@anisinha.ca>
> > ---
> >  tests/avocado/acpi-bits/bits-tests/testacpi.py2 | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/avocado/acpi-bits/bits-tests/testacpi.py2 
> > b/tests/avocado/acpi-bits/bits-tests/testacpi.py2
> > index 18dc818d62..f818a9cce6 100644
> > --- a/tests/avocado/acpi-bits/bits-tests/testacpi.py2
> > +++ b/tests/avocado/acpi-bits/bits-tests/testacpi.py2
> > @@ -40,8 +40,8 @@ import time
> >  
> >  def register_tests():
> >      testsuite.add_test("ACPI _MAT (Multiple APIC Table Entry) under 
> > Processor objects", test_mat, submenu="ACPI Tests")
> > -    testsuite.add_test("ACPI _PSS (Pstate) table conformance tests", 
> > test_pss, submenu="ACPI Tests")
> > -    testsuite.add_test("ACPI _PSS (Pstate) runtime tests", test_pstates, 
> > submenu="ACPI Tests")
> > +#    testsuite.add_test("ACPI _PSS (Pstate) table conformance tests", 
> > test_pss, submenu="ACPI Tests")
> > +#    testsuite.add_test("ACPI _PSS (Pstate) runtime tests", test_pstates, 
> > submenu="ACPI Tests")
> >      testsuite.add_test("ACPI DSDT (Differentiated System Description 
> > Table)", test_dsdt, submenu="ACPI Tests")
> >      testsuite.add_test("ACPI FACP (Fixed ACPI Description Table)", 
> > test_facp, submenu="ACPI Tests")
> >      testsuite.add_test("ACPI HPET (High Precision Event Timer Table)", 
> > test_hpet, submenu="ACPI Tests")
> 
> I think this breaks bisection so should probably be included in the
> commit that add the test with a comment in the commit message.

IIUC, this is all unreachable code until Patch 08 adds the avocado
test providing the entry point, so bisect ought to be safe.

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|