>>> On 25.02.16 at 13:14, <quan...@intel.com> wrote: > On February 25, 2016 4:59pm, <jbeul...@suse.com> wrote: >> I'd >> really suggest investigating alternatives. One that comes to mind would be to >> move acquiring/releasing pcidevs_lock into a helper function, and setting a >> per-CPU flag indicating ownership. > > To me, this might be fine. > Does Per-CPU flag refer to this_cpu(iommu_dont_flush_iotlb) or variant?
Yes. But I'd prefer ... >> However, the same effect could be achieved >> by making the lock a recursive one, which would then seem to more >> conventional approach (but requiring as much code to be touched). >> Both approached would eliminate the need to pass down "locked" >> flags. ... this one (the more that the other won't mean less changes). Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel