> -----Original Message----- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 13 September 2018 15:51 > To: Paul Durrant <paul.durr...@citrix.com> > Cc: George Dunlap <george.dun...@citrix.com>; Wei Liu > <wei.l...@citrix.com>; Kevin Tian <kevin.t...@intel.com>; xen-devel <xen- > de...@lists.xenproject.org> > Subject: RE: [PATCH v8 7/7] vtd: add lookup_page method to iommu_ops > > >>> On 13.09.18 at 16:04, <paul.durr...@citrix.com> wrote: > >> -----Original Message----- > >> From: Jan Beulich [mailto:jbeul...@suse.com] > >> Sent: 13 September 2018 14:57 > >> To: Paul Durrant <paul.durr...@citrix.com> > >> Cc: George Dunlap <george.dun...@citrix.com>; Wei Liu > >> <wei.l...@citrix.com>; Kevin Tian <kevin.t...@intel.com>; xen-devel > <xen- > >> de...@lists.xenproject.org> > >> Subject: RE: [PATCH v8 7/7] vtd: add lookup_page method to iommu_ops > >> > >> >>> On 13.09.18 at 15:50, <paul.durr...@citrix.com> wrote: > >> > Ok. I'll spell it out in the header if you think it is non-obvious. > >> > >> Obvious or not - do we _have_ any such outer locking in place right > now? > >> > > > > Yes. The locking is either via the P2M or grant table locks for all > current > > uses or map and unmap that I can see. > > But two different locks still means no guarantees at all. >
So, are you saying the current implementation is not fit for purpose? Do you want me to add locking at the wrapper level? Paul > Jan > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
