]> xenbits.xensource.com Git - people/dwmw2/xen.git/commitdiff
VT-d: change bogus return value of intel_iommu_lookup_page()
authorJan Beulich <jbeulich@suse.com>
Fri, 31 May 2019 09:39:49 +0000 (11:39 +0200)
committerJan Beulich <jbeulich@suse.com>
Fri, 31 May 2019 09:39:49 +0000 (11:39 +0200)
The function passes 0 as "alloc" argument to addr_to_dma_page_maddr(),
so -ENOMEM simply makes no sense (and its use was probably simply a
copy-and-paste effect originating at intel_iommu_map_page()).

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Acked-by: Kevin Tian <kevin.tian@intel.com>
xen/drivers/passthrough/vtd/iommu.c

index 7b9e09a084eaadb4ee82ac175365ddb3e1575ddd..4caf63c7fa4a01df6df07a68d9173be42ff1ecc7 100644 (file)
@@ -1870,7 +1870,7 @@ static int intel_iommu_lookup_page(struct domain *d, dfn_t dfn, mfn_t *mfn,
     if ( !pg_maddr )
     {
         spin_unlock(&hd->arch.mapping_lock);
-        return -ENOMEM;
+        return -ENOENT;
     }
 
     page = map_vtd_domain_page(pg_maddr);