> From: Jan Beulich <[email protected]> > Sent: Wednesday, September 15, 2021 5:13 PM > > Doing the cleanup also for phantom devices is at best redundant with > doing it for the corresponding real device. I couldn't force myself into > checking all the code paths whether it really is: It seems better to > explicitly skip this step in such cases. > > Signed-off-by: Jan Beulich <[email protected]>
Reviewed-by: Kevin Tian <[email protected]> > > --- a/xen/drivers/passthrough/vtd/iommu.c > +++ b/xen/drivers/passthrough/vtd/iommu.c > @@ -1733,7 +1733,7 @@ static int domain_context_unmap(struct d > return -EINVAL; > } > > - if ( ret || QUARANTINE_SKIP(domain) ) > + if ( ret || QUARANTINE_SKIP(domain) || pdev->devfn != devfn ) > return ret; > > /*
