aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Herrmann <[email protected]>2008-10-16 16:27:36 +0200
committerDavid Woodhouse <[email protected]>2008-10-18 14:29:30 +0100
commitf609891f428e1c20e270e7c350daf8c93cc459d7 (patch)
treec08fe134b55bdb13b7d0f4c4105cd683037d7cfb
parent5b6985ce8ec7127b4d60ad450b64ca8b82748a3b (diff)
amd_iommu: fix nasty bug that caused ILLEGAL_DEVICE_TABLE_ENTRY errors
We are on 64-bit so better use u64 instead of u32 to deal with addresses: static void __init iommu_set_device_table(struct amd_iommu *iommu) { u64 entry; ... entry = virt_to_phys(amd_iommu_dev_table); ... (I am wondering why gcc 4.2.x did not warn about the assignment between u32 and unsigned long.) Cc: [email protected] Cc: [email protected] Signed-off-by: Andreas Herrmann <[email protected]> Signed-off-by: Joerg Roedel <[email protected]> Signed-off-by: David Woodhouse <[email protected]>
-rw-r--r--arch/x86/kernel/amd_iommu_init.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/kernel/amd_iommu_init.c b/arch/x86/kernel/amd_iommu_init.c
index 4cd8083c58be..0cdcda35a05f 100644
--- a/arch/x86/kernel/amd_iommu_init.c
+++ b/arch/x86/kernel/amd_iommu_init.c
@@ -212,7 +212,7 @@ static void __init iommu_set_exclusion_range(struct amd_iommu *iommu)
/* Programs the physical address of the device table into the IOMMU hardware */
static void __init iommu_set_device_table(struct amd_iommu *iommu)
{
- u32 entry;
+ u64 entry;
BUG_ON(iommu->mmio_base == NULL);