Re: [PATCH 3/6] iommu/amd: Take domain->lock for complete attach/detach path

2019-09-25 Thread Jerry Snitselaar
On Wed Sep 25 19, Joerg Roedel wrote: From: Joerg Roedel The code-paths before __attach_device() and __detach_device() are called also access and modify domain state, so take the domain lock there too. This allows to get rid of the __detach_device() function. Fixes: 92d420ec028d ("iommu/amd: R

Re: [PATCH 3/6] iommu/amd: Take domain->lock for complete attach/detach path

2019-09-25 Thread Sironi, Filippo via iommu
> On 25. Sep 2019, at 06:22, Joerg Roedel wrote: > > From: Joerg Roedel > > The code-paths before __attach_device() and __detach_device() are called > also access and modify domain state, so take the domain lock there too. > This allows to get rid of the __detach_device() function. > > Fixe

[PATCH 3/6] iommu/amd: Take domain->lock for complete attach/detach path

2019-09-25 Thread Joerg Roedel
From: Joerg Roedel The code-paths before __attach_device() and __detach_device() are called also access and modify domain state, so take the domain lock there too. This allows to get rid of the __detach_device() function. Fixes: 92d420ec028d ("iommu/amd: Relax locking in dma_ops path") Signed-of