[coreboot-gerrit] Patch set updated for coreboot: ACPI: Add code to include ATSR structure in DMAR table

Werner Zeh (werner.zeh@siemens.com) gerrit at coreboot.org
Thu Jul 28 08:15:39 CEST 2016


Werner Zeh (werner.zeh at siemens.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15911

-gerrit

commit b5016cce3aad176be012a773c51f8742cfec1d36
Author: Werner Zeh <werner.zeh at siemens.com>
Date:   Wed Jul 27 06:56:36 2016 +0200

    ACPI: Add code to include ATSR structure in DMAR table
    
    When creating DMAR table one needs to report all PCI root ports in the
    scope of a ATSR structure. Add code to create this ATS reporting
    structure. In addition, a function to fix up the size of the ATSR
    structure is added as this is a new type and using the function
    acpi_dmar_drhd_fixup() can lead to confusion.
    
    Change-Id: Idc3f6025f597048151f0fd5ea6be04843041e1ab
    Signed-off-by: Werner Zeh <werner.zeh at siemens.com>
---
 src/arch/x86/acpi.c              | 19 +++++++++++++++++++
 src/arch/x86/include/arch/acpi.h | 11 +++++++++++
 2 files changed, 30 insertions(+)

diff --git a/src/arch/x86/acpi.c b/src/arch/x86/acpi.c
index 3661297..3aee2a1 100644
--- a/src/arch/x86/acpi.c
+++ b/src/arch/x86/acpi.c
@@ -433,12 +433,31 @@ unsigned long acpi_create_dmar_drhd(unsigned long current, u8 flags,
 	return drhd->length;
 }
 
+unsigned long acpi_create_dmar_atsr(unsigned long current, u8 flags,
+	u16 segment)
+{
+	dmar_atsr_entry_t *atsr = (dmar_atsr_entry_t *)current;
+	memset(atsr, 0, sizeof(*atsr));
+	atsr->type = DMAR_ATSR;
+	atsr->length = sizeof(*atsr); /* will be fixed up later */
+	atsr->flags = flags;
+	atsr->segment = segment;
+
+	return atsr->length;
+}
+
 void acpi_dmar_drhd_fixup(unsigned long base, unsigned long current)
 {
 	dmar_entry_t *drhd = (dmar_entry_t *)base;
 	drhd->length = current - base;
 }
 
+void acpi_dmar_atsr_fixup(unsigned long base, unsigned long current)
+{
+	dmar_atsr_entry_t *atsr = (dmar_atsr_entry_t *)base;
+	atsr->length = current - base;
+}
+
 static unsigned long acpi_create_dmar_drhd_ds(unsigned long current,
 	enum dev_scope_type type, u8 enumeration_id, u8 bus, u8 dev, u8 fn)
 {
diff --git a/src/arch/x86/include/arch/acpi.h b/src/arch/x86/include/arch/acpi.h
index dfbffb3..b67213d 100644
--- a/src/arch/x86/include/arch/acpi.h
+++ b/src/arch/x86/include/arch/acpi.h
@@ -285,6 +285,14 @@ typedef struct dmar_entry {
 	u64 bar;
 } __attribute__ ((packed)) dmar_entry_t;
 
+typedef struct dmar_atsr_entry {
+	u16 type;
+	u16 length;
+	u8 flags;
+	u8 reserved;
+	u16 segment;
+} __attribute__ ((packed)) dmar_atsr_entry_t;
+
 /* DMAR (DMA Remapping Reporting Structure) */
 typedef struct acpi_dmar {
 	struct acpi_table_header header;
@@ -607,7 +615,10 @@ void acpi_create_dmar(acpi_dmar_t *dmar, enum dmar_flags flags,
 		      unsigned long (*acpi_fill_dmar) (unsigned long));
 unsigned long acpi_create_dmar_drhd(unsigned long current, u8 flags,
 				    u16 segment, u32 bar);
+unsigned long acpi_create_dmar_atsr(unsigned long current, u8 flags,
+				    u16 segment);
 void acpi_dmar_drhd_fixup(unsigned long base, unsigned long current);
+void acpi_dmar_atsr_fixup(unsigned long base, unsigned long current);
 unsigned long acpi_create_dmar_drhd_ds_pci(unsigned long current,
 					   u8 bus, u8 dev, u8 fn);
 unsigned long acpi_create_dmar_drhd_ds_ioapic(unsigned long current,



More information about the coreboot-gerrit mailing list