diff options
| author | Jan Beulich <jbeulich@suse.com> | 2013-03-25 16:55:22 +0100 |
|---|---|---|
| committer | Jan Beulich <jbeulich@suse.com> | 2013-03-25 16:55:22 +0100 |
| commit | 92b8bc03bd4b582cb524db51494d0dba7607e7ac (patch) | |
| tree | b9c4c82dddaa6ae8f46c051188e49691f76b3ac2 /xen/drivers/passthrough | |
| parent | 6890cebc6a987d0e896f5d23a8de11a3934101cf (diff) | |
| download | xen-92b8bc03bd4b582cb524db51494d0dba7607e7ac.tar.gz xen-92b8bc03bd4b582cb524db51494d0dba7607e7ac.tar.bz2 xen-92b8bc03bd4b582cb524db51494d0dba7607e7ac.zip | |
AMD IOMMU: allow disabling only interrupt remapping when certain IVRS consistency checks fail
After some more thought on the XSA-36 and specifically the comments we
got regarding disabling the IOMMU in this situation altogether making
things worse instead of better, I came to the conclusion that we can
actually restrict the action in affected cases to just disabling
interrupt remapping. That doesn't make the situation worse than prior
to the XSA-36 fixes (where interrupt remapping didn't really protect
domains from one another), but allows at least DMA isolation to still
be utilized.
To do so, disabling of interrupt remapping must be explicitly requested
on the command line - respective checks will then be skipped.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Suravee Suthikulanit <suravee.suthikulpanit@amd.com>
Diffstat (limited to 'xen/drivers/passthrough')
| -rw-r--r-- | xen/drivers/passthrough/amd/iommu_acpi.c | 4 | ||||
| -rw-r--r-- | xen/drivers/passthrough/amd/iommu_init.c | 5 |
2 files changed, 6 insertions, 3 deletions
diff --git a/xen/drivers/passthrough/amd/iommu_acpi.c b/xen/drivers/passthrough/amd/iommu_acpi.c index 8a6038cc86..6a2cbb6190 100644 --- a/xen/drivers/passthrough/amd/iommu_acpi.c +++ b/xen/drivers/passthrough/amd/iommu_acpi.c @@ -659,6 +659,8 @@ static u16 __init parse_ivhd_device_special( switch ( special->variety ) { case ACPI_IVHD_IOAPIC: + if ( !iommu_intremap ) + break; /* * Some BIOSes have IOAPIC broken entries so we check for IVRS * consistency here --- whether entry's IOAPIC ID is valid and @@ -921,7 +923,7 @@ static int __init parse_ivrs_table(struct acpi_table_header *table) } /* Each IO-APIC must have been mentioned in the table. */ - for ( apic = 0; !error && apic < nr_ioapics; ++apic ) + for ( apic = 0; !error && iommu_intremap && apic < nr_ioapics; ++apic ) { if ( !nr_ioapic_entries[apic] || ioapic_sbdf[IO_APIC_ID(apic)].pin_setup ) diff --git a/xen/drivers/passthrough/amd/iommu_init.c b/xen/drivers/passthrough/amd/iommu_init.c index e0986d85d1..73d9ce4666 100644 --- a/xen/drivers/passthrough/amd/iommu_init.c +++ b/xen/drivers/passthrough/amd/iommu_init.c @@ -1194,7 +1194,8 @@ int __init amd_iommu_init(void) BUG_ON( !iommu_found() ); - if ( amd_iommu_perdev_intremap && amd_sp5100_erratum28() ) + if ( iommu_intremap && amd_iommu_perdev_intremap && + amd_sp5100_erratum28() ) goto error_out; ivrs_bdf_entries = amd_iommu_get_ivrs_dev_entries(); @@ -1211,7 +1212,7 @@ int __init amd_iommu_init(void) goto error_out; /* initialize io-apic interrupt remapping entries */ - if ( amd_iommu_setup_ioapic_remapping() != 0 ) + if ( iommu_intremap && amd_iommu_setup_ioapic_remapping() != 0 ) goto error_out; /* allocate and initialize a global device table shared by all iommus */ |