| Message ID | 20200114165138.15716-1-shameerali.kolothum.thodi@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | tests: acpi: update path in rebuild-expected-aml | expand |
On 14/01/2020 17.51, Shameer Kolothum wrote: > Since commit 1e8a1fae7464("test: Move qtests to a separate > directory") qtests are now placed in a separate folder and > this breaks the script used to rebuild the expected ACPI > tables for bios-tables-test. Update the script with correct > path. > > Fixes: 1e8a1fae7464("test: Move qtests to a separate directory") > Signed-off-by: Shameer Kolothum <shameerali.kolothum.thodi@huawei.com> > --- > tests/data/acpi/rebuild-expected-aml.sh | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/tests/data/acpi/rebuild-expected-aml.sh b/tests/data/acpi/rebuild-expected-aml.sh > index f89d4624bc..d44e511533 100755 > --- a/tests/data/acpi/rebuild-expected-aml.sh > +++ b/tests/data/acpi/rebuild-expected-aml.sh > @@ -14,7 +14,7 @@ > > qemu_bins="x86_64-softmmu/qemu-system-x86_64 aarch64-softmmu/qemu-system-aarch64" > > -if [ ! -e "tests/bios-tables-test" ]; then > +if [ ! -e "tests/qtest/bios-tables-test" ]; then > echo "Test: bios-tables-test is required! Run make check before this script." > echo "Run this script from the build directory." > exit 1; > @@ -26,11 +26,11 @@ for qemu in $qemu_bins; do > echo "Also, run this script from the build directory." > exit 1; > fi > - TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu tests/bios-tables-test > + TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu tests/qtest/bios-tables-test > done > > eval `grep SRC_PATH= config-host.mak` > > -echo '/* List of comma-separated changed AML files to ignore */' > ${SRC_PATH}/tests/bios-tables-test-allowed-diff.h > +echo '/* List of comma-separated changed AML files to ignore */' > ${SRC_PATH}/tests/qtest/bios-tables-test-allowed-diff.h > > echo "The files were rebuilt and can be added to git." Oh, sorry for missing that in my patch series ... is there maybe a way that we could test this script in one of our CI pipelines so that it is not so easy to miss? Anyway, for this patch: Reviewed-by: Thomas Huth <thuth@redhat.com>
Patchew URL: https://patchew.org/QEMU/20200114165138.15716-1-shameerali.kolothum.thodi@huawei.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PATCH] tests: acpi: update path in rebuild-expected-aml
Type: series
Message-id: 20200114165138.15716-1-shameerali.kolothum.thodi@huawei.com
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
From https://github.com/patchew-project/qemu
* [new tag] patchew/20200114165138.15716-1-shameerali.kolothum.thodi@huawei.com -> patchew/20200114165138.15716-1-shameerali.kolothum.thodi@huawei.com
Switched to a new branch 'test'
d4dcd84 tests: acpi: update path in rebuild-expected-aml
=== OUTPUT BEGIN ===
ERROR: line over 90 characters
#42: FILE: tests/data/acpi/rebuild-expected-aml.sh:34:
+echo '/* List of comma-separated changed AML files to ignore */' > ${SRC_PATH}/tests/qtest/bios-tables-test-allowed-diff.h
total: 1 errors, 0 warnings, 21 lines checked
Commit d4dcd849a42e (tests: acpi: update path in rebuild-expected-aml) has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20200114165138.15716-1-shameerali.kolothum.thodi@huawei.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
> -----Original Message----- > From: Thomas Huth [mailto:thuth@redhat.com] > Sent: 14 January 2020 17:08 > To: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@huawei.com>; > qemu-devel@nongnu.org; imammedo@redhat.com; mst@redhat.com > Cc: xuwei (O) <xuwei5@huawei.com>; Linuxarm <linuxarm@huawei.com> > Subject: Re: [PATCH] tests: acpi: update path in rebuild-expected-aml > > On 14/01/2020 17.51, Shameer Kolothum wrote: > > Since commit 1e8a1fae7464("test: Move qtests to a separate > > directory") qtests are now placed in a separate folder and > > this breaks the script used to rebuild the expected ACPI > > tables for bios-tables-test. Update the script with correct > > path. > > > > Fixes: 1e8a1fae7464("test: Move qtests to a separate directory") > > Signed-off-by: Shameer Kolothum <shameerali.kolothum.thodi@huawei.com> > > --- > > tests/data/acpi/rebuild-expected-aml.sh | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/tests/data/acpi/rebuild-expected-aml.sh > b/tests/data/acpi/rebuild-expected-aml.sh > > index f89d4624bc..d44e511533 100755 > > --- a/tests/data/acpi/rebuild-expected-aml.sh > > +++ b/tests/data/acpi/rebuild-expected-aml.sh > > @@ -14,7 +14,7 @@ > > > > qemu_bins="x86_64-softmmu/qemu-system-x86_64 > aarch64-softmmu/qemu-system-aarch64" > > > > -if [ ! -e "tests/bios-tables-test" ]; then > > +if [ ! -e "tests/qtest/bios-tables-test" ]; then > > echo "Test: bios-tables-test is required! Run make check before this > script." > > echo "Run this script from the build directory." > > exit 1; > > @@ -26,11 +26,11 @@ for qemu in $qemu_bins; do > > echo "Also, run this script from the build directory." > > exit 1; > > fi > > - TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu > tests/bios-tables-test > > + TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu > tests/qtest/bios-tables-test > > done > > > > eval `grep SRC_PATH= config-host.mak` > > > > -echo '/* List of comma-separated changed AML files to ignore */' > > ${SRC_PATH}/tests/bios-tables-test-allowed-diff.h > > +echo '/* List of comma-separated changed AML files to ignore */' > > ${SRC_PATH}/tests/qtest/bios-tables-test-allowed-diff.h > > > > echo "The files were rebuilt and can be added to git." > > Oh, sorry for missing that in my patch series ... is there maybe a way > that we could test this script in one of our CI pipelines so that it is > not so easy to miss? Right. That will be a useful one. I am also seeing another error when I run "make check-qtest" on x86_64. This doesn’t seems to be related to the recent changes. I have gone back to 4.1.0 and it is still there. TEST check-qtest-x86_64: tests/boot-order-test TEST check-qtest-x86_64: tests/bios-tables-test Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator Could not access KVM kernel module: No such file or directory qemu-system-x86_64: failed to initialize KVM: No such file or directory qemu-system-x86_64: Back to tcg accelerator acpi-test: Warning! FACP binary file mismatch. Actual [aml:/tmp/aml-2Q9EE0], Expected [aml:tests/data/acpi/pc/FACP.bridge]. acpi-test: Warning! FACP mismatch. Actual [asl:/tmp/asl-CQ9EE0.dsl, aml:/tmp/aml-2Q9EE0], Expected [asl:/tmp/asl-N18EE0.dsl, aml:tests/data/acpi/pc/FACP.bridge]. ** ERROR:tests/bios-tables-test.c:447:test_acpi_asl: assertion failed: (all_tables_match) ERROR - Bail out! ERROR:tests/bios-tables-test.c:447:test_acpi_asl: assertion failed: (all_tables_match) Aborted (core dumped) /home/shameer/qemu/tests/Makefile.include:899: recipe for target 'check-qtest-x86_64' failed make: *** [check-qtest-x86_64] Error 1 FACP seems to have changed and it looks like need to run the script to generate a new one. ~/qemu$ diff -u /tmp/asl-CQ9EE0.dsl /tmp/asl-N18EE0.dsl --- /tmp/asl-CQ9EE0.dsl 2020-01-15 10:52:03.018448627 +0000 +++ /tmp/asl-N18EE0.dsl 2020-01-15 10:52:03.022448627 +0000 @@ -3,7 +3,7 @@ * AML/ASL+ Disassembler version 20180105 (64-bit version) * Copyright (c) 2000 - 2018 Intel Corporation * - * Disassembly of /tmp/aml-2Q9EE0, Wed Jan 15 10:52:03 2020 + * Disassembly of tests/data/acpi/pc/FACP.bridge, Wed Jan 15 10:52:03 + 2020 * * ACPI Data Table [FACP] * @@ -13,7 +13,7 @@ [000h 0000 4] Signature : "FACP" [Fixed ACPI Description Table (FADT)] [004h 0004 4] Table Length : 00000074 [008h 0008 1] Revision : 01 -[009h 0009 1] Checksum : A1 +[009h 0009 1] Checksum : D3 [00Ah 0010 6] Oem ID : "BOCHS " [010h 0016 8] Oem Table ID : "BXPCFACP" [018h 0024 4] Oem Revision : 00000001 @@ -54,7 +54,7 @@ [069h 0105 1] Duty Cycle Width : 00 [06Ah 0106 1] RTC Day Alarm Index : 00 [06Bh 0107 1] RTC Month Alarm Index : 00 -[06Ch 0108 1] RTC Century Index : 32 +[06Ch 0108 1] RTC Century Index : 00 [06Dh 0109 2] Boot Flags (decoded below) : 0000 Legacy Devices Supported (V2) : 0 8042 Present on ports 60/64 (V2) : 0 @@ -89,11 +89,11 @@ Not sure this is an already reported one or I am missing something. Please let me know. Thanks, Shameer > Anyway, for this patch: > > Reviewed-by: Thomas Huth <thuth@redhat.com>
On Wed, Jan 15, 2020 at 11:01:44AM +0000, Shameerali Kolothum Thodi wrote: > > > > -----Original Message----- > > From: Thomas Huth [mailto:thuth@redhat.com] > > Sent: 14 January 2020 17:08 > > To: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@huawei.com>; > > qemu-devel@nongnu.org; imammedo@redhat.com; mst@redhat.com > > Cc: xuwei (O) <xuwei5@huawei.com>; Linuxarm <linuxarm@huawei.com> > > Subject: Re: [PATCH] tests: acpi: update path in rebuild-expected-aml > > > > On 14/01/2020 17.51, Shameer Kolothum wrote: > > > Since commit 1e8a1fae7464("test: Move qtests to a separate > > > directory") qtests are now placed in a separate folder and > > > this breaks the script used to rebuild the expected ACPI > > > tables for bios-tables-test. Update the script with correct > > > path. > > > > > > Fixes: 1e8a1fae7464("test: Move qtests to a separate directory") > > > Signed-off-by: Shameer Kolothum <shameerali.kolothum.thodi@huawei.com> > > > --- > > > tests/data/acpi/rebuild-expected-aml.sh | 6 +++--- > > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/tests/data/acpi/rebuild-expected-aml.sh > > b/tests/data/acpi/rebuild-expected-aml.sh > > > index f89d4624bc..d44e511533 100755 > > > --- a/tests/data/acpi/rebuild-expected-aml.sh > > > +++ b/tests/data/acpi/rebuild-expected-aml.sh > > > @@ -14,7 +14,7 @@ > > > > > > qemu_bins="x86_64-softmmu/qemu-system-x86_64 > > aarch64-softmmu/qemu-system-aarch64" > > > > > > -if [ ! -e "tests/bios-tables-test" ]; then > > > +if [ ! -e "tests/qtest/bios-tables-test" ]; then > > > echo "Test: bios-tables-test is required! Run make check before this > > script." > > > echo "Run this script from the build directory." > > > exit 1; > > > @@ -26,11 +26,11 @@ for qemu in $qemu_bins; do > > > echo "Also, run this script from the build directory." > > > exit 1; > > > fi > > > - TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu > > tests/bios-tables-test > > > + TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu > > tests/qtest/bios-tables-test > > > done > > > > > > eval `grep SRC_PATH= config-host.mak` > > > > > > -echo '/* List of comma-separated changed AML files to ignore */' > > > ${SRC_PATH}/tests/bios-tables-test-allowed-diff.h > > > +echo '/* List of comma-separated changed AML files to ignore */' > > > ${SRC_PATH}/tests/qtest/bios-tables-test-allowed-diff.h > > > > > > echo "The files were rebuilt and can be added to git." > > > > Oh, sorry for missing that in my patch series ... is there maybe a way > > that we could test this script in one of our CI pipelines so that it is > > not so easy to miss? > > Right. That will be a useful one. > > I am also seeing another error when I run "make check-qtest" on x86_64. > This doesn’t seems to be related to the recent changes. I have gone back > to 4.1.0 and it is still there. > > TEST check-qtest-x86_64: tests/boot-order-test > TEST check-qtest-x86_64: tests/bios-tables-test > Could not access KVM kernel module: No such file or directory > qemu-system-x86_64: failed to initialize KVM: No such file or directory > qemu-system-x86_64: Back to tcg accelerator > Could not access KVM kernel module: No such file or directory > qemu-system-x86_64: failed to initialize KVM: No such file or directory > qemu-system-x86_64: Back to tcg accelerator > Could not access KVM kernel module: No such file or directory > qemu-system-x86_64: failed to initialize KVM: No such file or directory > qemu-system-x86_64: Back to tcg accelerator > acpi-test: Warning! FACP binary file mismatch. Actual [aml:/tmp/aml-2Q9EE0], Expected [aml:tests/data/acpi/pc/FACP.bridge]. > acpi-test: Warning! FACP mismatch. Actual [asl:/tmp/asl-CQ9EE0.dsl, aml:/tmp/aml-2Q9EE0], Expected [asl:/tmp/asl-N18EE0.dsl, aml:tests/data/acpi/pc/FACP.bridge]. > ** > ERROR:tests/bios-tables-test.c:447:test_acpi_asl: assertion failed: (all_tables_match) > ERROR - Bail out! ERROR:tests/bios-tables-test.c:447:test_acpi_asl: assertion failed: (all_tables_match) > Aborted (core dumped) > /home/shameer/qemu/tests/Makefile.include:899: recipe for target 'check-qtest-x86_64' failed > make: *** [check-qtest-x86_64] Error 1 Well make check seems to pass for me ... What's different for you? > FACP seems to have changed and it looks like need to run the script to generate > a new one. > > ~/qemu$ diff -u /tmp/asl-CQ9EE0.dsl /tmp/asl-N18EE0.dsl > --- /tmp/asl-CQ9EE0.dsl 2020-01-15 10:52:03.018448627 +0000 > +++ /tmp/asl-N18EE0.dsl 2020-01-15 10:52:03.022448627 +0000 > @@ -3,7 +3,7 @@ > * AML/ASL+ Disassembler version 20180105 (64-bit version) > * Copyright (c) 2000 - 2018 Intel Corporation > * > - * Disassembly of /tmp/aml-2Q9EE0, Wed Jan 15 10:52:03 2020 > + * Disassembly of tests/data/acpi/pc/FACP.bridge, Wed Jan 15 10:52:03 > + 2020 > * > * ACPI Data Table [FACP] > * > @@ -13,7 +13,7 @@ > [000h 0000 4] Signature : "FACP" [Fixed ACPI > Description Table (FADT)] > [004h 0004 4] Table Length : 00000074 > [008h 0008 1] Revision : 01 > -[009h 0009 1] Checksum : A1 > +[009h 0009 1] Checksum : D3 > [00Ah 0010 6] Oem ID : "BOCHS " > [010h 0016 8] Oem Table ID : "BXPCFACP" > [018h 0024 4] Oem Revision : 00000001 > @@ -54,7 +54,7 @@ > [069h 0105 1] Duty Cycle Width : 00 > [06Ah 0106 1] RTC Day Alarm Index : 00 > [06Bh 0107 1] RTC Month Alarm Index : 00 > -[06Ch 0108 1] RTC Century Index : 32 > +[06Ch 0108 1] RTC Century Index : 00 > [06Dh 0109 2] Boot Flags (decoded below) : 0000 > Legacy Devices Supported (V2) : 0 > 8042 Present on ports 60/64 (V2) : 0 @@ -89,11 +89,11 @@ > > > Not sure this is an already reported one or I am missing something. > Please let me know. > > Thanks, > Shameer > > > Anyway, for this patch: > > > > Reviewed-by: Thomas Huth <thuth@redhat.com> >
> -----Original Message----- > From: Michael S. Tsirkin [mailto:mst@redhat.com] > Sent: 15 January 2020 13:46 > To: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@huawei.com> > Cc: Thomas Huth <thuth@redhat.com>; qemu-devel@nongnu.org; > imammedo@redhat.com; xuwei (O) <xuwei5@huawei.com>; Linuxarm > <linuxarm@huawei.com> > Subject: Re: [PATCH] tests: acpi: update path in rebuild-expected-aml > > On Wed, Jan 15, 2020 at 11:01:44AM +0000, Shameerali Kolothum Thodi > wrote: > > > > > > > -----Original Message----- > > > From: Thomas Huth [mailto:thuth@redhat.com] > > > Sent: 14 January 2020 17:08 > > > To: Shameerali Kolothum Thodi > > > <shameerali.kolothum.thodi@huawei.com>; > > > qemu-devel@nongnu.org; imammedo@redhat.com; mst@redhat.com > > > Cc: xuwei (O) <xuwei5@huawei.com>; Linuxarm <linuxarm@huawei.com> > > > Subject: Re: [PATCH] tests: acpi: update path in > > > rebuild-expected-aml > > > > > > On 14/01/2020 17.51, Shameer Kolothum wrote: > > > > Since commit 1e8a1fae7464("test: Move qtests to a separate > > > > directory") qtests are now placed in a separate folder and this > > > > breaks the script used to rebuild the expected ACPI tables for > > > > bios-tables-test. Update the script with correct path. > > > > > > > > Fixes: 1e8a1fae7464("test: Move qtests to a separate directory") > > > > Signed-off-by: Shameer Kolothum > > > > <shameerali.kolothum.thodi@huawei.com> > > > > --- > > > > tests/data/acpi/rebuild-expected-aml.sh | 6 +++--- > > > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > > > > > diff --git a/tests/data/acpi/rebuild-expected-aml.sh > > > b/tests/data/acpi/rebuild-expected-aml.sh > > > > index f89d4624bc..d44e511533 100755 > > > > --- a/tests/data/acpi/rebuild-expected-aml.sh > > > > +++ b/tests/data/acpi/rebuild-expected-aml.sh > > > > @@ -14,7 +14,7 @@ > > > > > > > > qemu_bins="x86_64-softmmu/qemu-system-x86_64 > > > aarch64-softmmu/qemu-system-aarch64" > > > > > > > > -if [ ! -e "tests/bios-tables-test" ]; then > > > > +if [ ! -e "tests/qtest/bios-tables-test" ]; then > > > > echo "Test: bios-tables-test is required! Run make check > > > > before this > > > script." > > > > echo "Run this script from the build directory." > > > > exit 1; > > > > @@ -26,11 +26,11 @@ for qemu in $qemu_bins; do > > > > echo "Also, run this script from the build directory." > > > > exit 1; > > > > fi > > > > - TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu > > > tests/bios-tables-test > > > > + TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu > > > tests/qtest/bios-tables-test > > > > done > > > > > > > > eval `grep SRC_PATH= config-host.mak` > > > > > > > > -echo '/* List of comma-separated changed AML files to ignore */' > > > > > > > > ${SRC_PATH}/tests/bios-tables-test-allowed-diff.h > > > > +echo '/* List of comma-separated changed AML files to ignore */' > > > > +> > > > ${SRC_PATH}/tests/qtest/bios-tables-test-allowed-diff.h > > > > > > > > echo "The files were rebuilt and can be added to git." > > > > > > Oh, sorry for missing that in my patch series ... is there maybe a > > > way that we could test this script in one of our CI pipelines so > > > that it is not so easy to miss? > > > > Right. That will be a useful one. > > > > I am also seeing another error when I run "make check-qtest" on x86_64. > > This doesn’t seems to be related to the recent changes. I have gone > > back to 4.1.0 and it is still there. > > > > TEST check-qtest-x86_64: tests/boot-order-test > > TEST check-qtest-x86_64: tests/bios-tables-test > > Could not access KVM kernel module: No such file or directory > > qemu-system-x86_64: failed to initialize KVM: No such file or > > directory > > qemu-system-x86_64: Back to tcg accelerator Could not access KVM > > kernel module: No such file or directory > > qemu-system-x86_64: failed to initialize KVM: No such file or > > directory > > qemu-system-x86_64: Back to tcg accelerator Could not access KVM > > kernel module: No such file or directory > > qemu-system-x86_64: failed to initialize KVM: No such file or > > directory > > qemu-system-x86_64: Back to tcg accelerator > > acpi-test: Warning! FACP binary file mismatch. Actual [aml:/tmp/aml-2Q9EE0], > Expected [aml:tests/data/acpi/pc/FACP.bridge]. > > acpi-test: Warning! FACP mismatch. Actual [asl:/tmp/asl-CQ9EE0.dsl, > aml:/tmp/aml-2Q9EE0], Expected [asl:/tmp/asl-N18EE0.dsl, > aml:tests/data/acpi/pc/FACP.bridge]. > > ** > > ERROR:tests/bios-tables-test.c:447:test_acpi_asl: assertion failed: > > (all_tables_match) ERROR - Bail out! > > ERROR:tests/bios-tables-test.c:447:test_acpi_asl: assertion failed: > > (all_tables_match) Aborted (core dumped) > > /home/shameer/qemu/tests/Makefile.include:899: recipe for target > > 'check-qtest-x86_64' failed > > make: *** [check-qtest-x86_64] Error 1 > > Well make check seems to pass for me ... What's different for you? I tried a fresh git clone of qemu and that seems to work fine. So I guess it might be something to do with my earlier setup. Ignore and sorry for the noise. Thanks, Shameer
diff --git a/tests/data/acpi/rebuild-expected-aml.sh b/tests/data/acpi/rebuild-expected-aml.sh index f89d4624bc..d44e511533 100755 --- a/tests/data/acpi/rebuild-expected-aml.sh +++ b/tests/data/acpi/rebuild-expected-aml.sh @@ -14,7 +14,7 @@ qemu_bins="x86_64-softmmu/qemu-system-x86_64 aarch64-softmmu/qemu-system-aarch64" -if [ ! -e "tests/bios-tables-test" ]; then +if [ ! -e "tests/qtest/bios-tables-test" ]; then echo "Test: bios-tables-test is required! Run make check before this script." echo "Run this script from the build directory." exit 1; @@ -26,11 +26,11 @@ for qemu in $qemu_bins; do echo "Also, run this script from the build directory." exit 1; fi - TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu tests/bios-tables-test + TEST_ACPI_REBUILD_AML=y QTEST_QEMU_BINARY=$qemu tests/qtest/bios-tables-test done eval `grep SRC_PATH= config-host.mak` -echo '/* List of comma-separated changed AML files to ignore */' > ${SRC_PATH}/tests/bios-tables-test-allowed-diff.h +echo '/* List of comma-separated changed AML files to ignore */' > ${SRC_PATH}/tests/qtest/bios-tables-test-allowed-diff.h echo "The files were rebuilt and can be added to git."
Since commit 1e8a1fae7464("test: Move qtests to a separate directory") qtests are now placed in a separate folder and this breaks the script used to rebuild the expected ACPI tables for bios-tables-test. Update the script with correct path. Fixes: 1e8a1fae7464("test: Move qtests to a separate directory") Signed-off-by: Shameer Kolothum <shameerali.kolothum.thodi@huawei.com> --- tests/data/acpi/rebuild-expected-aml.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)