Re: [PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-13 Thread Alexey Kardashevskiy
On 05/14/2015 12:53 PM, Alex Williamson wrote: On Thu, 2015-05-14 at 12:34 +1000, Alexey Kardashevskiy wrote: On 05/14/2015 09:27 AM, Gavin Shan wrote: On Wed, May 13, 2015 at 02:51:36PM +0200, Thomas Huth wrote: On Wed, 13 May 2015 16:30:16 +1000 Alexey Kardashevskiy wrote: On 05/13/2015 0

Re: [PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-13 Thread Alex Williamson
On Thu, 2015-05-14 at 12:34 +1000, Alexey Kardashevskiy wrote: > On 05/14/2015 09:27 AM, Gavin Shan wrote: > > On Wed, May 13, 2015 at 02:51:36PM +0200, Thomas Huth wrote: > >> On Wed, 13 May 2015 16:30:16 +1000 > >> Alexey Kardashevskiy wrote: > >> > >>> On 05/13/2015 03:33 PM, Gavin Shan wrote:

Re: [PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-13 Thread Alexey Kardashevskiy
On 05/14/2015 09:27 AM, Gavin Shan wrote: On Wed, May 13, 2015 at 02:51:36PM +0200, Thomas Huth wrote: On Wed, 13 May 2015 16:30:16 +1000 Alexey Kardashevskiy wrote: On 05/13/2015 03:33 PM, Gavin Shan wrote: On Tue, May 12, 2015 at 01:38:54AM +1000, Alexey Kardashevskiy wrote: At the moment

Re: [PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-13 Thread Gavin Shan
On Wed, May 13, 2015 at 02:51:36PM +0200, Thomas Huth wrote: >On Wed, 13 May 2015 16:30:16 +1000 >Alexey Kardashevskiy wrote: > >> On 05/13/2015 03:33 PM, Gavin Shan wrote: >> > On Tue, May 12, 2015 at 01:38:54AM +1000, Alexey Kardashevskiy wrote: >> >> At the moment iommu_free_table() only releas

Re: [PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-13 Thread Thomas Huth
On Wed, 13 May 2015 16:30:16 +1000 Alexey Kardashevskiy wrote: > On 05/13/2015 03:33 PM, Gavin Shan wrote: > > On Tue, May 12, 2015 at 01:38:54AM +1000, Alexey Kardashevskiy wrote: > >> At the moment iommu_free_table() only releases memory if > >> the table was initialized for the platform code u

Re: [PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-12 Thread Alexey Kardashevskiy
On 05/13/2015 03:33 PM, Gavin Shan wrote: On Tue, May 12, 2015 at 01:38:54AM +1000, Alexey Kardashevskiy wrote: At the moment iommu_free_table() only releases memory if the table was initialized for the platform code use, i.e. it had it_map initialized (which purpose is to track DMA memory space

Re: [PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-12 Thread Gavin Shan
On Tue, May 12, 2015 at 01:38:54AM +1000, Alexey Kardashevskiy wrote: >At the moment iommu_free_table() only releases memory if >the table was initialized for the platform code use, i.e. it had >it_map initialized (which purpose is to track DMA memory space use). > >With dynamic DMA windows, we wil

[PATCH kernel v10 05/34] powerpc/iommu: Always release iommu_table in iommu_free_table()

2015-05-11 Thread Alexey Kardashevskiy
At the moment iommu_free_table() only releases memory if the table was initialized for the platform code use, i.e. it had it_map initialized (which purpose is to track DMA memory space use). With dynamic DMA windows, we will need to be able to release iommu_table even if it was used for VFIO in wh