linux-stable.git/drivers/base/dma-buf.c, branch linux-3.13.y

dma-buf: Expose buffer size to userspace (v2)

2013-09-10T06:06:45+00:00

Each dma-buf has an associated size and it's reasonable for userspace
to want to know what it is.

Since userspace already has an fd, expose the size using the
size = lseek(fd, SEEK_END, 0); lseek(fd, SEEK_CUR, 0);
idiom.

v2: Added Daniel's sugeested documentation, with minor fixups

Signed-off-by: Christopher James Halse Rogers 
Reviewed-by: Daniel Vetter 
Tested-by: Daniel Vetter 
Signed-off-by: Sumit Semwal

dma-buf: Check return value of anon_inode_getfile

2013-09-10T05:59:21+00:00

anon_inode_getfile might fail, so check its return value.

Signed-off-by: Tuomas Tynkkynen 
Signed-off-by: Sumit Semwal

dma-buf: Replace PTR_RET with PTR_ERR_OR_ZERO

2013-07-16T06:36:01+00:00

PTR_RET is now deprecated. Use PTR_ERR_OR_ZERO instead.

Signed-off-by: Sachin Kamat 
Signed-off-by: Rusty Russell

[media] dma-buf: Cocci spatch "ptr_ret.spatch"

2013-06-08T23:00:29+00:00

Signed-off-by: Thomas Meyer 
Signed-off-by: Mauro Carvalho Chehab

dma-buf: Add debugfs support

2013-05-01T11:06:22+00:00

Add debugfs support to make it easier to print debug information
about the dma-buf buffers.

Cc: Dave Airlie 
 [minor fixes on init and warning fix]
Cc: Dan Carpenter 
 [remove double unlock in fail case]
Signed-off-by: Sumit Semwal

dma-buf: replace dma_buf_export() with dma_buf_export_named()

2013-05-01T11:05:36+00:00

For debugging purposes, it is useful to have a name-string added
while exporting buffers. Hence, dma_buf_export() is replaced with
dma_buf_export_named(), which additionally takes 'exp_name' as a
parameter.

For backward compatibility, and for lazy exporters who don't wish to
name themselves, a #define dma_buf_export() is also made available,
which adds a __FILE__ instead of 'exp_name'.

Cc: Daniel Vetter 
  [Thanks for the idea!]
Reviewed-by: Daniel Vetter 
Signed-off-by: Sumit Semwal

CHROMIUM: dma-buf: restore args on failure of dma_buf_mmap

2013-02-27T09:44:02+00:00

Callers to dma_buf_mmap expect to fput() the vma struct's vm_file
themselves on failure.  Not restoring the struct's data on failure
causes a double-decrement of the vm_file's refcount.

Signed-off-by: John Sheu 
Reviewed-by: Daniel Vetter 
Signed-off-by: Sumit Semwal

dma-buf: implement vmap refcounting in the interface logic

2013-02-27T09:43:36+00:00

All drivers which implement this need to have some sort of refcount to
allow concurrent vmap usage. Hence implement this in the dma-buf core.

To protect against concurrent calls we need a lock, which potentially
causes new funny locking inversions. But this shouldn't be a problem
for exporters with statically allocated backing storage, and more
dynamic drivers have decent issues already anyway.

Inspired by some refactoring patches from Aaron Plattner, who
implemented the same idea, but only for drm/prime drivers.

v2: Check in dma_buf_release that no dangling vmaps are left.
Suggested by Aaron Plattner. We might want to do similar checks for
attachments, but that's for another patch. Also fix up ERR_PTR return
for vmap.

v3: Check whether the passed-in vmap address matches with the cached
one for vunmap. Eventually we might want to remove that parameter -
compared to the kmap functions there's no need for the vaddr for
unmapping.  Suggested by Chris Wilson.

v4: Fix a brown-paper-bag bug spotted by Aaron Plattner.

Cc: Aaron Plattner 
Reviewed-by: Aaron Plattner 
Tested-by: Aaron Plattner 
Reviewed-by: Rob Clark 
Signed-off-by: Daniel Vetter 
Signed-off-by: Sumit Semwal

dma_buf: Cleanup dma_buf_fd

2013-01-17T20:57:07+00:00

Remove redundant 'error' variable.

Signed-off-by: Borislav Petkov 
Signed-off-by: Greg Kroah-Hartman

dma-buf: might_sleep() in dma_buf_unmap_attachment()

2012-12-20T06:35:05+00:00

We never really clarified if unmap could be done in atomic context.
But since mapping might require sleeping, this implies mutex in use
to synchronize mapping/unmapping, so unmap could sleep as well.  Add
a might_sleep() to clarify this.

Signed-off-by: Rob Clark 
Acked-by: Daniel Vetter 
Reviewed-by: Maarten Lankhorst 
Signed-off-by: Sumit Semwal