[Intel-gfx] [PATCH v5 0/7] Reorganise calls to vmap() GEM objects

Dave Gordon david.s.gordon at intel.com
Mon Feb 29 11:13:07 UTC 2016

Alex Dai and Chris Wilson have both recently posted patches to
rationalise the use of vmap() for mapping GEM objects into kernel
virtual space. However, they addressed different areas, with Alex's
patch being derived from the copy_batch() code, whereas Chris' patch
refactored the dma-buf and ringbuffer code.

So this patchset unifies the two, copying Chris' interfaces which
unite pin-and-vmap for convenient lifecycle management, but using
Alex's code underneath to permit partial mappings. And finally
there's a little optimisation I've added for "small" objects e.g.
ringbuffers and contexts, which are expected to be the objects most
commonly handled by this code.

  Added another of Chris' patches, introducing drm_malloc_gfp().

  Split Chris' original patch into three, of which two are actually
  minor unrelated improvements, and only one is actually addresses
  the vmap() reorganisation [Tvrtko Ursulin]

  Decided not to hold onto vmappings after the pin count goes to
  zero. This may reduce the benefit of Chris' scheme somewhat, but
  does avoid any increased risk of exhausting kernel vmap space on
  32-bit kernels [Tvrtko Ursulin]. Potentially, the vunmap() could
  be moved back to the put_pages() stage (thus extending the cache
  lifetime) if a suitable notifier were written, but that's not
  included here.

Signed-off-by: Dave Gordon <david.s.gordon at intel.com>
Cc: Alex Dai <yu.dai at intel.com>
Cc: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
Cc: Daniel Vetter <daniel.vetter at ffwll.ch>

Alex Dai (1):
  drm/i915: introduce and use i915_gem_object_vmap_range()

Chris Wilson (3):
  drm,i915: introduce drm_malloc_gfp()
  drm/i915: move locking in i915_gem_unmap_dma_buf()
  drm/i915: deduplicate intel_pin_and_map_ringbuffer_obj() error

Dave Gordon (3):
  drm: add parameter-order checking to drm memory allocators
  drm/i915: optimise i915_gem_object_vmap_range() for small objects
  drm/i915: refactor duplicate object vmap functions (reworked some

 drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c |  2 +-
 drivers/gpu/drm/i915/i915_cmd_parser.c       | 28 +--------
 drivers/gpu/drm/i915/i915_drv.h              | 26 ++++++--
 drivers/gpu/drm/i915/i915_gem.c              | 88 ++++++++++++++++++++++++++++
 drivers/gpu/drm/i915/i915_gem_dmabuf.c       | 53 ++---------------
 drivers/gpu/drm/i915/i915_gem_execbuffer.c   | 12 ++--
 drivers/gpu/drm/i915/i915_gem_gtt.c          |  5 +-
 drivers/gpu/drm/i915/i915_gem_userptr.c      | 15 ++---
 drivers/gpu/drm/i915/intel_ringbuffer.c      | 52 +++++-----------
 include/drm/drm_mem_util.h                   | 47 ++++++++++++++-
 10 files changed, 190 insertions(+), 138 deletions(-)


More information about the Intel-gfx mailing list