]> git.baikalelectronics.ru Git - kernel.git/commit
drm/i915: Allow i915_gem_object_get_page() on userptr as well
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 22 Jan 2016 18:32:31 +0000 (18:32 +0000)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Wed, 3 Feb 2016 18:21:24 +0000 (10:21 -0800)
commit6b5cff7e19e1ed5f664cec5e61b4802d2dca68da
tree1edb28e1ca191d3d8053cde723008b2179fad3d3
parenta6a33cf606c84db29e7d4f00d9329085f6cc989e
drm/i915: Allow i915_gem_object_get_page() on userptr as well

commit d5fa9077b065388ba3e754a21db156770a273a5b
Author: Dave Gordon <david.s.gordon@intel.com>
Date:   Thu Dec 10 18:51:23 2015 +0000

    drm/i915: mark GEM object pages dirty when mapped & written by the CPU

introduced a check into i915_gem_object_get_dirty_pages() that returned
a NULL pointer when called with a bad object, one that was not backed by
shmemfs. This WARN was too strict as we can work on all struct page
backed objects, and resulted in a WARN + GPF for existing userspace. In
order to differentiate the various types of objects, add a new flags field
to the i915_gem_object_ops struct to describe their capabilities, with
the first flag being whether the object has struct pages.

v2: Drop silly const before an integer in the structure declaration.

Testcase: igt/gem_userptr_blits/relocations
Reported-and-tested-by: Kristian Høgsberg Kristensen <krh@bitplanet.net>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Dave Gordon <david.s.gordon@intel.com>
Cc: Kristian Høgsberg Kristensen <krh@bitplanet.net>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Dave Gordon <david.s.gordon@intel.com>
Reviewed-by: Kristian Høgsberg Kristensen <krh@bitplanet.net>
Tested-by: Michal Winiarski <michal.winiarski@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1453487551-16799-1-git-send-email-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_drv.h
drivers/gpu/drm/i915/i915_gem.c
drivers/gpu/drm/i915/i915_gem_userptr.c