projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
718659a
)
drm/i915: Add a check that the VMA instance we lookup matches the request
author
Chris Wilson
<chris@chris-wilson.co.uk>
Mon, 16 Jan 2017 15:21:29 +0000
(15:21 +0000)
committer
Chris Wilson
<chris@chris-wilson.co.uk>
Thu, 19 Jan 2017 10:17:24 +0000
(10:17 +0000)
Just as added paranoia against our future-selves add another check that
the lookup/created VMA instance matches the request.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link:
http://patchwork.freedesktop.org/patch/msgid/20170116152131.18089-3-chris@chris-wilson.co.uk
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
drivers/gpu/drm/i915/i915_vma.c
patch
|
blob
|
history
diff --git
a/drivers/gpu/drm/i915/i915_vma.c
b/drivers/gpu/drm/i915/i915_vma.c
index
87273b0
..
b4d7b51
100644
(file)
--- a/
drivers/gpu/drm/i915/i915_vma.c
+++ b/
drivers/gpu/drm/i915/i915_vma.c
@@
-242,6
+242,7
@@
i915_vma_instance(struct drm_i915_gem_object *obj,
vma = i915_vma_create(obj, vm, view);
GEM_BUG_ON(!IS_ERR(vma) && i915_vma_is_closed(vma));
+ GEM_BUG_ON(!IS_ERR(vma) && i915_vma_compare(vma, vm, view));
GEM_BUG_ON(!IS_ERR(vma) && i915_vma_lookup(obj, vm, view) != vma);
return vma;
}