[PATCH 17/17] iommu/vt-d: Remove update code for old ir-table

From: Joerg Roedel
Date: Fri Jun 05 2015 - 10:16:51 EST


From: Joerg Roedel <jroedel@xxxxxxx>

The code to update the irq remapping table of the old kernel
is not necessary anymore because we switch immidiatly to the
new table after we copied the data over.

Tested-by: Baoquan He <bhe@xxxxxxxxxx>
Signed-off-by: Joerg Roedel <jroedel@xxxxxxx>
---
drivers/iommu/intel_irq_remapping.c | 44 -------------------------------------
include/linux/intel-iommu.h | 2 --
2 files changed, 46 deletions(-)

diff --git a/drivers/iommu/intel_irq_remapping.c b/drivers/iommu/intel_irq_remapping.c
index 07a8e82..b49aa8c 100644
--- a/drivers/iommu/intel_irq_remapping.c
+++ b/drivers/iommu/intel_irq_remapping.c
@@ -22,7 +22,6 @@
#include "irq_remapping.h"

static int iommu_load_old_irte(struct intel_iommu *iommu);
-static int __iommu_update_old_irte(struct intel_iommu *iommu, int index);
static void iommu_check_pre_ir_status(struct intel_iommu *iommu);
static void iommu_disable_irq_remapping(struct intel_iommu *iommu);

@@ -202,9 +201,6 @@ static int modify_irte(int irq, struct irte *irte_modified)
set_64bit(&irte->low, irte_modified->low);
set_64bit(&irte->high, irte_modified->high);

- if (iommu->pre_enabled_ir)
- __iommu_update_old_irte(iommu, index);
-
__iommu_flush_cache(iommu, irte, sizeof(*irte));

rc = qi_flush_iec(iommu, index, 0);
@@ -266,9 +262,6 @@ static int clear_entries(struct irq_2_iommu *irq_iommu)
bitmap_release_region(iommu->ir_table->bitmap, index,
irq_iommu->irte_mask);

- if (iommu->pre_enabled_ir)
- __iommu_update_old_irte(iommu, -1);
-
return qi_flush_iec(iommu, index, irq_iommu->irte_mask);
}

@@ -1327,9 +1320,6 @@ static int iommu_load_old_irte(struct intel_iommu *iommu)

__iommu_flush_cache(iommu, iommu->ir_table->base, size);

- iommu->ir_table->base_old_phys = irt_phys;
- iommu->ir_table->base_old_virt = old_ir_table;
-
/*
* Now check the table for used entries and mark those as
* allocated in the bitmap
@@ -1342,40 +1332,6 @@ static int iommu_load_old_irte(struct intel_iommu *iommu)
return 0;
}

-static int __iommu_update_old_irte(struct intel_iommu *iommu, int index)
-{
- int start;
- unsigned long size;
- void __iomem *to;
-void *from;
-
- if ((!iommu)
- || (!iommu->ir_table)
- || (!iommu->ir_table->base)
- || (!iommu->ir_table->base_old_phys)
- || (!iommu->ir_table->base_old_virt))
- return -1;
-
- if (index < -1 || index >= INTR_REMAP_TABLE_ENTRIES)
- return -1;
-
- if (index == -1) {
- start = 0;
- size = INTR_REMAP_TABLE_ENTRIES * sizeof(struct irte);
- } else {
- start = index * sizeof(struct irte);
- size = sizeof(struct irte);
- }
-
- to = iommu->ir_table->base_old_virt;
- from = iommu->ir_table->base;
- memcpy(to + start, from + start, size);
-
- __iommu_flush_cache(iommu, to + start, size);
-
- return 0;
-}
-
static void iommu_check_pre_ir_status(struct intel_iommu *iommu)
{
u32 sts;
diff --git a/include/linux/intel-iommu.h b/include/linux/intel-iommu.h
index 5aa8834..9fcdd9a 100644
--- a/include/linux/intel-iommu.h
+++ b/include/linux/intel-iommu.h
@@ -302,8 +302,6 @@ struct q_inval {
struct ir_table {
struct irte *base;
unsigned long *bitmap;
- void __iomem *base_old_virt;
- unsigned long base_old_phys;
};
#endif

--
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/