iommu/vt-d: Don't do early domain assignment if kdump kernel
authorJoerg Roedel <jroedel@suse.de>
Fri, 12 Jun 2015 10:32:54 +0000 (12:32 +0200)
committerJoerg Roedel <jroedel@suse.de>
Tue, 16 Jun 2015 08:59:35 +0000 (10:59 +0200)
When we copied over context tables from an old kernel, we
need to defer assignment of devices to domains until the
device driver takes over. So skip this part of
initialization when we copied over translation tables from
the old kernel.

Tested-by: ZhenHua Li <zhen-hual@hp.com>
Tested-by: Baoquan He <bhe@redhat.com>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/intel-iommu.c

index b43f049..3b32aa5 100644 (file)
@@ -3001,6 +3001,7 @@ static int __init init_dmars(void)
 {
        struct dmar_drhd_unit *drhd;
        struct dmar_rmrr_unit *rmrr;
+       bool copied_tables = false;
        struct device *dev;
        struct intel_iommu *iommu;
        int i, ret;
@@ -3091,6 +3092,7 @@ static int __init init_dmars(void)
                        } else {
                                pr_info("Copied translation tables from previous kernel for %s\n",
                                        iommu->name);
+                               copied_tables = true;
                        }
                }
 
@@ -3119,6 +3121,15 @@ static int __init init_dmars(void)
        check_tylersburg_isoch();
 
        /*
+        * If we copied translations from a previous kernel in the kdump
+        * case, we can not assign the devices to domains now, as that
+        * would eliminate the old mappings. So skip this part and defer
+        * the assignment to device driver initialization time.
+        */
+       if (copied_tables)
+               goto domains_done;
+
+       /*
         * If pass through is not set or not enabled, setup context entries for
         * identity mappings for rmrr, gfx, and isa and may fall back to static
         * identity mapping if iommu_identity_mapping is set.
@@ -3157,6 +3168,8 @@ static int __init init_dmars(void)
 
        iommu_prepare_isa();
 
+domains_done:
+
        /*
         * for each drhd
         *   enable fault log