Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-19 Thread Leizhen (ThunderTown)
On 2016/3/17 19:59, Catalin Marinas wrote: > On Thu, Mar 17, 2016 at 07:06:27PM +0800, Leizhen (ThunderTown) wrote: >> On 2016/3/16 9:56, Leizhen (ThunderTown) wrote: >>> On 2016/3/15 23:37, Catalin Marinas wrote: On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: > diff --git

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-19 Thread Leizhen (ThunderTown)
On 2016/3/17 19:59, Catalin Marinas wrote: > On Thu, Mar 17, 2016 at 07:06:27PM +0800, Leizhen (ThunderTown) wrote: >> On 2016/3/16 9:56, Leizhen (ThunderTown) wrote: >>> On 2016/3/15 23:37, Catalin Marinas wrote: On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: > diff --git

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-19 Thread Leizhen (ThunderTown)
On 2016/3/16 9:56, Leizhen (ThunderTown) wrote: > > > On 2016/3/15 23:37, Catalin Marinas wrote: >> On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: >>> 1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly >>>return vaddr on success, and pass vaddr to free_pages

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-19 Thread Leizhen (ThunderTown)
On 2016/3/16 9:56, Leizhen (ThunderTown) wrote: > > > On 2016/3/15 23:37, Catalin Marinas wrote: >> On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: >>> 1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly >>>return vaddr on success, and pass vaddr to free_pages

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-18 Thread Catalin Marinas
On Thu, Mar 17, 2016 at 07:06:27PM +0800, Leizhen (ThunderTown) wrote: > On 2016/3/16 9:56, Leizhen (ThunderTown) wrote: > > On 2016/3/15 23:37, Catalin Marinas wrote: > >> On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: > >>> diff --git a/arch/arm64/mm/dma-mapping.c

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-18 Thread Catalin Marinas
On Thu, Mar 17, 2016 at 07:06:27PM +0800, Leizhen (ThunderTown) wrote: > On 2016/3/16 9:56, Leizhen (ThunderTown) wrote: > > On 2016/3/15 23:37, Catalin Marinas wrote: > >> On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: > >>> diff --git a/arch/arm64/mm/dma-mapping.c

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-15 Thread Leizhen (ThunderTown)
On 2016/3/15 23:37, Catalin Marinas wrote: > On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: >> 1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly >>return vaddr on success, and pass vaddr to free_pages on failure. >> 2. So, we can directly transparent pass

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-15 Thread Leizhen (ThunderTown)
On 2016/3/15 23:37, Catalin Marinas wrote: > On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: >> 1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly >>return vaddr on success, and pass vaddr to free_pages on failure. >> 2. So, we can directly transparent pass

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-15 Thread Catalin Marinas
On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: > 1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly >return vaddr on success, and pass vaddr to free_pages on failure. > 2. So, we can directly transparent pass vaddr from __dma_free to >swiotlb_free_coherent,

Re: [PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-15 Thread Catalin Marinas
On Tue, Mar 15, 2016 at 10:12:11AM +0800, Zhen Lei wrote: > 1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly >return vaddr on success, and pass vaddr to free_pages on failure. > 2. So, we can directly transparent pass vaddr from __dma_free to >swiotlb_free_coherent,

[PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-14 Thread Zhen Lei
1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly return vaddr on success, and pass vaddr to free_pages on failure. 2. So, we can directly transparent pass vaddr from __dma_free to swiotlb_free_coherent, keep consistent with swiotlb_alloc_coherent. This patch have no

[PATCH 1/1] arm64/dma-mapping: remove an unnecessary conversion

2016-03-14 Thread Zhen Lei
1. In swiotlb_alloc_coherent, the branch of __get_free_pages. Directly return vaddr on success, and pass vaddr to free_pages on failure. 2. So, we can directly transparent pass vaddr from __dma_free to swiotlb_free_coherent, keep consistent with swiotlb_alloc_coherent. This patch have no