[PATCH 0/4] acpi: fix short OEM [Table] ID padding

Igor Mammedov posted 4 patches 2 weeks, 1 day ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220112130332.1648664-1-imammedo@redhat.com
Maintainers: Igor Mammedov <imammedo@redhat.com>, Ani Sinha <ani@anisinha.ca>, "Michael S. Tsirkin" <mst@redhat.com>
hw/acpi/aml-build.c              |   4 ++--
tests/data/acpi/pc/SSDT.dimmpxm  | Bin 734 -> 734 bytes
tests/data/acpi/q35/FACP.slic    | Bin 244 -> 244 bytes
tests/data/acpi/q35/SSDT.dimmpxm | Bin 734 -> 734 bytes
tests/data/acpi/virt/SSDT.memhp  | Bin 736 -> 736 bytes
tests/qtest/bios-tables-test.c   |  15 ++++++---------
6 files changed, 8 insertions(+), 11 deletions(-)

[PATCH 0/4] acpi: fix short OEM [Table] ID padding

Posted by Igor Mammedov 2 weeks, 1 day ago
Since 6.0 the commit:
  602b458201 ("acpi: Permit OEM ID and OEM table ID fields to be changed")
regressed values of OEM [Table] ID fields in ACPI tables
by padding them with whitespace is a value is shorter then
max possible. That depending on vendor broke OEM [Table] ID patching
with SLIC table values and as result licensing of Windows guests.

First reported here https://gitlab.com/qemu-project/qemu/-/issues/707

CC: Marian Postevca <posteuca@mutex.one>
CC: Michael S. Tsirkin <mst@redhat.com>
CC: Ani Sinha <ani@anisinha.ca>

Igor Mammedov (4):
  tests: acpi: manually pad OEM_ID/OEM_TABLE_ID for test_oem_fields()
    test
  tests: acpi: whitelist nvdimm's SSDT and FACP.slic expected blobs
  acpi: fix OEM ID/OEM Table ID padding
  tests: acpi: update expected blobs

 hw/acpi/aml-build.c              |   4 ++--
 tests/data/acpi/pc/SSDT.dimmpxm  | Bin 734 -> 734 bytes
 tests/data/acpi/q35/FACP.slic    | Bin 244 -> 244 bytes
 tests/data/acpi/q35/SSDT.dimmpxm | Bin 734 -> 734 bytes
 tests/data/acpi/virt/SSDT.memhp  | Bin 736 -> 736 bytes
 tests/qtest/bios-tables-test.c   |  15 ++++++---------
 6 files changed, 8 insertions(+), 11 deletions(-)

-- 
2.31.1