From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38039) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zb6dp-00045d-Np for qemu-devel@nongnu.org; Sun, 13 Sep 2015 08:44:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zb6do-0000In-4z for qemu-devel@nongnu.org; Sun, 13 Sep 2015 08:44:21 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38892) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zb6do-0000IV-0e for qemu-devel@nongnu.org; Sun, 13 Sep 2015 08:44:20 -0400 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (Postfix) with ESMTPS id B28B08EA47 for ; Sun, 13 Sep 2015 12:44:19 +0000 (UTC) From: Laszlo Ersek Date: Sun, 13 Sep 2015 14:43:44 +0200 Message-Id: <1442148227-17343-11-git-send-email-lersek@redhat.com> In-Reply-To: <1442148227-17343-1-git-send-email-lersek@redhat.com> References: <55F5647C.6030901@redhat.com> <1442148227-17343-1-git-send-email-lersek@redhat.com> Subject: [Qemu-devel] [PATCH FYI 10/13] hw/acpi: add AML generator function for DataTableRegion() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Gal Hammer , Paolo Bonzini , "Michael S. Tsirkin" , Igor Mammedov This ASL operator (and the underlying AML) enables named ACPI data tables to be located from AML code, and to be accessed field-wise, like an operation region. This is useful for passing down "parameter tables" to the guest; the ACPI linker/loader can relocate pointers in them, and then the AML code can read valid pointer values from the fields in the tables. Cc: Paolo Bonzini Cc: Gal Hammer Cc: Igor Mammedov Cc: "Michael S. Tsirkin" Signed-off-by: Laszlo Ersek --- include/hw/acpi/aml-build.h | 2 ++ hw/acpi/aml-build.c | 14 ++++++++++++++ 2 files changed, 16 insertions(+) diff --git a/include/hw/acpi/aml-build.h b/include/hw/acpi/aml-build.h index f8f96ec..dc4d215 100644 --- a/include/hw/acpi/aml-build.h +++ b/include/hw/acpi/aml-build.h @@ -225,6 +225,8 @@ Aml *aml_io(AmlIODecode dec, uint16_t min_base, uint16_t max_base, uint8_t aln, uint8_t len); Aml *aml_operation_region(const char *name, AmlRegionSpace rs, uint32_t offset, uint32_t len); +Aml *aml_data_table_region(const char *name, Aml *sig, Aml *oem_id, + Aml *oem_table_id); Aml *aml_irq_no_flags(uint8_t irq); Aml *aml_named_field(const char *name, unsigned length); Aml *aml_reserved_field(unsigned length); diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c index a0f187e..2dd2f33 100644 --- a/hw/acpi/aml-build.c +++ b/hw/acpi/aml-build.c @@ -788,6 +788,20 @@ Aml *aml_operation_region(const char *name, AmlRegionSpace rs, return var; } +/* ACPI 2.0: 17.2.4.2 Named Objects Encoding: DefDataRegion */ +Aml *aml_data_table_region(const char *name, Aml *sig, Aml *oem_id, + Aml *oem_table_id) +{ + Aml *var = aml_alloc(); + build_append_byte(var->buf, 0x5B); /* ExtOpPrefix */ + build_append_byte(var->buf, 0x88); /* DataRegionOp */ + build_append_namestring(var->buf, "%s", name); + aml_append(var, sig); + aml_append(var, oem_id); + aml_append(var, oem_table_id); + return var; +} + /* ACPI 1.0b: 16.2.5.2 Named Objects Encoding: NamedField */ Aml *aml_named_field(const char *name, unsigned length) { -- 1.8.3.1