Commit ea97c4ca authored by Chris Wilson's avatar Chris Wilson

drm/i915/gem: Suppress some random warnings

Leave the error propagation in place, but limit the warnings to only
show up in CI if the unlikely errors are hit.
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarMika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200525141957.3061-2-chris@chris-wilson.co.uk
parent 1d93949e
...@@ -1626,8 +1626,7 @@ eb_relocate_entry(struct i915_execbuffer *eb, ...@@ -1626,8 +1626,7 @@ eb_relocate_entry(struct i915_execbuffer *eb,
err = i915_vma_bind(target->vma, err = i915_vma_bind(target->vma,
target->vma->obj->cache_level, target->vma->obj->cache_level,
PIN_GLOBAL, NULL); PIN_GLOBAL, NULL);
if (drm_WARN_ONCE(&i915->drm, err, if (err)
"Unexpected failure to bind target VMA!"))
return err; return err;
} }
} }
......
...@@ -29,8 +29,7 @@ static int i915_gem_object_get_pages_phys(struct drm_i915_gem_object *obj) ...@@ -29,8 +29,7 @@ static int i915_gem_object_get_pages_phys(struct drm_i915_gem_object *obj)
void *dst; void *dst;
int i; int i;
if (drm_WARN_ON(obj->base.dev, if (GEM_WARN_ON(i915_gem_object_needs_bit17_swizzle(obj)))
i915_gem_object_needs_bit17_swizzle(obj)))
return -EINVAL; return -EINVAL;
/* /*
......
...@@ -147,8 +147,7 @@ static int shmem_get_pages(struct drm_i915_gem_object *obj) ...@@ -147,8 +147,7 @@ static int shmem_get_pages(struct drm_i915_gem_object *obj)
last_pfn = page_to_pfn(page); last_pfn = page_to_pfn(page);
/* Check that the i965g/gm workaround works. */ /* Check that the i965g/gm workaround works. */
drm_WARN_ON(&i915->drm, GEM_BUG_ON(gfp & __GFP_DMA32 && last_pfn >= 0x00100000UL);
(gfp & __GFP_DMA32) && (last_pfn >= 0x00100000UL));
} }
if (sg) { /* loop terminated early; short sg table */ if (sg) { /* loop terminated early; short sg table */
sg_page_sizes |= sg->length; sg_page_sizes |= sg->length;
......
...@@ -235,7 +235,7 @@ i915_gem_userptr_init__mmu_notifier(struct drm_i915_gem_object *obj, ...@@ -235,7 +235,7 @@ i915_gem_userptr_init__mmu_notifier(struct drm_i915_gem_object *obj,
if (flags & I915_USERPTR_UNSYNCHRONIZED) if (flags & I915_USERPTR_UNSYNCHRONIZED)
return capable(CAP_SYS_ADMIN) ? 0 : -EPERM; return capable(CAP_SYS_ADMIN) ? 0 : -EPERM;
if (drm_WARN_ON(obj->base.dev, obj->userptr.mm == NULL)) if (GEM_WARN_ON(!obj->userptr.mm))
return -EINVAL; return -EINVAL;
mn = i915_mmu_notifier_find(obj->userptr.mm); mn = i915_mmu_notifier_find(obj->userptr.mm);
......
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