While refactoring i386/FADT generation to build_append_int_noprefix() and testing it, It turned out that FADT is only tested for valid checksum but actual test for unintended changes isn't applied to it even though we have reference tables in tree. So here goes a couple of cleanups to reflect what fuctions do + some comments and actual fix. Note to maintainer: FADT reference table is out of sync and should be updated along with series applied. CC: "Michael S. Tsirkin" <mst@redhat.com> CC: Marcel Apfelbaum <marcel@redhat.com> Igor Mammedov (5): tests: acpi: move tested tables array allocation outside of test_acpi_dsdt_table() tests: acpi: init table descriptor in test_dst_table() tests: acpi: rename test_acpi_tables()/test_dst_table() to reflect its usage tests: acpi: add comments to fetch_rsdt_referenced_tables/data->tables usage tests: acpi: fix FADT not being compared to reference table tests/bios-tables-test.c | 30 ++++++++++++++++++------------ 1 file changed, 18 insertions(+), 12 deletions(-) -- 2.7.4
On Mon, Jul 31, 2017 at 05:40:47PM +0200, Igor Mammedov wrote: > While refactoring i386/FADT generation to build_append_int_noprefix() > and testing it, It turned out that FADT is only tested for valid > checksum but actual test for unintended changes isn't applied to it > even though we have reference tables in tree. > So here goes a couple of cleanups to reflect what fuctions do + > some comments and actual fix. > > Note to maintainer: > FADT reference table is out of sync and should be updated along with > series applied. > > CC: "Michael S. Tsirkin" <mst@redhat.com> > CC: Marcel Apfelbaum <marcel@redhat.com> Absolutely good stuff, but not a bugfix as such (it's not that the test is wrong, it's that we skip FADT for now) so I don't think this is 2.10 material. > Igor Mammedov (5): > tests: acpi: move tested tables array allocation outside of > test_acpi_dsdt_table() > tests: acpi: init table descriptor in test_dst_table() > tests: acpi: rename test_acpi_tables()/test_dst_table() to reflect its > usage > tests: acpi: add comments to fetch_rsdt_referenced_tables/data->tables > usage > tests: acpi: fix FADT not being compared to reference table > > tests/bios-tables-test.c | 30 ++++++++++++++++++------------ > 1 file changed, 18 insertions(+), 12 deletions(-) > > -- > 2.7.4
On Wed, 2 Aug 2017 00:14:18 +0300 "Michael S. Tsirkin" <mst@redhat.com> wrote: > On Mon, Jul 31, 2017 at 05:40:47PM +0200, Igor Mammedov wrote: > > While refactoring i386/FADT generation to build_append_int_noprefix() > > and testing it, It turned out that FADT is only tested for valid > > checksum but actual test for unintended changes isn't applied to it > > even though we have reference tables in tree. > > So here goes a couple of cleanups to reflect what fuctions do + > > some comments and actual fix. > > > > Note to maintainer: > > FADT reference table is out of sync and should be updated along with > > series applied. > > > > CC: "Michael S. Tsirkin" <mst@redhat.com> > > CC: Marcel Apfelbaum <marcel@redhat.com> > > Absolutely good stuff, but not a bugfix as such (it's not that the > test is wrong, it's that we skip FADT for now) > so I don't think this is 2.10 material. Agreed, it could go in when 2.11 merge window is open. > > > Igor Mammedov (5): > > tests: acpi: move tested tables array allocation outside of > > test_acpi_dsdt_table() > > tests: acpi: init table descriptor in test_dst_table() > > tests: acpi: rename test_acpi_tables()/test_dst_table() to reflect its > > usage > > tests: acpi: add comments to fetch_rsdt_referenced_tables/data->tables > > usage > > tests: acpi: fix FADT not being compared to reference table > > > > tests/bios-tables-test.c | 30 ++++++++++++++++++------------ > > 1 file changed, 18 insertions(+), 12 deletions(-) > > > > -- > > 2.7.4
On Wed, Aug 02, 2017 at 09:51:22AM +0200, Igor Mammedov wrote: > On Wed, 2 Aug 2017 00:14:18 +0300 > "Michael S. Tsirkin" <mst@redhat.com> wrote: > > > On Mon, Jul 31, 2017 at 05:40:47PM +0200, Igor Mammedov wrote: > > > While refactoring i386/FADT generation to build_append_int_noprefix() > > > and testing it, It turned out that FADT is only tested for valid > > > checksum but actual test for unintended changes isn't applied to it > > > even though we have reference tables in tree. > > > So here goes a couple of cleanups to reflect what fuctions do + > > > some comments and actual fix. > > > > > > Note to maintainer: > > > FADT reference table is out of sync and should be updated along with > > > series applied. > > > > > > CC: "Michael S. Tsirkin" <mst@redhat.com> > > > CC: Marcel Apfelbaum <marcel@redhat.com> > > > > Absolutely good stuff, but not a bugfix as such (it's not that the > > test is wrong, it's that we skip FADT for now) > > so I don't think this is 2.10 material. > Agreed, it could go in when 2.11 merge window is open. thanks,pls remember to repost or ping then. > > > > > Igor Mammedov (5): > > > tests: acpi: move tested tables array allocation outside of > > > test_acpi_dsdt_table() > > > tests: acpi: init table descriptor in test_dst_table() > > > tests: acpi: rename test_acpi_tables()/test_dst_table() to reflect its > > > usage > > > tests: acpi: add comments to fetch_rsdt_referenced_tables/data->tables > > > usage > > > tests: acpi: fix FADT not being compared to reference table > > > > > > tests/bios-tables-test.c | 30 ++++++++++++++++++------------ > > > 1 file changed, 18 insertions(+), 12 deletions(-) > > > > > > -- > > > 2.7.4
© 2016 - 2024 Red Hat, Inc.