[v1] tests: acpi: improve tests reproducability

[Qemu-devel] [PATCH 0/2] tests: acpi: improve tests reproducability

Igor Mammedov posted 2 patches 4 years, 10 months ago

Diff against v3
Download series mbox

Test s390x passed

Test checkpatch passed

Test asan passed

Test docker-mingw@fedora passed

Test FreeBSD passed

Test docker-clang@ubuntu passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190704082736.25258-1-imammedo@redhat.com

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <lvivier@redhat.com>, Igor Mammedov <imammedo@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>

There is a newer version of this series

tests/bios-tables-test.c | 23 +++++++++++++++++------
1 file changed, 17 insertions(+), 6 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/2] tests: acpi: improve tests reproducability

Posted by Igor Mammedov 4 years, 10 months ago

series should help to make tests more reproducable and not depend
on IASL being installed. IASL will be required only is user needs
to get textual diff of mismatching files.

Igor Mammedov (2):
  tests: acpi: do not require IASL for dumping AML blobs
  tests: acpi: do not skip tests when IASL is not installed

 tests/bios-tables-test.c | 23 +++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

-- 
2.18.1

Re: [Qemu-devel] [PATCH 0/2] tests: acpi: improve tests reproducability

Posted by Michael S. Tsirkin 4 years, 10 months ago

On Thu, Jul 04, 2019 at 04:27:34AM -0400, Igor Mammedov wrote:
> 
> series should help to make tests more reproducable and not depend
> on IASL being installed. IASL will be required only is user needs
> to get textual diff of mismatching files.


I like this very much but pls send v3 with both patches so
I can apply it easily.

> Igor Mammedov (2):
>   tests: acpi: do not require IASL for dumping AML blobs
>   tests: acpi: do not skip tests when IASL is not installed
> 
>  tests/bios-tables-test.c | 23 +++++++++++++++++------
>  1 file changed, 17 insertions(+), 6 deletions(-)
> 
> -- 
> 2.18.1