iommu/vt-d: Allocate space for ACPI devices
authorDavid Woodhouse <David.Woodhouse@intel.com>
Fri, 7 Mar 2014 14:39:27 +0000 (14:39 +0000)
committerDavid Woodhouse <David.Woodhouse@intel.com>
Thu, 20 Mar 2014 14:25:33 +0000 (14:25 +0000)
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
drivers/iommu/dmar.c

index eb95020c2314c78b0826382e0ac3e08aaeb2cf22..4c6297d1b421a7d87cf1021917afa212d2947261 100644 (file)
@@ -84,7 +84,8 @@ void *dmar_alloc_dev_scope(void *start, void *end, int *cnt)
        *cnt = 0;
        while (start < end) {
                scope = start;
-               if (scope->entry_type == ACPI_DMAR_SCOPE_TYPE_ENDPOINT ||
+               if (scope->entry_type == ACPI_DMAR_SCOPE_TYPE_ACPI ||
+                   scope->entry_type == ACPI_DMAR_SCOPE_TYPE_ENDPOINT ||
                    scope->entry_type == ACPI_DMAR_SCOPE_TYPE_BRIDGE)
                        (*cnt)++;
                else if (scope->entry_type != ACPI_DMAR_SCOPE_TYPE_IOAPIC &&
@@ -342,21 +343,18 @@ dmar_parse_one_drhd(struct acpi_dmar_header *header)
        dmaru->reg_base_addr = drhd->address;
        dmaru->segment = drhd->segment;
        dmaru->include_all = drhd->flags & 0x1; /* BIT0: INCLUDE_ALL */
-       if (!dmaru->include_all) {
-               dmaru->devices = dmar_alloc_dev_scope((void *)(drhd + 1),
-                                       ((void *)drhd) + drhd->header.length,
-                                       &dmaru->devices_cnt);
-               if (dmaru->devices_cnt && dmaru->devices == NULL) {
-                       kfree(dmaru);
-                       return -ENOMEM;
-               }
+       dmaru->devices = dmar_alloc_dev_scope((void *)(drhd + 1),
+                                             ((void *)drhd) + drhd->header.length,
+                                             &dmaru->devices_cnt);
+       if (dmaru->devices_cnt && dmaru->devices == NULL) {
+               kfree(dmaru);
+               return -ENOMEM;
        }
 
        ret = alloc_iommu(dmaru);
        if (ret) {
-               if (!dmaru->include_all)
-                       dmar_free_dev_scope(&dmaru->devices,
-                                           &dmaru->devices_cnt);
+               dmar_free_dev_scope(&dmaru->devices,
+                                   &dmaru->devices_cnt);
                kfree(dmaru);
                return ret;
        }