|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [xen master] AMD/IOMMU: improve (extended) feature detection
commit 87caa1c3fb872c21297fee936f6247eb09cfe840
Author: Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Fri Oct 15 12:45:16 2021 +0200
Commit: Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Fri Oct 15 12:45:16 2021 +0200
AMD/IOMMU: improve (extended) feature detection
First of all the documentation is very clear about ACPI table data
superseding raw register data. Use raw register data only if EFRSup is
clear in the ACPI tables (which may still go too far). Additionally if
this flag is clear, the IVRS type 11H table is reserved and hence may
not be recognized.
Furthermore propagate IVRS type 10H data into the feature flags
recorded, as the full extended features field is available in type 11H
only.
Note that this also makes necessary to stop the bad practice of us
finding a type 11H IVHD entry, but still processing the type 10H one
in detect_iommu_acpi()'s invocation of amd_iommu_detect_one_acpi().
Note also that the features.raw check in amd_iommu_prepare_one() needs
replacing, now that the field can also be populated by different means.
Key IOMMUv2 availability off of IVHD type not being 10H, and then move
it a function layer up, so that it would be set only once all IOMMUs
have been successfully prepared.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Reviewed-by: Paul Durrant <paul@xxxxxxx>
---
xen/drivers/passthrough/amd/iommu.h | 1 +
xen/drivers/passthrough/amd/iommu_acpi.c | 8 +++--
xen/drivers/passthrough/amd/iommu_detect.c | 48 ++++++++++++++++++++++++++----
xen/drivers/passthrough/amd/iommu_init.c | 7 +++--
xen/include/acpi/actbl2.h | 3 ++
5 files changed, 56 insertions(+), 11 deletions(-)
diff --git a/xen/drivers/passthrough/amd/iommu.h
b/xen/drivers/passthrough/amd/iommu.h
index b0e6322906..3cec012103 100644
--- a/xen/drivers/passthrough/amd/iommu.h
+++ b/xen/drivers/passthrough/amd/iommu.h
@@ -306,6 +306,7 @@ extern struct hpet_sbdf {
} init;
} hpet_sbdf;
+extern unsigned int amd_iommu_acpi_info;
extern int amd_iommu_min_paging_mode;
extern void *shared_intremap_table;
diff --git a/xen/drivers/passthrough/amd/iommu_acpi.c
b/xen/drivers/passthrough/amd/iommu_acpi.c
index fdc601d498..acca2c685d 100644
--- a/xen/drivers/passthrough/amd/iommu_acpi.c
+++ b/xen/drivers/passthrough/amd/iommu_acpi.c
@@ -1062,7 +1062,8 @@ static unsigned int __initdata nr_ivmd;
static inline bool_t is_ivhd_block(u8 type)
{
return (type == ACPI_IVRS_TYPE_HARDWARE ||
- type == ACPI_IVRS_TYPE_HARDWARE_11H);
+ ((amd_iommu_acpi_info & ACPI_IVRS_EFR_SUP) &&
+ type == ACPI_IVRS_TYPE_HARDWARE_11H));
}
static inline bool_t is_ivmd_block(u8 type)
@@ -1176,7 +1177,7 @@ static int __init detect_iommu_acpi(struct
acpi_table_header *table)
ivrs_block = (struct acpi_ivrs_header *)((u8 *)table + length);
if ( table->length < (length + ivrs_block->length) )
return -ENODEV;
- if ( ivrs_block->type == ACPI_IVRS_TYPE_HARDWARE &&
+ if ( ivrs_block->type == ivhd_type &&
amd_iommu_detect_one_acpi(to_ivhd_block(ivrs_block)) != 0 )
return -ENODEV;
length += ivrs_block->length;
@@ -1316,6 +1317,9 @@ get_supported_ivhd_type(struct acpi_table_header *table)
return -ENODEV;
}
+ amd_iommu_acpi_info = container_of(table, const struct acpi_table_ivrs,
+ header)->info;
+
while ( table->length > (length + sizeof(*ivrs_block)) )
{
ivrs_block = (struct acpi_ivrs_header *)((u8 *)table + length);
diff --git a/xen/drivers/passthrough/amd/iommu_detect.c
b/xen/drivers/passthrough/amd/iommu_detect.c
index d05bc6a5bb..b017575325 100644
--- a/xen/drivers/passthrough/amd/iommu_detect.c
+++ b/xen/drivers/passthrough/amd/iommu_detect.c
@@ -60,14 +60,14 @@ void __init get_iommu_features(struct amd_iommu *iommu)
const struct amd_iommu *first;
ASSERT( iommu->mmio_base );
- if ( !iommu_has_cap(iommu, PCI_CAP_EFRSUP_SHIFT) )
+ if ( !(amd_iommu_acpi_info & ACPI_IVRS_EFR_SUP) )
{
- iommu->features.raw = 0;
- return;
- }
+ if ( !iommu_has_cap(iommu, PCI_CAP_EFRSUP_SHIFT) )
+ return;
- iommu->features.raw =
- readq(iommu->mmio_base + IOMMU_EXT_FEATURE_MMIO_OFFSET);
+ iommu->features.raw =
+ readq(iommu->mmio_base + IOMMU_EXT_FEATURE_MMIO_OFFSET);
+ }
/* Don't log the same set of features over and over. */
first = list_first_entry(&amd_iommu_head, struct amd_iommu, list);
@@ -164,6 +164,42 @@ int __init amd_iommu_detect_one_acpi(
iommu->cap_offset = ivhd_block->capability_offset;
iommu->mmio_base_phys = ivhd_block->base_address;
+ if ( ivhd_type != ACPI_IVRS_TYPE_HARDWARE )
+ iommu->features.raw = ivhd_block->efr_image;
+ else if ( amd_iommu_acpi_info & ACPI_IVRS_EFR_SUP )
+ {
+ union {
+ uint32_t raw;
+ struct {
+ unsigned int xt_sup:1;
+ unsigned int nx_sup:1;
+ unsigned int gt_sup:1;
+ unsigned int glx_sup:2;
+ unsigned int ia_sup:1;
+ unsigned int ga_sup:1;
+ unsigned int he_sup:1;
+ unsigned int pas_max:5;
+ unsigned int pn_counters:4;
+ unsigned int pn_banks:6;
+ unsigned int msi_num_ppr:5;
+ unsigned int gats:2;
+ unsigned int hats:2;
+ };
+ } attr = { .raw = ivhd_block->iommu_attr };
+
+ iommu->features.flds.xt_sup = attr.xt_sup;
+ iommu->features.flds.nx_sup = attr.nx_sup;
+ iommu->features.flds.gt_sup = attr.gt_sup;
+ iommu->features.flds.glx_sup = attr.glx_sup;
+ iommu->features.flds.ia_sup = attr.ia_sup;
+ iommu->features.flds.ga_sup = attr.ga_sup;
+ iommu->features.flds.pas_max = attr.pas_max;
+ iommu->features.flds.gats = attr.gats;
+ iommu->features.flds.hats = attr.hats;
+ }
+ else if ( list_empty(&amd_iommu_head) )
+ AMD_IOMMU_DEBUG("EFRSup not set in ACPI table; will fall back to
hardware\n");
+
/* override IOMMU HT flags */
iommu->ht_flags = ivhd_block->header.flags;
diff --git a/xen/drivers/passthrough/amd/iommu_init.c
b/xen/drivers/passthrough/amd/iommu_init.c
index 7eb5013756..a4f152c150 100644
--- a/xen/drivers/passthrough/amd/iommu_init.c
+++ b/xen/drivers/passthrough/amd/iommu_init.c
@@ -29,6 +29,7 @@ static bool __initdata pci_init;
static void do_amd_iommu_irq(void *data);
static DECLARE_SOFTIRQ_TASKLET(amd_iommu_irq_tasklet, do_amd_iommu_irq, NULL);
+unsigned int __read_mostly amd_iommu_acpi_info;
unsigned int __read_mostly ivrs_bdf_entries;
u8 __read_mostly ivhd_type;
static struct radix_tree_root ivrs_maps;
@@ -1375,9 +1376,6 @@ static int __init amd_iommu_prepare_one(struct amd_iommu
*iommu)
get_iommu_features(iommu);
- if ( iommu->features.raw )
- iommuv2_enabled = true;
-
return 0;
}
@@ -1419,6 +1417,9 @@ int __init amd_iommu_prepare(bool xt)
has_xt = false;
}
+ if ( ivhd_type != ACPI_IVRS_TYPE_HARDWARE )
+ iommuv2_enabled = true;
+
for_each_amd_iommu ( iommu )
{
/* NB: There's no need to actually write these out right here. */
diff --git a/xen/include/acpi/actbl2.h b/xen/include/acpi/actbl2.h
index 42beac4e26..4252c90a3f 100644
--- a/xen/include/acpi/actbl2.h
+++ b/xen/include/acpi/actbl2.h
@@ -716,6 +716,9 @@ struct acpi_table_ivrs {
/* Values for Info field above */
+#define ACPI_IVRS_EFR_SUP 0x00000001 /* extended feature support */
+#define ACPI_IVRS_PREBOOT_DMA_REMAP 0x00000002 /* pre-boot DMA remapping in
use */
+#define ACPI_IVRS_GVA_SIZE 0x000000E0 /* 3 bits, guest VA size */
#define ACPI_IVRS_PHYSICAL_SIZE 0x00007F00 /* 7 bits, physical address
size */
#define ACPI_IVRS_VIRTUAL_SIZE 0x003F8000 /* 7 bits, virtual address size
*/
#define ACPI_IVRS_ATS_RESERVED 0x00400000 /* ATS address translation
range reserved */
--
generated by git-patchbot for /home/xen/git/xen.git#master
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |