Most GFX drivers don't call standard PCI DMA APIs to allocate DMA buffer, Such drivers will be broken with IOMMU enabled. To workaround this issue, we added two options. Once graphics devices are converted over to use the DMA-API's this entire patch can be removed... a. intel_iommu=igfx_off. With this option, DMAR who has just gfx devices under it will be ignored. This mostly affect intergated gfx devices. If the DMAR is ignored, gfx device under it will get physical address for DMA. b. intel_iommu=gfx_workaround. With this option, we will setup 1:1 mapping for whole memory for gfx devices, that is physical address equals to virtual address.In this way, gfx will use physical address for DMA, this is primarily for add-in card GFX device. Signed-off-by: Ashok Raj Signed-off-by: Shaohua Li Index: 2.6.21-rc6/arch/x86_64/kernel/e820.c =================================================================== --- 2.6.21-rc6.orig/arch/x86_64/kernel/e820.c 2007-04-20 11:03:01.000000000 +0800 +++ 2.6.21-rc6/arch/x86_64/kernel/e820.c 2007-04-20 11:45:56.000000000 +0800 @@ -730,3 +730,22 @@ __init void e820_setup_gap(void) printk(KERN_INFO "Allocating PCI resources starting at %lx (gap: %lx:%lx)\n", pci_mem_start, gapstart, gapsize); } + +int __init arch_get_ram_range(int slot, u64 *addr, u64 *size) +{ + int i; + + if (slot < 0 || slot >= e820.nr_map) + return -1; + for (i = slot; i < e820.nr_map; i++) { + if(e820.map[i].type != E820_RAM) + continue; + break; + } + if (i == e820.nr_map || e820.map[i].addr > (max_pfn << PAGE_SHIFT)) + return -1; + *addr = e820.map[i].addr; + *size = min_t(u64, e820.map[i].size + e820.map[i].addr, + max_pfn << PAGE_SHIFT) - *addr; + return i + 1; +} Index: 2.6.21-rc6/drivers/pci/dmar.h =================================================================== --- 2.6.21-rc6.orig/drivers/pci/dmar.h 2007-04-20 11:38:30.000000000 +0800 +++ 2.6.21-rc6/drivers/pci/dmar.h 2007-04-20 11:45:56.000000000 +0800 @@ -35,6 +35,7 @@ struct dmar_drhd_unit { int devices_cnt; u8 include_all:1; struct iommu *iommu; + int ignored:1; /* the drhd should be ignored */ }; struct dmar_rmrr_unit { Index: 2.6.21-rc6/drivers/pci/intel-iommu.c =================================================================== --- 2.6.21-rc6.orig/drivers/pci/intel-iommu.c 2007-04-20 11:45:52.000000000 +0800 +++ 2.6.21-rc6/drivers/pci/intel-iommu.c 2007-04-20 11:45:56.000000000 +0800 @@ -36,6 +36,7 @@ #include "iova.h" #include "pci.h" +#define IS_GFX_DEVICE(pdev) ((pdev->class >> 16) == PCI_BASE_CLASS_DISPLAY) #define IOAPIC_RANGE_START (0xfee00000) #define IOAPIC_RANGE_END (0xfeefffff) #define IOAPIC_RANGE_SIZE (IOAPIC_RANGE_END - IOAPIC_RANGE_START + 1) @@ -85,6 +86,7 @@ struct iommu { }; static int dmar_disabled, dmar_force_rw; +static int dmar_map_gfx = 1, dmar_no_gfx_identity_map = 1; static char *get_fault_reason(u8 fault_reason) { @@ -105,7 +107,14 @@ static int __init intel_iommu_setup(char } else if (!strncmp(str, "forcerw", 7)) { dmar_force_rw = 1; printk(KERN_INFO"Intel-IOMMU: force R/W for W/O mapping\n"); + } else if (!strncmp(str, "igfx_off", 8)) { + dmar_map_gfx = 0; + printk(KERN_INFO"Intel-IOMMU: disable GFX device mapping\n"); + } else if (!strncmp(str, "gfx_workaround", 14)) { + dmar_no_gfx_identity_map = 0; + printk(KERN_INFO"Intel-IOMMU: do 1-1 mapping whole physical memory for GFX device\n"); } + str += strcspn(str, ","); while (*str == ',') str++; @@ -1318,6 +1327,7 @@ struct device_domain_info { struct pci_dev *dev; /* it's NULL for PCIE-to-PCI bridge */ struct domain *domain; }; +#define DUMMY_DEVICE_DOMAIN_INFO ((struct device_domain_info *)(-1)) static DEFINE_SPINLOCK(device_domain_lock); static LIST_HEAD(device_domain_list); @@ -1538,10 +1548,40 @@ error: static inline int iommu_prepare_rmrr_dev(struct dmar_rmrr_unit *rmrr, struct pci_dev *pdev) { + if (pdev->sysdata == DUMMY_DEVICE_DOMAIN_INFO) + return 0; return iommu_prepare_identity_map(pdev, rmrr->base_address, rmrr->end_address + 1); } +static void iommu_prepare_gfx_mapping(void) +{ + struct pci_dev *pdev = NULL; + u64 base, size; + int slot; + int ret; + + if (dmar_no_gfx_identity_map) + return; + + for_each_pci_dev(pdev) { + if (pdev->sysdata == DUMMY_DEVICE_DOMAIN_INFO || + !IS_GFX_DEVICE(pdev)) + continue; + printk(KERN_INFO "IOMMU: gfx device %s 1-1 mapping\n", + pci_name(pdev)); + slot = 0; + while ((slot = arch_get_ram_range(slot, &base, &size)) >= 0) { + ret = iommu_prepare_identity_map(pdev, base, base + size); + if (ret) + goto error; + } + continue; +error: + printk(KERN_ERR "IOMMU: mapping reserved region failed\n"); + } +} + int __init init_dmars(void) { struct dmar_drhd_unit *drhd; @@ -1557,6 +1597,8 @@ int __init init_dmars(void) * endfor */ for_each_drhd_unit(drhd) { + if (drhd->ignored) + continue; iommu = alloc_iommu(drhd); if (!iommu) { ret = -ENOMEM; @@ -1595,6 +1637,8 @@ int __init init_dmars(void) printk(KERN_ERR "IOMMU: mapping reserved region failed\n"); end_for_each_rmrr_device(rmrr, pdev) + iommu_prepare_gfx_mapping(); + /* * for each drhd * enable fault log @@ -1603,6 +1647,8 @@ int __init init_dmars(void) * enable translation */ for_each_drhd_unit(drhd) { + if (drhd->ignored) + continue; iommu = drhd->iommu; sprintf (iommu->name, "dmar%d", unit++); @@ -1625,6 +1671,8 @@ int __init init_dmars(void) return 0; error: for_each_drhd_unit(drhd) { + if (drhd->ignored) + continue; iommu = drhd->iommu; free_iommu(iommu); } @@ -1763,6 +1811,9 @@ static dma_addr_t intel_map_single(struc u64 flush_addr; unsigned int flush_size; + if (pdev->sysdata == DUMMY_DEVICE_DOMAIN_INFO) + return virt_to_bus(addr); + ret = __intel_map_single(dev, addr, size, dir, &flush_addr, &flush_size); if (ret) { domain = find_domain(pdev); @@ -1810,6 +1861,9 @@ static void intel_unmap_single(struct de u64 flush_addr; unsigned int flush_size; + if (pdev->sysdata == DUMMY_DEVICE_DOMAIN_INFO) + return; + domain = find_domain(pdev); __intel_unmap_single(dev, dev_addr, size, dir, &flush_addr, &flush_size); if (flush_size == 0) @@ -1833,6 +1887,7 @@ static void * intel_alloc_coherent(struc if (!ret) return NULL; memset(ret, 0, size); + *dma_handle = intel_map_single(hwdev, ret, size, DMA_BIDIRECTIONAL); if (*dma_handle) return ret; @@ -1861,6 +1916,9 @@ static void intel_unmap_sg(struct device u64 flush_addr; unsigned int flush_size; + if (pdev->sysdata == DUMMY_DEVICE_DOMAIN_INFO) + return; + domain = find_domain(pdev); for (i = 0; i < nelems; i++, sg++) __intel_unmap_single(hwdev, sg->dma_address, @@ -1871,6 +1929,20 @@ static void intel_unmap_sg(struct device } #define SG_ENT_VIRT_ADDRESS(sg) (page_address((sg)->page) + (sg)->offset) +static int intel_nontranslate_map_sg(struct device* hddev, + struct scatterlist *sg, int nelems, int dir) +{ + int i; + + for (i = 0; i < nelems; i++) { + struct scatterlist *s = &sg[i]; + BUG_ON(!s->page); + s->dma_address = virt_to_bus(SG_ENT_VIRT_ADDRESS(s)); + s->dma_length = s->length; + } + return nelems; +} + static int intel_map_sg(struct device *hwdev, struct scatterlist *sg, int nelems, int dir) { @@ -1883,6 +1955,8 @@ static int intel_map_sg(struct device *h unsigned int flush_size; BUG_ON(dir == DMA_NONE); + if (pdev->sysdata == DUMMY_DEVICE_DOMAIN_INFO) + return intel_nontranslate_map_sg(hwdev, sg, nelems, dir); for (i = 0; i < nelems; i++, sg++) { addr = SG_ENT_VIRT_ADDRESS(sg); @@ -1971,6 +2045,49 @@ void __init detect_intel_iommu(void) } } +static void __init init_no_remapping_devices(void) +{ + struct dmar_drhd_unit *drhd; + + for_each_drhd_unit(drhd) { + int i; + if (drhd->devices_cnt == 0 && !drhd->include_all) { + drhd->ignored = 1; + continue; + } + for (i = 0; i < drhd->devices_cnt; i++) + if (drhd->devices[i] != NULL) + break; + if (!drhd->include_all && i == drhd->devices_cnt) { + drhd->ignored = 1; + continue; + } + } + + if (dmar_map_gfx) + return; + + for_each_drhd_unit(drhd) { + int i; + if (drhd->ignored || drhd->include_all) + continue; + + for (i = 0; i < drhd->devices_cnt; i++) + if (drhd->devices[i] && !IS_GFX_DEVICE(drhd->devices[i])) + break; + if (i < drhd->devices_cnt) + continue; + + /* bypass IOMMU if it is just for gfx devices */ + drhd->ignored = 1; + for (i = 0; i < drhd->devices_cnt; i++) { + if (!drhd->devices[i]) + continue; + drhd->devices[i]->sysdata = DUMMY_DEVICE_DOMAIN_INFO; + } + } +} + int __init intel_iommu_init(void) { int ret = 0; @@ -1987,6 +2104,8 @@ int __init intel_iommu_init(void) iommu_init_mempool(); dmar_init_reserved_ranges(); + init_no_remapping_devices(); + ret = init_dmars(); if (ret) { printk(KERN_ERR "IOMMU: dmar init failed\n"); Index: 2.6.21-rc6/include/linux/intel-iommu.h =================================================================== --- 2.6.21-rc6.orig/include/linux/intel-iommu.h 2007-04-20 11:45:52.000000000 +0800 +++ 2.6.21-rc6/include/linux/intel-iommu.h 2007-04-20 11:45:56.000000000 +0800 @@ -263,6 +263,8 @@ extern void dmar_msi_read(int irq, struc extern void dmar_msi_write(int irq, struct msi_msg *msg); extern int arch_setup_dmar_msi(unsigned int irq); +extern int arch_get_ram_range(int slot, u64 *addr, u64 *size); + extern int init_dmars(void); extern void detect_intel_iommu(void); extern int early_dmar_detect(void); -- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/