mbox series

[v2,0/9] drm/msm: separate iova allocation and mapping

Message ID 20181107223554.27308-1-jcrouse@codeaurora.org (mailing list archive)
Headers show
Series drm/msm: separate iova allocation and mapping | expand

Message

Jordan Crouse Nov. 7, 2018, 10:35 p.m. UTC
Currently in the msm driver iova addresses are mapped in the IOMMU at
allocation time and stay there for the life of the buffer. This may not
be desirable for long lived user space buffers that could be temporarily
swapped or moved.

This first set of patches breaks up the allocation and mapping into
distinct steps and adds reference counting for pinning and unpinning
the memory. Future code can use this information to manipulate the
memory when it isn't in use.

Revision 2 adds an additional 2 patches to add a description to each
buffer object so the output from the debugfs file gives a clue as to
what the buffer is used for. This is useful for debugging leaks and
will also be helpful when we start to figure out how to shrink or
move memory around.

The last patch fixes an annoyance where the ringbuffer iova was
refcounted every time the GPU started. It isn't harmful but it
sticks out like a sore thumb when debugging.

Jordan Crouse (9):
  drm/msm: Add a common function to free kernel buffer objects
  drm/msm: Remove sgt from the mmu unmap function
  drm/msm: Split msm_gem_get_iova into two steps
  drm/msm: Clean up and enhance the output of the 'gem' debugfs node
  drm/msm: Add msm_gem_get_and_pin_iova()
  drm/msm: Count how many times iova memory is pinned
  drm/msm: Add a name field for gem objects
  drm/msm: Add a name field to struct drm_msm_gem_new
  drm/msm/gpu: Map the ringbuffer in the iova at create time

 drivers/gpu/drm/msm/adreno/a5xx_debugfs.c |   6 +-
 drivers/gpu/drm/msm/adreno/a5xx_gpu.c     |  33 ++--
 drivers/gpu/drm/msm/adreno/a5xx_power.c   |  15 +-
 drivers/gpu/drm/msm/adreno/a5xx_preempt.c |  16 +-
 drivers/gpu/drm/msm/adreno/a6xx_gpu.c     |   5 +-
 drivers/gpu/drm/msm/adreno/adreno_gpu.c   |   8 -
 drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c |   6 +-
 drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c  |   4 +-
 drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c |   4 +-
 drivers/gpu/drm/msm/dsi/dsi_host.c        |   6 +-
 drivers/gpu/drm/msm/msm_drv.c             |   9 +-
 drivers/gpu/drm/msm/msm_drv.h             |  20 ++-
 drivers/gpu/drm/msm/msm_fb.c              |   6 +-
 drivers/gpu/drm/msm/msm_fbdev.c           |   2 +-
 drivers/gpu/drm/msm/msm_gem.c             | 193 ++++++++++++++++------
 drivers/gpu/drm/msm/msm_gem.h             |   4 +
 drivers/gpu/drm/msm/msm_gem_submit.c      |   4 +-
 drivers/gpu/drm/msm/msm_gem_vma.c         |  95 ++++++++---
 drivers/gpu/drm/msm/msm_gpu.c             |  18 +-
 drivers/gpu/drm/msm/msm_iommu.c           |   3 +-
 drivers/gpu/drm/msm/msm_mmu.h             |   3 +-
 drivers/gpu/drm/msm/msm_ringbuffer.c      |  14 +-
 include/uapi/drm/msm_drm.h                |   1 +
 23 files changed, 302 insertions(+), 173 deletions(-)