9 files changed, 70 insertions, 41 deletions

diff --git a/drivers/iommu/Kconfig b/drivers/iommu/Kconfig
index fe302e33f72e..9fd51e51e78b 100644
--- a/drivers/iommu/Kconfig
+++ b/drivers/iommu/Kconfig
@@ -52,7 +52,7 @@ config AMD_IOMMU
         select PCI_PRI
         select PCI_PASID
         select IOMMU_API
-        depends on X86_64 && PCI && ACPI && X86_IO_APIC
+        depends on X86_64 && PCI && ACPI
         ---help---
           With this option you can enable support for AMD IOMMU hardware in
           your system. An IOMMU is a hardware component which provides
@@ -206,7 +206,7 @@ config SHMOBILE_IPMMU_TLB
 config SHMOBILE_IOMMU
         bool "IOMMU for Renesas IPMMU/IPMMUI"
         default n
-        depends on (ARM && ARCH_SHMOBILE)
+        depends on ARM || COMPILE_TEST
         select IOMMU_API
         select ARM_DMA_USE_IOMMU
         select SHMOBILE_IPMMU
diff --git a/drivers/iommu/Makefile b/drivers/iommu/Makefile
index 14c1f474cf11..5d58bf16e9e3 100644
--- a/drivers/iommu/Makefile
+++ b/drivers/iommu/Makefile
@@ -1,4 +1,5 @@
 obj-$(CONFIG_IOMMU_API) += iommu.o
+obj-$(CONFIG_IOMMU_API) += iommu-traces.o
 obj-$(CONFIG_OF_IOMMU)  += of_iommu.o
 obj-$(CONFIG_MSM_IOMMU) += msm_iommu.o msm_iommu_dev.o
 obj-$(CONFIG_AMD_IOMMU) += amd_iommu.o amd_iommu_init.o
diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c
index 785675a56a10..da2d0d926e40 100644
--- a/drivers/iommu/dmar.c
+++ b/drivers/iommu/dmar.c
@@ -403,7 +403,7 @@ dmar_find_matched_drhd_unit(struct pci_dev *dev)
 
         dev = pci_physfn(dev);
 
