Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-22 Thread Hiroshi Doyu
Hi Marek, On Tue, 21 Aug 2012 17:01:08 +0200 Marek Szyprowski wrote: > > > > -__iommu_alloc_remap(struct page **pages, size_t size, gfp_t gfp, > > > > pgprot_t prot) > > > > +__iommu_alloc_remap(struct page **pages, size_t size, gfp_t gfp, > > > > pgprot_t prot, > > > > +

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-22 Thread Hiroshi Doyu
Hi Marek, On Tue, 21 Aug 2012 17:01:08 +0200 Marek Szyprowski m.szyprow...@samsung.com wrote: -__iommu_alloc_remap(struct page **pages, size_t size, gfp_t gfp, pgprot_t prot) +__iommu_alloc_remap(struct page **pages, size_t size, gfp_t gfp, pgprot_t prot, +

RE: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Marek Szyprowski
Hi Hiroshi, On Tuesday, August 21, 2012 2:15 PM Hiroshi Doyu wrote: > Hiroshi Doyu wrote @ Tue, 21 Aug 2012 13:22:35 +0200: > > > Hi, > > > > On Mon, 30 Jul 2012 10:28:19 +0200 > > Marek Szyprowski wrote: > > > > > This patch changes dma-mapping subsystem to use generic vmalloc areas > > >

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Hiroshi Doyu
Russell King - ARM Linux wrote @ Tue, 21 Aug 2012 14:34:51 +0200: > On Tue, Aug 21, 2012 at 02:22:35PM +0300, Hiroshi Doyu wrote: > > The following "__get_vm_area_node()" can take gfp_mask, it means that > > this function is expected to be called from atomic context, but why > > it's _NOT_

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Russell King - ARM Linux
On Tue, Aug 21, 2012 at 02:22:35PM +0300, Hiroshi Doyu wrote: > The following "__get_vm_area_node()" can take gfp_mask, it means that > this function is expected to be called from atomic context, but why > it's _NOT_ allowed _ONLY_ from interrupt context? One reason is it takes read/write locks

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Hiroshi Doyu
Hiroshi Doyu wrote @ Tue, 21 Aug 2012 13:22:35 +0200: > Hi, > > On Mon, 30 Jul 2012 10:28:19 +0200 > Marek Szyprowski wrote: > > > This patch changes dma-mapping subsystem to use generic vmalloc areas > > for all consistent dma allocations. This increases the total size limit > > of the

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Hiroshi Doyu
Hi, On Mon, 30 Jul 2012 10:28:19 +0200 Marek Szyprowski wrote: > This patch changes dma-mapping subsystem to use generic vmalloc areas > for all consistent dma allocations. This increases the total size limit > of the consistent allocations and removes platform hacks and a lot of > duplicated

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Hiroshi Doyu
Hi, On Mon, 30 Jul 2012 10:28:19 +0200 Marek Szyprowski m.szyprow...@samsung.com wrote: This patch changes dma-mapping subsystem to use generic vmalloc areas for all consistent dma allocations. This increases the total size limit of the consistent allocations and removes platform hacks and a

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Hiroshi Doyu
Hiroshi Doyu hd...@nvidia.com wrote @ Tue, 21 Aug 2012 13:22:35 +0200: Hi, On Mon, 30 Jul 2012 10:28:19 +0200 Marek Szyprowski m.szyprow...@samsung.com wrote: This patch changes dma-mapping subsystem to use generic vmalloc areas for all consistent dma allocations. This increases the

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Russell King - ARM Linux
On Tue, Aug 21, 2012 at 02:22:35PM +0300, Hiroshi Doyu wrote: The following __get_vm_area_node() can take gfp_mask, it means that this function is expected to be called from atomic context, but why it's _NOT_ allowed _ONLY_ from interrupt context? One reason is it takes read/write locks

Re: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Hiroshi Doyu
Russell King - ARM Linux li...@arm.linux.org.uk wrote @ Tue, 21 Aug 2012 14:34:51 +0200: On Tue, Aug 21, 2012 at 02:22:35PM +0300, Hiroshi Doyu wrote: The following __get_vm_area_node() can take gfp_mask, it means that this function is expected to be called from atomic context, but why

RE: [PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-08-21 Thread Marek Szyprowski
Hi Hiroshi, On Tuesday, August 21, 2012 2:15 PM Hiroshi Doyu wrote: Hiroshi Doyu hd...@nvidia.com wrote @ Tue, 21 Aug 2012 13:22:35 +0200: Hi, On Mon, 30 Jul 2012 10:28:19 +0200 Marek Szyprowski m.szyprow...@samsung.com wrote: This patch changes dma-mapping subsystem to use

[PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-07-30 Thread Marek Szyprowski
This patch changes dma-mapping subsystem to use generic vmalloc areas for all consistent dma allocations. This increases the total size limit of the consistent allocations and removes platform hacks and a lot of duplicated code. Atomic allocations are served from special pool preallocated on

[PATCHv6 2/2] ARM: dma-mapping: remove custom consistent dma region

2012-07-30 Thread Marek Szyprowski
This patch changes dma-mapping subsystem to use generic vmalloc areas for all consistent dma allocations. This increases the total size limit of the consistent allocations and removes platform hacks and a lot of duplicated code. Atomic allocations are served from special pool preallocated on