Re: [PATCH v5 15/21] dma-buf: Move dma_buf_vmap() to dynamic locking specification

2022-09-20 Thread Sumit Semwal
return; > > + dma_resv_assert_held(dmabuf->resv); > + > BUG_ON(iosys_map_is_null(&dmabuf->vmap_ptr)); > BUG_ON(dmabuf->vmapping_counter == 0); > BUG_ON(!iosys_map_is_equal(&dmabuf->vmap_ptr, map)); > -- > 2.37.3 > -- Thanks and regards, Sumit Semwal (he / him) Tech Lead - LCG, Vertical Technologies Linaro.org │ Open source software for ARM SoCs

Re: [PATCH v5 00/21] Move all drivers to a common dma-buf locking convention

2022-09-20 Thread Sumit Semwal
| 17 +- > drivers/infiniband/core/umem_dmabuf.c | 7 +- > .../common/videobuf2/videobuf2-dma-contig.c | 22 +- > .../media/common/videobuf2/videobuf2-dma-sg.c | 19 +- > .../common/videobuf2/videobuf2-vmalloc.c | 17 +- > .../platform/nvidia/tegra-vde/dmabuf-c

Re: [PATCH 1/6] dma-buf: add peer2peer flag

2020-03-31 Thread Sumit Semwal
by: Christian König > > On the series: > > Acked-by: Daniel Vetter > Fwiw, for the series, Acked-by: Sumit Semwal > --- > > drivers/dma-buf/dma-buf.c | 2 ++ > > include/linux/dma-buf.h | 10 ++ > > 2 files changed, 12 insertions(+) > > &

Re: [PATCH 01/12] dma-buf: add dynamic caching of sg_table

2019-05-22 Thread Sumit Semwal
Hi Christian, On Sat, 27 Apr 2019 at 05:31, Liam Mark wrote: > > On Tue, 16 Apr 2019, Christian König wrote: > > > To allow a smooth transition from pinning buffer objects to dynamic > > invalidation we first start to cache the sg_table for an attachment > > unless the driver explicitly says to n

Re: [PATCH 1/7] sync_file: get rid of internal reference count.

2017-04-17 Thread Sumit Semwal
eed it later. >> >> [airlied: fix buildbot warnings] >> >> Reviewed-by: Chris Wilson >> Signed-off-by: Dave Airlie >> --- >> drivers/dma-buf/sync_file.c | 13 ++--- >> include/linux/sync_file.h | 3 --- >> 2 files changed, 2 insert

Re: [PATCH 1/3] dma-buf: return index of the first signaled fence

2016-11-04 Thread Sumit Semwal
gned-off-by: Alex Deucher > Cc: Sumit Semwal > --- > drivers/dma-buf/fence.c | 19 ++- > include/linux/fence.h | 2 +- > 2 files changed, 15 insertions(+), 6 deletions(-) > > diff --git a/drivers/dma-buf/fence.c b/drivers/dma-buf/fence.c > index 64478f9.

Re: [PATCH 2/3] drm/amdgpu: add the interface of waiting multiple fences (v3)

2016-11-04 Thread Sumit Semwal
On 4 November 2016 at 11:14, Sumit Semwal wrote: > Hi Alex, > > Thanks for the patch! > > On 3 November 2016 at 11:41, Alex Deucher wrote: >> From: Junwei Zhang >> >> v2: agd: rebase and squash in all the previous optimizations and >> changes so everyt

Re: [PATCH 2/3] drm/amdgpu: add the interface of waiting multiple fences (v3)

2016-11-04 Thread Sumit Semwal
Hi Alex, Thanks for the patch! On 3 November 2016 at 11:41, Alex Deucher wrote: > From: Junwei Zhang > > v2: agd: rebase and squash in all the previous optimizations and > changes so everything compiles. > v3: squash in Slava's 32bit build fix > > Signed-off-by: Junwei Zhang > Reviewed-by: Mon

Re: [PATCH 1/2] dma-buf: return index of the first signaled fence (v2)

2016-11-04 Thread Sumit Semwal
gt; Signed-off-by: monk.liu > Signed-off-by: Alex Deucher > Cc: Sumit Semwal > --- > > This is the same patch set I send out yesterday, I just > squashed the amdgpu patches together and rebased everything on > the fence -> dma_fence renaming. This is used by our VK driver

Re: [PATCH 1/2] dma-buf: return index of the first signaled fence (v2)

2016-11-07 Thread Sumit Semwal
Hi Alex, On 07-Nov-2016 11:14 PM, "Alex Deucher" wrote: > > On Fri, Nov 4, 2016 at 6:03 PM, Sumit Semwal wrote: > > Hi Alex, > > > > Thanks for the patches. > > > > On 4 November 2016 at 14:16, Alex Deucher wrote: > >> From: "monk.

Re: [PATCH 1/2] dma-buf: return index of the first signaled fence (v2)

2016-11-08 Thread Sumit Semwal
Hi Alex, Christian, On 8 November 2016 at 04:12, Sumit Semwal wrote: > Hi Alex, > > On 07-Nov-2016 11:14 PM, "Alex Deucher" wrote: >> >> On Fri, Nov 4, 2016 at 6:03 PM, Sumit Semwal >> wrote: >> > Hi Alex, >> > >> > Thanks for

Re: [PATCH 1/4] dma-buf/fence: make timeout handling in fence_default_wait consistent (v2)

2016-11-08 Thread Sumit Semwal
Hi Alex, Christian, On 8 November 2016 at 02:46, Alex Deucher wrote: > Kernel functions taking a timeout usually return 1 on success even > when they get a zero timeout. > > v2: agd: rebase on drm-next > > Reviewed-by: Alex Deucher > Signen-off-by: Christian König > Reviewed-by: Chunming Zhou

Re: [PATCH 1/2] dma-buf: keep only not signaled fence in reservation_object_add_shared_replace v2

2017-11-02 Thread Sumit Semwal
Hi Christian, On 30 October 2017 at 20:29, Christian König wrote: > From: Christian König > > The amdgpu issue to also need signaled fences in the reservation objects > should be fixed by now. > > Optimize the list by keeping only the not signaled yet fences around. I think this warrants some m

Re: [PATCH 0/3] dma-buf changes for ttm and amdgpu

2017-08-09 Thread Sumit Semwal
x27;t exist anymore, all the dma-buf stuff is in drm-misc now). >> And you have commit rights for that even :-) > > Yeah, I just didn't want to step on Sumit's toes in case he preferred to > handle dma-buf patches himself. > Apologies for missing your ping! Please fee

Re: [PATCH 0/3] dma-buf changes for ttm and amdgpu

2017-08-11 Thread Sumit Semwal
NULL safe > dma-buf: add reservation_object_copy_fences > dma-buf: fix reservation_object_wait_timeout_rcu to wait correctly v2 > Looks like my reply on your last set of patches didn't reach you? Anyways, I'm on a short vacation, so please feel free to take this via drm

Re: [PATCH 2/5] dma-buf: remove kmap_atomic interface

2018-06-19 Thread Sumit Semwal
was out of office for some time) - this looks ok to me, please feel free to add my Acked-by: Sumit Semwal > >> --- > >> drivers/dma-buf/dma-buf.c | 44 > >> -- > >> drivers/gpu/drm/amd/amdgpu/amdgpu_prime.c