* [PATCH v3 1/5] hw/smbios: add core_count2 to smbios table type 4
2022-10-11 11:17 [PATCH v3 0/5] hw/smbios: add core_count2 to smbios table type 4 Julia Suvorova
@ 2022-10-11 11:17 ` Julia Suvorova
2022-10-11 11:17 ` [PATCH v3 2/5] bios-tables-test: teach test to use smbios 3.0 tables Julia Suvorova
` (3 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Julia Suvorova @ 2022-10-11 11:17 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Igor Mammedov, Ani Sinha, Julia Suvorova
In order to use the increased number of cpus, we need to bring smbios
tables in line with the SMBIOS 3.0 specification. This allows us to
introduce core_count2 which acts as a duplicate of core_count if we have
fewer cores than 256, and contains the actual core number per socket if
we have more.
core_enabled2 and thread_count2 fields work the same way.
Signed-off-by: Julia Suvorova <jusual@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Message-Id: <20220731162141.178443-2-jusual@redhat.com>
---
hw/smbios/smbios.c | 19 ++++++++++++++++---
hw/smbios/smbios_build.h | 9 +++++++--
include/hw/firmware/smbios.h | 12 ++++++++++++
3 files changed, 35 insertions(+), 5 deletions(-)
diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
index 4c9f664830..591481d449 100644
--- a/hw/smbios/smbios.c
+++ b/hw/smbios/smbios.c
@@ -681,8 +681,14 @@ static void smbios_build_type_3_table(void)
static void smbios_build_type_4_table(MachineState *ms, unsigned instance)
{
char sock_str[128];
+ size_t tbl_len = SMBIOS_TYPE_4_LEN_V28;
- SMBIOS_BUILD_TABLE_PRE(4, T4_BASE + instance, true); /* required */
+ if (smbios_ep_type == SMBIOS_ENTRY_POINT_TYPE_64) {
+ tbl_len = SMBIOS_TYPE_4_LEN_V30;
+ }
+
+ SMBIOS_BUILD_TABLE_PRE_SIZE(4, T4_BASE + instance,
+ true, tbl_len); /* required */
snprintf(sock_str, sizeof(sock_str), "%s%2x", type4.sock_pfx, instance);
SMBIOS_TABLE_SET_STR(4, socket_designation_str, sock_str);
@@ -709,8 +715,15 @@ static void smbios_build_type_4_table(MachineState *ms, unsigned instance)
SMBIOS_TABLE_SET_STR(4, serial_number_str, type4.serial);
SMBIOS_TABLE_SET_STR(4, asset_tag_number_str, type4.asset);
SMBIOS_TABLE_SET_STR(4, part_number_str, type4.part);
- t->core_count = t->core_enabled = ms->smp.cores;
- t->thread_count = ms->smp.threads;
+
+ t->core_count = (ms->smp.cores > 255) ? 0xFF : ms->smp.cores;
+ t->core_enabled = t->core_count;
+
+ t->core_count2 = t->core_enabled2 = cpu_to_le16(ms->smp.cores);
+
+ t->thread_count = (ms->smp.threads > 255) ? 0xFF : ms->smp.threads;
+ t->thread_count2 = cpu_to_le16(ms->smp.threads);
+
t->processor_characteristics = cpu_to_le16(0x02); /* Unknown */
t->processor_family2 = cpu_to_le16(0x01); /* Other */
diff --git a/hw/smbios/smbios_build.h b/hw/smbios/smbios_build.h
index 56b5a1e3f3..351660024e 100644
--- a/hw/smbios/smbios_build.h
+++ b/hw/smbios/smbios_build.h
@@ -27,6 +27,11 @@ extern unsigned smbios_table_max;
extern unsigned smbios_table_cnt;
#define SMBIOS_BUILD_TABLE_PRE(tbl_type, tbl_handle, tbl_required) \
+ SMBIOS_BUILD_TABLE_PRE_SIZE(tbl_type, tbl_handle, tbl_required, \
+ sizeof(struct smbios_type_##tbl_type))\
+
+#define SMBIOS_BUILD_TABLE_PRE_SIZE(tbl_type, tbl_handle, \
+ tbl_required, tbl_len) \
struct smbios_type_##tbl_type *t; \
size_t t_off; /* table offset into smbios_tables */ \
int str_index = 0; \
@@ -39,12 +44,12 @@ extern unsigned smbios_table_cnt;
/* use offset of table t within smbios_tables */ \
/* (pointer must be updated after each realloc) */ \
t_off = smbios_tables_len; \
- smbios_tables_len += sizeof(*t); \
+ smbios_tables_len += tbl_len; \
smbios_tables = g_realloc(smbios_tables, smbios_tables_len); \
t = (struct smbios_type_##tbl_type *)(smbios_tables + t_off); \
\
t->header.type = tbl_type; \
- t->header.length = sizeof(*t); \
+ t->header.length = tbl_len; \
t->header.handle = cpu_to_le16(tbl_handle); \
} while (0)
diff --git a/include/hw/firmware/smbios.h b/include/hw/firmware/smbios.h
index 4b7ad77a44..9615446f5d 100644
--- a/include/hw/firmware/smbios.h
+++ b/include/hw/firmware/smbios.h
@@ -18,6 +18,8 @@
#define SMBIOS_MAX_TYPE 127
+#define offsetofend(TYPE, MEMBER) \
+ (offsetof(TYPE, MEMBER) + sizeof_field(TYPE, MEMBER))
/* memory area description, used by type 19 table */
struct smbios_phys_mem_area {
@@ -187,8 +189,18 @@ struct smbios_type_4 {
uint8_t thread_count;
uint16_t processor_characteristics;
uint16_t processor_family2;
+ /* SMBIOS spec 3.0.0, Table 21 */
+ uint16_t core_count2;
+ uint16_t core_enabled2;
+ uint16_t thread_count2;
} QEMU_PACKED;
+typedef enum smbios_type_4_len_ver {
+ SMBIOS_TYPE_4_LEN_V28 = offsetofend(struct smbios_type_4,
+ processor_family2),
+ SMBIOS_TYPE_4_LEN_V30 = offsetofend(struct smbios_type_4, thread_count2),
+} smbios_type_4_len_ver;
+
/* SMBIOS type 11 - OEM strings */
struct smbios_type_11 {
struct smbios_structure_header header;
--
2.37.3
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v3 2/5] bios-tables-test: teach test to use smbios 3.0 tables
2022-10-11 11:17 [PATCH v3 0/5] hw/smbios: add core_count2 to smbios table type 4 Julia Suvorova
2022-10-11 11:17 ` [PATCH v3 1/5] " Julia Suvorova
@ 2022-10-11 11:17 ` Julia Suvorova
2022-10-11 11:17 ` [PATCH v3 3/5] tests/acpi: allow changes for core_count2 test Julia Suvorova
` (2 subsequent siblings)
4 siblings, 0 replies; 10+ messages in thread
From: Julia Suvorova @ 2022-10-11 11:17 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Igor Mammedov, Ani Sinha, Julia Suvorova
Introduce the 64-bit entry point. Since we no longer have a total
number of structures, stop checking for the new ones at the EOF
structure (type 127).
Signed-off-by: Julia Suvorova <jusual@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Message-Id: <20220731162141.178443-3-jusual@redhat.com>
---
tests/qtest/bios-tables-test.c | 100 +++++++++++++++++++++++++--------
1 file changed, 76 insertions(+), 24 deletions(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index 2ebeb530b2..f5fffdc348 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest/bios-tables-test.c
@@ -88,8 +88,8 @@ typedef struct {
uint64_t rsdp_addr;
uint8_t rsdp_table[36 /* ACPI 2.0+ RSDP size */];
GArray *tables;
- uint32_t smbios_ep_addr;
- struct smbios_21_entry_point smbios_ep_table;
+ uint64_t smbios_ep_addr[SMBIOS_ENTRY_POINT_TYPE__MAX];
+ SmbiosEntryPoint smbios_ep_table;
uint16_t smbios_cpu_max_speed;
uint16_t smbios_cpu_curr_speed;
uint8_t *required_struct_types;
@@ -533,10 +533,9 @@ static void test_acpi_asl(test_data *data)
free_test_data(&exp_data);
}
-static bool smbios_ep_table_ok(test_data *data)
+static bool smbios_ep2_table_ok(test_data *data, uint32_t addr)
{
- struct smbios_21_entry_point *ep_table = &data->smbios_ep_table;
- uint32_t addr = data->smbios_ep_addr;
+ struct smbios_21_entry_point *ep_table = &data->smbios_ep_table.ep21;
qtest_memread(data->qts, addr, ep_table, sizeof(*ep_table));
if (memcmp(ep_table->anchor_string, "_SM_", 4)) {
@@ -559,13 +558,29 @@ static bool smbios_ep_table_ok(test_data *data)
return true;
}
-static void test_smbios_entry_point(test_data *data)
+static bool smbios_ep3_table_ok(test_data *data, uint64_t addr)
+{
+ struct smbios_30_entry_point *ep_table = &data->smbios_ep_table.ep30;
+
+ qtest_memread(data->qts, addr, ep_table, sizeof(*ep_table));
+ if (memcmp(ep_table->anchor_string, "_SM3_", 5)) {
+ return false;
+ }
+
+ if (acpi_calc_checksum((uint8_t *)ep_table, sizeof *ep_table)) {
+ return false;
+ }
+
+ return true;
+}
+
+static SmbiosEntryPointType test_smbios_entry_point(test_data *data)
{
uint32_t off;
/* find smbios entry point structure */
for (off = 0xf0000; off < 0x100000; off += 0x10) {
- uint8_t sig[] = "_SM_";
+ uint8_t sig[] = "_SM_", sig3[] = "_SM3_";
int i;
for (i = 0; i < sizeof sig - 1; ++i) {
@@ -574,14 +589,30 @@ static void test_smbios_entry_point(test_data *data)
if (!memcmp(sig, "_SM_", sizeof sig)) {
/* signature match, but is this a valid entry point? */
- data->smbios_ep_addr = off;
- if (smbios_ep_table_ok(data)) {
+ if (smbios_ep2_table_ok(data, off)) {
+ data->smbios_ep_addr[SMBIOS_ENTRY_POINT_TYPE_32] = off;
+ }
+ }
+
+ for (i = 0; i < sizeof sig3 - 1; ++i) {
+ sig3[i] = qtest_readb(data->qts, off + i);
+ }
+
+ if (!memcmp(sig3, "_SM3_", sizeof sig3)) {
+ if (smbios_ep3_table_ok(data, off)) {
+ data->smbios_ep_addr[SMBIOS_ENTRY_POINT_TYPE_64] = off;
+ /* found 64-bit entry point, no need to look for 32-bit one */
break;
}
}
}
- g_assert_cmphex(off, <, 0x100000);
+ /* found at least one entry point */
+ g_assert_true(data->smbios_ep_addr[SMBIOS_ENTRY_POINT_TYPE_32] ||
+ data->smbios_ep_addr[SMBIOS_ENTRY_POINT_TYPE_64]);
+
+ return data->smbios_ep_addr[SMBIOS_ENTRY_POINT_TYPE_64] ?
+ SMBIOS_ENTRY_POINT_TYPE_64 : SMBIOS_ENTRY_POINT_TYPE_32;
}
static inline bool smbios_single_instance(uint8_t type)
@@ -625,16 +656,23 @@ static bool smbios_cpu_test(test_data *data, uint32_t addr)
return true;
}
-static void test_smbios_structs(test_data *data)
+static void test_smbios_structs(test_data *data, SmbiosEntryPointType ep_type)
{
DECLARE_BITMAP(struct_bitmap, SMBIOS_MAX_TYPE+1) = { 0 };
- struct smbios_21_entry_point *ep_table = &data->smbios_ep_table;
- uint32_t addr = le32_to_cpu(ep_table->structure_table_address);
- int i, len, max_len = 0;
+
+ SmbiosEntryPoint *ep_table = &data->smbios_ep_table;
+ int i = 0, len, max_len = 0;
uint8_t type, prv, crt;
+ uint64_t addr;
+
+ if (ep_type == SMBIOS_ENTRY_POINT_TYPE_32) {
+ addr = le32_to_cpu(ep_table->ep21.structure_table_address);
+ } else {
+ addr = le64_to_cpu(ep_table->ep30.structure_table_address);
+ }
/* walk the smbios tables */
- for (i = 0; i < le16_to_cpu(ep_table->number_of_structures); i++) {
+ do {
/* grab type and formatted area length from struct header */
type = qtest_readb(data->qts, addr);
@@ -660,19 +698,33 @@ static void test_smbios_structs(test_data *data)
}
/* keep track of max. struct size */
- if (max_len < len) {
+ if (ep_type == SMBIOS_ENTRY_POINT_TYPE_32 && max_len < len) {
max_len = len;
- g_assert_cmpuint(max_len, <=, ep_table->max_structure_size);
+ g_assert_cmpuint(max_len, <=, ep_table->ep21.max_structure_size);
}
/* start of next structure */
addr += len;
- }
- /* total table length and max struct size must match entry point values */
- g_assert_cmpuint(le16_to_cpu(ep_table->structure_table_length), ==,
- addr - le32_to_cpu(ep_table->structure_table_address));
- g_assert_cmpuint(le16_to_cpu(ep_table->max_structure_size), ==, max_len);
+ /*
+ * Until all structures have been scanned (ep21)
+ * or an EOF structure is found (ep30)
+ */
+ } while (ep_type == SMBIOS_ENTRY_POINT_TYPE_32 ?
+ ++i < le16_to_cpu(ep_table->ep21.number_of_structures) :
+ type != 127);
+
+ if (ep_type == SMBIOS_ENTRY_POINT_TYPE_32) {
+ /*
+ * Total table length and max struct size
+ * must match entry point values
+ */
+ g_assert_cmpuint(le16_to_cpu(ep_table->ep21.structure_table_length), ==,
+ addr - le32_to_cpu(ep_table->ep21.structure_table_address));
+
+ g_assert_cmpuint(le16_to_cpu(ep_table->ep21.max_structure_size), ==,
+ max_len);
+ }
/* required struct types must all be present */
for (i = 0; i < data->required_struct_types_len; i++) {
@@ -756,8 +808,8 @@ static void test_acpi_one(const char *params, test_data *data)
* https://bugs.launchpad.net/qemu/+bug/1821884
*/
if (!use_uefi) {
- test_smbios_entry_point(data);
- test_smbios_structs(data);
+ SmbiosEntryPointType ep_type = test_smbios_entry_point(data);
+ test_smbios_structs(data, ep_type);
}
qtest_quit(data->qts);
--
2.37.3
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v3 3/5] tests/acpi: allow changes for core_count2 test
2022-10-11 11:17 [PATCH v3 0/5] hw/smbios: add core_count2 to smbios table type 4 Julia Suvorova
2022-10-11 11:17 ` [PATCH v3 1/5] " Julia Suvorova
2022-10-11 11:17 ` [PATCH v3 2/5] bios-tables-test: teach test to use smbios 3.0 tables Julia Suvorova
@ 2022-10-11 11:17 ` Julia Suvorova
2022-10-14 14:22 ` Igor Mammedov
2022-10-11 11:17 ` [PATCH v3 4/5] bios-tables-test: add test for number of cores > 255 Julia Suvorova
2022-10-11 11:17 ` [PATCH v3 5/5] tests/acpi: update tables for new core count test Julia Suvorova
4 siblings, 1 reply; 10+ messages in thread
From: Julia Suvorova @ 2022-10-11 11:17 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Igor Mammedov, Ani Sinha, Julia Suvorova
Signed-off-by: Julia Suvorova <jusual@redhat.com>
Message-Id: <20220731162141.178443-4-jusual@redhat.com>
---
tests/data/acpi/q35/APIC.core-count2 | 0
tests/data/acpi/q35/DSDT.core-count2 | 0
tests/data/acpi/q35/FACP.core-count2 | 0
tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
4 files changed, 3 insertions(+)
create mode 100644 tests/data/acpi/q35/APIC.core-count2
create mode 100644 tests/data/acpi/q35/DSDT.core-count2
create mode 100644 tests/data/acpi/q35/FACP.core-count2
diff --git a/tests/data/acpi/q35/APIC.core-count2 b/tests/data/acpi/q35/APIC.core-count2
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/q35/DSDT.core-count2 b/tests/data/acpi/q35/DSDT.core-count2
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/data/acpi/q35/FACP.core-count2 b/tests/data/acpi/q35/FACP.core-count2
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index dfb8523c8b..e81dc67a2e 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1 +1,4 @@
/* List of comma-separated changed AML files to ignore */
+"tests/data/acpi/q35/APIC.core-count2",
+"tests/data/acpi/q35/DSDT.core-count2",
+"tests/data/acpi/q35/FACP.core-count2",
--
2.37.3
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v3 3/5] tests/acpi: allow changes for core_count2 test
2022-10-11 11:17 ` [PATCH v3 3/5] tests/acpi: allow changes for core_count2 test Julia Suvorova
@ 2022-10-14 14:22 ` Igor Mammedov
0 siblings, 0 replies; 10+ messages in thread
From: Igor Mammedov @ 2022-10-14 14:22 UTC (permalink / raw)
To: Julia Suvorova; +Cc: qemu-devel, Michael S. Tsirkin, Ani Sinha
On Tue, 11 Oct 2022 13:17:29 +0200
Julia Suvorova <jusual@redhat.com> wrote:
> Signed-off-by: Julia Suvorova <jusual@redhat.com>
> Message-Id: <20220731162141.178443-4-jusual@redhat.com>
Acked-by: Igor Mammedov <imammedo@redhat.com>
> ---
> tests/data/acpi/q35/APIC.core-count2 | 0
> tests/data/acpi/q35/DSDT.core-count2 | 0
> tests/data/acpi/q35/FACP.core-count2 | 0
> tests/qtest/bios-tables-test-allowed-diff.h | 3 +++
> 4 files changed, 3 insertions(+)
> create mode 100644 tests/data/acpi/q35/APIC.core-count2
> create mode 100644 tests/data/acpi/q35/DSDT.core-count2
> create mode 100644 tests/data/acpi/q35/FACP.core-count2
>
> diff --git a/tests/data/acpi/q35/APIC.core-count2 b/tests/data/acpi/q35/APIC.core-count2
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/q35/DSDT.core-count2 b/tests/data/acpi/q35/DSDT.core-count2
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/data/acpi/q35/FACP.core-count2 b/tests/data/acpi/q35/FACP.core-count2
> new file mode 100644
> index 0000000000..e69de29bb2
> diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
> index dfb8523c8b..e81dc67a2e 100644
> --- a/tests/qtest/bios-tables-test-allowed-diff.h
> +++ b/tests/qtest/bios-tables-test-allowed-diff.h
> @@ -1 +1,4 @@
> /* List of comma-separated changed AML files to ignore */
> +"tests/data/acpi/q35/APIC.core-count2",
> +"tests/data/acpi/q35/DSDT.core-count2",
> +"tests/data/acpi/q35/FACP.core-count2",
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 4/5] bios-tables-test: add test for number of cores > 255
2022-10-11 11:17 [PATCH v3 0/5] hw/smbios: add core_count2 to smbios table type 4 Julia Suvorova
` (2 preceding siblings ...)
2022-10-11 11:17 ` [PATCH v3 3/5] tests/acpi: allow changes for core_count2 test Julia Suvorova
@ 2022-10-11 11:17 ` Julia Suvorova
2022-10-14 14:22 ` Igor Mammedov
2022-10-11 11:17 ` [PATCH v3 5/5] tests/acpi: update tables for new core count test Julia Suvorova
4 siblings, 1 reply; 10+ messages in thread
From: Julia Suvorova @ 2022-10-11 11:17 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Igor Mammedov, Ani Sinha, Julia Suvorova
The new test is run with a large number of cpus and checks if the
core_count field in smbios_cpu_test (structure type 4) is correct.
Choose q35 as it allows to run with -smp > 255.
Signed-off-by: Julia Suvorova <jusual@redhat.com>
Message-Id: <20220731162141.178443-5-jusual@redhat.com>
---
tests/qtest/bios-tables-test.c | 58 ++++++++++++++++++++++++++--------
1 file changed, 45 insertions(+), 13 deletions(-)
diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
index f5fffdc348..4a76befc93 100644
--- a/tests/qtest/bios-tables-test.c
+++ b/tests/qtest/bios-tables-test.c
@@ -92,6 +92,8 @@ typedef struct {
SmbiosEntryPoint smbios_ep_table;
uint16_t smbios_cpu_max_speed;
uint16_t smbios_cpu_curr_speed;
+ uint8_t smbios_core_count;
+ uint16_t smbios_core_count2;
uint8_t *required_struct_types;
int required_struct_types_len;
QTestState *qts;
@@ -631,29 +633,42 @@ static inline bool smbios_single_instance(uint8_t type)
}
}
-static bool smbios_cpu_test(test_data *data, uint32_t addr)
+static void smbios_cpu_test(test_data *data, uint32_t addr,
+ SmbiosEntryPointType ep_type)
{
- uint16_t expect_speed[2];
- uint16_t real;
+ uint8_t core_count, expected_core_count = data->smbios_core_count;
+ uint16_t speed, expected_speed[2];
+ uint16_t core_count2, expected_core_count2 = data->smbios_core_count2;
int offset[2];
int i;
/* Check CPU speed for backward compatibility */
offset[0] = offsetof(struct smbios_type_4, max_speed);
offset[1] = offsetof(struct smbios_type_4, current_speed);
- expect_speed[0] = data->smbios_cpu_max_speed ? : 2000;
- expect_speed[1] = data->smbios_cpu_curr_speed ? : 2000;
+ expected_speed[0] = data->smbios_cpu_max_speed ? : 2000;
+ expected_speed[1] = data->smbios_cpu_curr_speed ? : 2000;
for (i = 0; i < 2; i++) {
- real = qtest_readw(data->qts, addr + offset[i]);
- if (real != expect_speed[i]) {
- fprintf(stderr, "Unexpected SMBIOS CPU speed: real %u expect %u\n",
- real, expect_speed[i]);
- return false;
- }
+ speed = qtest_readw(data->qts, addr + offset[i]);
+ g_assert_cmpuint(speed, ==, expected_speed[i]);
}
- return true;
+ core_count = qtest_readb(data->qts,
+ addr + offsetof(struct smbios_type_4, core_count));
+
+ if (expected_core_count) {
+ g_assert_cmpuint(core_count, ==, expected_core_count);
+ }
+
+ if (ep_type == SMBIOS_ENTRY_POINT_TYPE_64) {
+ core_count2 = qtest_readw(data->qts,
+ addr + offsetof(struct smbios_type_4, core_count2));
+
+ /* Core Count has reached its limit, checking Core Count 2 */
+ if (expected_core_count == 0xFF && expected_core_count2) {
+ g_assert_cmpuint(core_count2, ==, expected_core_count2);
+ }
+ }
}
static void test_smbios_structs(test_data *data, SmbiosEntryPointType ep_type)
@@ -686,7 +701,7 @@ static void test_smbios_structs(test_data *data, SmbiosEntryPointType ep_type)
set_bit(type, struct_bitmap);
if (type == 4) {
- g_assert(smbios_cpu_test(data, addr));
+ smbios_cpu_test(data, addr, ep_type);
}
/* seek to end of unformatted string area of this struct ("\0\0") */
@@ -908,6 +923,21 @@ static void test_acpi_q35_tcg(void)
free_test_data(&data);
}
+static void test_acpi_q35_tcg_core_count2(void)
+{
+ test_data data = {
+ .machine = MACHINE_Q35,
+ .variant = ".core-count2",
+ .required_struct_types = base_required_struct_types,
+ .required_struct_types_len = ARRAY_SIZE(base_required_struct_types),
+ .smbios_core_count = 0xFF,
+ .smbios_core_count2 = 275,
+ };
+
+ test_acpi_one("-machine smbios-entry-point-type=64 -smp 275", &data);
+ free_test_data(&data);
+}
+
static void test_acpi_q35_tcg_bridge(void)
{
test_data data;
@@ -1859,6 +1889,8 @@ int main(int argc, char *argv[])
qtest_add_func("acpi/q35/tpm12-tis",
test_acpi_q35_tcg_tpm12_tis);
}
+ qtest_add_func("acpi/q35/core-count2",
+ test_acpi_q35_tcg_core_count2);
qtest_add_func("acpi/q35/bridge", test_acpi_q35_tcg_bridge);
qtest_add_func("acpi/q35/multif-bridge",
test_acpi_q35_multif_bridge);
--
2.37.3
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH v3 4/5] bios-tables-test: add test for number of cores > 255
2022-10-11 11:17 ` [PATCH v3 4/5] bios-tables-test: add test for number of cores > 255 Julia Suvorova
@ 2022-10-14 14:22 ` Igor Mammedov
0 siblings, 0 replies; 10+ messages in thread
From: Igor Mammedov @ 2022-10-14 14:22 UTC (permalink / raw)
To: Julia Suvorova; +Cc: qemu-devel, Michael S. Tsirkin, Ani Sinha
On Tue, 11 Oct 2022 13:17:30 +0200
Julia Suvorova <jusual@redhat.com> wrote:
> The new test is run with a large number of cpus and checks if the
> core_count field in smbios_cpu_test (structure type 4) is correct.
>
> Choose q35 as it allows to run with -smp > 255.
>
> Signed-off-by: Julia Suvorova <jusual@redhat.com>
> Message-Id: <20220731162141.178443-5-jusual@redhat.com>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
> ---
> tests/qtest/bios-tables-test.c | 58 ++++++++++++++++++++++++++--------
> 1 file changed, 45 insertions(+), 13 deletions(-)
>
> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
> index f5fffdc348..4a76befc93 100644
> --- a/tests/qtest/bios-tables-test.c
> +++ b/tests/qtest/bios-tables-test.c
> @@ -92,6 +92,8 @@ typedef struct {
> SmbiosEntryPoint smbios_ep_table;
> uint16_t smbios_cpu_max_speed;
> uint16_t smbios_cpu_curr_speed;
> + uint8_t smbios_core_count;
> + uint16_t smbios_core_count2;
> uint8_t *required_struct_types;
> int required_struct_types_len;
> QTestState *qts;
> @@ -631,29 +633,42 @@ static inline bool smbios_single_instance(uint8_t type)
> }
> }
>
> -static bool smbios_cpu_test(test_data *data, uint32_t addr)
> +static void smbios_cpu_test(test_data *data, uint32_t addr,
> + SmbiosEntryPointType ep_type)
> {
> - uint16_t expect_speed[2];
> - uint16_t real;
> + uint8_t core_count, expected_core_count = data->smbios_core_count;
> + uint16_t speed, expected_speed[2];
> + uint16_t core_count2, expected_core_count2 = data->smbios_core_count2;
> int offset[2];
> int i;
>
> /* Check CPU speed for backward compatibility */
> offset[0] = offsetof(struct smbios_type_4, max_speed);
> offset[1] = offsetof(struct smbios_type_4, current_speed);
> - expect_speed[0] = data->smbios_cpu_max_speed ? : 2000;
> - expect_speed[1] = data->smbios_cpu_curr_speed ? : 2000;
> + expected_speed[0] = data->smbios_cpu_max_speed ? : 2000;
> + expected_speed[1] = data->smbios_cpu_curr_speed ? : 2000;
>
> for (i = 0; i < 2; i++) {
> - real = qtest_readw(data->qts, addr + offset[i]);
> - if (real != expect_speed[i]) {
> - fprintf(stderr, "Unexpected SMBIOS CPU speed: real %u expect %u\n",
> - real, expect_speed[i]);
> - return false;
> - }
> + speed = qtest_readw(data->qts, addr + offset[i]);
> + g_assert_cmpuint(speed, ==, expected_speed[i]);
> }
>
> - return true;
> + core_count = qtest_readb(data->qts,
> + addr + offsetof(struct smbios_type_4, core_count));
> +
> + if (expected_core_count) {
> + g_assert_cmpuint(core_count, ==, expected_core_count);
> + }
> +
> + if (ep_type == SMBIOS_ENTRY_POINT_TYPE_64) {
> + core_count2 = qtest_readw(data->qts,
> + addr + offsetof(struct smbios_type_4, core_count2));
> +
> + /* Core Count has reached its limit, checking Core Count 2 */
> + if (expected_core_count == 0xFF && expected_core_count2) {
> + g_assert_cmpuint(core_count2, ==, expected_core_count2);
> + }
> + }
> }
>
> static void test_smbios_structs(test_data *data, SmbiosEntryPointType ep_type)
> @@ -686,7 +701,7 @@ static void test_smbios_structs(test_data *data, SmbiosEntryPointType ep_type)
> set_bit(type, struct_bitmap);
>
> if (type == 4) {
> - g_assert(smbios_cpu_test(data, addr));
> + smbios_cpu_test(data, addr, ep_type);
> }
>
> /* seek to end of unformatted string area of this struct ("\0\0") */
> @@ -908,6 +923,21 @@ static void test_acpi_q35_tcg(void)
> free_test_data(&data);
> }
>
> +static void test_acpi_q35_tcg_core_count2(void)
> +{
> + test_data data = {
> + .machine = MACHINE_Q35,
> + .variant = ".core-count2",
> + .required_struct_types = base_required_struct_types,
> + .required_struct_types_len = ARRAY_SIZE(base_required_struct_types),
> + .smbios_core_count = 0xFF,
> + .smbios_core_count2 = 275,
> + };
> +
> + test_acpi_one("-machine smbios-entry-point-type=64 -smp 275", &data);
> + free_test_data(&data);
> +}
> +
> static void test_acpi_q35_tcg_bridge(void)
> {
> test_data data;
> @@ -1859,6 +1889,8 @@ int main(int argc, char *argv[])
> qtest_add_func("acpi/q35/tpm12-tis",
> test_acpi_q35_tcg_tpm12_tis);
> }
> + qtest_add_func("acpi/q35/core-count2",
> + test_acpi_q35_tcg_core_count2);
> qtest_add_func("acpi/q35/bridge", test_acpi_q35_tcg_bridge);
> qtest_add_func("acpi/q35/multif-bridge",
> test_acpi_q35_multif_bridge);
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 5/5] tests/acpi: update tables for new core count test
2022-10-11 11:17 [PATCH v3 0/5] hw/smbios: add core_count2 to smbios table type 4 Julia Suvorova
` (3 preceding siblings ...)
2022-10-11 11:17 ` [PATCH v3 4/5] bios-tables-test: add test for number of cores > 255 Julia Suvorova
@ 2022-10-11 11:17 ` Julia Suvorova
2022-10-27 6:11 ` Michael S. Tsirkin
4 siblings, 1 reply; 10+ messages in thread
From: Julia Suvorova @ 2022-10-11 11:17 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Igor Mammedov, Ani Sinha, Julia Suvorova
Changes in the tables (for 275 cores):
FACP:
+ Use APIC Cluster Model (V4) : 1
APIC:
+[02Ch 0044 1] Subtable Type : 00 [Processor Local APIC]
+[02Dh 0045 1] Length : 08
+[02Eh 0046 1] Processor ID : 00
+[02Fh 0047 1] Local Apic ID : 00
+[030h 0048 4] Flags (decoded below) : 00000001
+ Processor Enabled : 1
...
+
+[81Ch 2076 1] Subtable Type : 00 [Processor Local APIC]
+[81Dh 2077 1] Length : 08
+[81Eh 2078 1] Processor ID : FE
+[81Fh 2079 1] Local Apic ID : FE
+[820h 2080 4] Flags (decoded below) : 00000001
+ Processor Enabled : 1
+ Runtime Online Capable : 0
+
+[824h 2084 1] Subtable Type : 09 [Processor Local x2APIC]
+[825h 2085 1] Length : 10
+[826h 2086 2] Reserved : 0000
+[828h 2088 4] Processor x2Apic ID : 000000FF
+[82Ch 2092 4] Flags (decoded below) : 00000001
+ Processor Enabled : 1
+[830h 2096 4] Processor UID : 000000FF
...
DSDT:
+ Processor (C001, 0x01, 0x00000000, 0x00)
+ {
+ Method (_STA, 0, Serialized) // _STA: Status
+ {
+ Return (CSTA (One))
+ }
+
+ Name (_MAT, Buffer (0x08) // _MAT: Multiple APIC Table Entry
+ {
+ 0x00, 0x08, 0x01, 0x01, 0x01, 0x00, 0x00, 0x00 // ........
+ })
+ Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
+ {
+ CEJ0 (One)
+ }
+
+ Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
+ {
+ COST (One, Arg0, Arg1, Arg2)
+ }
+ }
...
+ Processor (C0FE, 0xFE, 0x00000000, 0x00)
+ {
+ Method (_STA, 0, Serialized) // _STA: Status
+ {
+ Return (CSTA (0xFE))
+ }
+
+ Name (_MAT, Buffer (0x08) // _MAT: Multiple APIC Table Entry
+ {
+ 0x00, 0x08, 0xFE, 0xFE, 0x01, 0x00, 0x00, 0x00 // ........
+ })
+ Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
+ {
+ CEJ0 (0xFE)
+ }
+
+ Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
+ {
+ COST (0xFE, Arg0, Arg1, Arg2)
+ }
+ }
+
+ Device (C0FF)
+ {
+ Name (_HID, "ACPI0007" /* Processor Device */) // _HID: Hardware ID
+ Name (_UID, 0xFF) // _UID: Unique ID
+ Method (_STA, 0, Serialized) // _STA: Status
+ {
+ Return (CSTA (0xFF))
+ }
+
+ Name (_MAT, Buffer (0x10) // _MAT: Multiple APIC Table Entry
+ {
+ /* 0000 */ 0x09, 0x10, 0x00, 0x00, 0xFF, 0x00, 0x00, 0x00, // ........
+ /* 0008 */ 0x01, 0x00, 0x00, 0x00, 0xFF, 0x00, 0x00, 0x00 // ........
+ })
+ Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
+ {
+ CEJ0 (0xFF)
+ }
+
+ Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
+ {
+ COST (0xFF, Arg0, Arg1, Arg2)
+ }
+ }
+
...
Signed-off-by: Julia Suvorova <jusual@redhat.com>
Message-Id: <20220731162141.178443-6-jusual@redhat.com>
---
tests/data/acpi/q35/APIC.core-count2 | Bin 0 -> 2478 bytes
tests/data/acpi/q35/DSDT.core-count2 | Bin 0 -> 32414 bytes
tests/data/acpi/q35/FACP.core-count2 | Bin 0 -> 244 bytes
tests/qtest/bios-tables-test-allowed-diff.h | 3 ---
4 files changed, 3 deletions(-)
diff --git a/tests/data/acpi/q35/APIC.core-count2 b/tests/data/acpi/q35/APIC.core-count2
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..a255082ef5bc39f0d92d3e372b91f09dd6d0d9a1 100644
GIT binary patch
literal 2478
zcmXZeWl$AS7=Youz=a#M-K}6ZwgLuNAQ;$~*xjvQcY@uCVs{~Sf`WpLVt2Rbe!ORA
z_B`J^b9R56*&pj2=<ge2)-*$cPk^sqaDJbVK;QiOWzaNDW>M2p(=#;b`y@>U1KQZ2
ztu5Nwq0xx;_UPb%CKH;?XtAKxijI!x<b=-7=;DH|uIT25?(Uc=6K2kgS+Zc(te7nu
zX3vf}a$wG!m@60N&W(BUVBWl#FCTI)nyEkmx?n*pR0s<f#v(<qXi+Ry3_U#1(-Vsq
z#}Xy5WJxSl3QL#9GG(xASu9r$%a_Lr6|iDOtW*grS4J-{tWpK5R>f-7uzGc@Q3Gq%
z#9Fnmc5SRv2fe+~#|M4+PE2*{()H?L{rcFT0s8r&zdtr?h>aRy<Hp#e2{vtt0Rb2o
zh|QW|P!I+OWAo<Nq6M~WiLF{;NC>uWjcwXs+qT%Q9ky?e9Xepgju;w>ojPIX&e)|3
zcI}GYx?%V37#4;-dSK6<*sB-z?u~u=VBfyjuOIgBj{^qaz=1eu5Dp%ULx$kcp*U<9
z4j+yqM&QViIBFD*9*twh;MlP^ZXAvuj}s=~#ECd*5{8FkL<CNrj8mrI)Tuaa8cv^%
zGiKn-nK)|}&Yq2P=HT49IBy=#pN|U`;KGHtXb~=6j7yeaWF$sK;nJnJY#A<Jjw@E+
z%9Xfk6|P>5Yu4b}wYY8_u3wKEHsHpMxM>q^-i%we;MT3UZ5u{M<M!>iV+Y2>;Le@6
zYZva`jeGXs-o3bQAMW3e2M*xDgLvo=9zKjmj^NRwc<dM+KaM9(;K`F18;hq-VO$)Z
zK8<J2;Mucy?i`*!j~6cB#fy095?;QHSFYgIt9b1i#>Znq0$#t4H*R2JA|@r_&6{}Z
z7A7ZSN($b-jd$+g-Me`29^Su?4<6vdhnSj*j~?OU$C#FePoCh@r}*p{K7WocUf|1@
z`05qDevNP5;M=$O?j62=j~_nZ$B+2w6Mp`TU%ueiulVg7e*ca?e&Ela`0E$`{*8bB
z;NQQPo-UeQHSM3S%%ZeJ#vXl<HmDY*ZB&cWwyH&GJJq7JQMD*-uUeFLP%TQEREyGP
z)uOaTwJ2>>ElNA87Nwn3i_*@jMQIn+qO_}OQQA$lDDAE~Lr49*wAgf6Z7ljNgG@%F
cu9Hk={TGbMqHkcbS~Dh#{`5cn(qE|k2lzA_5C8xG
literal 0
HcmV?d00001
diff --git a/tests/data/acpi/q35/DSDT.core-count2 b/tests/data/acpi/q35/DSDT.core-count2
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..aa1be16f4efe23711b776b16fe39b84086028bef 100644
GIT binary patch
literal 32414
zcmb8&cYKpo7YFbsU8HH+rlo9z3b+U9Zrzz}ftJ!1QH!<(RE8r8sE8;OMZ}2}+*+{i
zy|?1tiW~Rdd*keT&hy;l{+@H|=kv-R&AG|F-*ax7^fXT%nrKV3^^)T)%kx8ca-^=M
zIJ{3&q?rDBEX(4UF9Y=%)2+6p(Qs|MCnqh#)3z+fj_FI-TeM>|Jhj$y-t3OG9UTi!
z?(lRbtQqSAYi4vNmapuXVezDeGZH=@2Z2~#drP>ztt;Ydiqr?a?RE80e`ur6>urxX
zQM$K16m7P=?NjTcR?(yZ?UP#~8LQ@5k?6FNp2p1X*#!fAfwqRX%6=O?a<8@17KAqJ
z{`x_y=J_&cLMdH(>9Q=rUpf|d_(pZ~1kUcB<5|MX?b}Lv2A<t*-|So$;9C+Me9bz)
za}`r1z4O|-s-w-Jh+SiDQ)5#w5-g#*BF!xYqkSV!9%5O3zH|8BO?`cR4?3Y0SY!nn
zyv{CT-nyn(YjT$!+8ghxZE0OoYlG#ttd0NVSQ%~00z4kLcdRj5+|(RxFD|NVZ)pu#
z3BINy!B=%2+u;jz^!QhIcUavWIemS-R!`pQ?oR8RPQUfRFPXhouP@)Jt-zPJBAaX3
z_>YxgSyWA=U}r9e@9?&+c%Z%|)NbF&RKRE|rm4PpN}8|3=WpvOpBRqPZO%6(oyw&$
zlI~F7pjuCd)nR!&wVqklEV>+%Mtob>xQ5A7Q>*d^x>k98nPd1aY3nMT)YMv%v@@J<
zu*Wqw;LEM`9Fu-by4O2qsV$dACO4$kWYTQ9`}$e4Iel}w<sB2S*KF0?8c9`?qrPD_
zov;#?$2-RI(0$Sf_sP<xNK30HXM^A8rSAm~B6MF(Ev7Hg;<lwNu?h6Bf70)x?^F&C
zBH>_c0^Ja9Y-sl6Y;>0KAQEnz+ECn7AKj<8%@dv!nP?Sl)f9<MNNXF*qrSexY@U%=
z<e6i!(Y6g=-D?%?5TZ5x&0;ITUpmy>j-qH*Q?#WauV<2aE^MrN>XVj=V;fcv`Lbcy
zBawy#TZbIl(KEJtuGKTFdyX%S9`Z}<2acZ}r+rp$LAqxdf3emL=^4~L*Eb+hn3-nf
zTAhiGm7SK=+v#npkHoDae>+`z60>;}IAdQAKZ*j8FYRY;Q*&!m;l%k%3v)8);b={2
zoEWp5*h6vLD)jU9zBC%tCI{&>sIv-lGpG)F>NxY%h*#0XJm)rh9ZgLRc_sPw+%$V7
z%~c~_nVgzt*(+(Py^@B!lB%)irrRr%6{K6dGC4KfSxHmvl{Dm)$t?6la^~7jhP^U5
zHN#n%ES`qA&rH`oGwqehshQ5oWbrh_eP+4#nPsm`PR(-flZLp@0j_-xaP4z|d!IDK
zeR^H{^t$%xb?=jgxKH|#1NU^cYoFQfebNy3>2vMV=h~;wy-ynAK670A%yI2A$GuM)
z;y&p|AACRku6_F5`=lZ6GuO4xT-QEx-TR~=?laG|&pg*Y^W6KSA?`EZwa<LlKJ(rC
zq#^Dz;M!-vwa<WipESgM7P$6V;M!+_d!IDKeHOa*S?Jnlp?jY+#C;BQ?Q@`Op99_d
zq#^FJ$hFTR*FKBf`=lZ6bC7GFgIxO@<lZL@ai4=-`yA}r=V13fX^8uzUmfVtm;5<2
z#I?^M?tRh__c_$H&!Mh;4t4L7hPcmR_R5g@NjS`28CE~+hB+%EddMrIYVL5?%Hgh+
z!`&-sh$~09R*rD39N}I`LtHu1wQ{6u<w*BR8sf?=Tr0P5t=z)Bl7_f)l)W<9Wk%U6
zlRx7}IV+PtxM|2Mli$J7uAHM?IY+y5($LB2r3M;;23p%vPpR!@>Y<nts|HE~O6v^L
zX^>1x4b+)uw<M3ntCID3D5*0w)j(;O+!{4dXKu2A!c=ac&eT)`r6E<5Y@p6uHR7Du
zK%J?n21-M!$ev3L)S0VBYLE6*ZlKQ8<Pi5s4b)-nmDE6CDmPFXCc7dxP#PxpNe$GQ
zYp<jR3RAg(I#cbO+(2oV+$S|qXRf`H8YoQV2I@?;S8@ZTVRE0;K%KevN@}1ml^dut
z)n3UBl!nRgCpA!KuDy~PC`{!B>P)p)as#Dda-Y;dow@c(YM?Nc8>lnYUdauVhRJ<W
z19j%wE2)9PRBoWoRC^^iP#PxpNe$GQYp<jR3RAg(I#cbH+(2oV+$S|qXRf`H8YoQV
z2I@?;S8@ZTVRE0;K%KevN@}1ml^dut)n3UBl!nQDQUi77+AFDn!c=ac&QyCPH&7ZT
z_el-ZnQO141`1QTfjU#|mE1sSnA|5dP-m{ak{T#X<p%0ZwO4WjrD1ZP)IgoN_DX7?
zFqIpqGu2+n4U~q-eNqE;=GrT%fx=X7pw3i#B{xtSCih7V)R}9qqy`F8xq&)U?Umd>
zX_(w6HBe`+y^<OzOyvgZOtn{X1EpbdpVUB|x%NtGpfHsis58}G$qkf-$$e4-b>`YD
zse!^&ZlKOodnGqe8YcHi4b+)yucQVFQ@MdUQ|*=9KxvrVCpAzSK?9``Hc%R31EnD}
zP*_O~6jpKrg_YbuVI?(CSV;{OR&oP{mE1sKB{fi3NevWMas!2x+(2O^HBeYd4HQ;#
z1BI2`Kw%{{P#Pt_yVO8wMBg#{&g2G4L;5x(hrBX*-%<mmQS!c}21=vkeaj7$hROSu
z8z>Fw*wFBHYN1w-zuSJX+RHCU*JbfVsBPIU<J0+7Dj(|L!yBHa<|!fSxKqOXCBk2#
zUW*R4Oo?$E=P$MVrOsP4uzm_14d}^U>{~jUj;dt$`f`fO>!(DTvuQ$euprNub4r$#
zZ6EqsWUb=se3|sMKCyVVFW=&bifE=oEt3Y&?+BsfArC(tpW&lFUMjn#V6boCDS1|Q
zI(0XTetui@lbDWP6y@6oetNkCswPxa&>pRCv6gV`TWVjE&j+9^nh|NX$3xm%np-@p
zW~&-FJzuY~hedWec=CT|4@35_ojpwTVfJuQ|JlQBT@Ac<=hzt?-&1?{rTvHW5lA1g
z(?>-5$iJkIsPs{>c^@6n`oDJ{h4fK7eN?26{!99(N*|NyGyX&R7^IKc>0=^&>|fHy
zRQk9~pZOos$02>(P9GQP<NuOAuF}`a^jZHQeJ!M~wbR#%^tJzzzE-8Llj#TihxB!j
zzRpfxC(_sbOZqxWpTR%MWcq-UKKa9geyyOVyK}(SeO*zO{gkH*eqJk1K8<|^ZjOG}
zFgWMBemUcLS?K2I7Z8JUuIraGo@aCjn4f3KjnQu<2IpMYFK0Y&irgIis$y`?b^UV2
z^I(vhqu*Z)&bh8%&Ujo8c60PgjlntB^~)KL(;;q-e$z2H=Q=kR7y-xl^IICi^xg`+
zkK#|KL;QhL$I_#T-*u_;O&_10d@rD>Io6VVnEL41NzExY(loWW?(hYPLlz!feB^=i
z4_|oX%me3JZJtQeq>1X`n(On`dN%nt`8|F*tWQmekDYJw)7uBjw&g*{DvmV88a<Kv
z#uiVcxnXY)9RUyWm$=7Dk3<@xbd<f2XQoG5Vv~w#dNkBpOb46m8|%FZi?3|!S~)ap
zy1gT+$f+_n6ltmt21|lmhx1-hX3?Y<BCYX#(|UZ}v#k}8U@$m`Ll1`~b9!>RXZzAP
zE_LE`j?0`lgX3~1&g8hliL*Gabm9RVS2?kl<7y|)=D5a*eH@3JIEUk~6Z<)iIB_n=
zQ76vhIOfFp9LJqFKye9`&ovittm-f1Sk*s}V^x0<$EyB89IN^VbFAtg!m+A<D95V)
zVH~UahjXmzAHlJze<a7M{w+9G^^f9M)jyhJRev$XrK<idIac+L;aJr_mSa`_IF42Q
zTXC%F-<o4p|27<}`nTm+)xRCbs{ZXcR`u_|v8sPZj#d3Tajfd!nPXM|c#c*5L5j;%
z{Usc$`b#-h^_Ow1>M!S5)nCD}s=tzBReu%7s{U$@RsA&_tNM50Sk=EP$EyC_I9B!V
z&atY04~|v+dvdJm-;3gMRey+MRezXcReywIRezLYRey|QRezjgRevqVs{T5TRsHoG
ztNJH!tm>c0v8sO($EyAYj#d4Q9IN^#bFAubqPRlUKZRpee>2Cb{uYi^{jD6U`loWN
z>ff7VRsS@ORsH*Ltm@yFV^#lt9IN`<I9Byf=UCOhKgX*612|Uow{xuO@1VF+)!)gn
zs(%K@s{WZAtNLegtm;3IV^#k_9IN^d=2+D~n`2e~9FA4}b2(P^&*NCte+b8_{zExd
z^&iHus{e3~Rs9Kyt5p3*aIES-l4Di>e2!K93piHwFXUL&e-y{6{-Zfo^&i8rs(%s3
zs{Ug+R`oCDSk-?V$EyA%9IN_|=UCN$0>`TUr4(1I`j>I6>hI!M)xVr$RsV?`tNKsk
zSk=FRV^#mj9IN_Qa;)k<g=1C!DvnkCr*f?7KaFEm|LGj7`p@84)qf_(s{XSmu2J=`
z=2+E#Hpi;|ZjM#`=Wwj*U&FDge=WzV{&P82^`FPFs{eeBRs9!mtm?mzV^#k}9IN^-
z=2+E#3CF7bOF35c_fQ;C^{?Yt)xVx&RsUritNMF6R`p-bv8w+Hj#d3va;)mViepv(
z)f}t(ui;qLe=WzV{_8ka^<U4is{aO#RsA<|tm?ms;;^d!W{y?;w{WcLzm;QE|7{$r
z`ful0)qe-as{T7UR`uV-v8w-Wj#d5laIEUTmt$4`eH^R$@8?+6{{Y9T{s%c$^>3g!
zqUwK$V^#md9IN^t;aJuGD95V)$2eB?KhCkL{|Sy&{ZDeN>VJx3RsYi*tNNeeSk?b5
z$EyD4I9ByP&#|ii1&&qy8!3*e`d{Q&)&CO5s{WTbR`tKav8w-7j#d4yajfcponux1
zCXQA8Z*Z*Yf0JWX|63fZ`rqbQ)&CC1s{VI5R`tKfv8w-liesw&4>(r!f5@?_|09l7
z{U39z>i>jeRsW|PtNK6VSk?bI$EyA>I9Bz4$+4>cD~?tDUvsSL|Au2#|F;~g`oH5?
z)&D)kaaI2h9IN_&<XF}J6UVClpE*|b|H84V|5uJx{l9Un>fg+<s{eP6RsDZ(tm^-h
zV^#lO9IN{O=2+GL567zhK8~q=fA?&9?_x!ygn$3%fyW-u4tS7e4`>TKNVf;H2OebD
z1KI=+GVKBFf(KdlfVRPd0rr6Q!2_>7ppEb#+aAzPc;K@Kv=tuY*aO-N5B&CkHp7El
zdqBJ4L7qLJ?eHMq9;p2Uc)%|R_?7~Dp!QQ}4@&10)n(DCN}=qwvD3$A;EOtbvBu{b
z$*bvwRl@4%8Qv{#;FoI&Up}2<WZW`oVsA%LLngn3`)`Rnk|LpBzo>3-q$Sem{g-Wc
zBaN|Xg=u{WYi7bTYkj=1dzL3*%}RJW5?1H>M*mse9iBBaa|Y6t9f_GTsl8=+CA|dg
z8L~!ROD{raUfz@4-RbM2M3*P}yS-<eao_Q)+{Db4TxF-Vu#@UqSJhKtOtW7H(@WKv
zzVw7~*U*i$ZsANmaWa<SoH@zh3~MI8yq!r2+DiK0oMBNt3up9yM>k|gCbzWu|K*PE
z|M6n@bmt}J^mE@Nd};Dpo;p`vNq2L<)wM0JrWeCOI^043=wJXnvc0|h_@<Zp4WZV8
z9G};sH&^+vdr=;p6xy1@o3UokncKEJmO)EP>?887rF67AwbbLWm*%vqgAqXwFdmz!
zW2The9p@amx&l6IopRFBY{#E+d}(xSFj<c;O-p5Tx|x*d2y{wHrxWn(ttKn-rK6Ot
zrE+@rTuO8>IVELeNGSuQ3@ugg`we1ubX+(kWoAk#6QxWoRnoibQlf)MDJd&UN?9mn
zX{n0dZI==qtxZV-21scDN&~c1O%D($(V4|5$?KJp7bUNjYV0#;-H(E7kh1Off9PSF
z`rc)ul&z%@zq=#uaUV!NpRCD;l21!veve2<IUrHb;hUkR9F%gj6rp$TrQ`?6Z=cwV
zk{>0%mZJ1-zLau7%FUHE<)W0Ur5L@tFQq(?^75pVhf<!F;`DC6l<2*;)T1CjUrPBX
z<)@^QARnd|_c$HMPf3A*lmaLPv{b@J<Ap@W@KaJjfs_hRD$r6XABYzc9llRVg@sZo
zM5$0qWqb@?NOa^rB@G-XrGY37)KWPgeisrQv`<MzMN%q4sYpu|eB@n7bi6(#4H_h+
zK`0H<QY9aB7ZM$+Pf3FZOKC7lgSAw}$J>QON9j}2kRehUg3=HzRnyrAQW^@<(4kTq
ziqcRm)zH}pQW^%*uwhafhSD%Ch3ND?DGdi{_;4u=M`^g0!gRKRltzFwVuX}Npfo~D
z5juN8N+UrUIZ{d^Q5vbGD4oq9r7b|(Vhbs4fzlRQiqY8(QW^!)s8LcHh0-W3#p!GZ
zDUAkc^k^xKMrm|PDh(E!QZY!*|Kh=)<HaZyYpG;QQ`!=wEw_|4ZHdyBS}Gl5N@GA8
zGe$~dP#UA9vazN#7NoIbr8E|$v05r0XG-Hh8aGZ#<4_u>rHZXgX)BPn+Db}Wp|q8j
zDz`SJtwGv)YbkAw($-q4+QyW&0co3Uq_ho6+i0nJTT|K=q;0pA(zYmVtEHOlOldoi
zw%blh+o80bmO|T`()J*2zrB>UM`?R4g?BKe9YES)2Py4<(hgdR>}X0mg0$m~QrZ!v
z9kmqQ$&_{iX{Vi}v=d4@X(_g|DeVl>&O1wKXOwo<QhdBAjR$G`cqxrXX?#j5Q!nRS
zuchdH=~U+o2Bj24DX68A60;`yk6|gP#D4XMKj}(PD$!DDsVS9$R9Y%)Dn+SOOJ!xI
zR0dL6nUu;<D$`PVxha){R9-Hna+J!oR8e6{6(ChqNT~v)3N2Muno=c5m6cMeM5$6s
zRaK@`1yWU&l&Vmw(o%J`DOH11T`i?*l&ZB<Q)5asAl1}JsRpGQEroV5rCmVUWfv*!
zg3>Np3h!!4yMnaqu2R|+rCqfY+0B%818KM2q_i7KyJ;!9yD9As((b!UX?K)%*HUZ`
zQ`!ThJ@$~&9w_ahrTCtvv?oY=?kS}`QQ9*ll?V4SrM*DfYcDD7h0<PHDhZhqodcHo
z`4kFCDTGo;OQm5`qSL=pQaCInJ}WbI{3)!ZvWO|snO`X>V*lT6{F%uoSf-?imdc~1
zL??Zvq-a#u#OF$;q^OoEVx~mrd!?jUOiFy(V@ir?sWNU#bgEZMiqqLf^r8)phVU7T
zDJia{s#;T`v%FGLZLO5}WW<zItEK8XQ=${RQc_)=l=!^DlvJmsntD^J2dTbZO7$q!
zYbi9rlqP^QVS<z<pfo{C;fba+5u}L|r8E(xiCT(GGNnl%O`0U7NhnRyQnbO8=s(}5
z?(v2ODK((fpru%&DK&!B*eIn&lp3`ZpKMB#L7F^SN|RBVoRTVnO{UZYQd5(Znow%e
zQpprkngY_4DN>q((iAO~Hk(p2NX^YsYDTG9OJyylM5p?tzIQDxQffh|MN8$arbK^r
zAtkl8N~sm4RxMRbHKnN_O`R&GsVGg=Qsv&Jv^Pk5?=7XhQQBKeRntsq8c5ToNog8N
z)3j8*k16c~(mwl0X&;pK(NfL6rnE0e`|c~HeNoz1OQHQtX+Mzm+fPdSp|qct!fmG1
z22xv_l-f{g(^6!*DNP4y`gAExM`^m2qWhcD{vhqYzm)bzX@4!n4ltzyKsw+6DII{)
z0a}W;n-cxOjMTHGy<JM}D7B}g%3y~nb%4~-A*BwKI<!>MX-b_Sb#_Xr6QxcqmCi7w
z86eG=A*C58&CpWWOjDW((#)Asnu*d(EtStQrCA`&nkA)KD9zGR#et@DAV>!uD5V2Y
zI#5fM2bt1AARTm&lnz4aAT3oLY)S`%bnwAaIvAycwNyRZlxBl8d$yEjqcmGfHFHd9
z4oGw6NNEmAbF>tiYf5uLnmboYb5WYBrSLpcng`Onc~Y8((mX9i4l$)eKsw|QDIJ2+
zAzF$aYD$NKbm*Z{IuxZtwG=zdlnw*wu*0Nu7)pm}DSo&q9S+jrhfC>jlnzfxRl$TQ
zB|u6fq?AA@p{0@|Oz8-ajyOU}N1$|smP(H_r6WN)@<=HiiPDi;Dw}Ug^Ff+FUrO^)
zny;nu1*WtBqy-D4v;d_ATB=xRN((_+xKK(9QCg^_%A-u_D3FdiN=iqebd;8=jy9#E
zK|1<qDIJZ{(ORlL#*~f$>6l}rbPP(zXsKqADJ=qN(IP1=LTQnfLdTlYu^=6Ltdx#L
z=~yj=7n{;zkQOhN(qfbrYbkP^DIEvWamPvNIFydlQgn$aEdgoC5-BY~X^EC%$D7je
zART|al#WN~crC?GFr^bfI^hH<oq*B_DXBWR)RdNjv~;PImZG#&OC`%pX&Fe%mPu(D
zO3SoV+GR>zAa!*~sSBkpEtM@drR5+kUoNHPC@t4g`H7}<B1k8mD5VooI#Ej%Cz;Yo
zAf0rQluknFBrR30Fr^hBtym$Y6)3IHQq{?(bTUXMpDd-5Q94;m)hkVDB}gk*N@*oZ
zE45T}iYc7}(kZ7%=@gVs(NbuYDXju&)ha2iLTQzj!l#<jsUV$ts+3Me=~OL6PBW#`
zKsxO-DV>JWX<CY&Zc3+vbo%L1Ivu6cwG=zUl+FO@j5DNk21;jWDSoCYoe9#JXG-Zz
zl+H{^HNmq?=`4`WI!j7tp>&p(N>-cFYLHg1meOjJR%@yBY*RWLq_fYK(%C4Tt);SV
zQ|bn(yIV@#D0OS8{2Ws{2c&b(k<vLRouj3SHKw!%q%~`#v<9U$TB=-YN^3z{yH-kT
zQCh2|s&h^0T#(K^S4!uibgq`F&oia-KsxU{DV>MXd0MJD-;~Y=>HPDhbUsSwYbkVr
zDO~{41s6!^0+cS$Qusnsx)7ubFO<@SC|#(f$VH}f5l9zZB&CZ`x=2gWi%scbkS@Mh
zN*AMav6f<&n9?O6U2=((E<x!MEyXW2rAtA&^inBZiqfSiDHQB6r5=!adZg5YQjeBO
z)|t{ekk+k}(mIsZX{mI*DXj--{dy^_M`^v5$}Tgd%Rsv9GAUh#(q&pI?=_`fka~Ni
z)QeKDmMShcrOQFO{BkK>j?(2?s=UIKt^nzZE2MM<N>^y9>Pl0(5~M4yl+u+bU8$w&
zt4!%CkgmE)N>`zDm6mF*Hl?dUy83D<U5(P!S_)lbO4opN%{5ZG2BmAX6u#D!t_A7Z
zYo&B8O4n*Ba-At%2hw%dN$EP2uG3QVdQ-X{r0cJj()B1^ucg=xrgQ^HH{2kl8&JAI
zOYs{`=|+%lyirOwqI6?Q3I}g8rJF#y=_V=Ngwjn~D!JK|ZU*V*o27I!N;hk%^cGXP
z1*BVUk<u+F-J+$kTTSU!kZ!$IO1GkPtCq@dGo{-=y6rY8-G<U_TB^9+lx_#<_S>a&
zJ4&}}sqzj}x&x#;?vT<QDBYo@syj{TPLS@rQ%ZNDbf=c8?=q#kK)UNLDcyzAU0SNS
z+m!AG>F&FwbT>+OYbkV(Dcu9oJ@-iI9+d9UQutm|x)-E-@0HTMDBY{2$bF`CA4vDz
zC#CyPx=%~d`%US7knX=<O8295zm{STn9>6vJ@9~(9zf{<EyW)+r3XQJ@Ifg(h|+^8
zDH7aZN*h4hut7>2P}-oSl7~#`A&?$=NJ<Z(^pKWHA2y|jL3;RMDLstR!&)kP#FQQZ
z>5)gI^ax6iXsP^BQ+gDnM<12aqbNP9rHaQ)=`oNVdrV4?q4b!RDjzqc$3c4haVb5H
z(&JjHdcu^R0O^S*r1S(zPiU$7NmF_fq$i)0(vv7Xsim5yOzA0*o_b13PoeabmO@XP
z($gS4{j`*xM(JrSg`Y8{XFz)987V!3(lc6$JZnnNg7oaOQhFApXSEc4&Xk@5>AB~m
z^c+ghX({%+DLoI;^Uq7^d6b^lQv3x|dI6*tUXaoYD7}!9qQQ-(v=O9@8>O@nrHxuD
zdC`<!1nI>WrSu|7FKVguB~y9{q?cZj(n~14q@}W#P3dKjUVd3hFQfFbmdal-rB^_D
z<rOKtg3>Ems(96uUIppZSEckSO0R0E@-<U>4W!pzlhSJ_y{4tA*G=hlkY0aXO0T2z
zx|XUpnbIbZHf@s9CX_a5spbt+dIO|4-jLE8D7~Sj(3__8CP;6-DWx}2dQ(f`w@m3R
zkluPrN^ha`mX;!Ko6_4Lz5TY7-bU$dEk)lkrFTGj=N&1%gVH-%ioI(}?}GI1yHa`=
zrFXRyf6tWO1L?i@r1Tz2@1>+z@O@KyAEfu+m(u$vy|1N`4@~I;kUsc8N*|!~ftE@?
zG^Gzg`tU<3eTdSBS}Oa<ls*FKqmQKY5lSCvsr+M8`WU2-KbF$RD1EG@icd`G6Ocan
zL`t8a^of=#KQ*OKLHhJlDSe94r&_A|%#=O@>9fzI^chN@X{q{iQ~DgF&p(&a=O}%y
zrJ65H=?jp)_(Dowp!9{7LSLHFmmq!lrIfxz=}RqzzcQt-K>F${DSd^~S6YgEZAxE*
z^!3+L`WmILwG{ovl)eG!n{TA_4NBi=DfX=?eGAgJ-%9CQl)lwc{5w<n4y5nClhSu6
zeV3Br!S7A!dyu~WUP|Ai^u3l!elVpUK>FbaDgA)b4_Yey(Ug7!>Bk?X^dm|?YN_le
zQ~C*{pMH|kPbmGQrShLm>1U9B{#i;tqx7?uDt<AgUqJfh7b*RM(l1)7{MD3x1?ksc
zrSvOGziO%KH&glzq~Cs%(r+mJrlso5rnDKP&6}mP8Kuoys`=fNeh2CI-=*|BO22C<
z^oJ?^0n#6TNa+uh{?JnRPgD96q(A?Z(w`{(sinwYrt}v`fBhw;zfk&1OVPhg>2Hw!
z{##0aqx83yV*i-ZKOp_{kCgsF=^vK5_R8WvxMu&+X!YlqgH@LOm!o}suK(rR=g9O?
zo!J4)>Z8=G^CkX|tUk=pXJ?3J@n3Q4F9ZFBH~NcBsXq%vLst$?z&ZE}m6H6sJiIE9
z`Y%>A(pgIS&Ej!q@wl^iR2II(EVK%;cpyt@zgg1US<>8D(o`0{#4NN5vZO(lvVOCq
zyR)Rbv!tsme2H0T6=X?=Eam-X$#7@MaA(O-S@;sO&??B10a+^g&64TP!XF08|6@Tt
z!ZKABzQioF3bJHEmdbv!WVy3sxwB-cEPRPsXcc71f-F`2W*OklGQgc>fXc#`n1xnB
zmI08Zy5B5bcNVWZi&tggOUy#6Ad45W)byJr+npuboh4gk;Y-Xys~}4@WC`_~#pllA
zb7%3XEPRPsXcc7fL6&g8S#sQ2a@<*RR2II(EVK%;<Up24zghh5EPi(uzskawn1xnB
z7C&T(_M0Wwoh8?uC0AwPOUy#6AWJS}iS?T$&z&XDoh46Y;Y-Xys~}4rWa+oJ&>zNh
z{p8PgXUSJt_!6_wD#((rvy}ATUjptd0e6;w%EFhJg;qh90A%UEzZAH$6u7e#s4RSm
zS!fkxDS#~f_m@I<mO^)yLY0LtF$=ANEQOq<Xas!>Lv2&6Jr*pr)Q3m#M>6oA><moK
z;G0|zj7qDhhUS)7{8uJZA2s28`Y~M56lt{o45VvDpka{nnGf~{4BH>HJt4<;$I4tQ
z&Ci|P{{4YI8_KiUznPvjbiIc@--7=AMmKNvr^dPTdBm3Wll=+Z_U-%;OMQJgJeB^X
zS+39S_R>cpJMZ{g)>W=sHqeK9JHItq8}?YW1AX4N(j15GJW~7myv{K7mB;*n*O3gm
zbJRnPKR!GZtaN@?%H`|f<29TQ!On3$4aWX>jpD50ERX%E*%LCF>YAFJyV#A;oz%}q
zb(q!${|3KjD1GGhcGD?+OIdqiRtEq2LUDO}p*O>q=1a3j6?_ZtBCxCeHner^9G@0Z
zPs=H>hN<*7I!k#ieK0sy9LNl0@&|$h8Z)Nb9}^wV;O}BvS58YX*zSC$dt2A`Ex~GM
cK;JQsRYXon)S?f-rl2$#l-WTRKSZqm0l)xpv;Y7A
literal 0
HcmV?d00001
diff --git a/tests/data/acpi/q35/FACP.core-count2 b/tests/data/acpi/q35/FACP.core-count2
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..31fa5dd19c213034eef4eeefa6a04e61dadd8a2a 100644
GIT binary patch
literal 244
zcmZ>BbPo8!z`($~*~#D8BUr&HBEVSz2pEB4AU24G0Y(N+hD|^Y6El!tgNU*~X%LSC
z$X0-fGcm9T0LA|E|L2FOWMD92VqjR>!otAF!NBm72O<iWged~jj0!*k$y^{03>bk1
YBHITON2VDSAnpK(F*YFF1LDH~0O^Si0RR91
literal 0
HcmV?d00001
diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h
index e81dc67a2e..dfb8523c8b 100644
--- a/tests/qtest/bios-tables-test-allowed-diff.h
+++ b/tests/qtest/bios-tables-test-allowed-diff.h
@@ -1,4 +1 @@
/* List of comma-separated changed AML files to ignore */
-"tests/data/acpi/q35/APIC.core-count2",
-"tests/data/acpi/q35/DSDT.core-count2",
-"tests/data/acpi/q35/FACP.core-count2",
--
2.37.3
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH v3 5/5] tests/acpi: update tables for new core count test
2022-10-11 11:17 ` [PATCH v3 5/5] tests/acpi: update tables for new core count test Julia Suvorova
@ 2022-10-27 6:11 ` Michael S. Tsirkin
2022-10-27 14:11 ` Igor Mammedov
0 siblings, 1 reply; 10+ messages in thread
From: Michael S. Tsirkin @ 2022-10-27 6:11 UTC (permalink / raw)
To: Julia Suvorova; +Cc: qemu-devel, Igor Mammedov, Ani Sinha
On Tue, Oct 11, 2022 at 01:17:31PM +0200, Julia Suvorova wrote:
> Changes in the tables (for 275 cores):
> FACP:
> + Use APIC Cluster Model (V4) : 1
>
> APIC:
> +[02Ch 0044 1] Subtable Type : 00 [Processor Local APIC]
> +[02Dh 0045 1] Length : 08
> +[02Eh 0046 1] Processor ID : 00
> +[02Fh 0047 1] Local Apic ID : 00
> +[030h 0048 4] Flags (decoded below) : 00000001
> + Processor Enabled : 1
> ...
> +
> +[81Ch 2076 1] Subtable Type : 00 [Processor Local APIC]
> +[81Dh 2077 1] Length : 08
> +[81Eh 2078 1] Processor ID : FE
> +[81Fh 2079 1] Local Apic ID : FE
> +[820h 2080 4] Flags (decoded below) : 00000001
> + Processor Enabled : 1
> + Runtime Online Capable : 0
> +
> +[824h 2084 1] Subtable Type : 09 [Processor Local x2APIC]
> +[825h 2085 1] Length : 10
> +[826h 2086 2] Reserved : 0000
> +[828h 2088 4] Processor x2Apic ID : 000000FF
> +[82Ch 2092 4] Flags (decoded below) : 00000001
> + Processor Enabled : 1
> +[830h 2096 4] Processor UID : 000000FF
> ...
>
> DSDT:
> + Processor (C001, 0x01, 0x00000000, 0x00)
> + {
> + Method (_STA, 0, Serialized) // _STA: Status
> + {
> + Return (CSTA (One))
> + }
> +
> + Name (_MAT, Buffer (0x08) // _MAT: Multiple APIC Table Entry
> + {
> + 0x00, 0x08, 0x01, 0x01, 0x01, 0x00, 0x00, 0x00 // ........
> + })
> + Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
> + {
> + CEJ0 (One)
> + }
> +
> + Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
> + {
> + COST (One, Arg0, Arg1, Arg2)
> + }
> + }
> ...
> + Processor (C0FE, 0xFE, 0x00000000, 0x00)
> + {
> + Method (_STA, 0, Serialized) // _STA: Status
> + {
> + Return (CSTA (0xFE))
> + }
> +
> + Name (_MAT, Buffer (0x08) // _MAT: Multiple APIC Table Entry
> + {
> + 0x00, 0x08, 0xFE, 0xFE, 0x01, 0x00, 0x00, 0x00 // ........
> + })
> + Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
> + {
> + CEJ0 (0xFE)
> + }
> +
> + Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
> + {
> + COST (0xFE, Arg0, Arg1, Arg2)
> + }
> + }
> +
> + Device (C0FF)
> + {
> + Name (_HID, "ACPI0007" /* Processor Device */) // _HID: Hardware ID
> + Name (_UID, 0xFF) // _UID: Unique ID
> + Method (_STA, 0, Serialized) // _STA: Status
> + {
> + Return (CSTA (0xFF))
> + }
> +
> + Name (_MAT, Buffer (0x10) // _MAT: Multiple APIC Table Entry
> + {
> + /* 0000 */ 0x09, 0x10, 0x00, 0x00, 0xFF, 0x00, 0x00, 0x00, // ........
> + /* 0008 */ 0x01, 0x00, 0x00, 0x00, 0xFF, 0x00, 0x00, 0x00 // ........
> + })
> + Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
> + {
> + CEJ0 (0xFF)
> + }
> +
> + Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
> + {
> + COST (0xFF, Arg0, Arg1, Arg2)
> + }
> + }
> +
> ...
>
> Signed-off-by: Julia Suvorova <jusual@redhat.com>
> Message-Id: <20220731162141.178443-6-jusual@redhat.com>
I had to rebase pushed last update to my tree. Could you pls
disassemble and verify it's correct? Thanks!
--
MST
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH v3 5/5] tests/acpi: update tables for new core count test
2022-10-27 6:11 ` Michael S. Tsirkin
@ 2022-10-27 14:11 ` Igor Mammedov
0 siblings, 0 replies; 10+ messages in thread
From: Igor Mammedov @ 2022-10-27 14:11 UTC (permalink / raw)
To: Michael S. Tsirkin; +Cc: Julia Suvorova, qemu-devel, Ani Sinha
On Thu, 27 Oct 2022 02:11:23 -0400
"Michael S. Tsirkin" <mst@redhat.com> wrote:
> On Tue, Oct 11, 2022 at 01:17:31PM +0200, Julia Suvorova wrote:
> > Changes in the tables (for 275 cores):
> > FACP:
> > + Use APIC Cluster Model (V4) : 1
> >
> > APIC:
> > +[02Ch 0044 1] Subtable Type : 00 [Processor Local APIC]
> > +[02Dh 0045 1] Length : 08
> > +[02Eh 0046 1] Processor ID : 00
> > +[02Fh 0047 1] Local Apic ID : 00
> > +[030h 0048 4] Flags (decoded below) : 00000001
> > + Processor Enabled : 1
> > ...
> > +
> > +[81Ch 2076 1] Subtable Type : 00 [Processor Local APIC]
> > +[81Dh 2077 1] Length : 08
> > +[81Eh 2078 1] Processor ID : FE
> > +[81Fh 2079 1] Local Apic ID : FE
> > +[820h 2080 4] Flags (decoded below) : 00000001
> > + Processor Enabled : 1
> > + Runtime Online Capable : 0
> > +
> > +[824h 2084 1] Subtable Type : 09 [Processor Local x2APIC]
> > +[825h 2085 1] Length : 10
> > +[826h 2086 2] Reserved : 0000
> > +[828h 2088 4] Processor x2Apic ID : 000000FF
> > +[82Ch 2092 4] Flags (decoded below) : 00000001
> > + Processor Enabled : 1
> > +[830h 2096 4] Processor UID : 000000FF
> > ...
> >
> > DSDT:
> > + Processor (C001, 0x01, 0x00000000, 0x00)
> > + {
> > + Method (_STA, 0, Serialized) // _STA: Status
> > + {
> > + Return (CSTA (One))
> > + }
> > +
> > + Name (_MAT, Buffer (0x08) // _MAT: Multiple APIC Table Entry
> > + {
> > + 0x00, 0x08, 0x01, 0x01, 0x01, 0x00, 0x00, 0x00 // ........
> > + })
> > + Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
> > + {
> > + CEJ0 (One)
> > + }
> > +
> > + Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
> > + {
> > + COST (One, Arg0, Arg1, Arg2)
> > + }
> > + }
> > ...
> > + Processor (C0FE, 0xFE, 0x00000000, 0x00)
> > + {
> > + Method (_STA, 0, Serialized) // _STA: Status
> > + {
> > + Return (CSTA (0xFE))
> > + }
> > +
> > + Name (_MAT, Buffer (0x08) // _MAT: Multiple APIC Table Entry
> > + {
> > + 0x00, 0x08, 0xFE, 0xFE, 0x01, 0x00, 0x00, 0x00 // ........
> > + })
> > + Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
> > + {
> > + CEJ0 (0xFE)
> > + }
> > +
> > + Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
> > + {
> > + COST (0xFE, Arg0, Arg1, Arg2)
> > + }
> > + }
> > +
> > + Device (C0FF)
> > + {
> > + Name (_HID, "ACPI0007" /* Processor Device */) // _HID: Hardware ID
> > + Name (_UID, 0xFF) // _UID: Unique ID
> > + Method (_STA, 0, Serialized) // _STA: Status
> > + {
> > + Return (CSTA (0xFF))
> > + }
> > +
> > + Name (_MAT, Buffer (0x10) // _MAT: Multiple APIC Table Entry
> > + {
> > + /* 0000 */ 0x09, 0x10, 0x00, 0x00, 0xFF, 0x00, 0x00, 0x00, // ........
> > + /* 0008 */ 0x01, 0x00, 0x00, 0x00, 0xFF, 0x00, 0x00, 0x00 // ........
> > + })
> > + Method (_EJ0, 1, NotSerialized) // _EJx: Eject Device, x=0-9
> > + {
> > + CEJ0 (0xFF)
> > + }
> > +
> > + Method (_OST, 3, Serialized) // _OST: OSPM Status Indication
> > + {
> > + COST (0xFF, Arg0, Arg1, Arg2)
> > + }
> > + }
> > +
> > ...
> >
> > Signed-off-by: Julia Suvorova <jusual@redhat.com>
> > Message-Id: <20220731162141.178443-6-jusual@redhat.com>
>
>
>
> I had to rebase pushed last update to my tree. Could you pls
> disassemble and verify it's correct? Thanks!
new lobs look correct,
though I'd mention in commit log relevant FACP bits:
+ Use APIC Cluster Model (V4) : 1
^ permalink raw reply [flat|nested] 10+ messages in thread