Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1158326 > unrolled thread
| Started by | Daniel J Blueman <daniel@numascale.com> |
|---|---|
| First post | 2015-06-04 10:10 +0200 |
| Last post | 2015-06-04 10:20 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH -next] iommu: Fix build failure without INTEL_IOMMU Daniel J Blueman <daniel@numascale.com> - 2015-06-04 10:10 +0200
Re: [PATCH -next] iommu: Fix build failure without INTEL_IOMMU Jiang Liu <jiang.liu@linux.intel.com> - 2015-06-04 10:20 +0200
| From | Daniel J Blueman <daniel@numascale.com> |
|---|---|
| Date | 2015-06-04 10:10 +0200 |
| Subject | [PATCH -next] iommu: Fix build failure without INTEL_IOMMU |
| Message-ID | <pxynU-37f-9@gated-at.bofh.it> |
Fix Intel IOMMU build failure in linux-next when CONFIG_INTEL_IOMMU is not enabled.
Signed-off-by: Daniel J Blueman <daniel@numascale.com>
---
drivers/iommu/intel_irq_remapping.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/iommu/intel_irq_remapping.c b/drivers/iommu/intel_irq_remapping.c
index 24f7a35..ec337e7 100644
--- a/drivers/iommu/intel_irq_remapping.c
+++ b/drivers/iommu/intel_irq_remapping.c
@@ -146,8 +146,10 @@ static int modify_irte(struct irq_2_iommu *irq_iommu,
set_64bit(&irte->low, irte_modified->low);
set_64bit(&irte->high, irte_modified->high);
+#ifdef CONFIG_INTEL_IOMMU
if (iommu->pre_enabled_ir)
__iommu_update_old_irte(iommu, index);
+#endif
__iommu_flush_cache(iommu, irte, sizeof(*irte));
@@ -210,8 +212,10 @@ static int clear_entries(struct irq_2_iommu *irq_iommu)
bitmap_release_region(iommu->ir_table->bitmap, index,
irq_iommu->irte_mask);
+#ifdef CONFIG_INTEL_IOMMU
if (iommu->pre_enabled_ir)
__iommu_update_old_irte(iommu, -1);
+#endif
return qi_flush_iec(iommu, index, irq_iommu->irte_mask);
}
@@ -650,6 +654,7 @@ static int __init intel_enable_irq_remapping(void)
* Setup Interrupt-remapping for all the DRHD's now.
*/
for_each_iommu(iommu, drhd) {
+#ifdef CONFIG_INTEL_IOMMU
if (iommu->pre_enabled_ir) {
unsigned long long q;
@@ -660,6 +665,7 @@ static int __init intel_enable_irq_remapping(void)
INTR_REMAP_TABLE_ENTRIES*sizeof(struct irte));
__iommu_load_old_irte(iommu);
} else
+#endif
iommu_set_irq_remapping(iommu, eim);
setup = true;
@@ -1374,6 +1380,7 @@ static int __iommu_update_old_irte(struct intel_iommu *iommu, int index)
static void iommu_check_pre_ir_status(struct intel_iommu *iommu)
{
+#ifdef CONFIG_INTEL_IOMMU
u32 sts;
sts = readl(iommu->reg + DMAR_GSTS_REG);
@@ -1381,4 +1388,5 @@ static void iommu_check_pre_ir_status(struct intel_iommu *iommu)
pr_info("IR is enabled prior to OS.\n");
iommu->pre_enabled_ir = 1;
}
+#endif
}
--
2.1.4
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Jiang Liu <jiang.liu@linux.intel.com> |
|---|---|
| Date | 2015-06-04 10:20 +0200 |
| Message-ID | <pxyxA-3iT-17@gated-at.bofh.it> |
| In reply to | #1158326 |
Hi Daniel,
Seems there's already a patch to fix this issue posted by Joerg
http://www.spinics.net/lists/kernel/msg2004510.html
Thanks!
Gerry
On 2015/6/4 16:07, Daniel J Blueman wrote:
> Fix Intel IOMMU build failure in linux-next when CONFIG_INTEL_IOMMU is not enabled.
>
> Signed-off-by: Daniel J Blueman <daniel@numascale.com>
> ---
> drivers/iommu/intel_irq_remapping.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/iommu/intel_irq_remapping.c b/drivers/iommu/intel_irq_remapping.c
> index 24f7a35..ec337e7 100644
> --- a/drivers/iommu/intel_irq_remapping.c
> +++ b/drivers/iommu/intel_irq_remapping.c
> @@ -146,8 +146,10 @@ static int modify_irte(struct irq_2_iommu *irq_iommu,
> set_64bit(&irte->low, irte_modified->low);
> set_64bit(&irte->high, irte_modified->high);
>
> +#ifdef CONFIG_INTEL_IOMMU
> if (iommu->pre_enabled_ir)
> __iommu_update_old_irte(iommu, index);
> +#endif
>
> __iommu_flush_cache(iommu, irte, sizeof(*irte));
>
> @@ -210,8 +212,10 @@ static int clear_entries(struct irq_2_iommu *irq_iommu)
> bitmap_release_region(iommu->ir_table->bitmap, index,
> irq_iommu->irte_mask);
>
> +#ifdef CONFIG_INTEL_IOMMU
> if (iommu->pre_enabled_ir)
> __iommu_update_old_irte(iommu, -1);
> +#endif
>
> return qi_flush_iec(iommu, index, irq_iommu->irte_mask);
> }
> @@ -650,6 +654,7 @@ static int __init intel_enable_irq_remapping(void)
> * Setup Interrupt-remapping for all the DRHD's now.
> */
> for_each_iommu(iommu, drhd) {
> +#ifdef CONFIG_INTEL_IOMMU
> if (iommu->pre_enabled_ir) {
> unsigned long long q;
>
> @@ -660,6 +665,7 @@ static int __init intel_enable_irq_remapping(void)
> INTR_REMAP_TABLE_ENTRIES*sizeof(struct irte));
> __iommu_load_old_irte(iommu);
> } else
> +#endif
> iommu_set_irq_remapping(iommu, eim);
>
> setup = true;
> @@ -1374,6 +1380,7 @@ static int __iommu_update_old_irte(struct intel_iommu *iommu, int index)
>
> static void iommu_check_pre_ir_status(struct intel_iommu *iommu)
> {
> +#ifdef CONFIG_INTEL_IOMMU
> u32 sts;
>
> sts = readl(iommu->reg + DMAR_GSTS_REG);
> @@ -1381,4 +1388,5 @@ static void iommu_check_pre_ir_status(struct intel_iommu *iommu)
> pr_info("IR is enabled prior to OS.\n");
> iommu->pre_enabled_ir = 1;
> }
> +#endif
> }
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web