Re: [PATCH v4] udmabuf: add vmap and vunmap methods to udmabuf_ops

2022-11-18 Thread Lukasz Wiecaszek
On Thu, Nov 17, 2022 at 07:01:05PM +0100, Christian König wrote: > Am 17.11.22 um 18:32 schrieb Dmitry Osipenko: > > On 11/17/22 20:08, Lukasz Wiecaszek wrote: > > > On Thu, Nov 17, 2022 at 12:04:35PM +0300, Dmitry Osipenko wrote: > > > > Hi, > > > > &

[PATCH v5] udmabuf: add vmap and vunmap methods to udmabuf_ops

2022-11-17 Thread Lukasz Wiecaszek
e. .vunmap removes mapping created earlier by .vmap. All locking and 'vmapping counting' is done in dma_buf.c so it seems to be redundant/unnecessary in .vmap/.vunmap. Reviewed-by: Dmitry Osipenko Acked-by: Christian König Signed-off-by: Lukasz Wiecaszek --- v1: https://lore.ke

Re: [PATCH v4] udmabuf: add vmap and vunmap methods to udmabuf_ops

2022-11-17 Thread Lukasz Wiecaszek
On Thu, Nov 17, 2022 at 12:04:35PM +0300, Dmitry Osipenko wrote: > Hi, > > On 11/17/22 07:58, Lukasz Wiecaszek wrote: > > The reason behind that patch is associated with videobuf2 subsystem > > (or more genrally with v4l2 framework) and user created > > dma

[PATCH v4] udmabuf: add vmap and vunmap methods to udmabuf_ops

2022-11-16 Thread Lukasz Wiecaszek
e. .vunmap removes mapping created earlier by .vmap. All locking and 'vmapping counting' is done in dma_buf.c so it seems to be redundant/unnecessary in .vmap/.vunmap. Signed-off-by: Lukasz Wiecaszek --- v1: https://lore.kernel.org/linux-media/202211120352.g7wpasop-...@inte

Re: [PATCH v3] udmabuf: add vmap and vunmap methods to udmabuf_ops

2022-11-16 Thread Lukasz Wiecaszek
On Wed, Nov 16, 2022 at 01:01:46PM +0100, Christian König wrote: > Am 15.11.22 um 21:04 schrieb Lukasz Wiecaszek: > > The reason behind that patch is associated with videobuf2 subsystem > > (or more genrally with v4l2 framework) and user created > > dma buffers (udmabuf).

[PATCH v3] udmabuf: add vmap and vunmap methods to udmabuf_ops

2022-11-15 Thread Lukasz Wiecaszek
e. .vunmap removes mapping created earlier by .vmap. All locking and 'vmapping counting' is done in dma_buf.c so it seems to be redundant/unnecessary in .vmap/.vunmap. Signed-off-by: Lukasz Wiecaszek Reported-by: kernel test robot --- v1: https://lore.kernel.org/linux-media/2

Re: [PATCH v2] udmabuf: add vmap method to udmabuf_ops

2022-11-13 Thread Lukasz Wiecaszek
On Sun, Nov 13, 2022 at 07:35:20PM +0300, Dmitry Osipenko wrote: > On 11/13/22 18:05, Lukasz Wiecaszek wrote: > > +static int vmap_udmabuf(struct dma_buf *buf, struct iosys_map *map) > > +{ > > + struct udmabuf *ubuf = buf->priv; > > + > > + if (!ubuf-&

[PATCH v2] udmabuf: add vmap method to udmabuf_ops

2022-11-13 Thread Lukasz Wiecaszek
hod itself uses vm_map_ram() to map pages linearly into the kernel virtual address space (only if such mapping hasn't been created yet). Signed-off-by: Lukasz Wiecaszek Reported-by: kernel test robot --- v1: https://lore.kernel.org/linux-media/202211120352.g7wpasop-...@intel.com/T/#t v1 ->

Re: [PATCH] udmabuf: add vmap method to udmabuf_ops

2022-11-13 Thread Lukasz Wiecaszek
On Fri, Nov 11, 2022 at 03:31:15PM +0300, Dmitry Osipenko wrote: > On 11/11/22 15:05, Christian König wrote: > > Adding Dmitry as well. > > > > Am 11.11.22 um 12:45 schrieb Lukasz Wiecaszek: > >> The reason behind that patch is associated with videobuf2 subsystem &g

[PATCH] udmabuf: add vmap method to udmabuf_ops

2022-11-11 Thread Lukasz Wiecaszek
hod itself uses vm_map_ram() to map pages linearly into the kernel virtual address space (only if such mapping hasn't been created yet). Signed-off-by: Lukasz Wiecaszek --- drivers/dma-buf/udmabuf.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/drivers/dma-buf/udmabuf.