From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B1DD3C3524A for ; Sun, 2 Feb 2020 14:01:59 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6957220679 for ; Sun, 2 Feb 2020 14:01:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726900AbgBBOBz (ORCPT ); Sun, 2 Feb 2020 09:01:55 -0500 Received: from szxga05-in.huawei.com ([45.249.212.191]:9681 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726881AbgBBOBz (ORCPT ); Sun, 2 Feb 2020 09:01:55 -0500 Received: from DGGEMS413-HUB.china.huawei.com (unknown [172.30.72.59]) by Forcepoint Email with ESMTP id 7079A892D86C4AF33C67; Sun, 2 Feb 2020 22:01:51 +0800 (CST) Received: from [127.0.0.1] (10.142.68.147) by DGGEMS413-HUB.china.huawei.com (10.3.19.213) with Microsoft SMTP Server id 14.3.439.0; Sun, 2 Feb 2020 22:01:41 +0800 Subject: Re: [PATCH v22 3/9] ACPI: Build related register address fields via hardware error fw_cfg blob To: Igor Mammedov CC: , , , , , , , , , , , , , , , References: <1578483143-14905-1-git-send-email-gengdongjiu@huawei.com> <1578483143-14905-4-git-send-email-gengdongjiu@huawei.com> <20200123161451.07c4a291@redhat.com> From: gengdongjiu Message-ID: Date: Sun, 2 Feb 2020 22:01:39 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <20200123161451.07c4a291@redhat.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.142.68.147] X-CFilter-Loop: Reflected Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org On 2020/1/23 23:14, Igor Mammedov wrote: > On Wed, 8 Jan 2020 19:32:17 +0800 > Dongjiu Geng wrote: > >> This patch builds error_block_address and read_ack_register fields >> in hardware errors table , the error_block_address points to Generic >> Error Status Block(GESB) via bios_linker. The max size for one GESB >> is 0x1000 in bytes, For more detailed information, please refer to > > s/0x1000/4Kb/ will update it. > >> document: docs/specs/acpi_hest_ghes.rst >> >> Now we only support one Error source, if necessary, we can extend to >> support more. >> >> Suggested-by: Laszlo Ersek >> Signed-off-by: Dongjiu Geng >> Signed-off-by: Xiang Zheng >> Reviewed-by: Michael S. Tsirkin >> --- >> default-configs/arm-softmmu.mak | 1 + >> hw/acpi/Kconfig | 4 ++ >> hw/acpi/Makefile.objs | 1 + >> hw/acpi/aml-build.c | 2 + >> hw/acpi/ghes.c | 94 +++++++++++++++++++++++++++++++++++++++++ >> hw/arm/virt-acpi-build.c | 6 +++ >> include/hw/acpi/aml-build.h | 1 + >> include/hw/acpi/ghes.h | 26 ++++++++++++ >> 8 files changed, 135 insertions(+) >> create mode 100644 hw/acpi/ghes.c >> create mode 100644 include/hw/acpi/ghes.h >> >> diff --git a/default-configs/arm-softmmu.mak b/default-configs/arm-softmmu.mak >> index 1f2e0e7..5722f31 100644 >> --- a/default-configs/arm-softmmu.mak >> +++ b/default-configs/arm-softmmu.mak >> @@ -40,3 +40,4 @@ CONFIG_FSL_IMX25=y >> CONFIG_FSL_IMX7=y >> CONFIG_FSL_IMX6UL=y >> CONFIG_SEMIHOSTING=y >> +CONFIG_ACPI_APEI=y >> diff --git a/hw/acpi/Kconfig b/hw/acpi/Kconfig >> index 12e3f1e..ed8c34d 100644 >> --- a/hw/acpi/Kconfig >> +++ b/hw/acpi/Kconfig >> @@ -23,6 +23,10 @@ config ACPI_NVDIMM >> bool >> depends on ACPI >> >> +config ACPI_APEI >> + bool >> + depends on ACPI >> + >> config ACPI_PCI >> bool >> depends on ACPI && PCI >> diff --git a/hw/acpi/Makefile.objs b/hw/acpi/Makefile.objs >> index 9925305..7b5db4b 100644 >> --- a/hw/acpi/Makefile.objs >> +++ b/hw/acpi/Makefile.objs >> @@ -5,6 +5,7 @@ common-obj-$(CONFIG_ACPI_CPU_HOTPLUG) += cpu_hotplug.o >> common-obj-$(CONFIG_ACPI_MEMORY_HOTPLUG) += memory_hotplug.o >> common-obj-$(CONFIG_ACPI_CPU_HOTPLUG) += cpu.o >> common-obj-$(CONFIG_ACPI_NVDIMM) += nvdimm.o >> +common-obj-$(CONFIG_ACPI_APEI) += ghes.o >> common-obj-$(CONFIG_ACPI_VMGENID) += vmgenid.o >> common-obj-$(CONFIG_ACPI_HW_REDUCED) += generic_event_device.o >> common-obj-$(call lnot,$(CONFIG_ACPI_X86)) += acpi-stub.o >> diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c >> index 2c3702b..3681ec6 100644 >> --- a/hw/acpi/aml-build.c >> +++ b/hw/acpi/aml-build.c >> @@ -1578,6 +1578,7 @@ void acpi_build_tables_init(AcpiBuildTables *tables) >> tables->table_data = g_array_new(false, true /* clear */, 1); >> tables->tcpalog = g_array_new(false, true /* clear */, 1); >> tables->vmgenid = g_array_new(false, true /* clear */, 1); >> + tables->hardware_errors = g_array_new(false, true /* clear */, 1); >> tables->linker = bios_linker_loader_init(); >> } >> >> @@ -1588,6 +1589,7 @@ void acpi_build_tables_cleanup(AcpiBuildTables *tables, bool mfre) >> g_array_free(tables->table_data, true); >> g_array_free(tables->tcpalog, mfre); >> g_array_free(tables->vmgenid, mfre); >> + g_array_free(tables->hardware_errors, mfre); >> } >> >> /* >> diff --git a/hw/acpi/ghes.c b/hw/acpi/ghes.c >> new file mode 100644 >> index 0000000..b7fdbbb >> --- /dev/null >> +++ b/hw/acpi/ghes.c >> @@ -0,0 +1,94 @@ >> +/* >> + * Support for generating APEI tables and recording CPER for Guests >> + * >> + * Copyright (c) 2019 HUAWEI TECHNOLOGIES CO., LTD. > s/2019/2020/ > > the same for other places that add this string within this series sure, thanks for the pointing out. > >> + * >> + * Author: Dongjiu Geng >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License as published by >> + * the Free Software Foundation; either version 2 of the License, or >> + * (at your option) any later version. >> + >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + >> + * You should have received a copy of the GNU General Public License along >> + * with this program; if not, see . >> + */ >> + >> +#include "qemu/osdep.h" >> +#include "hw/acpi/acpi.h" >> +#include "hw/acpi/ghes.h" >> +#include "hw/acpi/aml-build.h" >> +#include "hw/nvram/fw_cfg.h" >> +#include "sysemu/sysemu.h" >> +#include "qemu/error-report.h" >> + >> +#include "hw/acpi/bios-linker-loader.h" >> + >> +#define ACPI_GHES_ERRORS_FW_CFG_FILE "etc/hardware_errors" >> +#define ACPI_GHES_DATA_ADDR_FW_CFG_FILE "etc/hardware_errors_addr" >> + >> +/* The max size in bytes for one error block */ >> +#define ACPI_GHES_MAX_RAW_DATA_LENGTH 0x400 > > using 1 * KiB would make it more readable sure > > >> + >> +/* Now only support ARMv8 SEA notification type error source */ >> +#define ACPI_GHES_ERROR_SOURCE_COUNT 1 >> + >> +/* >> + * Build table for the hardware error fw_cfg blob. >> + * Initialize "etc/hardware_errors" and "etc/hardware_errors_addr" fw_cfg blobs. >> + * See docs/specs/acpi_hest_ghes.rst for blobs format. >> + */ >> +void build_ghes_error_table(GArray *hardware_errors, BIOSLinker *linker) >> +{ >> + int i, error_status_block_offset; >> + >> + /* Build error_block_address */ >> + for (i = 0; i < ACPI_GHES_ERROR_SOURCE_COUNT; i++) { >> + build_append_int_noprefix(hardware_errors, 0, sizeof(uint64_t)); >> + } >> + >> + /* Build read_ack_register */ >> + for (i = 0; i < ACPI_GHES_ERROR_SOURCE_COUNT; i++) { >> + /* >> + * Initialize the value of read_ack_register to 1, so GHES can be >> + * writeable after (re)boot. >> + * ACPI 6.2: 18.3.2.8 Generic Hardware Error Source version 2 >> + * (GHESv2 - Type 10) >> + */ >> + build_append_int_noprefix(hardware_errors, 1, sizeof(uint64_t)); >> + } >> + >> + /* Generic Error Status Block offset in the hardware error fw_cfg blob */ >> + error_status_block_offset = hardware_errors->len; >> + >> + /* Reserve space for Error Status Data Block */ >> + acpi_data_push(hardware_errors, >> + ACPI_GHES_MAX_RAW_DATA_LENGTH * ACPI_GHES_ERROR_SOURCE_COUNT); >> + >> + /* Tell guest firmware to place hardware_errors blob into RAM */ >> + bios_linker_loader_alloc(linker, ACPI_GHES_ERRORS_FW_CFG_FILE, >> + hardware_errors, 1, false); > ^^ > /* > * bios_linker_loader_alloc: ask guest to load file into guest memory. > [...] > * @alloc_align: required minimal alignment in bytes. Must be a power of 2. > Maybe use 8 here to make 64bit addresses naturally aligned? I think we can use "sizeof(uint64_t)" to make it aligned. > >> + >> + for (i = 0; i < ACPI_GHES_ERROR_SOURCE_COUNT; i++) { >> + /* >> + * Tell firmware to patch error_block_address entries to point to >> + * corresponding "Generic Error Status Block" >> + */ >> + bios_linker_loader_add_pointer(linker, >> + ACPI_GHES_ERRORS_FW_CFG_FILE, sizeof(uint64_t) * i, >> + sizeof(uint64_t), ACPI_GHES_ERRORS_FW_CFG_FILE, >> + error_status_block_offset + i * ACPI_GHES_MAX_RAW_DATA_LENGTH); >> + } >> + >> + /* >> + * tell firmware to write hardware_errors GPA into >> + * hardware_errors_addr fw_cfg, once the former has been initialized. >> + */ >> + bios_linker_loader_write_pointer(linker, ACPI_GHES_DATA_ADDR_FW_CFG_FILE, >> + 0, sizeof(uint64_t), ACPI_GHES_ERRORS_FW_CFG_FILE, 0); >> +} >> diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c >> index bd5f771..6819fcf 100644 >> --- a/hw/arm/virt-acpi-build.c >> +++ b/hw/arm/virt-acpi-build.c >> @@ -48,6 +48,7 @@ >> #include "sysemu/reset.h" >> #include "kvm_arm.h" >> #include "migration/vmstate.h" >> +#include "hw/acpi/ghes.h" >> >> #define ARM_SPI_BASE 32 >> >> @@ -830,6 +831,11 @@ void virt_acpi_build(VirtMachineState *vms, AcpiBuildTables *tables) >> acpi_add_table(table_offsets, tables_blob); >> build_spcr(tables_blob, tables->linker, vms); >> >> + if (vms->ras) { >> + acpi_add_table(table_offsets, tables_blob); >> + build_ghes_error_table(tables->hardware_errors, tables->linker); >> + } >> + >> if (ms->numa_state->num_nodes > 0) { >> acpi_add_table(table_offsets, tables_blob); >> build_srat(tables_blob, tables->linker, vms); >> diff --git a/include/hw/acpi/aml-build.h b/include/hw/acpi/aml-build.h >> index de4a406..8f13620 100644 >> --- a/include/hw/acpi/aml-build.h >> +++ b/include/hw/acpi/aml-build.h >> @@ -220,6 +220,7 @@ struct AcpiBuildTables { >> GArray *rsdp; >> GArray *tcpalog; >> GArray *vmgenid; >> + GArray *hardware_errors; >> BIOSLinker *linker; >> } AcpiBuildTables; >> >> diff --git a/include/hw/acpi/ghes.h b/include/hw/acpi/ghes.h >> new file mode 100644 >> index 0000000..3dbda3f >> --- /dev/null >> +++ b/include/hw/acpi/ghes.h >> @@ -0,0 +1,26 @@ >> +/* >> + * Support for generating APEI tables and recording CPER for Guests >> + * >> + * Copyright (c) 2019 HUAWEI TECHNOLOGIES CO., LTD. >> + * >> + * Author: Dongjiu Geng >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License as published by >> + * the Free Software Foundation; either version 2 of the License, or >> + * (at your option) any later version. >> + >> + * This program is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the >> + * GNU General Public License for more details. >> + >> + * You should have received a copy of the GNU General Public License along >> + * with this program; if not, see . >> + */ >> + >> +#ifndef ACPI_GHES_H >> +#define ACPI_GHES_H >> + >> +void build_ghes_error_table(GArray *hardware_errors, BIOSLinker *linker); >> +#endif > > . >