ANDROID: drivers/arm-smmu-v3-kvm: Free io_pgtable struct on domain free

misaleh · misaleh · commit 04469eba65a0 · 2024-10-29T08:37:06.000Z
When a domain is freed, it should also free the page table struct
"arm_lpae_io_pgtable" which is per domain, as the domain is freed
and this memory would be leaked.

Bug: 277989609
Bug: 278749606

Change-Id: I6eaf00977d613d203f3918485d4cd8618ec40c9e
Signed-off-by: Mostafa Saleh &lt;smostafa@google.com&gt;
diff --git a/drivers/iommu/arm/arm-smmu-v3/pkvm/io-pgtable-arm.c b/drivers/iommu/arm/arm-smmu-v3/pkvm/io-pgtable-arm.c
@@ -136,6 +136,7 @@ int kvm_arm_io_pgtable_free(struct io_pgtable *iopt)
 		kvm_flush_dcache_to_poc(data->pgd, pgd_size);
 
 	__arm_lpae_free_pgtable(data, data->start_level, data->pgd);
+	hyp_free(data);
 	return 0;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -136,6 +136,7 @@ int kvm_arm_io_pgtable_free(struct io_pgtable *iopt)`
`136`	`136`	`kvm_flush_dcache_to_poc(data->pgd, pgd_size);`
`137`	`137`
`138`	`138`	`__arm_lpae_free_pgtable(data, data->start_level, data->pgd);`
	`139`	`+ hyp_free(data);`
`139`	`140`	`return 0;`
`140`	`141`	`}`
`141`	`142`