Re: [Xen-devel] [PATCH] gnttab: make sure grant map operations don't skip their IOMMU part

2019-11-29 Thread Andrew Cooper
On 22/11/2019 11:22, Durrant, Paul wrote: >> @@ -1144,27 +1140,22 @@ map_grant_ref( >> if ( need_iommu ) >> { >> unsigned int kind; >> -int err = 0; >> >> double_gt_lock(lgt, rgt); >> >> -/* We're not translated, so we know that gmfns and mfns are >> -

Re: [Xen-devel] [PATCH] gnttab: make sure grant map operations don't skip their IOMMU part

2019-11-22 Thread Durrant, Paul
; Ian Jackson > Subject: [Xen-devel] [PATCH] gnttab: make sure grant map operations don't > skip their IOMMU part > > Two almost simultaneous mapping requests need to make sure that at the > completion of the earlier one IOMMU mappings (established explicitly > here in t

[Xen-devel] [PATCH] gnttab: make sure grant map operations don't skip their IOMMU part

2019-11-21 Thread Jan Beulich
Two almost simultaneous mapping requests need to make sure that at the completion of the earlier one IOMMU mappings (established explicitly here in the PV case) have been put in place. Forever since the splitting of the grant table lock a violation of this has been possible (using simplified pin co