Re: [PATCH v3] vfio/type1: Remove locked page accounting workqueue

2017-04-14 Thread Alex Williamson
On Fri, 14 Apr 2017 14:58:14 -0600 Alex Williamson wrote: > On Sat, 15 Apr 2017 00:51:28 +0530 > Kirti Wankhede wrote: > > > On 4/12/2017 12:58 AM, Alex Williamson wrote: > > > If the mmap_sem is contented then the vfio type1 IOMMU backend will > > > defer locked page accounting updates to a

Re: [PATCH v3] vfio/type1: Remove locked page accounting workqueue

2017-04-14 Thread Alex Williamson
On Sat, 15 Apr 2017 00:51:28 +0530 Kirti Wankhede wrote: > On 4/12/2017 12:58 AM, Alex Williamson wrote: > > If the mmap_sem is contented then the vfio type1 IOMMU backend will > > defer locked page accounting updates to a workqueue task. This has a > > few problems and depending on which side t

Re: [PATCH v3] vfio/type1: Remove locked page accounting workqueue

2017-04-14 Thread Kirti Wankhede
On 4/12/2017 12:58 AM, Alex Williamson wrote: > If the mmap_sem is contented then the vfio type1 IOMMU backend will > defer locked page accounting updates to a workqueue task. This has a > few problems and depending on which side the user tries to play, they > might be over-penalized for unmaps

Re: [PATCH v3] vfio/type1: Remove locked page accounting workqueue

2017-04-12 Thread Auger Eric
Hi Alex, On 11/04/2017 21:28, Alex Williamson wrote: > If the mmap_sem is contented then the vfio type1 IOMMU backend will > defer locked page accounting updates to a workqueue task. This has a > few problems and depending on which side the user tries to play, they > might be over-penalized for u

Re: [PATCH v3] vfio/type1: Remove locked page accounting workqueue

2017-04-11 Thread Peter Xu
On Tue, Apr 11, 2017 at 01:28:06PM -0600, Alex Williamson wrote: > If the mmap_sem is contented then the vfio type1 IOMMU backend will > defer locked page accounting updates to a workqueue task. This has a > few problems and depending on which side the user tries to play, they > might be over-pena

[PATCH v3] vfio/type1: Remove locked page accounting workqueue

2017-04-11 Thread Alex Williamson
If the mmap_sem is contented then the vfio type1 IOMMU backend will defer locked page accounting updates to a workqueue task. This has a few problems and depending on which side the user tries to play, they might be over-penalized for unmaps that haven't yet been accounted or race the workqueue to