[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for 4.1 v2 09/13] tests: acpi: ignore SMBIOS tes
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-devel] [PATCH for 4.1 v2 09/13] tests: acpi: ignore SMBIOS tests when UEFI firmware is used |
Date: |
Tue, 26 Mar 2019 15:19:13 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 03/26/19 14:09, Igor Mammedov wrote:
> once FW provides a pointer to SMBIOS entry point like it does for
> RSDP it should be possible to enable this one the same way.
>
> Signed-off-by: Igor Mammedov <address@hidden>
> ---
> tests/bios-tables-test.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
> index 91846d6..097ef03 100644
> --- a/tests/bios-tables-test.c
> +++ b/tests/bios-tables-test.c
> @@ -569,8 +569,11 @@ static void test_acpi_one(const char *params, test_data
> *data)
> }
> }
>
> - test_smbios_entry_point(data);
> - test_smbios_structs(data);
> + /* TODO: make SMBIOS tests work with UEFI firmware */
> + if (!use_uefi) {
> + test_smbios_entry_point(data);
> + test_smbios_structs(data);
> + }
>
> assert(!global_qtest);
> qtest_quit(data->qts);
>
For now:
Reviewed-by: Laszlo Ersek <address@hidden>
Can you file a LP item about this, and assign it to me? (I can assign it
to myself as well if you send me the link.)
... In fact, if we have a LP ticket, we could reference it either in the
code comment, or in the commit message. I'll leave that up to you (you
can keep my R-b either way).
Thanks,
Laszlo
[Qemu-devel] [PATCH for 4.1 v2 03/13] tests: acpi: make pointer to RSDP 64bit, Igor Mammedov, 2019/03/26
[Qemu-devel] [PATCH for 4.1 v2 07/13] tests: acpi: add acpi_find_rsdp_address_uefi() helper, Igor Mammedov, 2019/03/26
[Qemu-devel] [PATCH for 4.1 v2 05/13] tests: acpi: skip FACS table if board uses hw reduced ACPI profile, Igor Mammedov, 2019/03/26
[Qemu-devel] [PATCH for 4.1 v2 04/13] tests: acpi: fetch X_DSDT if pointer to DSDT is 0, Igor Mammedov, 2019/03/26
[Qemu-devel] [PATCH for 4.1 v2 01/13] tests: acpi: make RSDT test routine handle XSDT, Igor Mammedov, 2019/03/26
[Qemu-devel] [PATCH for 4.1 v2 02/13] tests: acpi: rename acpi_parse_rsdp_table() into acpi_fetch_rsdp_table(), Igor Mammedov, 2019/03/26