Commit 40fa60c8 authored by Chris Wilson's avatar Chris Wilson

drm/i915: Catch premature unpinning of pages

Try to catch the violation of unpinning the backing storage whilst still
bound to the GPU.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Link: http://patchwork.freedesktop.org/patch/msgid/20161019101147.17342-5-chris@chris-wilson.co.ukReviewed-by: default avatarJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
parent 1dc16aac
...@@ -3181,14 +3181,15 @@ i915_gem_object_get_page(struct drm_i915_gem_object *obj, int n) ...@@ -3181,14 +3181,15 @@ i915_gem_object_get_page(struct drm_i915_gem_object *obj, int n)
static inline void i915_gem_object_pin_pages(struct drm_i915_gem_object *obj) static inline void i915_gem_object_pin_pages(struct drm_i915_gem_object *obj)
{ {
BUG_ON(obj->pages == NULL); GEM_BUG_ON(obj->pages == NULL);
obj->pages_pin_count++; obj->pages_pin_count++;
} }
static inline void i915_gem_object_unpin_pages(struct drm_i915_gem_object *obj) static inline void i915_gem_object_unpin_pages(struct drm_i915_gem_object *obj)
{ {
BUG_ON(obj->pages_pin_count == 0); GEM_BUG_ON(obj->pages_pin_count == 0);
obj->pages_pin_count--; obj->pages_pin_count--;
GEM_BUG_ON(obj->pages_pin_count < obj->bind_count);
} }
enum i915_map_type { enum i915_map_type {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment