diff options
| author | Joerg Roedel <joerg.roedel@amd.com> | 2008-12-17 16:36:44 +0100 |
|---|---|---|
| committer | Joerg Roedel <joerg.roedel@amd.com> | 2008-12-17 16:36:44 +0100 |
| commit | 84df81759590ad16b0024cf46b3423cca76b2e07 (patch) | |
| tree | ad2e21daa3960753dbb3a8c6793d0f6242c063b5 | |
| parent | AMD IOMMU: set cmd buffer pointers to zero manually (diff) | |
| download | linux-84df81759590ad16b0024cf46b3423cca76b2e07.tar.xz linux-84df81759590ad16b0024cf46b3423cca76b2e07.zip | |
AMD IOMMU: panic if completion wait loop fails
Impact: prevents data corruption after a failed completion wait loop
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
| -rw-r--r-- | arch/x86/kernel/amd_iommu.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/arch/x86/kernel/amd_iommu.c b/arch/x86/kernel/amd_iommu.c index a7b6dec6fc3f..0a60d60ed036 100644 --- a/arch/x86/kernel/amd_iommu.c +++ b/arch/x86/kernel/amd_iommu.c @@ -235,8 +235,9 @@ static int iommu_completion_wait(struct amd_iommu *iommu) status &= ~MMIO_STATUS_COM_WAIT_INT_MASK; writel(status, iommu->mmio_base + MMIO_STATUS_OFFSET); - if (unlikely((i == EXIT_LOOP_COUNT) && printk_ratelimit())) - printk(KERN_WARNING "AMD IOMMU: Completion wait loop failed\n"); + if (unlikely(i == EXIT_LOOP_COUNT)) + panic("AMD IOMMU: Completion wait loop failed\n"); + out: spin_unlock_irqrestore(&iommu->lock, flags); |