Re: [PATCH 03/12] dma-buf: lock the reservation object during (un)map_dma_buf v3

2019-05-27 Thread Hillf Danton
On Tue, 16 Apr 2019 20:38:32 +0200 Christian König wrote: > @@ -688,9 +689,9 @@ struct sg_table *dma_buf_map_attachment(struct > dma_buf_attachment *attach, > if (attach->sgt) > return attach->sgt; > > - sg_table = attach->dmabuf->ops->map_dma_buf(attach, direction); > -

Re: [PATCH 03/12] dma-buf: lock the reservation object during (un)map_dma_buf v3

2019-04-17 Thread Christian König
Am 17.04.19 um 16:26 schrieb Daniel Vetter: On Wed, Apr 17, 2019 at 04:14:32PM +0200, Christian König wrote: Am 17.04.19 um 16:08 schrieb Daniel Vetter: On Tue, Apr 16, 2019 at 08:38:32PM +0200, Christian König wrote: Make it mandatory for dynamic dma-buf callbacks to be called with the reserv

Re: [PATCH 03/12] dma-buf: lock the reservation object during (un)map_dma_buf v3

2019-04-17 Thread Daniel Vetter
On Wed, Apr 17, 2019 at 04:14:32PM +0200, Christian König wrote: > Am 17.04.19 um 16:08 schrieb Daniel Vetter: > > On Tue, Apr 16, 2019 at 08:38:32PM +0200, Christian König wrote: > > > Make it mandatory for dynamic dma-buf callbacks to be called with the > > > reservation lock held. > > > > > > F

Re: [PATCH 03/12] dma-buf: lock the reservation object during (un)map_dma_buf v3

2019-04-17 Thread Christian König
Am 17.04.19 um 16:08 schrieb Daniel Vetter: On Tue, Apr 16, 2019 at 08:38:32PM +0200, Christian König wrote: Make it mandatory for dynamic dma-buf callbacks to be called with the reservation lock held. For static dma-buf exporters we still have the fallback of using cached sgt. v2: reordered v

Re: [PATCH 03/12] dma-buf: lock the reservation object during (un)map_dma_buf v3

2019-04-17 Thread Daniel Vetter
On Tue, Apr 16, 2019 at 08:38:32PM +0200, Christian König wrote: > Make it mandatory for dynamic dma-buf callbacks to be called with the > reservation lock held. > > For static dma-buf exporters we still have the fallback of using cached sgt. > > v2: reordered > v3: rebased on sgt caching > v4: u

[PATCH 03/12] dma-buf: lock the reservation object during (un)map_dma_buf v3

2019-04-16 Thread Christian König
Make it mandatory for dynamic dma-buf callbacks to be called with the reservation lock held. For static dma-buf exporters we still have the fallback of using cached sgt. v2: reordered v3: rebased on sgt caching v4: use the cached sgt when possible Signed-off-by: Christian König --- drivers/dma