Re: [PATCH v2 17/25] intel-iommu: switch from ioremap_cache to memremap

2015-08-03 Thread Joerg Roedel
On Fri, Jul 24, 2015 at 10:39:33PM -0400, Dan Williams wrote:
> In preparation for deprecating ioremap_cache() convert its usage in
> intel-iommu to memremap.  This also eliminates the mishandling of the
> __iomem annotation in the implementation.
> 
> Cc: David Woodhouse 
> Cc: Joerg Roedel 
> Signed-off-by: Dan Williams 
> ---
>  drivers/iommu/intel-iommu.c |   10 ++
>  drivers/iommu/intel_irq_remapping.c |4 ++--
>  2 files changed, 8 insertions(+), 6 deletions(-)

Acked-by: Joerg Roedel 

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


Re: [PATCH v2 17/25] intel-iommu: switch from ioremap_cache to memremap

2015-08-03 Thread Joerg Roedel
On Fri, Jul 24, 2015 at 10:39:33PM -0400, Dan Williams wrote:
 In preparation for deprecating ioremap_cache() convert its usage in
 intel-iommu to memremap.  This also eliminates the mishandling of the
 __iomem annotation in the implementation.
 
 Cc: David Woodhouse dw...@infradead.org
 Cc: Joerg Roedel j...@8bytes.org
 Signed-off-by: Dan Williams dan.j.willi...@intel.com
 ---
  drivers/iommu/intel-iommu.c |   10 ++
  drivers/iommu/intel_irq_remapping.c |4 ++--
  2 files changed, 8 insertions(+), 6 deletions(-)

Acked-by: Joerg Roedel jroe...@suse.de

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


[PATCH v2 17/25] intel-iommu: switch from ioremap_cache to memremap

2015-07-24 Thread Dan Williams
In preparation for deprecating ioremap_cache() convert its usage in
intel-iommu to memremap.  This also eliminates the mishandling of the
__iomem annotation in the implementation.

Cc: David Woodhouse 
Cc: Joerg Roedel 
Signed-off-by: Dan Williams 
---
 drivers/iommu/intel-iommu.c |   10 ++
 drivers/iommu/intel_irq_remapping.c |4 ++--
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index a98a7b27aca1..028ad0ff6abb 100644
--- a/drivers/iommu/intel-iommu.c
+++ b/drivers/iommu/intel-iommu.c
@@ -34,6 +34,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -2871,7 +2872,8 @@ static int copy_context_table(struct intel_iommu *iommu,
}
 
ret = -ENOMEM;
-   old_ce = ioremap_cache(old_ce_phys, PAGE_SIZE);
+   old_ce = memremap(old_ce_phys, PAGE_SIZE,
+   MEMREMAP_CACHE);
if (!old_ce)
goto out;
 
@@ -2919,7 +2921,7 @@ static int copy_context_table(struct intel_iommu *iommu,
__iommu_flush_cache(iommu, new_ce, VTD_PAGE_SIZE);
 
 out_unmap:
-   iounmap(old_ce);
+   memunmap(old_ce);
 
 out:
return ret;
@@ -2953,7 +2955,7 @@ static int copy_translation_tables(struct intel_iommu 
*iommu)
if (!old_rt_phys)
return -EINVAL;
 
-   old_rt = ioremap_cache(old_rt_phys, PAGE_SIZE);
+   old_rt = memremap(old_rt_phys, PAGE_SIZE, MEMREMAP_CACHE);
if (!old_rt)
return -ENOMEM;
 
@@ -3002,7 +3004,7 @@ static int copy_translation_tables(struct intel_iommu 
*iommu)
ret = 0;
 
 out_unmap:
-   iounmap(old_rt);
+   memunmap(old_rt);
 
return ret;
 }
diff --git a/drivers/iommu/intel_irq_remapping.c 
b/drivers/iommu/intel_irq_remapping.c
index 27cdfa84ec5b..1da158d3b3ce 100644
--- a/drivers/iommu/intel_irq_remapping.c
+++ b/drivers/iommu/intel_irq_remapping.c
@@ -408,7 +408,7 @@ static int iommu_load_old_irte(struct intel_iommu *iommu)
size = INTR_REMAP_TABLE_ENTRIES*sizeof(struct irte);
 
