Commit f6019271 authored by Joerg Roedel's avatar Joerg Roedel

iommu/amd: Set global pointers to NULL after freeing them

Avoid any tries to double-free these pointers.
Signed-off-by: default avatarJoerg Roedel <jroedel@suse.de>
parent 151b0903
...@@ -2114,18 +2114,22 @@ static void __init free_iommu_resources(void) ...@@ -2114,18 +2114,22 @@ static void __init free_iommu_resources(void)
kmemleak_free(irq_lookup_table); kmemleak_free(irq_lookup_table);
free_pages((unsigned long)irq_lookup_table, free_pages((unsigned long)irq_lookup_table,
get_order(rlookup_table_size)); get_order(rlookup_table_size));
irq_lookup_table = NULL;
kmem_cache_destroy(amd_iommu_irq_cache); kmem_cache_destroy(amd_iommu_irq_cache);
amd_iommu_irq_cache = NULL; amd_iommu_irq_cache = NULL;
free_pages((unsigned long)amd_iommu_rlookup_table, free_pages((unsigned long)amd_iommu_rlookup_table,
get_order(rlookup_table_size)); get_order(rlookup_table_size));
amd_iommu_rlookup_table = NULL;
free_pages((unsigned long)amd_iommu_alias_table, free_pages((unsigned long)amd_iommu_alias_table,
get_order(alias_table_size)); get_order(alias_table_size));
amd_iommu_alias_table = NULL;
free_pages((unsigned long)amd_iommu_dev_table, free_pages((unsigned long)amd_iommu_dev_table,
get_order(dev_table_size)); get_order(dev_table_size));
amd_iommu_dev_table = NULL;
free_iommu_all(); free_iommu_all();
...@@ -2195,6 +2199,7 @@ static void __init free_dma_resources(void) ...@@ -2195,6 +2199,7 @@ static void __init free_dma_resources(void)
{ {
free_pages((unsigned long)amd_iommu_pd_alloc_bitmap, free_pages((unsigned long)amd_iommu_pd_alloc_bitmap,
get_order(MAX_DOMAIN_ID/8)); get_order(MAX_DOMAIN_ID/8));
amd_iommu_pd_alloc_bitmap = NULL;
free_unity_maps(); free_unity_maps();
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment