Re: [PATCH] media: vb2-dc: skip CPU sync in map/unmap dma_buf

2019-06-06 Thread Hans Verkuil
On 6/7/19 6:32 AM, Tomasz Figa wrote: > Hi Hans, Lucas, > > On Fri, May 3, 2019 at 9:38 PM Hans Verkuil wrote: >> >> Hi Lucas, >> >> As you mentioned there hasn't been any further review comments, so >> it is fair not to postpone this. >> > > Sorry for being late to the party. I didn't notice th

Re: [PATCH] media: vb2-dc: skip CPU sync in map/unmap dma_buf

2019-06-06 Thread Tomasz Figa
Hi Hans, Lucas, On Fri, May 3, 2019 at 9:38 PM Hans Verkuil wrote: > > Hi Lucas, > > As you mentioned there hasn't been any further review comments, so > it is fair not to postpone this. > Sorry for being late to the party. I didn't notice this patch before. (Perhaps it could be worth adding me

Re: [PATCH] media: vb2-dc: skip CPU sync in map/unmap dma_buf

2019-05-03 Thread Hans Verkuil
Hi Lucas, As you mentioned there hasn't been any further review comments, so it is fair not to postpone this. However, I have one new review comment myself that I would like to see addressed in a v2 before I merge this for 5.3: On 2/28/19 8:19 AM, Lucas Stach wrote: > This is rougly equivalent t

Re: [PATCH] media: vb2-dc: skip CPU sync in map/unmap dma_buf

2019-05-03 Thread Lucas Stach
Hi Hans, Am Dienstag, den 12.03.2019, 11:03 +0100 schrieb Lucas Stach: > Hi Hans, > > Am Dienstag, den 12.03.2019, 08:57 +0100 schrieb Hans Verkuil: > > Pawel and/or Marek, can you take a look at this? Seems nobody else is interested in this patch. Would you be willing to take it without further

Re: [PATCH] media: vb2-dc: skip CPU sync in map/unmap dma_buf

2019-03-12 Thread Lucas Stach
Hi Hans, Am Dienstag, den 12.03.2019, 08:57 +0100 schrieb Hans Verkuil: > Pawel and/or Marek, can you take a look at this? > > It looks sane to me, but I'd like to have a second opinion as well before > merging this. > > On 2/28/19 8:19 AM, Lucas Stach wrote: > > This is rougly equivalent to ca0

Re: [PATCH] media: vb2-dc: skip CPU sync in map/unmap dma_buf

2019-03-12 Thread Hans Verkuil
Pawel and/or Marek, can you take a look at this? It looks sane to me, but I'd like to have a second opinion as well before merging this. On 2/28/19 8:19 AM, Lucas Stach wrote: > This is rougly equivalent to ca0e68e21aae (drm/prime: skip CPU sync > in map/unmap dma_buf). The contig memory allocate

[PATCH] media: vb2-dc: skip CPU sync in map/unmap dma_buf

2019-02-27 Thread Lucas Stach
This is rougly equivalent to ca0e68e21aae (drm/prime: skip CPU sync in map/unmap dma_buf). The contig memory allocated is already device coherent memory, so there is no point in doing a CPU sync when mapping it to another device. Also most importers currently cache the mapping so the CPU sync would