/* Map the old IR table */
-   old_ir_table = ioremap_cache(irt_phys, size);
+   old_ir_table = memremap(irt_phys, size, MEMREMAP_CACHE);
if (!old_ir_table)
return -ENOMEM;
 
@@ -426,7 +426,7 @@ static int iommu_load_old_irte(struct intel_iommu *iommu)
bitmap_set(iommu->ir_table->bitmap, i, 1);
}
 
-   iounmap(old_ir_table);
+   memunmap(old_ir_table);
 
return 0;
 }

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


[PATCH v2 17/25] intel-iommu: switch from ioremap_cache to memremap

2015-07-24 Thread Dan Williams
In preparation for deprecating ioremap_cache() convert its usage in
intel-iommu to memremap.  This also eliminates the mishandling of the
__iomem annotation in the implementation.

Cc: David Woodhouse dw...@infradead.org
Cc: Joerg Roedel j...@8bytes.org
Signed-off-by: Dan Williams dan.j.willi...@intel.com
---
 drivers/iommu/intel-iommu.c |   10 ++
 drivers/iommu/intel_irq_remapping.c |4 ++--
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index a98a7b27aca1..028ad0ff6abb 100644
--- a/drivers/iommu/intel-iommu.c
+++ b/drivers/iommu/intel-iommu.c
@@ -34,6 +34,7 @@
 #include linux/mempool.h
 #include linux/memory.h
 #include linux/timer.h
+#include linux/io.h
 #include linux/iova.h
 #include linux/iommu.h
 #include linux/intel-iommu.h
@@ -2871,7 +2872,8 @@ static int copy_context_table(struct intel_iommu *iommu,
}
 
ret = -ENOMEM;
-   old_ce = ioremap_cache(old_ce_phys, PAGE_SIZE);
+   old_ce = memremap(old_ce_phys, PAGE_SIZE,
+   MEMREMAP_CACHE);
if (!old_ce)
goto out;
 
@@ -2919,7 +2921,7 @@ static int copy_context_table(struct intel_iommu *iommu,
__iommu_flush_cache(iommu, new_ce, VTD_PAGE_SIZE);
 
 out_unmap:
-   iounmap(old_ce);
+   memunmap(old_ce);
 
 out:
return ret;
@@ -2953,7 +2955,7 @@ static int copy_translation_tables(struct intel_iommu 
*iommu)
if (!old_rt_phys)
return -EINVAL;
 
-   old_rt = ioremap_cache(old_rt_phys, PAGE_SIZE);
+   old_rt = memremap(old_rt_phys, PAGE_SIZE, MEMREMAP_CACHE);
if (!old_rt)
return -ENOMEM;
 
@@ -3002,7 +3004,7 @@ static int copy_translation_tables(struct intel_iommu 
*iommu)
ret = 0;
 
 out_unmap:
-   iounmap(old_rt);
+   memunmap(old_rt);
 
return ret;
 }
diff --git a/drivers/iommu/intel_irq_remapping.c 
b/drivers/iommu/intel_irq_remapping.c
index 27cdfa84ec5b..1da158d3b3ce 100644
--- a/drivers/iommu/intel_irq_remapping.c
+++ b/drivers/iommu/intel_irq_remapping.c
@@ -408,7 +408,7 @@ static int iommu_load_old_irte(struct intel_iommu *iommu)
size = INTR_REMAP_TABLE_ENTRIES*sizeof(struct irte);
 
/* Map the old IR table */
-   old_ir_table = ioremap_cache(irt_phys, size);
+   old_ir_table = memremap(irt_phys, size, MEMREMAP_CACHE);
if (!old_ir_table)
return -ENOMEM;
 
@@ -426,7 +426,7 @@ static int iommu_load_old_irte(struct intel_iommu *iommu)
bitmap_set(iommu-ir_table-bitmap, i, 1);
}
 
-   iounmap(old_ir_table);
+   memunmap(old_ir_table);
 
return 0;
 }

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