[Xen-devel] [PATCH v3 45/62] arm/acpi: Prepare STAO table for Dom0
Stefano Stabellini
stefano.stabellini at eu.citrix.com
Fri Nov 27 03:24:38 PST 2015
On Tue, 17 Nov 2015, shannon.zhao at linaro.org wrote:
> From: Shannon Zhao <shannon.zhao at linaro.org>
>
> Create STAO table for Dom0. This table is used to tell Dom0 whether it
> should ignore UART defined in SPCR table or the ACPI namespace names.
>
> Look at below url for details:
> http://wiki.xenproject.org/mediawiki/images/0/02/Status-override-table.pdf
>
> Signed-off-by: Parth Dixit <parth.dixit at linaro.org>
> Signed-off-by: Shannon Zhao <shannon.zhao at linaro.org>
> ---
> xen/arch/arm/domain_build.c | 43 +++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 43 insertions(+)
>
> diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c
> index 4591955..fb3f150 100644
> --- a/xen/arch/arm/domain_build.c
> +++ b/xen/arch/arm/domain_build.c
> @@ -1359,6 +1359,45 @@ static int prepare_dtb(struct domain *d, struct kernel_info *kinfo)
> #ifdef CONFIG_ACPI
> #define XEN_HYPERVISOR_ID 0x000058656E564D4D /* "XenVMM" */
>
> +static int acpi_create_stao(struct domain *d, struct membank tbl_add[])
> +{
> + struct acpi_table_header *table = NULL;
> + struct acpi_table_stao *stao = NULL;
> + u64 table_size = sizeof(struct acpi_table_stao);
> + acpi_status status;
> + u8 *base_ptr;
> + u8 checksum;
> +
> + status = acpi_get_table(ACPI_SIG_MADT, 0, &table);
This needs a comment: why are we trying to get the MADT table while this
function is to create the STAO table?
> + if ( ACPI_FAILURE(status) )
> + {
> + const char *msg = acpi_format_exception(status);
> +
> + printk("Failed to get MADT table, %s\n", msg);
> + return -EINVAL;
> + }
> +
> + base_ptr = d->arch.efi_acpi_table
> + + acpi_get_table_offset(tbl_add, TBL_STAO);
> + /* Copy header of one table, here use MADT */
Please move the comment above. This is just to copy the OEM fields, I
guess?
> + ACPI_MEMCPY(base_ptr, table, sizeof(struct acpi_table_header));
> + stao = (struct acpi_table_stao *)base_ptr;
> +
> + ACPI_MEMCPY(stao->header.signature, ACPI_SIG_STAO, 4);
> + stao->header.revision = 1;
> + stao->header.length = table_size;
> + stao->ignore_uart = 1;
> + checksum = acpi_tb_checksum(ACPI_CAST_PTR(u8, stao), table_size);
> + stao->header.checksum = stao->header.checksum - checksum;
> +
> + tbl_add[TBL_STAO].start = d->arch.efi_acpi_gpa
> + + acpi_get_table_offset(tbl_add, TBL_STAO);
> + tbl_add[TBL_STAO].size = table_size;
> +
> + return 0;
> +}
> +
> static int acpi_create_madt(struct domain *d, struct membank tbl_add[])
> {
> struct acpi_table_header *table = NULL;
> @@ -1561,6 +1600,10 @@ static int prepare_acpi(struct domain *d, struct kernel_info *kinfo)
> if ( rc != 0 )
> return rc;
>
> + rc = acpi_create_stao(d, tbl_add);
> + if ( rc != 0 )
> + return rc;
> +
> return 0;
> }
> #else
> --
> 2.1.0
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel at lists.xen.org
> http://lists.xen.org/xen-devel
>
More information about the linux-arm-kernel
mailing list