[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-07 Thread Dave Airlie
>>> One that appears the same as a GEM object created by userspace. i.e. mmap >>> works. >> >> Oh, we have an mmap interface in the dma_buf thing for that, and iirc >> Rob Clark even bothered to implement the gem->dma_buf mmap forwarding >> somewhere. And iirc android's ion-on-dma_buf stuff is eve

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-07 Thread Dave Airlie
On Tue, May 7, 2013 at 1:59 AM, Daniel Vetter wrote: > On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: >> >> >> >> However if we don't set a dma mask on the usb device, the mapping >> >> ends up using swiotlb on machines that have it enabled, which >> >> is less than desireable. >> >>

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Daniel Vetter
On Mon, May 6, 2013 at 9:56 PM, Dave Airlie wrote: > On Tue, May 7, 2013 at 1:59 AM, Daniel Vetter wrote: >> On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: >>> >> >>> >> However if we don't set a dma mask on the usb device, the mapping >>> >> ends up using swiotlb on machines that h

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Rob Clark
On Mon, May 6, 2013 at 4:44 PM, Daniel Vetter wrote: > On Mon, May 6, 2013 at 9:56 PM, Dave Airlie wrote: >> On Tue, May 7, 2013 at 1:59 AM, Daniel Vetter wrote: >>> On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: >> >> However if we don't set a dma mask on the usb device,

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Daniel Vetter
On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: > >> > >> However if we don't set a dma mask on the usb device, the mapping > >> ends up using swiotlb on machines that have it enabled, which > >> is less than desireable. > >> > >> Signed-off-by: Dave Airlie > > > > Fyi for everyone el

Re: [PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Dave Airlie
>>> One that appears the same as a GEM object created by userspace. i.e. mmap >>> works. >> >> Oh, we have an mmap interface in the dma_buf thing for that, and iirc >> Rob Clark even bothered to implement the gem->dma_buf mmap forwarding >> somewhere. And iirc android's ion-on-dma_buf stuff is eve

Re: [PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Rob Clark
On Mon, May 6, 2013 at 4:44 PM, Daniel Vetter wrote: > On Mon, May 6, 2013 at 9:56 PM, Dave Airlie wrote: >> On Tue, May 7, 2013 at 1:59 AM, Daniel Vetter wrote: >>> On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: >> >> However if we don't set a dma mask on the usb device,

Re: [PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Daniel Vetter
On Mon, May 6, 2013 at 9:56 PM, Dave Airlie wrote: > On Tue, May 7, 2013 at 1:59 AM, Daniel Vetter wrote: >> On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: >>> >> >>> >> However if we don't set a dma mask on the usb device, the mapping >>> >> ends up using swiotlb on machines that h

Re: [PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Dave Airlie
On Tue, May 7, 2013 at 1:59 AM, Daniel Vetter wrote: > On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: >> >> >> >> However if we don't set a dma mask on the usb device, the mapping >> >> ends up using swiotlb on machines that have it enabled, which >> >> is less than desireable. >> >>

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Dave Airlie
>> >> However if we don't set a dma mask on the usb device, the mapping >> ends up using swiotlb on machines that have it enabled, which >> is less than desireable. >> >> Signed-off-by: Dave Airlie > > Fyi for everyone else who was not on irc when Dave&I discussed this: > This really shouldn't be

Re: [PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-06 Thread Daniel Vetter
On Mon, May 06, 2013 at 10:35:35AM +1000, Dave Airlie wrote: > >> > >> However if we don't set a dma mask on the usb device, the mapping > >> ends up using swiotlb on machines that have it enabled, which > >> is less than desireable. > >> > >> Signed-off-by: Dave Airlie > > > > Fyi for everyone el

Re: [PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-05 Thread Dave Airlie
>> >> However if we don't set a dma mask on the usb device, the mapping >> ends up using swiotlb on machines that have it enabled, which >> is less than desireable. >> >> Signed-off-by: Dave Airlie > > Fyi for everyone else who was not on irc when Dave&I discussed this: > This really shouldn't be

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-01 Thread Dave Airlie
Since we ask the dmabuf owner to map the dma-buf into our device address space, but for udl at present that is the CPU address space, since we don't DMA directly from the mapped buffer. However if we don't set a dma mask on the usb device, the mapping ends up using swiotlb on machines that have it

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-01 Thread Daniel Vetter
On Wed, May 1, 2013 at 6:30 AM, Dave Airlie wrote: > Since we ask the dmabuf owner to map the dma-buf into our device > address space, but for udl at present that is the CPU address space, > since we don't DMA directly from the mapped buffer. > > However if we don't set a dma mask on the usb devic

Re: [PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-05-01 Thread Daniel Vetter
On Wed, May 1, 2013 at 6:30 AM, Dave Airlie wrote: > Since we ask the dmabuf owner to map the dma-buf into our device > address space, but for udl at present that is the CPU address space, > since we don't DMA directly from the mapped buffer. > > However if we don't set a dma mask on the usb devic

[PATCH] drm/udl: avoid swiotlb for imported vmap buffers.

2013-04-30 Thread Dave Airlie
Since we ask the dmabuf owner to map the dma-buf into our device address space, but for udl at present that is the CPU address space, since we don't DMA directly from the mapped buffer. However if we don't set a dma mask on the usb device, the mapping ends up using swiotlb on machines that have it