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.
v5:
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.
v6:
Addressed a few review comments by Tvrtko & Chris; the main
functional change is that i915_gem_object_vmap_range() now takes
a range in bytes rather than pages, so we use sg_nents_for_len()
to validate it against the actual scatterlist.
Signed-off-by: Dave Gordon <david.s.gordon@xxxxxxxxx>
Cc: Alex Dai <yu.dai@xxxxxxxxx>
Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
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
handling
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 again)
drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c | 2 +-
drivers/gpu/drm/i915/i915_cmd_parser.c | 32 +--------
drivers/gpu/drm/i915/i915_drv.h | 26 +++++--
drivers/gpu/drm/i915/i915_gem.c | 103 +++++++++++++++++++++++++++
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 | 44 +++++++++++-
10 files changed, 202 insertions(+), 142 deletions(-)
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
