Re: [PATCH v1] iommu/hyperv: Don't free a paging domain the hypervisor did not delete
From: Jason Gunthorpe
Date: Thu Nov 05 2026 - 16:14:25 EST
On Thu, Nov 05, 2026 at 02:27:08PM -0500, Yuho Choi wrote:
> hv_iommu_paging_domain_free() frees all page-table pages with
> pt_iommu_deinit() and only then asks the hypervisor to delete the
> device domain that points at them. If HVCALL_DELETE_DEVICE_DOMAIN
> fails, hv_delete_device_domain() only logs the error, returns the
> domain ID to the IDA, and the domain is freed anyway.
Isn't there a WARN_ON on this path? I recall asking for one ..
So long as that is there this is the right way to continue
Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx>