> -----Original Message----- > From: Andrii Anisov [mailto:[email protected]] > Sent: 23 January 2019 12:50 > To: [email protected] > Cc: Julien Grall <[email protected]>; Stefano Stabellini > <[email protected]>; Paul Durrant <[email protected]>; Juergen > Gross <[email protected]>; Andrii Anisov <[email protected]> > Subject: [PATCH v2 for-4.12] arm/p2m: call iommu iotlb flush if iommu > exists and enabled > > From: Andrii Anisov <[email protected]> > > Taking decision by `need_iommu_pt_sync()` make us never kicking > `iommu_iotlb_flush()` for IOMMUs which do share P2M with CPU. > So check `has_iommu_pt()` instead. > > Signed-off-by: Andrii Anisov <[email protected]>
Reviewed-by: Paul Durrant <[email protected]> > --- > Changes in v2: > Fixed a missprint and a nit in a commit message > > xen/arch/arm/p2m.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c > index 2394f97..059a391 100644 > --- a/xen/arch/arm/p2m.c > +++ b/xen/arch/arm/p2m.c > @@ -1019,7 +1019,7 @@ static int __p2m_set_entry(struct p2m_domain *p2m, > !mfn_eq(lpae_get_mfn(*entry), lpae_get_mfn(orig_pte)) ) > p2m_free_entry(p2m, orig_pte, level); > > - if ( need_iommu_pt_sync(p2m->domain) && > + if ( has_iommu_pt(p2m->domain) && > (lpae_is_valid(orig_pte) || lpae_is_valid(*entry)) ) > { > unsigned int flush_flags = 0; > -- > 2.7.4 _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
