drm/i915: Add a check that the VMA instance we lookup matches the request
authorChris Wilson <chris@chris-wilson.co.uk>
Mon, 16 Jan 2017 15:21:29 +0000 (15:21 +0000)
committerChris 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

index 87273b0137ec830e46b252dfff849609d5d8f882..b4d7b51266d2337b0908075c6bca7866e83f1806 100644 (file)
@@ -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;
 }