Re: [PATCH v1 3/3] iommu/amd: Optimize the IOMMU queue flush

2017-06-27 Thread Jan Vesely
On Mon, 2017-06-26 at 14:14 +0200, Joerg Roedel wrote: > On Fri, Jun 23, 2017 at 10:20:47AM -0400, Jan Vesely wrote: > > I was able to trigger "Completion-Wait loop timed out" messages in the > > following situation: > > Hung OpenCL task running on dGPU. > > dG

Re: [PATCH v1 3/3] iommu/amd: Optimize the IOMMU queue flush

2017-06-23 Thread Jan Vesely
On Thu, 2017-06-22 at 23:57 +0200, Joerg Roedel wrote: > On Thu, Jun 22, 2017 at 11:13:09AM -0400, Jan Vesely wrote: > > It looks like I tested different patches. > > linux-4.10.17 with both > > "iommu/amd: Optimize iova queue flushing" > > This patch isn'

Re: [PATCH v1 3/3] iommu/amd: Optimize the IOMMU queue flush

2017-06-22 Thread Jan Vesely
On Thu, 2017-06-22 at 11:20 +0200, Joerg Roedel wrote: > On Wed, Jun 21, 2017 at 05:09:31PM -0400, Jan Vesely wrote: > > On Wed, 2017-06-21 at 12:01 -0500, Tom Lendacky wrote: > > > On 6/21/2017 11:20 AM, Jan Vesely wrote: > > > > Hi Arindam, > > >

Re: [PATCH v1 3/3] iommu/amd: Optimize the IOMMU queue flush

2017-06-21 Thread Jan Vesely
On Wed, 2017-06-21 at 12:01 -0500, Tom Lendacky wrote: > On 6/21/2017 11:20 AM, Jan Vesely wrote: > > Hi Arindam, > > > > has this patch been replaced by Joerg's "[PATCH 0/7] iommu/amd: > > Optimize iova queue flushing" series? > > Yes, Joerg's pa

Re: [PATCH v1 3/3] iommu/amd: Optimize the IOMMU queue flush

2017-06-21 Thread Jan Vesely
Hi Arindam, has this patch been replaced by Joerg's "[PATCH 0/7] iommu/amd: Optimize iova queue flushing" series? Jan On Thu, 2017-06-08 at 22:33 +0200, Jan Vesely wrote: > On Tue, 2017-06-06 at 10:02 +, Nath, Arindam wrote: > > > -Original Message- > &

Re: [PATCH v1 3/3] iommu/amd: Optimize the IOMMU queue flush

2017-06-08 Thread Jan Vesely
; @@ -2819,6 +2854,12 @@ int __init amd_iommu_init_api(void) > > goto out_put_iova; > > > > spin_lock_init(>lock); > > + > > + pd_queue->entries = kzalloc(FLUSH_QUEUE_SIZE * > > +

Re: [PATCH] iommu/amd: flush IOTLB for specific domains only (v2)

2017-05-19 Thread Jan Vesely
mu_types.h > @@ -454,6 +454,8 @@ struct protection_domain { > bool updated; /* complete domain flush required */ > unsigned dev_cnt; /* devices assigned to this domain */ > unsigned dev_iommu[MAX_IOMMUS]; /* per-IOMMU reference count */ > + bool a

[PATCH 1/2] iommu/amd: Remove entry from the list before freeing it

2016-05-21 Thread Jan Vesely
From: Jan Vesely <jan.ves...@amd.com> Signed-off-by: Jan Vesely <jan.ves...@amd.com> --- drivers/iommu/amd_iommu.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/iommu/amd_iommu.c b/drivers/iommu/amd_iommu.c index 634f636..17c76f2 100644 --- a/d

[PATCH 2/2] iommu/amd: Destroy api_lock mutex when freeing domain

2016-05-21 Thread Jan Vesely
From: Jan Vesely <jan.ves...@amd.com> Signed-off-by: Jan Vesely <jan.ves...@amd.com> --- drivers/iommu/amd_iommu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/iommu/amd_iommu.c b/drivers/iommu/amd_iommu.c index 17c76f2..4ff5e40 100644 --- a/drivers/iommu/amd_iommu.c +