On Monday 13 June 2011 17:30:44 KyongHo Cho wrote:
On Tue, Jun 14, 2011 at 12:07 AM, Arnd Bergmann arnd@arndb.de wrote:
I'm sure that the graphics people will disagree with you on that. Having the frame buffer mapped in write-combine mode is rather important when you want to efficiently output videos from your CPU.
I agree with you. But I am discussing about dma_alloc_writecombine() in ARM. You can see that only ARM and AVR32 implement it and there are few drivers which use it. No function in dma_map_ops corresponds to dma_alloc_writecombine(). That's why Marek tried to add 'alloc_writecombine' to dma_map_ops.
Yes, and I think Marek's patch is really necessary. The reason we need dma_alloc_writecombine on ARM is because the page attributes in the kernel need to match the ones in user space, while other architectures either handle the writecombine flag outside of the page table or can have multiple conflicting mappings.
The reason that I suspect AVR32 needs it is to share device drivers with ARM.
I can understand that there are arguments why mapping a DMA buffer into user space doesn't belong into dma_map_ops, but I don't see how the presence of an IOMMU is one of them.
The entire purpose of dma_map_ops is to hide from the user whether you have an IOMMU or not, so that would be the main argument for putting it in there, not against doing so.
I also understand the reasons why dma_map_ops maps a buffer into user space. Mapping in device and user space at the same time or in a simple approach may look good. But I think mapping to user must be and driver-specific. Moreover, kernel already provides various ways to map physical memory to user space.
I believe the idea of providing dma_mmap_... is to ensure that the page attributes are not conflicting and the DMA code is the place that decides on the page attributes for the kernel mapping, so no other place in the kernel can really know what it should be in user space.
And I think that remapping DMA address that is in device address space to user space is not a good idea because DMA address is not same to physical address semantically if features of IOMMU are implemented.
I'm totally not following this argument. This has nothing to do with IOMMU or not. If you have an IOMMU, the dma code will know where the pages are anyway, so it can always map them into user space. The dma code might have an easier way to do it other than follwoing the page tables.
Arnd