powerpc/pseries/iommu: Use correct vfree for it_map

aik · mpe · commit 41ee7232fa5f · 2021-10-25T11:41:14.000+11:00
The it_map array is vzalloc'ed so use vfree() for it when creating a huge DMA window failed for whatever reason. While at this, write zero to it_map. Fixes: 381ceda ("powerpc/pseries/iommu: Make use of DDW for indirect mapping") Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> Link: https://lore.kernel.org/r/20211020132315.2287178-3-aik@ozlabs.ru
diff --git a/arch/powerpc/platforms/pseries/iommu.c b/arch/powerpc/platforms/pseries/iommu.c
@@ -1440,7 +1440,8 @@ static bool enable_ddw(struct pci_dev *dev, struct device_node *pdn)
 		/* Keep default DMA window stuct if removed */
 		if (default_win_removed) {
 			tbl->it_size = 0;
-			kfree(tbl->it_map);
+			vfree(tbl->it_map);
+			tbl->it_map = NULL;
 		}
 
 		set_iommu_table_base(&dev->dev, newtbl);

Original file line number	Diff line number	Diff line change
`@@ -1440,7 +1440,8 @@ static bool enable_ddw(struct pci_dev dev, struct device_node pdn)`
`1440`	`1440`	`/* Keep default DMA window stuct if removed */`
`1441`	`1441`	`if (default_win_removed) {`
`1442`	`1442`	`tbl->it_size = 0;`
`1443`		`- kfree(tbl->it_map);`
	`1443`	`+ vfree(tbl->it_map);`
	`1444`	`+ tbl->it_map = NULL;`
`1444`	`1445`	`}`
`1445`	`1446`
`1446`	`1447`	`set_iommu_table_base(&dev->dev, newtbl);`