Prev: [PATCH] ubi: init even if mtd device cannot be attached, if built into kernel
Next: [PATCH 8/8] power_supply: Block suspend while power supply change notifications are pending
From: Andrew Morton on 30 Apr 2010 18:30
On Mon, 19 Apr 2010 14:37:46 -0700
"Tom Lyon" <pugs(a)cisco.com> wrote:

>
> Bug: intel_iommu_map_range didn't allow allocation at the very
> end of the address space.
> Signed-off-by: Tom Lyon <pugs(a)cisco.com>
> ---
> V1 and V2 of this patch were patch of a larger patch, this is now independent.
>
> --- linux-2.6.33/drivers/pci/intel-iommu.c 2010-02-24 10:52:17.000000000 -0800
> +++ mylinux-2.6.33/drivers/pci/intel-iommu.c 2010-04-13 16:51:55.000000000 -0700
> @@ -3632,7 +3631,6 @@ static int intel_iommu_map_range(struct
> {
> struct dmar_domain *dmar_domain = domain->priv;
> u64 max_addr;
> - int addr_width;
> int prot = 0;
> int ret;
>
> @@ -3645,18 +3643,14 @@ static int intel_iommu_map_range(struct
>
> max_addr = iova + size;
> if (dmar_domain->max_addr < max_addr) {
> - int min_agaw;
> u64 end;
>
> /* check if minimum agaw is sufficient for mapped address */
> - min_agaw = vm_domain_min_agaw(dmar_domain);
> - addr_width = agaw_to_width(min_agaw);
> - end = DOMAIN_MAX_ADDR(addr_width);
> - end = end & VTD_PAGE_MASK;
> + end = __DOMAIN_MAX_ADDR(dmar_domain->gaw) + 1;
> if (end < max_addr) {
> - printk(KERN_ERR "%s: iommu agaw (%d) is not "
> + printk(KERN_ERR "%s: iommu width (%d) is not "
> "sufficient for the mapped address (%llx)\n",
> - __func__, min_agaw, max_addr);
> + __func__, dmar_domain->gaw, max_addr);
> return -EFAULT;
> }
> dmar_domain->max_addr = max_addr;

OK, I split my copy of the patches up. I also added your Signed-off-by:
to my copy of this patch.

David? Jesse? Anyone?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: [PATCH] ubi: init even if mtd device cannot be attached, if built into kernel
Next: [PATCH 8/8] power_supply: Block suspend while power supply change notifications are pending