-        list_for_each_entry(dmaru, &dmar_drhd_units, list) {
+        for_each_drhd_unit(dmaru) {
                 drhd = container_of(dmaru->hdr,
                                     struct acpi_dmar_hardware_unit,
                                     header);
diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index 15e9b57e9cf0..43b9bfea48fa 100644
--- a/drivers/iommu/intel-iommu.c
+++ b/drivers/iommu/intel-iommu.c
@@ -782,7 +782,11 @@ static struct dma_pte *pfn_to_dma_pte(struct dmar_domain *domain,
         int offset;
 
         BUG_ON(!domain->pgd);
-        BUG_ON(addr_width < BITS_PER_LONG && pfn >> addr_width);
+
+        if (addr_width < BITS_PER_LONG && pfn >> addr_width)
+                /* Address beyond IOMMU's addressing capabilities. */
+                return NULL;
+
         parent = domain->pgd;
 
         while (level > 0) {
@@ -3777,11 +3781,10 @@ static void iommu_detach_dependent_devices(struct intel_iommu *iommu,
 static void domain_remove_one_dev_info(struct dmar_domain *domain,
                                           struct pci_dev *pdev)
 {
-        struct device_domain_info *info;
+        struct device_domain_info *info, *tmp;
         struct intel_iommu *iommu;
         unsigned long flags;
         int found = 0;
-        struct list_head *entry, *tmp;
 
         iommu = device_to_iommu(pci_domain_nr(pdev->bus), pdev->bus->number,
                                 pdev->devfn);
@@ -3789,8 +3792,7 @@ static void domain_remove_one_dev_info(struct dmar_domain *domain,
                 return;
 
         spin_lock_irqsave(&device_domain_lock, flags);
-        list_for_each_safe(entry, tmp, &domain->devices) {
-                info = list_entry(entry, struct device_domain_info, link);
+        list_for_each_entry_safe(info, tmp, &domain->devices, link) {
                 if (info->segment == pci_domain_nr(pdev->bus) &&
                     info->bus == pdev->bus->number &&
                     info->devfn == pdev->devfn) {
diff --git a/drivers/iommu/intel_irq_remapping.c b/drivers/iommu/intel_irq_remapping.c
index f71673dbb23d..b97d70b1abe0 100644
--- a/drivers/iommu/intel_irq_remapping.c
+++ b/drivers/iommu/intel_irq_remapping.c
@@ -525,12 +525,13 @@ static int __init intel_irq_remapping_supported(void)
         if (disable_irq_remap)
                 return 0;
         if (irq_remap_broken) {
-                WARN_TAINT(1, TAINT_FIRMWARE_WORKAROUND,
-                           "This system BIOS has enabled interrupt remapping\n"
-                           "on a chipset that contains an erratum making that\n"
-                           "feature unstable.  To maintain system stability\n"
-                           "interrupt remapping is being disabled.  Please\n"
-                           "contact your BIOS vendor for an update\n");
+                printk(KERN_WARNING
+                        "This system BIOS has enabled interrupt remapping\n"
+                        "on a chipset that contains an erratum making that\n"
+                        "feature unstable.  To maintain system stability\n"
+                        "interrupt remapping is being disabled.  Please\n"
+                        "contact your BIOS vendor for an update\n");
+                add_taint(TAINT_FIRMWARE_WORKAROUND, LOCKDEP_STILL_OK);
                 disable_irq_remap = 1;
                 return 0;
         }
diff --git a/drivers/iommu/iommu-traces.c b/drivers/iommu/iommu-traces.c
new file mode 100644
index 000000000000..bf3b317ff0c1
--- /dev/null
+++ b/drivers/iommu/iommu-traces.c
@@ -0,0 +1,27 @@
+/*
+ * iommu trace points
+ *
+ * Copyright (C) 2013 Shuah Khan <shuah.kh@samsung.com>
+ *
+ */
+
+#include <linux/string.h>
+#include <linux/types.h>
+
+#define CREATE_TRACE_POINTS
+#include <trace/events/iommu.h>
+
+/* iommu_group_event */
+EXPORT_TRACEPOINT_SYMBOL_GPL(add_device_to_group);
+EXPORT_TRACEPOINT_SYMBOL_GPL(remove_device_from_group);
+
+/* iommu_device_event */
+EXPORT_TRACEPOINT_SYMBOL_GPL(attach_device_to_domain);
+EXPORT_TRACEPOINT_SYMBOL_GPL(detach_device_from_domain);
+
+/* iommu_map_unmap */
+EXPORT_TRACEPOINT_SYMBOL_GPL(map);
+EXPORT_TRACEPOINT_SYMBOL_GPL(unmap);
+
+/* iommu_error */
+EXPORT_TRACEPOINT_SYMBOL_GPL(io_page_fault);
diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
index fbe9ca734f8f..e5555fcfe703 100644
--- a/drivers/iommu/iommu.c
+++ b/drivers/iommu/iommu.c
@@ -29,6 +29,7 @@
 #include <linux/idr.h>
 #include <linux/notifier.h>
 #include <linux/err.h>
+#include <trace/events/iommu.h>
 
 static struct kset *iommu_group_kset;
 static struct ida iommu_group_ida;
@@ -363,6 +364,8 @@ rename:
         /* Notify any listeners about change to group. */
         blocking_notifier_call_chain(&group->notifier,
                                      IOMMU_GROUP_NOTIFY_ADD_DEVICE, dev);
+
+        trace_add_device_to_group(group->id, dev);
         return 0;
 }
 EXPORT_SYMBOL_GPL(iommu_group_add_device);
@@ -399,6 +402,8 @@ void iommu_group_remove_device(struct device *dev)
         sysfs_remove_link(group->devices_kobj, device->name);
         sysfs_remove_link(&dev->kobj, "iommu_group");
 
+        trace_remove_device_from_group(group->id, dev);
+
         kfree(device->name);
         kfree(device);
         dev->iommu_group = NULL;
@@ -680,10 +685,14 @@ EXPORT_SYMBOL_GPL(iommu_domain_free);
 
 int iommu_attach_device(struct iommu_domain *domain, struct device *dev)
 {
+        int ret;
         if (unlikely(domain->ops->attach_dev == NULL))
                 return -ENODEV;
 
-        return domain->ops->attach_dev(domain, dev);
+        ret = domain->ops->attach_dev(domain, dev);
+        if (!ret)
+                trace_attach_device_to_domain(dev);
+        return ret;
 }
 EXPORT_SYMBOL_GPL(iommu_attach_device);
 
@@ -693,6 +702,7 @@ void iommu_detach_device(struct iommu_domain *domain, struct device *dev)
                 return;
 
         domain->ops->detach_dev(domain, dev);
+        trace_detach_device_from_domain(dev);
 }
 EXPORT_SYMBOL_GPL(iommu_detach_device);
 
@@ -807,17 +817,17 @@ int iommu_map(struct iommu_domain *domain, unsigned long iova,
          * size of the smallest page supported by the hardware
          */
         if (!IS_ALIGNED(iova | paddr | size, min_pagesz)) {
-                pr_err("unaligned: iova 0x%lx pa 0x%pa size 0x%zx min_pagesz 0x%x\n",
+                pr_err("unaligned: iova 0x%lx pa %pa size 0x%zx min_pagesz 0x%x\n",
                        iova, &paddr, size, min_pagesz);
                 return -EINVAL;
         }
 
-        pr_debug("map: iova 0x%lx pa 0x%pa size 0x%zx\n", iova, &paddr, size);
+        pr_debug("map: iova 0x%lx pa %pa size 0x%zx\n", iova, &paddr, size);
 
         while (size) {
                 size_t pgsize = iommu_pgsize(domain, iova | paddr, size);
 
-                pr_debug("mapping: iova 0x%lx pa 0x%pa pgsize 0x%zx\n",
+                pr_debug("mapping: iova 0x%lx pa %pa pgsize 0x%zx\n",
                          iova, &paddr, pgsize);
 
                 ret = domain->ops->map(domain, iova, paddr, pgsize, prot);
@@ -832,6 +842,8 @@ int iommu_map(struct iommu_domain *domain, unsigned long iova,
         /* unroll mapping in case something went wrong */
         if (ret)
                 iommu_unmap(domain, orig_iova, orig_size - size);
+        else
+                trace_map(iova, paddr, size);
 
         return ret;
 }
@@ -880,6 +892,7 @@ size_t iommu_unmap(struct iommu_domain *domain, unsigned long iova, size_t size)
                 unmapped += unmapped_page;
         }
 
+        trace_unmap(iova, 0, size);
         return unmapped;
 }
 EXPORT_SYMBOL_GPL(iommu_unmap);
diff --git a/drivers/iommu/tegra-gart.c b/drivers/iommu/tegra-gart.c
index 108c0e9c24d9..dba1a9fd5070 100644
--- a/drivers/iommu/tegra-gart.c
+++ b/drivers/iommu/tegra-gart.c
@@ -252,7 +252,7 @@ static int gart_iommu_map(struct iommu_domain *domain, unsigned long iova,
         spin_lock_irqsave(&gart->pte_lock, flags);
         pfn = __phys_to_pfn(pa);
         if (!pfn_valid(pfn)) {
-                dev_err(gart->dev, "Invalid page: %08x\n", pa);
+                dev_err(gart->dev, "Invalid page: %pa\n", &pa);
                 spin_unlock_irqrestore(&gart->pte_lock, flags);
                 return -EINVAL;
         }
@@ -295,8 +295,8 @@ static phys_addr_t gart_iommu_iova_to_phys(struct iommu_domain *domain,
 
         pa = (pte & GART_PAGE_MASK);
         if (!pfn_valid(__phys_to_pfn(pa))) {
-                dev_err(gart->dev, "No entry for %08llx:%08x\n",
-                         (unsigned long long)iova, pa);
+                dev_err(gart->dev, "No entry for %08llx:%pa\n",
+                         (unsigned long long)iova, &pa);
                 gart_dump_table(gart);
                 return -EINVAL;
         }
@@ -351,7 +351,6 @@ static int tegra_gart_probe(struct platform_device *pdev)
         struct gart_device *gart;
         struct resource *res, *res_remap;
         void __iomem *gart_regs;
-        int err;
         struct device *dev = &pdev->dev;
 
         if (gart_handle)
@@ -376,8 +375,7 @@ static int tegra_gart_probe(struct platform_device *pdev)
         gart_regs = devm_ioremap(dev, res->start, resource_size(res));
         if (!gart_regs) {
                 dev_err(dev, "failed to remap GART registers\n");
-                err = -ENXIO;
-                goto fail;
+                return -ENXIO;
         }
 
         gart->dev = &pdev->dev;
@@ -391,8 +389,7 @@ static int tegra_gart_probe(struct platform_device *pdev)
         gart->savedata = vmalloc(sizeof(u32) * gart->page_count);
         if (!gart->savedata) {
                 dev_err(dev, "failed to allocate context save area\n");
-                err = -ENOMEM;
-                goto fail;
+                return -ENOMEM;
         }
 
         platform_set_drvdata(pdev, gart);
@@ -401,32 +398,20 @@ static int tegra_gart_probe(struct platform_device *pdev)
         gart_handle = gart;
         bus_set_iommu(&platform_bus_type, &gart_iommu_ops);
         return 0;
-
-fail:
-        if (gart_regs)
-                devm_iounmap(dev, gart_regs);
-        if (gart && gart->savedata)
-                vfree(gart->savedata);
-        devm_kfree(dev, gart);
-        return err;
 }
 
 static int tegra_gart_remove(struct platform_device *pdev)
 {
         struct gart_device *gart = platform_get_drvdata(pdev);
-        struct device *dev = gart->dev;
 
         writel(0, gart->regs + GART_CONFIG);
         if (gart->savedata)
                 vfree(gart->savedata);
-        if (gart->regs)
-                devm_iounmap(dev, gart->regs);
-        devm_kfree(dev, gart);
         gart_handle = NULL;
         return 0;
 }
 
-const struct dev_pm_ops tegra_gart_pm_ops = {
+static const struct dev_pm_ops tegra_gart_pm_ops = {
         .suspend        = tegra_gart_suspend,
         .resume         = tegra_gart_resume,
 };
diff --git a/drivers/iommu/tegra-smmu.c b/drivers/iommu/tegra-smmu.c
index e0665603afd9..605b5b46a903 100644
--- a/drivers/iommu/tegra-smmu.c
+++ b/drivers/iommu/tegra-smmu.c
@@ -731,7 +731,7 @@ static int smmu_iommu_map(struct iommu_domain *domain, unsigned long iova,
         unsigned long pfn = __phys_to_pfn(pa);
         unsigned long flags;
 
-        dev_dbg(as->smmu->dev, "[%d] %08lx:%08x\n", as->asid, iova, pa);
+        dev_dbg(as->smmu->dev, "[%d] %08lx:%pa\n", as->asid, iova, &pa);
 
         if (!pfn_valid(pfn))
                 return -ENOMEM;
@@ -1254,7 +1254,7 @@ static int tegra_smmu_remove(struct platform_device *pdev)
         return 0;
 }
 
-const struct dev_pm_ops tegra_smmu_pm_ops = {
+static const struct dev_pm_ops tegra_smmu_pm_ops = {
         .suspend        = tegra_smmu_suspend,
         .resume         = tegra_smmu_resume,
 };