During normal shutdown the iommu translation is disabled. Since the root table is preserved during live update, it needs to be cleaned up and the context entries of the unpreserved devices and root entries for the unpreserved context tables need to be cleared. Signed-off-by: Samiullah Khawaja --- drivers/iommu/intel/iommu.c | 15 +++- drivers/iommu/intel/iommu.h | 5 ++ drivers/iommu/intel/liveupdate.c | 140 +++++++++++++++++++++++++++++++ 3 files changed, 158 insertions(+), 2 deletions(-) diff --git a/drivers/iommu/intel/iommu.c b/drivers/iommu/intel/iommu.c index aeae0563dfd0..eca3944d9cf5 100644 --- a/drivers/iommu/intel/iommu.c +++ b/drivers/iommu/intel/iommu.c @@ -1832,6 +1832,14 @@ static int iommu_suspend(void *data) iommu_flush_all(); + /* + * Note that IOMMU suspend doesn't affect live update. The state + * preserved during live update is not released and remains valid during + * suspend and reused during IOMMU resume. + * + * Also note deployment of suspend/resume and live updated use case + * should be mostly mutually exclusive. + */ for_each_active_iommu(iommu, drhd) { iommu_disable_translation(iommu); @@ -2377,8 +2385,11 @@ void intel_iommu_shutdown(void) /* Disable PMRs explicitly here. */ iommu_disable_protect_mem_regions(iommu); - /* Make sure the IOMMUs are switched off */ - iommu_disable_translation(iommu); + /* Make sure the IOMMUs are switched off if not preserved. */ + if (iommu_preserved_state(&iommu->iommu)) + clear_unpreserved_context_entries(iommu); + else + iommu_disable_translation(iommu); } } diff --git a/drivers/iommu/intel/iommu.h b/drivers/iommu/intel/iommu.h index 4906cce1e66e..6c971f04ead3 100644 --- a/drivers/iommu/intel/iommu.h +++ b/drivers/iommu/intel/iommu.h @@ -1306,6 +1306,7 @@ int intel_iommu_preserve(struct iommu_device *iommu, struct iommu_hw_ser *iommu_ser); void intel_iommu_unpreserve(struct iommu_device *iommu, struct iommu_hw_ser *iommu_ser); +void clear_unpreserved_context_entries(struct intel_iommu *iommu); #else static inline int intel_iommu_preserve_device(struct device *dev, struct iommu_device_ser *device_ser) @@ -1328,6 +1329,10 @@ static inline void intel_iommu_unpreserve(struct iommu_device *iommu, struct iommu_hw_ser *iommu_ser) { } + +static inline void clear_unpreserved_context_entries(struct intel_iommu *iommu) +{ +} #endif #ifdef CONFIG_INTEL_IOMMU_SVM diff --git a/drivers/iommu/intel/liveupdate.c b/drivers/iommu/intel/liveupdate.c index ffdcebf2b773..b5aaebeeb5c1 100644 --- a/drivers/iommu/intel/liveupdate.c +++ b/drivers/iommu/intel/liveupdate.c @@ -77,6 +77,146 @@ static int preserve_context_table(struct intel_iommu *iommu, return 0; } +static void clear_unpreserved_context_root_entries(struct intel_iommu *iommu, + struct iommu_hw_ser *ser) +{ + struct root_entry *root; + int i; + + for (i = 0; i < ROOT_ENTRY_NR; i++) { + root = &iommu->root_entry[i]; + + if (!is_context_table_preserved(iommu, ser, i, 0) && (root->lo & 1)) { + root->lo = 0; + __iommu_flush_cache(iommu, + &root->lo, + sizeof(root->lo)); + } + + if (!sm_supported(iommu)) + continue; + + if (!is_context_table_preserved(iommu, ser, i, 0x80) && (root->hi & 1)) { + root->hi = 0; + __iommu_flush_cache(iommu, + &root->hi, + sizeof(root->hi)); + } + } +} + +static void clear_unpreserved_context(struct device_domain_info *info, u8 bus, u8 devfn) +{ + struct context_entry *context; + + /* + * This cleanup is done during shutdown, so it should be fine to only + * clear the entries here and issue one global invalidation later to + * invalidate all cleared entries. + * + * Note that the device IOTLB invalidation for unpreserved devices is + * skipped this way, but that should not be needed as the devices are + * quiesced at this point. This should improve the performance of the + * cleanup process and avoids any invalidation timeouts because drivers + * might have moved devices to D3 state. + */ + context = iommu_context_addr(info->iommu, bus, devfn, 0); + if (context) { + context_clear_entry(context); + __iommu_flush_cache(info->iommu, context, sizeof(*context)); + } +} + +static int clear_unpreserved_alias_cb(struct pci_dev *pdev, u16 alias, void *data) +{ + struct device_domain_info *info = data; + + clear_unpreserved_context(info, PCI_BUS_NUM(alias), alias & 0xff); + return 0; +} + +static int clear_unpreserve_context_entry_fn(struct device *dev, + struct iommu_device *iommu_dev, + void *arg) +{ + struct device_domain_info *info; + struct context_entry *context; + + info = dev_iommu_priv_get(dev); + if (!info) + return 0; + + if (!dev_is_pci(dev) || !dev_iommu_preserved_state(dev)) + goto out_unpreserved; + + /* + * PRE use cases are not supported with Live Update and a preservation + * attempt on such domains returns an error. But Intel IOMMU driver + * enables PRE by default on all devices that support it. For preserved + * entries, the PRE needs to be disabled so preserved PCI devices do not + * generate PRQs, during kexec, as translations are kept enabled during + * live update. There is no need to disable these for DMA aliases. + */ + if (sm_supported(info->iommu)) { + context = iommu_context_addr(info->iommu, info->bus, info->devfn, 0); + if (context) { + context_clear_sm_pre(context); + __iommu_flush_cache(info->iommu, context, sizeof(*context)); + } + } + + return 0; + +out_unpreserved: + if (dev_is_pci(dev)) + pci_for_each_dma_alias(to_pci_dev(dev), + clear_unpreserved_alias_cb, info); + else + clear_unpreserved_context(info, info->bus, info->devfn); + + return 0; +} + +/** + * clear_unpreserved_context_entries() - Clear context entries for unpreserved devices + * @iommu: Target IOMMU + * + * Clear the context entries of unpreserved devices during shutdown before kexec. + */ +void clear_unpreserved_context_entries(struct intel_iommu *iommu) +{ + struct iommu_dev_iter iter = { + .fn = clear_unpreserve_context_entry_fn, + .iommu = &iommu->iommu, + .arg = NULL, + + }; + + /* + * Clear context entries for unpreserved devices. + * + * Note that the error can be ignored as the iterator function does not + * fail. + */ + iommu_for_each_dev(&iter); + + /* Clear reference to unpreserved context tables */ + clear_unpreserved_context_root_entries(iommu, + iommu_preserved_state(&iommu->iommu)); + + /* + * Some devices might not have teardown/detached properly depending on + * whether a proper device remove is done before kexec is triggered. + * Also unpreserved context tables and entries are removed during + * shutdown. So issue global invalidations to remove references to + * unpreserved tables and entries. + */ + iommu->flush.flush_context(iommu, 0, 0, 0, DMA_CCMD_GLOBAL_INVL); + if (sm_supported(iommu)) + qi_flush_pasid_cache(iommu, 0, QI_PC_GLOBAL, 0); + iommu->flush.flush_iotlb(iommu, 0, 0, 0, DMA_TLB_GLOBAL_FLUSH); +} + static void unpreserve_iommu_context_tables(struct intel_iommu *iommu, struct iommu_hw_ser *ser) { -- 2.55.0.679.g6767b8d81c-goog