Blame SOURCES/kvm-tests-acpi-manually-pad-OEM_ID-OEM_TABLE_ID-for-test.patch

4841a6
From d94b3278c84cf7451489631d804a6b5cbd28a59d Mon Sep 17 00:00:00 2001
4841a6
From: Jon Maloy <jmaloy@redhat.com>
4841a6
Date: Wed, 30 Mar 2022 14:52:34 -0400
4841a6
Subject: [PATCH 13/18] tests: acpi: manually pad OEM_ID/OEM_TABLE_ID for
60061b
 test_oem_fields() test
60061b
4841a6
RH-Author: Jon Maloy <jmaloy@redhat.com>
4841a6
RH-MergeRequest: 141: acpi: fix QEMU crash when started with SLIC table
4841a6
RH-Commit: [5/10] 4ec8c738acec178c2f005f189b0c2a77a7af4088 (jmaloy/qemu-kvm)
4841a6
RH-Bugzilla: 2062611
4841a6
RH-Acked-by: Igor Mammedov <imammedo@redhat.com>
4841a6
4841a6
Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=2062611
4841a6
Upstream: Merged
4841a6
4841a6
commit a849522f726767022203ef2b6c395ea19facb866
4841a6
Author: Igor Mammedov <imammedo@redhat.com>
4841a6
Date:   Wed Jan 12 08:03:29 2022 -0500
4841a6
4841a6
    tests: acpi: manually pad OEM_ID/OEM_TABLE_ID for test_oem_fields() test
4841a6
4841a6
    The next commit will revert OEM fields padding with whitespace to
4841a6
    padding with '\0' as it was before [1]. As result test_oem_fields() will
4841a6
    fail due to unexpectedly smaller ID sizes read from QEMU ACPI tables.
4841a6
4841a6
    Pad OEM_ID/OEM_TABLE_ID manually with spaces so that values the test
4841a6
    puts on QEMU CLI and expected values match.
4841a6
4841a6
    1) 602b458201 ("acpi: Permit OEM ID and OEM table ID fields to be changed")
4841a6
    Signed-off-by: Igor Mammedov <imammedo@redhat.com>
4841a6
    Message-Id: <20220112130332.1648664-2-imammedo@redhat.com>
4841a6
    Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
4841a6
    Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
4841a6
60061b
(cherry picked from commit a849522f726767022203ef2b6c395ea19facb866)
4841a6
Signed-off-by: Jon Maloy <jmaloy@redhat.com>
60061b
---
60061b
 tests/qtest/bios-tables-test.c | 15 ++++++---------
60061b
 1 file changed, 6 insertions(+), 9 deletions(-)
60061b
60061b
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
60061b
index e159b71136..348fdbd202 100644
60061b
--- a/tests/qtest/bios-tables-test.c
60061b
+++ b/tests/qtest/bios-tables-test.c
60061b
@@ -71,9 +71,10 @@
60061b
 
60061b
 #define ACPI_REBUILD_EXPECTED_AML "TEST_ACPI_REBUILD_AML"
60061b
 
60061b
-#define OEM_ID             "TEST"
60061b
-#define OEM_TABLE_ID       "OEM"
60061b
-#define OEM_TEST_ARGS      "-machine x-oem-id="OEM_ID",x-oem-table-id="OEM_TABLE_ID
60061b
+#define OEM_ID             "TEST  "
60061b
+#define OEM_TABLE_ID       "OEM     "
60061b
+#define OEM_TEST_ARGS      "-machine x-oem-id='" OEM_ID "',x-oem-table-id='" \
60061b
+                           OEM_TABLE_ID "'"
60061b
 
60061b
 typedef struct {
60061b
     bool tcg_only;
60061b
@@ -1484,11 +1485,7 @@ static void test_acpi_q35_slic(void)
60061b
 static void test_oem_fields(test_data *data)
60061b
 {
60061b
     int i;
60061b
-    char oem_id[6];
60061b
-    char oem_table_id[8];
60061b
 
60061b
-    strpadcpy(oem_id, sizeof oem_id, OEM_ID, ' ');
60061b
-    strpadcpy(oem_table_id, sizeof oem_table_id, OEM_TABLE_ID, ' ');
60061b
     for (i = 0; i < data->tables->len; ++i) {
60061b
         AcpiSdtTable *sdt;
60061b
 
60061b
@@ -1498,8 +1495,8 @@ static void test_oem_fields(test_data *data)
60061b
             continue;
60061b
         }
60061b
 
60061b
-        g_assert(memcmp(sdt->aml + 10, oem_id, 6) == 0);
60061b
-        g_assert(memcmp(sdt->aml + 16, oem_table_id, 8) == 0);
60061b
+        g_assert(memcmp(sdt->aml + 10, OEM_ID, 6) == 0);
60061b
+        g_assert(memcmp(sdt->aml + 16, OEM_TABLE_ID, 8) == 0);
60061b
     }
60061b
 }
60061b
 
60061b
-- 
60061b
2.27.0
60061b