powerpc/powernv/npu: Allocate enough memory in pnv_try_setup_npu_table_group()

Dan Carpenter · mpe · commit d7b6cc199b2d · 2019-01-11T23:44:54.000+11:00
There is a typo so we accidentally allocate enough memory for a pointer when we wanted to allocate enough for a struct. Fixes: 0bd9716 ("powerpc/powernv/npu: Add compound IOMMU groups") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Alexey Kardashevskiy <aik@ozlabs.ru> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
diff --git a/arch/powerpc/platforms/powernv/npu-dma.c b/arch/powerpc/platforms/powernv/npu-dma.c
@@ -564,7 +564,7 @@ struct iommu_table_group *pnv_try_setup_npu_table_group(struct pnv_ioda_pe *pe)
 		}
 	} else {
 		/* Create a group for 1 GPU and attached NPUs for POWER8 */
-		pe->npucomp = kzalloc(sizeof(pe->npucomp), GFP_KERNEL);
+		pe->npucomp = kzalloc(sizeof(*pe->npucomp), GFP_KERNEL);
 		table_group = &pe->npucomp->table_group;
 		table_group->ops = &pnv_npu_peers_ops;
 		iommu_register_group(table_group, hose->global_number,

Original file line number	Diff line number	Diff line change
`@@ -564,7 +564,7 @@ struct iommu_table_group pnv_try_setup_npu_table_group(struct pnv_ioda_pe pe)`
`564`	`564`	`}`
`565`	`565`	`} else {`
`566`	`566`	`/* Create a group for 1 GPU and attached NPUs for POWER8 */`
`567`		`- pe->npucomp = kzalloc(sizeof(pe->npucomp), GFP_KERNEL);`
	`567`	`+ pe->npucomp = kzalloc(sizeof(*pe->npucomp), GFP_KERNEL);`
`568`	`568`	`table_group = &pe->npucomp->table_group;`
`569`	`569`	`table_group->ops = &pnv_npu_peers_ops;`
`570`	`570`	`iommu_register_group(table_group, hose->global_number,`